Tom, On Thu, Sep 13, 2012 at 2:04 PM, Tom Rini <tr...@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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot