Re: [U-Boot] [PATCH 1/4] ARM: tegra: don't use CONFIG_SPL_TEXT_BASE when no SPL
-Original Message- From: Stephen Warren [mailto:swar...@wwwdotorg.org] Sent: Wednesday, January 3, 2018 3:44 PM To: Tom Warren Cc: Simon Glass ; Tom Rini ; U-Boot Mailing List ; Stephen Warren Subject: Re: [PATCH 1/4] ARM: tegra: don't use CONFIG_SPL_TEXT_BASE when no SPL On 01/03/2018 03:43 PM, Tom Warren wrote: > Stephen, > > -Original Message- > From: s...@google.com [mailto:s...@google.com] On Behalf Of Simon Glass > Sent: Thursday, December 28, 2017 8:14 PM > To: Stephen Warren > Cc: Tom Warren ; Tom Rini ; > U-Boot Mailing List ; Stephen Warren > > Subject: Re: [PATCH 1/4] ARM: tegra: don't use CONFIG_SPL_TEXT_BASE > when no SPL > > On 19 December 2017 at 18:30, Stephen Warren wrote: >> From: Stephen Warren >> >> 64-bit Tegra don't use SPL, and soon won't define >> CONFIG_SPL_TEXT_BASE when building. Fix the binman .dts file so that >> it doesn't use undefined values. >> >> Signed-off-by: Stephen Warren >> --- >> arch/arm/dts/tegra-u-boot.dtsi | 15 +-- >> 1 file changed, 9 insertions(+), 6 deletions(-) > > Reviewed-by: Simon Glass > > I've applied this series to u-boot-tegra/master. I'll wait a bit to see how > your new series (RAM bank/mem parsing) is received and then I'll apply it, > too, and post a PR. > I assume you expect the Linux kernel image header patches to go in thru TomR? I think it's best to push them all through the Tegra path, since there will be a stream of common and Tegra patches all intermixed. OK, I'll apply them as they get reviewed/ACK'd. Thanks. --- This email message is for the sole use of the intended recipient(s) and may contain confidential information. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply email and destroy all copies of the original message. --- ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH 1/4] ARM: tegra: don't use CONFIG_SPL_TEXT_BASE when no SPL
On 01/03/2018 03:43 PM, Tom Warren wrote: Stephen, -Original Message- From: s...@google.com [mailto:s...@google.com] On Behalf Of Simon Glass Sent: Thursday, December 28, 2017 8:14 PM To: Stephen Warren Cc: Tom Warren ; Tom Rini ; U-Boot Mailing List ; Stephen Warren Subject: Re: [PATCH 1/4] ARM: tegra: don't use CONFIG_SPL_TEXT_BASE when no SPL On 19 December 2017 at 18:30, Stephen Warren wrote: From: Stephen Warren 64-bit Tegra don't use SPL, and soon won't define CONFIG_SPL_TEXT_BASE when building. Fix the binman .dts file so that it doesn't use undefined values. Signed-off-by: Stephen Warren --- arch/arm/dts/tegra-u-boot.dtsi | 15 +-- 1 file changed, 9 insertions(+), 6 deletions(-) Reviewed-by: Simon Glass I've applied this series to u-boot-tegra/master. I'll wait a bit to see how your new series (RAM bank/mem parsing) is received and then I'll apply it, too, and post a PR. I assume you expect the Linux kernel image header patches to go in thru TomR? I think it's best to push them all through the Tegra path, since there will be a stream of common and Tegra patches all intermixed. ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH 1/4] ARM: tegra: don't use CONFIG_SPL_TEXT_BASE when no SPL
Stephen, -Original Message- From: s...@google.com [mailto:s...@google.com] On Behalf Of Simon Glass Sent: Thursday, December 28, 2017 8:14 PM To: Stephen Warren Cc: Tom Warren ; Tom Rini ; U-Boot Mailing List ; Stephen Warren Subject: Re: [PATCH 1/4] ARM: tegra: don't use CONFIG_SPL_TEXT_BASE when no SPL On 19 December 2017 at 18:30, Stephen Warren wrote: > From: Stephen Warren > > 64-bit Tegra don't use SPL, and soon won't define CONFIG_SPL_TEXT_BASE > when building. Fix the binman .dts file so that it doesn't use > undefined values. > > Signed-off-by: Stephen Warren > --- > arch/arm/dts/tegra-u-boot.dtsi | 15 +-- > 1 file changed, 9 insertions(+), 6 deletions(-) Reviewed-by: Simon Glass I've applied this series to u-boot-tegra/master. I'll wait a bit to see how your new series (RAM bank/mem parsing) is received and then I'll apply it, too, and post a PR. I assume you expect the Linux kernel image header patches to go in thru TomR? Tom -- nvpublic ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH 1/4] ARM: tegra: don't use CONFIG_SPL_TEXT_BASE when no SPL
On 19 December 2017 at 18:30, Stephen Warren wrote: > From: Stephen Warren > > 64-bit Tegra don't use SPL, and soon won't define CONFIG_SPL_TEXT_BASE > when building. Fix the binman .dts file so that it doesn't use undefined > values. > > Signed-off-by: Stephen Warren > --- > arch/arm/dts/tegra-u-boot.dtsi | 15 +-- > 1 file changed, 9 insertions(+), 6 deletions(-) Reviewed-by: Simon Glass ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH 1/4] ARM: tegra: don't use CONFIG_SPL_TEXT_BASE when no SPL
Stephen, > -Original Message- > From: Stephen Warren [mailto:swar...@wwwdotorg.org] > Sent: Tuesday, December 19, 2017 6:31 PM > To: Tom Warren > Cc: Simon Glass ; Tom Rini ; u- > b...@lists.denx.de; Stephen Warren > Subject: [PATCH 1/4] ARM: tegra: don't use CONFIG_SPL_TEXT_BASE when no > SPL > > From: Stephen Warren > > 64-bit Tegra don't use SPL, and soon won't define CONFIG_SPL_TEXT_BASE > when building. Fix the binman .dts file so that it doesn't use undefined > values. > > Signed-off-by: Stephen Warren > --- > arch/arm/dts/tegra-u-boot.dtsi | 15 +-- > 1 file changed, 9 insertions(+), 6 deletions(-) > This series: Reviewed-by: Tom Warren I'll wait a bit for other reviews & apply it to u-boot-tegra today or tomorrow. Tom > diff --git a/arch/arm/dts/tegra-u-boot.dtsi b/arch/arm/dts/tegra-u-boot.dtsi > index cde591c5fca4..4f692ee97572 100644 > --- a/arch/arm/dts/tegra-u-boot.dtsi > +++ b/arch/arm/dts/tegra-u-boot.dtsi > @@ -1,5 +1,11 @@ > #include > > +#ifdef CONFIG_SPL_TEXT_BASE > +#define U_BOOT_OFFSET (CONFIG_SYS_TEXT_BASE - > CONFIG_SPL_TEXT_BASE) > +#else #define U_BOOT_OFFSET 0 #endif > + > / { > binman { > multiple-images; > @@ -9,8 +15,7 @@ > u-boot-spl { > }; > u-boot { > - pos = <(CONFIG_SYS_TEXT_BASE - > - CONFIG_SPL_TEXT_BASE)>; > + pos = <(U_BOOT_OFFSET)>; > }; > }; > > @@ -21,8 +26,7 @@ > u-boot-spl { > }; > u-boot { > - pos = <(CONFIG_SYS_TEXT_BASE - > - CONFIG_SPL_TEXT_BASE)>; > + pos = <(U_BOOT_OFFSET)>; > }; > }; > > @@ -32,8 +36,7 @@ > u-boot-spl { > }; > u-boot-nodtb { > - pos = <(CONFIG_SYS_TEXT_BASE - > - CONFIG_SPL_TEXT_BASE)>; > + pos = <(U_BOOT_OFFSET)>; > }; > }; > }; > -- > 2.15.1 -- nvpublic ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
[U-Boot] [PATCH 1/4] ARM: tegra: don't use CONFIG_SPL_TEXT_BASE when no SPL
From: Stephen Warren 64-bit Tegra don't use SPL, and soon won't define CONFIG_SPL_TEXT_BASE when building. Fix the binman .dts file so that it doesn't use undefined values. Signed-off-by: Stephen Warren --- arch/arm/dts/tegra-u-boot.dtsi | 15 +-- 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/arch/arm/dts/tegra-u-boot.dtsi b/arch/arm/dts/tegra-u-boot.dtsi index cde591c5fca4..4f692ee97572 100644 --- a/arch/arm/dts/tegra-u-boot.dtsi +++ b/arch/arm/dts/tegra-u-boot.dtsi @@ -1,5 +1,11 @@ #include +#ifdef CONFIG_SPL_TEXT_BASE +#define U_BOOT_OFFSET (CONFIG_SYS_TEXT_BASE - CONFIG_SPL_TEXT_BASE) +#else +#define U_BOOT_OFFSET 0 +#endif + / { binman { multiple-images; @@ -9,8 +15,7 @@ u-boot-spl { }; u-boot { - pos = <(CONFIG_SYS_TEXT_BASE - - CONFIG_SPL_TEXT_BASE)>; + pos = <(U_BOOT_OFFSET)>; }; }; @@ -21,8 +26,7 @@ u-boot-spl { }; u-boot { - pos = <(CONFIG_SYS_TEXT_BASE - - CONFIG_SPL_TEXT_BASE)>; + pos = <(U_BOOT_OFFSET)>; }; }; @@ -32,8 +36,7 @@ u-boot-spl { }; u-boot-nodtb { - pos = <(CONFIG_SYS_TEXT_BASE - - CONFIG_SPL_TEXT_BASE)>; + pos = <(U_BOOT_OFFSET)>; }; }; }; -- 2.15.1 ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot