[U-Boot] [PATCH v7 4/5] imx6: add spl in the header file
Fabio Estevam
festevam at gmail.com
Thu Nov 13 13:41:27 CET 2014
Hi John,
On Thu, Nov 13, 2014 at 6:58 AM, Stefano Babic <sbabic at 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
More information about the U-Boot
mailing list