[U-Boot] [PATCH] arm, imx: fix spl compile for mxs boards

From: Heiko Schocher <hs@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] arm, imx: fix spl compile for mxs boards
Date: Tue, 17 Mar 2015 10:40:02 +0100	[thread overview]
Message-ID: <5507F672.3030105@denx.de> (raw)
In-Reply-To: <5507E55A.8000107@denx.de>

Hello Stefano,

Am 17.03.2015 09:27, schrieb Stefano Babic:
> Hi Heiko, Daniel,
>
>
> On 17/03/2015 08:20, Heiko Schocher wrote:
>> files in arch/arm/imx-common/ get not yet compiled for
>> SPL case as "mxs" is missing in filter rule.
>>
>
> This is possible, but...

Yes .. I thought so too ...

>> Signed-off-by: Heiko Schocher <hs@denx.de>
>>
>> ---
>>
>> Fixes build error on for example the mx28evk_auart_console board:
>>
>> Building mx28evk_auart_console board...
>>     text    data     bss     dec     hex filename
>>   442689   36826  327648  807163   c50fb ./u-boot
>
>
> ...I do not understand why I can compile clean (u-boot-imx):
>
> ./tools/buildman/buildman mx28evk
> boards.cfg is up to date. Nothing to do.
> Building current source for 4 boards (4 threads, 2 jobs per thread)
>      4    0    0 /4      0:00:48  : mx28evk_auart_console
>
> And all mxb boards are compiled clean, too.

Hmm... I just did a "make mrproper" and now I see them without this
patch compiling again ... but it seems bogus to me not to add "mxs"
also to the SPL build, as it is in the "else" ... did this boards
use puts, printf in SPL ?

spl/drivers/serial/mxs_auart.o is not compiled in the SPL case ...

Hmm... no idea, why I got this error ...

bye,
Heiko
>
>> make[1]: *** [spl/u-boot-spl] Error 1
>> make: *** [spl/u-boot-spl] Error 2
>> drivers/serial/built-in.o: In function `mxs_auart_init':
>> /home/hs/zug/u-boot/drivers/serial/mxs_auart.c:84: undefined reference to `mxs_reset_block'
>> make[1]: *** [spl/u-boot-spl] Error 1
>> make: *** [spl/u-boot-spl] Error 2
>> make: *** Warte auf noch nicht beendete Prozesse...
>>
>>   arch/arm/Makefile | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/Makefile b/arch/arm/Makefile
>> index 08946de..55fe509 100644
>> --- a/arch/arm/Makefile
>> +++ b/arch/arm/Makefile
>> @@ -37,7 +37,7 @@ libs-y += arch/arm/cpu/
>>   libs-y += arch/arm/lib/
>>
>>   ifeq ($(CONFIG_SPL_BUILD),y)
>> -ifneq (,$(CONFIG_MX23)$(CONFIG_MX35)$(filter $(SOC), mx25 mx27 mx5 mx6 mx31 mx35))
>> +ifneq (,$(CONFIG_MX23)$(CONFIG_MX35)$(filter $(SOC), mx25 mx27 mx5 mx6 mx31 mx35 mxs))
>>   libs-y += arch/arm/imx-common/
>>   endif
>>   else
>
> Best regards,
> Stefano Babic
>
>

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany