From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 10986 invoked by alias); 17 Feb 2009 09:18:26 -0000 Received: (qmail 10977 invoked by uid 22791); 17 Feb 2009 09:18:25 -0000 X-SWARE-Spam-Status: No, hits=-2.2 required=5.0 tests=AWL,BAYES_00,SPF_PASS X-Spam-Check-By: sourceware.org Received: from hagrid.ecoscentric.com (HELO mail.ecoscentric.com) (212.13.207.197) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 17 Feb 2009 09:18:13 +0000 Received: from localhost (hagrid.ecoscentric.com [127.0.0.1]) by mail.ecoscentric.com (Postfix) with ESMTP id 749E860B8004 for ; Tue, 17 Feb 2009 09:18:11 +0000 (GMT) Received: from mail.ecoscentric.com ([127.0.0.1]) by localhost (hagrid.ecoscentric.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id Ms+uwax4vxRg; Tue, 17 Feb 2009 09:18:10 +0000 (GMT) Message-ID: <499A80DA.5030403@eCosCentric.com> Date: Tue, 17 Feb 2009 09:18:00 -0000 From: Jonathan Larmour User-Agent: Mozilla Thunderbird 1.0.8-1.1.fc3.4.legacy (X11/20060515) MIME-Version: 1.0 To: Bart Veer CC: ecos-maintainers@ecos.sourceware.org Subject: 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; format=flowed Content-Transfer-Encoding: 8bit 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/msg00028.txt.bz2 [ker-snip] Sorry I sounded a bit harsh there. It was late and I was tired. But I do believe that a .0 release should go out with an API which will work in future. 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. Jifl -- *See us at Embedded World 2009, Nürnberg, Germany, 3-5 Mar, Stand 11-300* eCosCentric Limited http://www.eCosCentric.com/ The eCos experts Barnwell House, Barnwell Drive, Cambridge, UK. Tel: +44 1223 245571 Registered in England and Wales: Reg No 4422071. ------["Si fractum non sit, noli id reficere"]------ Opinions==mine