[U-Boot] [PATCH v7 4/5] imx6: add spl in the header file
Fabio Estevam
festevam at gmail.com
Thu Nov 13 13:49:05 CET 2014
On Thu, Nov 13, 2014 at 10:41 AM, Fabio Estevam <festevam at gmail.com> wrote:
> 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
Sorry, I meant to remove only the ifdef:
--- a/include/configs/mx6sabre_common.h
+++ b/include/configs/mx6sabre_common.h
@@ -95,9 +95,7 @@
#define CONFIG_BOOTDELAY 1
#define CONFIG_LOADADDR 0x12000000
-#ifndef CONFIG_SYS_TEXT_BASE
#define CONFIG_SYS_TEXT_BASE 0x17800000
-#endif
More information about the U-Boot
mailing list