[U-Boot] [PATCH 05/10] am33xx: add ti814x specific register definitions

Peter Korsgaard jacmet at sunsite.dk
Mon Feb 18 16:21:30 CET 2013


>>>>> "Tom" == Tom Rini <trini at ti.com> writes:

Hi,

 >> Quite some of the base addresses are similar, but I wonder if it
 >> wouldn't be cleaner to simply have a hardware-am33xx.h /
 >> hardware-ti814x.h instead of all these ifdef / elif?

 Tom> Since I suspect the things common from ti814x and am33xx are also common
 Tom> to ti816x (which has been left as an exercise to whomever needs that
 Tom> next), I think we can re-structure this into something like that, but
 Tom> keeping the common parts within hardware.h still.

FYI, I might very well be that guy as I've recently started work on a
ti816x based project.

 >> Argh! Couldn't we just swap the meaning of mmc1/mmc2 or would that be
 >> too confusing?

 Tom> IMHO, that will lead to further confusion down the line.  I talked with
 Tom> Matt about this before and well, it's funky.

Ok.

-- 
Bye, Peter Korsgaard


More information about the U-Boot mailing list