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

Reply via email to