On Wed, 2008-03-19 at 18:29, Wolfgang Denk wrote: > Dear Grant, > > in http://news.gmane.org/gmane.comp.boot-loaders.u-boot > Jean-Christophe asked to get rid of the "#ifdef CONFIG_POST" parts in > the post code by using the "COBJS-$(CONFIG_POST) += " approach in the > Makefile. We tried, and it didn't work. > > The reason is that the Makefile assumes that the CONFIG_* variable > settings in include/autoconf.mk have a value of "y", so we can > collect objects to build in the "COBJS-y" make variable. > > However, this is not always true. Here is what we see for example on > the "lwmon5" board: > > -> grep -v =y include/autoconf.mk > CONFIG_SYS_CLK_FREQ=33300000 > CONFIG_POST_WATCHDOG="{ "Watchdog timer test", "watchdog", "This test checks > the watchdog timer.", POST_RAM | POST_POWERON | POST_SLOWTEST | POST_MANUAL | > POST_REBOOT, &lwmon5_watchdog_post_test, NULL, NULL, CFG_POST_WATCHDOG }" > CONFIG_BOOTDELAY=5 > CONFIG_ALT_LB_ADDR="(CFG_OCM_BASE)" > CONFIG_BOOTCOMMAND="run flash_self" > CONFIG_POST_KEY_MAGIC="3C+3E" > CONFIG_PHY_RESET_DELAY=300 > CONFIG_POST_BSPEC1="{ "dsPIC init test", "dspic_init", "This test returns > result of dsPIC READY test run earlier.", POST_RAM | POST_ALWAYS, > &dspic_init_post_test, NULL, NULL, CFG_POST_BSPEC1 }" > CONFIG_POST_BSPEC2="{ "dsPIC test", "dspic", "This test gets result of dsPIC > POST and dsPIC version.", POST_RAM | POST_ALWAYS, &dspic_post_test, NULL, > NULL, CFG_POST_BSPEC2 }" > CONFIG_POST_BSPEC3="{ "FPGA test", "fpga", "This test checks FPGA registers > and memory.", POST_RAM | POST_ALWAYS, &fpga_post_test, NULL, NULL, > CFG_POST_BSPEC3 }" > CONFIG_POST_BSPEC4="{ "GDC test", "gdc", "This test checks GDC registers and > memory.", POST_RAM | POST_ALWAYS, &gdc_post_test, NULL, NULL, CFG_POST_BSPEC4 > }" > CONFIG_POST_BSPEC5="{ "SYSMON1 test", "sysmon1", "This test checks > GPIO_62_EPX pin indicating power failure.", POST_RAM | POST_MANUAL | > POST_NORMAL | POST_SLOWTEST, &sysmon1_post_test, NULL, NULL, CFG_POST_BSPEC5 > }" > CONFIG_PHY_ADDR=3 > CONFIG_BAUDRATE=115200 > CONFIG_ALT_LH_ADDR="(CFG_PERIPHERAL_BASE + GPT0_COMP1)" > CONFIG_POST="(CFG_POST_CACHE | CFG_POST_CPU | CFG_POST_ECC_ON | > CFG_POST_ETHER | CFG_POST_FPU | CFG_POST_I2C | CFG_POST_MEMORY | CFG_POST_RTC > | CFG_POST_SPR | CFG_POST_UART | CFG_POST_SYSMON | CFG_POST_WATCHDOG | > CFG_POST_DSP | CFG_POST_BSPEC1 | CFG_POST_BSPEC2 | CFG_POST_BSPEC3 | > CFG_POST_BSPEC4 | CFG_POST_BSPEC5)" > CONFIG_PREBOOT="setenv bootdelay 15" > CONFIG_WD_PERIOD=40000
*sigh* Needless to say, exactly the same change that I performed last year for CONFIG_COMMAND -> CONFIG_CMD_* and for CONFIG_BOOTP needs to be done here now for CONFIG_POST as well. I am personally swamped right now and will not be able to get to it directly, but some enterprising young whipper-snapper might get there before I do... :-) jdl ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ U-Boot-Users mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/u-boot-users
