[U-Boot] [PATCH] ARM AT91 new board EB+CPUx9K2
Wolfgang Denk
wd at denx.de
Wed Oct 28 15:35:24 CET 2009
Dear Jens Scharsig,
In message <4AE81893.30506 at bus-elektronik.de> you wrote:
>
> > Close, but please also get rid of this "((AT91PS_PIO) AT91C_BASE_PIOA)"
> > thing, especialy the cast is nasty. Declare a real pointer variable to
> > the respective data structure, and use that.
> >
>
> I, think
>
> writel(AT91C_PA23_TXD2, AT91C_PIOA_OER);
>
> is the most correct way.
"most correct way" are big words. No, this is not "correct" at all.
The whole set of address / offset definitions in
include/asm-arm/arch-at91rm9200/AT91RM9200.h should be turned into a C
struct.
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
When all else fails, read the instructions.
More information about the U-Boot
mailing list