Dear Jens Scharsig,

In message <hc6pk5$tv...@ger.gmane.org> you wrote:
>
> > Please use I/O accessors to access device registers. Please fix
> > globally.
> Do you mean change to
> 
> writel(AT91C_PA23_TXD2, ((AT91PS_PIO) AT91C_BASE_PIOA)->PIO_OER);
> 
> like notation

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.

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: w...@denx.de
If the odds are a million to one against something occuring,  chances
are 50-50 it will.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to