[U-Boot] [PATCH 3/3] SMDK6400: Fix build error for smdk6400 nand_spl support

[seedshope]

On 01/09/2011 10:12 PM, Minkyu Kang wrote:
> Hello,
>
> On 9 January 2011 21:27, seedshope<bocui107 at gmail.com>  wrote:
>> On 01/09/2011 03:33 PM, Minkyu Kang wrote:
>>> Hi
>>>
>>> On 9 January 2011 10:14, seedshope<bocui107 at gmail.com>    wrote:
>>>> On 01/08/2011 11:15 PM, Minkyu Kang wrote:
>>>>> Dear seedshope,
>>>>>
>>>>> On 8 January 2011 00:53, seedshope<bocui107 at gmail.com>      wrote:
>>>>>> From: seedshope<bocui107 at gmail.com>
>>>>>>
>>>>>> Modify u-boot.lds from nand_spl/board/samsung/smdk6400.
>>>>>>
>>>>>> start.o: In function `clbss_l':
>>>>>> nand_spl/board/samsung/smdk6400/start.S:357: undefined reference to
>>>>>> `coloured_LED_init'
>>>>>> nand_spl/board/samsung/smdk6400/start.S:358: undefined reference to
>>>>>> `red_LED_on'
>>>>>> start.o: In function `_rel_dyn_start_ofs':
>>>>>> nand_spl/board/samsung/smdk6400/start.S:366: undefined reference to
>>>>>> `__rel_dyn_start'
>>>>>> start.o: In function `_rel_dyn_end_ofs':
>>>>>> nand_spl/board/samsung/smdk6400/start.S:366: undefined reference to
>>>>>> `__rel_dyn_end'
>>>>>> start.o: In function `_dynsym_start_ofs':
>>>>>> nand_spl/board/samsung/smdk6400/start.S:366: undefined reference to
>>>>>> `__dynsym_start'
>>>>>>
>>>>>> Signed-off-by: seedshope<bocui107 at googlemail.com>
>>>>>>
>>>>>> diff --git a/arch/arm/cpu/arm1176/start.S
>>>>>> b/arch/arm/cpu/arm1176/start.S
>>>>>> index 237dcfe..bde0357 100644
>>>>>> --- a/arch/arm/cpu/arm1176/start.S
>>>>>> +++ b/arch/arm/cpu/arm1176/start.S
>>>>>> @@ -353,10 +353,11 @@ clbss_l:str       r2, [r0]                /*
>>>>>> clear
>>>>>> loop...                    */
>>>>>>         add     r0, r0, #4
>>>>>>         cmp     r0, r1
>>>>>>         bne     clbss_l
>>>>>> -
>>>>>> +#ifndef CONFIG_NAND_SPL
>>>>> CONFIG_PRELOADER is better
>>>> No, The nand_spl support can't find the define of coloured_LED_init and
>>>> red_LED_on.
>>>> It is base on the build error:
>>>>
>>>> make smdk6400_config
>>>> make
>>>> the error information as following:
>>>>
>>>> nand_spl/board/samsung/smdk6400/start.S:357: undefined reference to
>>>> `coloured_LED_init'
>>>> nand_spl/board/samsung/smdk6400/start.S:358: undefined reference to
>>>> `red_LED_on'
>>>>
>>> Please add follow define at Makefile.
>>> AFLAGS  += -DCONFIG_PRELOADER -DCONFIG_NAND_SPL
>>> CFLAGS  += -DCONFIG_PRELOADER -DCONFIG_NAND_SPL
>> ok, It may be a good method.
>>
>> But I worried other people will be difficult  to see the process of the code
>> running.
>> So, if add #ifdef  CONFIG_NAND_SPL, It is very clear. What do you think
>> about it?
>>
> Ah, already used CONFIG_NAND_SPL at start.S.
> OK, we can do this work (change from CONFIG_NAND_SPL to
> CONFIG_PRELOADER) at another patch.
Ok, I will send RR2 for SMDK6400 relate patch.

Thanks,
seedshope
> Thanks
> Minkyu Kang