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

Ben Warren biggerbadderben at gmail.com
Wed Jan 16 16:00:21 CET 2008


Wolfgang Denk wrote:
> 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.
>
>
>   
Silly tabs.  Sorry for the noise.  I'll try to be less reflexive in the 
future.

regards,
Ben





More information about the U-Boot mailing list