From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 17998 invoked by alias); 17 Feb 2009 09:34:36 -0000 Received: (qmail 17988 invoked by uid 22791); 17 Feb 2009 09:34:35 -0000 X-SWARE-Spam-Status: No, hits=-2.3 required=5.0 tests=AWL,BAYES_00 X-Spam-Check-By: sourceware.org Received: from mtaout01-winn.ispmail.ntl.com (HELO mtaout01-winn.ispmail.ntl.com) (81.103.221.47) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 17 Feb 2009 09:34:27 +0000 Received: from aamtaout04-winn.ispmail.ntl.com ([81.103.221.35]) by mtaout01-winn.ispmail.ntl.com (InterMail vM.7.08.04.00 201-2186-134-20080326) with ESMTP id <20090217093423.BVQU2989.mtaout01-winn.ispmail.ntl.com@aamtaout04-winn.ispmail.ntl.com>; Tue, 17 Feb 2009 09:34:23 +0000 Received: from cog.dallaway.org.uk ([213.106.81.244]) by aamtaout04-winn.ispmail.ntl.com (InterMail vG.2.02.00.01 201-2161-120-102-20060912) with ESMTP id <20090217093423.KYXV22934.aamtaout04-winn.ispmail.ntl.com@cog.dallaway.org.uk>; Tue, 17 Feb 2009 09:34:23 +0000 Received: from cog.dallaway.org.uk (cog.dallaway.org.uk [127.0.0.1]) by cog.dallaway.org.uk (8.13.8/8.13.8) with ESMTP id n1H9YLFW021438; Tue, 17 Feb 2009 09:34:21 GMT Message-ID: <499A849D.6030809@dallaway.org.uk> Date: Tue, 17 Feb 2009 09:34:00 -0000 From: John Dallaway User-Agent: Thunderbird 2.0.0.19 (X11/20090107) MIME-Version: 1.0 To: Jonathan Larmour , Bart Veer CC: ecos-maintainers@ecos.sourceware.org Subject: Flash subsystem update [ was Re: eCos 3.0 beta 1 punch list #2 ] References: <4992951D.10004@dallaway.org.uk> <499A12DD.4060609@eCosCentric.com> In-Reply-To: <499A12DD.4060609@eCosCentric.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit X-IsSubscribed: yes Mailing-List: contact ecos-maintainers-help@ecos.sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Post: List-Help: , Sender: ecos-maintainers-owner@ecos.sourceware.org X-SW-Source: 2009-02/txt/msg00029.txt.bz2 Hi Jifl and Bart Regarding the prioritised constructor for Flash and consequent API changes, Jonathan Larmour wrote: > If the API is wrong we need to fix out before we make a major release. A > stable flash API was the second most critical change to make for eCos > 3.0 after the header updates. We can't flub it. This is a problem I > raised with you back in 2006. > > Let me know what you consider the problems to be and I will deal with > them. and later wrote: > More constructively, until I know what the "messy" bit is, I suggest > adding: > > externC cyg_flash_printf *cyg_flash_set_printf( const cyg_flashaddr_t > base, cyg_flash_printf *pf); > > which associates the printf function of flash at base 'base' with > function 'pf', returning its previous setting. There would also be: > #define CYG_FLASH_SET_PRINTF_ALL ((cyg_flashaddr_t)-1); > > which is used to set all devices' printf functions (and always return > NULL). > > We can instantly deprecate use of cyg_flash_init with non-NULL argument. > And if the messiness is too much to deal with after all, we can continue > with cyg_flash_init, and in future it would be more or less #defined to > cyg_flash_set_printf. FAOD, I'll take this as a showstopper issue for 3.0 beta 1 on the basis that we don't want API changes between beta and final. My concerns are: a) Further slippage to the beta release. b) The trunk is currently semi-frozen (no large/invasive changes) which prevents the maintainers from undertaking regular maintainer duties. The two issues are, of course, related. If we're able to address the Flash issue this week, I think it makes sense to cut the 3.0 release branch now (while we have stability) and double commit the Flash API changes to address issue (b). If the Flash issue will take longer, then we should hold back on cutting the branch and lift the semi-frozen status of the repository. We then risk some de-stabilisation. We need to contain further slippage as far as practically possible and push forward with eCos 3.0. I would therefore like to ensure we have a well-understood plan for addressing this issue in a realistic timescale. Firstly, we need to understand the issues that Bart encountered in detail. Bart, can you provide this detail today please? Does Jifl's proposed workaround work for you? John Dallaway