Re: [U-Boot] [PATCH V2 2/6] Tegra: Rework KConfig options to allow 64-bit builds (T210)

2015-07-22 Thread Tom Warren
Stephen,

> -Original Message-
> From: Stephen Warren [mailto:swar...@wwwdotorg.org]
> Sent: Wednesday, July 22, 2015 10:40 AM
> To: Tom Warren
> Cc: u-boot@lists.denx.de; Thierry Reding; Stephen Warren;
> tomcwarren3...@gmail.com
> Subject: Re: [U-Boot] [PATCH V2 2/6] Tegra: Rework KConfig options to allow
> 64-bit builds (T210)
> 
> On 07/20/2015 01:50 PM, Tom Warren wrote:
> > Moved Tegra config options to mach-tegra/Kconfig so that both 32-bit
> > and 64-bit builds can co-exist for Tegra SoCs.
> >
> > T210 will be 64-bit only (no SPL) and will requires a 32-bit AVP/BPMP
> > loader.
> 
> > diff --git a/arch/arm/mach-tegra/Kconfig b/arch/arm/mach-tegra/Kconfig
> 
> >   config TEGRA20
> > bool "Tegra20 family"
> > +   select SUPPORT_SPL
> > +   select SPL
> > +   select OF_CONTROL
> > +   select SPL_DISABLE_OF_CONTROL
> > +   select CPU_V7
> > +   select DM
> > +   select DM_SPI_FLASH
> > +   select DM_SERIAL
> > +   select DM_I2C
> > +   select DM_SPI
> > +   select DM_GPIO
> 
> This patch duplicates all those added lines for all current chips. Could we 
> add a
> new config variable to share these; something like:
> 
> config TEGRA_ARMV7_COMMON
>   select SUPPORT_SPL
>   select SPL
>   select OF_CONTROL
>   select SPL_DISABLE_OF_CONTROL
>   select CPU_V7
>   select DM
>   select DM_SPI_FLASH
>   select DM_SERIAL
>   select DM_I2C
>   select DM_SPI
>   select DM_GPIO
> 
> config TEGRA20
>   bool "Tegra20 family"
>   select TEGRA_ARMv7_COMMON
Great idea - I'll do that for V3. Thanks.

--
nvpublic
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH V2 2/6] Tegra: Rework KConfig options to allow 64-bit builds (T210)

2015-07-22 Thread Stephen Warren

On 07/20/2015 01:50 PM, Tom Warren wrote:

Moved Tegra config options to mach-tegra/Kconfig so that both
32-bit and 64-bit builds can co-exist for Tegra SoCs.

T210 will be 64-bit only (no SPL) and will requires a 32-bit
AVP/BPMP loader.



diff --git a/arch/arm/mach-tegra/Kconfig b/arch/arm/mach-tegra/Kconfig



  config TEGRA20
bool "Tegra20 family"
+   select SUPPORT_SPL
+   select SPL
+   select OF_CONTROL
+   select SPL_DISABLE_OF_CONTROL
+   select CPU_V7
+   select DM
+   select DM_SPI_FLASH
+   select DM_SERIAL
+   select DM_I2C
+   select DM_SPI
+   select DM_GPIO


This patch duplicates all those added lines for all current chips. Could 
we add a new config variable to share these; something like:


config TEGRA_ARMV7_COMMON
select SUPPORT_SPL
select SPL
select OF_CONTROL
select SPL_DISABLE_OF_CONTROL
select CPU_V7
select DM
select DM_SPI_FLASH
select DM_SERIAL
select DM_I2C
select DM_SPI
select DM_GPIO

config TEGRA20
bool "Tegra20 family"
select TEGRA_ARMv7_COMMON
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH V2 2/6] Tegra: Rework KConfig options to allow 64-bit builds (T210)

2015-07-20 Thread Tom Warren
Moved Tegra config options to mach-tegra/Kconfig so that both
32-bit and 64-bit builds can co-exist for Tegra SoCs.

T210 will be 64-bit only (no SPL) and will requires a 32-bit
AVP/BPMP loader.

Signed-off-by: Tom Warren 
---
 arch/arm/Kconfig| 11 ---
 arch/arm/mach-tegra/Kconfig | 44 
 2 files changed, 44 insertions(+), 11 deletions(-)

diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index 506463c..20ab398 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -681,17 +681,6 @@ config TARGET_XILINX_ZYNQMP
 
 config TEGRA
bool "NVIDIA Tegra"
-   select SUPPORT_SPL
-   select SPL
-   select OF_CONTROL
-   select SPL_DISABLE_OF_CONTROL
-   select CPU_V7
-   select DM
-   select DM_SPI_FLASH
-   select DM_SERIAL
-   select DM_I2C
-   select DM_SPI
-   select DM_GPIO
 
 config TARGET_VEXPRESS64_AEMV8A
bool "Support vexpress_aemv8a"
diff --git a/arch/arm/mach-tegra/Kconfig b/arch/arm/mach-tegra/Kconfig
index 54bd648..7494f8d 100644
--- a/arch/arm/mach-tegra/Kconfig
+++ b/arch/arm/mach-tegra/Kconfig
@@ -6,15 +6,59 @@ choice
 
 config TEGRA20
bool "Tegra20 family"
+   select SUPPORT_SPL
+   select SPL
+   select OF_CONTROL
+   select SPL_DISABLE_OF_CONTROL
+   select CPU_V7
+   select DM
+   select DM_SPI_FLASH
+   select DM_SERIAL
+   select DM_I2C
+   select DM_SPI
+   select DM_GPIO
 
 config TEGRA30
bool "Tegra30 family"
+   select SUPPORT_SPL
+   select SPL
+   select OF_CONTROL
+   select SPL_DISABLE_OF_CONTROL
+   select CPU_V7
+   select DM
+   select DM_SPI_FLASH
+   select DM_SERIAL
+   select DM_I2C
+   select DM_SPI
+   select DM_GPIO
 
 config TEGRA114
bool "Tegra114 family"
+   select SUPPORT_SPL
+   select SPL
+   select OF_CONTROL
+   select SPL_DISABLE_OF_CONTROL
+   select CPU_V7
+   select DM
+   select DM_SPI_FLASH
+   select DM_SERIAL
+   select DM_I2C
+   select DM_SPI
+   select DM_GPIO
 
 config TEGRA124
bool "Tegra124 family"
+   select SUPPORT_SPL
+   select SPL
+   select OF_CONTROL
+   select SPL_DISABLE_OF_CONTROL
+   select CPU_V7
+   select DM
+   select DM_SPI_FLASH
+   select DM_SERIAL
+   select DM_I2C
+   select DM_SPI
+   select DM_GPIO
 
 endchoice
 
-- 
1.8.2.1.610.g562af5b

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot