Re: [U-Boot] [PATCH 1/4] ARM: tegra: don't use CONFIG_SPL_TEXT_BASE when no SPL

[Tom Warren]

-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

[Stephen Warren]

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

[Tom Warren]

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

[Simon Glass]

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

[Tom Warren]

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