Hi John, On Thu, Nov 13, 2014 at 6:58 AM, Stefano Babic <sba...@denx.de> wrote:
>> #define CONFIG_LOADADDR 0x12000000 >> +#ifndef CONFIG_SYS_TEXT_BASE >> #define CONFIG_SYS_TEXT_BASE 0x17800000 >> +#endif >> > > Why is that needed ? SPL does not use it, because it use > CONFIG_SPL_TEXT_BASE, and this is defined in imx6_spl.h. And you do not > redefine it, meaning that overwriting the value is not necessary at all, > as we expect. Stefano is right. You can simply remove this piece: --- a/include/configs/mx6sabre_common.h +++ b/include/configs/mx6sabre_common.h @@ -95,9 +95,6 @@ #define CONFIG_BOOTDELAY 1 #define CONFIG_LOADADDR 0x12000000 -#ifndef CONFIG_SYS_TEXT_BASE -#define CONFIG_SYS_TEXT_BASE 0x17800000 -#endif #ifdef CONFIG_SUPPORT_EMMC_BOOT #define EMMC_ENV \ and the build will succeed as expected. Thanks _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot