[U-Boot-Users] resend#2 [PATCH 6/8] ColdFire: Add MCF547x_8x FEC driver

Wolfgang Denk wd at denx.de
Wed Jan 16 13:46:28 CET 2008


Dear Ben,
, the alignment is perfect. If it looked bad to you, i. e. if this is
what you saw:


in message <478D6D2A.5090608 at gmail.com> you wrote:
>
> > +/* Ethernet Transmit and Receive Buffers */
> > +#define DBUF_LENGTH		1520
> > +#define PKT_MAXBUF_SIZE		1518
> > +#define PKT_MINBUF_SIZE		64
> > +#define PKT_MAXBLR_SIZE		1536
> >   
> Please align better

Please don't get fooled. As you (probably) can see above, the
alignment is perfect. If it looked bad to you, i. e. if this is what
you saw:

> +/* Ethernet Transmit and Receive Buffers */
> +#define DBUF_LENGTH		1520
> +#define PKT_MAXBUF_SIZE		1518
> +#define PKT_MINBUF_SIZE		64
> +#define PKT_MAXBLR_SIZE		1536

...then this was caused by the fact that you did  not  see  the  code
alone,  but  with additional characters added at the beginning of the
line ("> +") - depending on where you are on a line, a TAB may or may
not get shifted then.

You have to *apply* the patch to really comment on formatting.

Best regards,

Wolfgang Denk

-- 
Software Engineering:  Embedded and Realtime Systems,  Embedded Linux
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
CAUTION:  The Mass of This Product Contains the Energy Equivalent  of
85 Million Tons of TNT per Net Ounce of Weight.




More information about the U-Boot mailing list