[U-Boot] [PATCH] OMAP3EVM: net_chip uses CS5 not CS6
Wolfgang Denk
wd at denx.de
Thu May 7 22:42:25 CEST 2009
Dear Scott,
In message <4A0333FC.6090900 at freescale.com> you wrote:
> Wolfgang Denk wrote:
> > Finally, and this is what I really compalin about, is that there is no
> > big structure which includes all the blocks that make up the CPU into
> > one big structure (as it's done for example for PowerPC systems in the
> > include/asm-ppc/*immap* files) - you still use code like
>
> Those immap structs are a huge pain to maintain (or to verify the
> correctness of), loaded with ifdeffery and magic numbers describing
> reserved spans. We should not be emulating them.
Well, #define'ing long lists of register offsets is not easier to
maintain or verify, and you don't have any typechecking by the
compiler.
> We used to have them in Linux, and got rid of them.
Hm... Seems I have missed this change... What's things like
struct qe_immap __iomem *qe_immr
or
cpm2_map_t __iomem *cpm2_immr
or
immap_t __iomem *mpc8xx_immr
then? Or what replaced the "immr" structs?
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
The manager will be continually amazed that policies he took for com-
mon knowledge are totally unknown by some member of his team. Since
his fundamental job is to keep everybody going in the same direction,
his chief daily task will be communication, not decision-making.
- Fred Brooks, "The Mythical Man Month"
More information about the U-Boot
mailing list