diff options
author | Tom Rini <trini@ti.com> | 2012-10-31 13:30:41 (GMT) |
---|---|---|
committer | Tom Rini <trini@ti.com> | 2012-11-04 18:00:38 (GMT) |
commit | 57a87a25f7563158d2b8a83d782568f1e6fc7fc5 (patch) | |
tree | cd2bf53e18bc9aae170d9cda7adaa3f1dd53ec5a | |
parent | ac63f2a2ecb24e309c2ca3b7b4c91b1941d3038b (diff) | |
download | u-boot-fsl-qoriq-57a87a25f7563158d2b8a83d782568f1e6fc7fc5.tar.xz |
usb gadget ether: Fix warning in is_eth_addr_valid()
The gadget ethernet driver needs to keep copies of the MAC address (at
both endpoints) as strings so it needs a custom function for validation
of the MAC. It was not however performing a totally correct check and
also was emitting a warning about a set but unused variable. The
solution to both is that after checking the string contents we use the
standard test for a valid MAC.
Cc: Joe Hershberger <joe.hershberger@ni.com>
Cc: Marek Vasut <marex@denx.de>
Signed-off-by: Tom Rini <trini@ti.com>
Acked-by: Joe Hershberger <joe.hershberger@ni.com>
-rw-r--r-- | drivers/usb/gadget/ether.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/usb/gadget/ether.c b/drivers/usb/gadget/ether.c index 1e187e5..8b24e00 100644 --- a/drivers/usb/gadget/ether.c +++ b/drivers/usb/gadget/ether.c @@ -1978,8 +1978,8 @@ static int is_eth_addr_valid(char *str) p = q; } - if (i == 6) /* it looks ok */ - return 1; + /* Now check the contents. */ + return is_valid_ether_addr(ea); } return 0; } |