From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Yan Date: Mon, 7 Aug 2017 16:45:55 +0800 Subject: [U-Boot] [U-Boot, v4, 60/66] rockchip: Kconfig: preset TPL_LDSCRIPT via Kconfig for the RK3368 In-Reply-To: <93EE8EEE-6F7B-46A4-BC81-59DCF23B6BB2@theobroma-systems.com> References: <1501706105-7490-61-git-send-email-philipp.tomsich@theobroma-systems.com> <4270BB75-844F-422A-990C-F56409F1CBA4@theobroma-systems.com> <3a834ddb-20b3-d513-e806-0798efba11fe@rock-chips.com> <93EE8EEE-6F7B-46A4-BC81-59DCF23B6BB2@theobroma-systems.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit To: u-boot@lists.denx.de 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 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 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 >>>>>> Reviewed-by: Simon Glass >>>>>> --- >>>>>> >>>>>> 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! >>>>> >>>>> >>>>> >>> >>> >> > > >