Dear Rabin VINCENT,

In message <20100408134135.ga4...@bnru02.bnr.st.com> you wrote:
> On Sun, Mar 28, 2010 at 07:49:15PM +0200, Tom wrote:
> > Rabin Vincent wrote:
> > > + /* UART2 */
> > > + nmk_gpio_af(29, GPIO_ALT_C);
> > > + nmk_gpio_af(30, GPIO_ALT_C);
> > 
> > Change these to immediates to logical #define's
> 
> I've changed this in the other places you mentioned, but for these
> GPIOs, I'd prefer to use the pin numbers directly.

Maybe you do, but we don't. Please don't use magic numbers.

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
"Tell the truth and run."                          - Yugoslav proverb
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to