[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