[U-Boot] [PATCH 1/5] Add support for SMSC95XX USB 2.0 10/100MBit Ethernet Adapter
Wolfgang Denk
wd at denx.de
Tue Apr 12 23:50:54 CEST 2011
Dear Simon Glass,
In message <1302642840-6958-1-git-send-email-sjg at chromium.org> you wrote:
> The SMSC95XX is a USB hub with a built-in Ethernet adapter. This adds support
> for this, using the USB host network framework.
>
> TEST=usb start; bootp; tftp ...
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
> drivers/usb/eth/Makefile | 1 +
> drivers/usb/eth/smsc95xx.c | 922 +++++++++++++++++++++++++++++++++++++++++++
> drivers/usb/eth/usb_ether.c | 7 +
> 3 files changed, 930 insertions(+), 0 deletions(-)
> create mode 100644 drivers/usb/eth/smsc95xx.c
$ checkpatch ~/Mail/U-Boot/6365
WARNING: space prohibited between function name and open parenthesis '('
#497: FILE: drivers/usb/eth/smsc95xx.c:383:
+ debug ("MAC %02x:%02x:%02x:%02x:%02x:%02x\n",
ERROR: space prohibited before open square bracket '['
#498: FILE: drivers/usb/eth/smsc95xx.c:384:
+ eth->enetaddr [0], eth->enetaddr [1],
ERROR: space prohibited before open square bracket '['
#498: FILE: drivers/usb/eth/smsc95xx.c:384:
+ eth->enetaddr [0], eth->enetaddr [1],
ERROR: space prohibited before open square bracket '['
#499: FILE: drivers/usb/eth/smsc95xx.c:385:
+ eth->enetaddr [2], eth->enetaddr [3],
ERROR: space prohibited before open square bracket '['
#499: FILE: drivers/usb/eth/smsc95xx.c:385:
+ eth->enetaddr [2], eth->enetaddr [3],
ERROR: space prohibited before open square bracket '['
#500: FILE: drivers/usb/eth/smsc95xx.c:386:
+ eth->enetaddr [4], eth->enetaddr [5]);
ERROR: space prohibited before open square bracket '['
#500: FILE: drivers/usb/eth/smsc95xx.c:386:
+ eth->enetaddr [4], eth->enetaddr [5]);
WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt
#822: FILE: drivers/usb/eth/smsc95xx.c:708:
+static int smsc95xx_send(struct eth_device *eth, volatile void* packet,
ERROR: code indent should use tabs where possible
#1046: FILE: drivers/usb/eth/usb_ether.c:49:
+ {$
WARNING: please, no spaces at the start of a line
#1046: FILE: drivers/usb/eth/usb_ether.c:49:
+ {$
ERROR: code indent should use tabs where possible
#1047: FILE: drivers/usb/eth/usb_ether.c:50:
+ .before_probe = smsc95xx_eth_before_probe,$
WARNING: please, no spaces at the start of a line
#1047: FILE: drivers/usb/eth/usb_ether.c:50:
+ .before_probe = smsc95xx_eth_before_probe,$
ERROR: code indent should use tabs where possible
#1048: FILE: drivers/usb/eth/usb_ether.c:51:
+ .probe = smsc95xx_eth_probe,$
WARNING: please, no spaces at the start of a line
#1048: FILE: drivers/usb/eth/usb_ether.c:51:
+ .probe = smsc95xx_eth_probe,$
ERROR: code indent should use tabs where possible
#1049: FILE: drivers/usb/eth/usb_ether.c:52:
+ .get_info = smsc95xx_eth_get_info,$
WARNING: please, no spaces at the start of a line
#1049: FILE: drivers/usb/eth/usb_ether.c:52:
+ .get_info = smsc95xx_eth_get_info,$
ERROR: code indent should use tabs where possible
#1050: FILE: drivers/usb/eth/usb_ether.c:53:
+ },$
WARNING: please, no spaces at the start of a line
#1050: FILE: drivers/usb/eth/usb_ether.c:53:
+ },$
total: 11 errors, 7 warnings, 942 lines checked
NOTE: whitespace errors detected, you may wish to use scripts/cleanpatch or
scripts/cleanfile
Please make sure to read http://www.denx.de/wiki/U-Boot/CodingStyle
and http://www.denx.de/wiki/view/U-Boot/Patches. Then clean up your
patches, and resubmit. Thanks.
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: wd at denx.de
How many seconds are there in a year? If I tell you there are 3.155 x
10^7, you won't even try to remember it. On the other hand, who could
forget that, to within half a percent, pi seconds is a nanocentury.
-- Tom Duff, Bell Labs
More information about the U-Boot
mailing list