[U-Boot] [U-Boot, v4, 60/66] rockchip: Kconfig: preset TPL_LDSCRIPT via Kconfig for the RK3368

[Andy Yan]

Hi PHilipp:


On 2017年08月07日 16:39, Dr. Philipp Tomsich wrote:
> Hm, ok: let me check again, maybe I can spot this now.
> I wonder why this didn’t show for me when I tried earlier.

     I also didn't found it in the early times build. When I run the 
"buildman rockchip" today, It show the warnings in red color,
Then I go back check the manual build (make xxxx_defconfig all), I found 
the warning message  also hide in it.:-)
>> On 07 Aug 2017, at 10:38, Andy Yan <andy.yan at rock-chips.com> wrote:
>>
>> Hi Philipp:
>>
>>
>> On 2017年08月07日 16:24, Dr. Philipp Tomsich wrote:
>>> Andy,
>>>
>>> Doing a non-buildman defconfig + make works fine.
>>> I’ve seen this too (I did a full buildman for all arm, x86 and powerpc boards) and it seems as if a shell-script is trying to source the generated auto.config file.
>>> I couldn’t spot where in buildman that happens, so I’ll need to leave this to someone with more insight into buildman to figure out…
>>
>>     I build with : make ARCH=arm CROSS_COMPILE=arm-linux-gnueabi- firefly-rk3288_defconfig all
>> Also can see the waring: inclue/config/auto.conf: line 209: ARCH: command not found.
>>> Regards,
>>> Phil.
>>>
>>>> On 07 Aug 2017, at 03:19, Andy Yan <andy.yan at rock-chips.com> wrote:
>>>>
>>>> Hi :
>>>>
>>>>     When run buildman rockchip on the master branch of current u-boot-rockchip tree, I got warnings about 16 rockchip arm32 based boards:
>>>>
>>>>    include/config/auto.config: line x: ARCH: command not found.
>>>>
>>>>    the line x is the definition of the: SPL_LDSCRITP "arch/$(ARCH)/cpu/u-boot-spl.lds"
>>>>
>>>>    not sure is there somethings wrong about this series change.
>>>>
>>>>
>>>> On 2017年08月07日 01:18, Philipp Tomsich wrote:
>>>>>> Set TPL_LDSCRIPT in Kconfig, so we don't have to pollute our
>>>>>> header file.
>>>>>>
>>>>>> Signed-off-by: Philipp Tomsich <philipp.tomsich at theobroma-systems.com>
>>>>>> Reviewed-by: Simon Glass <sjg at chromium.org>
>>>>>> ---
>>>>>>
>>>>>> Changes in v4:
>>>>>> - (added) sets TPL_LDSCRIPT via Kconfig
>>>>>>
>>>>>> Changes in v3: None
>>>>>> Changes in v2: None
>>>>>>
>>>>>>   arch/arm/mach-rockchip/Kconfig  | 7 +++++++
>>>>>>   include/configs/rk3368_common.h | 2 --
>>>>>>   2 files changed, 7 insertions(+), 2 deletions(-)
>>>>>>
>>>>> Applied to u-boot-rockchip, thanks!
>>>>>
>>>>>
>>>>>
>>>
>>>
>>
>
>
>