Re: eCos 3.0 beta 1 punch list #2

[Jonathan Larmour <jifl@eCosCentric.com>]

[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