[U-Boot] [uBoot] TI: configs: Commonize the boot of different devices

[Tom Rini]

On Wed, Dec 04, 2013 at 03:14:08PM +0100, Enric Balletbo Serra wrote:
> Hi Tom,
> 
> 2013/12/4 Tom Rini <trini at ti.com>:
> > On Wed, Nov 20, 2013 at 01:40:18PM -0600, Dan Murphy wrote:
> >
> >> Commonize in the ti_armv7_common.h the boot scripts for
> >> USB, MMC and NAND.
> >>
> >> Each board file can then select which BOOT_TARGETS are applicable
> >> for the target board.
> >> And any parameters based on that.
> >>
> >> Finally removed the findfdt from the common file and made this more board
> >> specific as omap4_common should not reference panda.
> >>
> >> This implemenation was adopted from the tegra-common-post.h file.
> >
> > OK, implementation wise, I think we might need to adapt the -post
> > concept here too.  That will let us remove from the config files
> > themselves a number of:
> > #ifdef FOO
> > #undef FOO
> > #endif
> > #define FOO
> >
> > And it will also let new boards like cm_t335 opt in to
> > ti_am335x_common.h but not have to pick our bootcmd if they don't want
> > to (tegra-common-post.h allows for that).  While not a big deal,
> > possibly, for cm_t335 it's probably a bigger deal for getting the
> > siemens boards adapted too.
> >
> > I think this also shows that we need to handle the DFU env settings
> > parts strictly inside of am335x_evm.h.  I'm toying with a patch to cover
> > this, now.
> >
> > Thanks!
> 
> Just a small question that is related with this. There is any plan to
> commonize the configuration code for OMAP3 platforms
> (ti_omap3_common.h)  like AM335x ?

I would love to see it happen, yes.  It's not high up on my priority
list right now (along with doing omap3_evm* stuff cleaner) so I'd also
love to see the patch come from someone else.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20131204/5ddadb82/attachment.pgp>