On Nov 30, 2010, at 9:23 AM, Joakim Tjernlund wrote: >> >> On Tue, 2010-11-30 at 08:00 -0700, Wolfgang Denk wrote: >>> Hello everybody. >>> >>> I apologise for being a bit late with this announcement: >>> >>> * U-Boot v2010.12-rc2 was released on Sunday, November 28. >>> >>> * Release "v2010.12" is (still) scheduled in 13 days: >>> on December 13, 2010. >>> >>> Please help testing, and check if all your relevant patches have been >>> included. >> >> Is it possible to pull Kumar's patch: >> http://lists.denx.de/pipermail/u-boot/2010-November/082099.html >> >> Since 8xxx ddr spd code calls hwconfig before relocating code, the most >> 85xx/86xx boards which use 8xxx ddr spd can not bring up without this >> patch. > > Does > const char *cpu_hwconfig __attribute__((weak)) = ""; > const char *board_hwconfig __attribute__((weak)) = ""; > need to ptrs instead of an array? > const char cpu_hwconfig[] __attribute__((weak)) = ""; > const char board_hwconfig[] __attribute__((weak)) = ""; > > All these static string pointers costs extra space, in this case > 2x4 bytes for the ptr and 2 fixup entries = 16 bytes > > Jocke
We can make them arrays instead. I don't see any reason they need to be pointers. - k _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot