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

2017-08-13 Thread Simon Glass
+Philipp On 9 August 2017 at 15:45, Masahiro Yamada wrote: > Hi. > > > 2017-08-07 23:05 GMT+09:00 Tom Rini : >> On Mon, Aug 07, 2017 at 10:48:12AM +0200, Dr. Philipp Tomsich wrote: >>> +Tom >>> >>> Ok, so the problem is

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

2017-08-09 Thread Masahiro Yamada
Hi. 2017-08-07 23:05 GMT+09:00 Tom Rini : > On Mon, Aug 07, 2017 at 10:48:12AM +0200, Dr. Philipp Tomsich wrote: >> +Tom >> >> Ok, so the problem is 'scripts/setlocalversion’, which does the following: >> > if test -e include/config/auto.conf; then >> > .

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

2017-08-08 Thread Dr. Philipp Tomsich
Masahiro & Tom, the proposed fix is at https://patchwork.ozlabs.org/patch/799172/ Thanks, Phil. > On 07 Aug 2017, at 16:05, Tom Rini wrote: > > On Mon, Aug 07, 2017 at 10:48:12AM +0200, Dr. Philipp Tomsich wrote: >> +Tom >> >> Ok, so the problem is

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

2017-08-07 Thread Dr. Philipp Tomsich
Tom, > On 07 Aug 2017, at 16:05, Tom Rini wrote: > > On Mon, Aug 07, 2017 at 10:48:12AM +0200, Dr. Philipp Tomsich wrote: >> +Tom >> >> Ok, so the problem is 'scripts/setlocalversion’, which does the following: >>> if test -e include/config/auto.conf; then >>>.

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

2017-08-07 Thread Tom Rini
On Mon, Aug 07, 2017 at 10:48:12AM +0200, Dr. Philipp Tomsich wrote: > +Tom > > Ok, so the problem is 'scripts/setlocalversion’, which does the following: > > if test -e include/config/auto.conf; then > > . include/config/auto.conf > > else > > echo "Error: kernelrelease not valid

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

2017-08-07 Thread Dr. Philipp Tomsich
+Tom Ok, so the problem is 'scripts/setlocalversion’, which does the following: > if test -e include/config/auto.conf; then > . include/config/auto.conf > else > echo "Error: kernelrelease not valid - run 'make prepare' to update > it" > exit 1 > fi in order to access the

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

2017-08-07 Thread 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

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

2017-08-07 Thread Dr. Philipp Tomsich
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. > 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

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

2017-08-07 Thread Andy Yan
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

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

2017-08-07 Thread Dr. Philipp Tomsich
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

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

2017-08-06 Thread Andy Yan
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

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

2017-08-06 Thread Philipp Tomsich
> 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 >