On Thursday, October 07, 2010 15:35:44 Wolfgang Denk wrote:
> Mike Frysinger wrote:
> > > Do you plan to post an update?
> > 
> > there isnt a clear indication of where to take this.  seems like we want
> > to do this, and we want it as the default moving forward, but we want
> > all existing boards to be unchanged.  so only reasonable way would be to
> > invert the logic, add a define for the arch lib/board.c files, and then
> > add that define to all existing boards.
> 
> I don't think we want to modify 550+ Board configurations and re-test
> on that many boards...

it would be ~100 boards.  board_init() is only called when CONFIG_CMD_NAND is 
defined.  so it should be as simple as:
        sed -i \
        '/define[[:space:]]*CONFIG_CMD_NAND/i#define CONFIG_NAND_EARLY_INIT' \
        include/configs/*

> I think we should rather enable the new feature by some #define, and
> recommend to enable this on new boards.

problem with recommendations is that people dont notice them
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to