[U-Boot] [PATCH 0/9] Add basic Tegra30 (T30) support

Tom Warren
2012-09-13T21:25:53Z

Tom,

On Thu, Sep 13, 2012 at 2:04 PM, Tom Rini <trini@ti.com> wrote:
> On Wed, Sep 12, 2012 at 03:10:46PM -0700, Tom Warren wrote:
>
>> This patch series adds basic (boot to cmd prompt) support for Tegra30.
>> This is based on the Tegra20 SPL, which initializes the AVP (ARM7TDMI
>> boot proc) first, then control is transferred to the CPU (A9 quad cluster).
>> It is based on current u-boot-tegra/next.
>>
>> Future patches will add support/drivers for MMC, USB, I2C, SPI, NAND,
>> and other peripherals. The Cardhu T30 board is supported initially.
>
> Let me put my TI Custodian hat on and say, I've been where you are and I
> have some advice. Make a local branch (or 5) of your working but needs
> re-factoring and re-cleaning tree, and start fresh from
> u-boot-tegra/next. Move more tegra20 files to the tegra common area.
> Strip (#if 0) the "new" include files bare. Bring it up again. If you
> bring in the framework so we can have <plat/foo.h> (which is in turn
> arch/$(ARCH)/include/plat-vendor) I will be so happy you beat me to the
> punch and with something a bit more (IMHO) clean than how the kernel
> handles <plat/>. You're masking a way a lot of things that are either
> identical (which is duplication which is bad) or subtly different (and
> you haven't hit it, yet). However you migrate from the current tree to
> the new tree (I've done a lot of ugly steps when I've done this locally,
> but it keeps the board working from change to change), do it and do it
> sooner rather than later. You will thank yourself in the end.
>

Seems like good advice. I'm off tomorrow, so I'll take a look at your
approach on Monday and see how it works for me.

Thanks,

Tom
> --
> Tom