On 12/07/2011 11:46 AM, Ilya Yanok wrote: > Hi Stefano, > > On 07.12.2011 13:06, Stefano Babic wrote: >>> +#ifndef CONFIG_SPL_NAND_WORKSPACE >>> +#define CONFIG_SPL_NAND_WORKSPACE (CONFIG_SYS_SDRAM_BASE + 0x10000) >>> +#endif >> >> Maybe it is better to not have a default value somewhere in the SDRAM >> and to get a compile error. If we do not want to fix also the related > > My intention was not to break something as I can't fix every board using > this driver. That's why I preserved the existing address for the case > where CONFIG_SPL_NAND_WORKSPACE is not defined.
How many boards are using the new SPL for NAND so far? It shouldn't be that bad to just fix them. -Scott _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
