[U-Boot] [PATCH 1/2] net: ks8851_mll: add ethernet support
Bo Shen
voice.shen at atmel.com
Wed Apr 24 03:20:31 CEST 2013
Hi Tom,
On 4/23/2013 23:34, Tom Rini wrote:
> On Tue, Apr 23, 2013 at 06:17:27PM +0800, Bo Shen wrote:
>
>> From: Roberto Cerati <roberto.cerati at bticino.it>
>>
>> The device interface is 16 bits wide.
>> All the available packets are read from the incoming fifo.
> [snip]
>> +/* union ks_tx_hdr - tx header data
>> + * @txb: The header as bytes
>
> /*
> * union ks_tx_hdr...
If I use this style comments, when use checkpatch.pl to check, it will
report:
--->8---
WARNING: networking block comments don't use an empty /* line, use /*
Comment...
#81: FILE: drivers/net/ks8851_mll.c:41:
+
+/*
---<8---
> Please fix globally.
So, do you think I should ignore the warning and fix this?
> [snip]
>> +#define RXFSHR_ERR (RXFSHR_RXCE | RXFSHR_RXRF |\
>
> #define<space> please, thanks.
I will fix it in next version.
>
> Looks fine aside from these style problems however, thanks!
Best Regards,
Bo Shen
More information about the U-Boot
mailing list