[U-Boot] [PATCH] lcd: Add support for CONFIG_LCD_NOSTDOUT

Hannes Petermaier hannes at petermaier.org
Sat Mar 8 20:46:06 CET 2014


Hi Gerhard,

On 2014-03-06 20:49, Gerhard Sittig wrote:
> In general U-Boot tries to get away from the multitude of ifdefs
> where possible.  I'm afraid adding a new one needs a very good
> reason to get perceived as welcome.
By the way i've found a passage within README, which tells to do so as 
id id:

'* If you modify existing code, make sure that your new code does not
   add to the memory footprint of the code ;-) Small is beautiful!

   When adding new features, these should compile conditionally only
   (using #ifdef), and the resulting code with the new feature
   disabled must not need more memory than the old code without your
   modification.
'
whats your opinion about this ?
> virtually yours
> Gerhard Sittig
best regards,
Hannes



More information about the U-Boot mailing list