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