Commit 10332dc2 authored by David S. Miller's avatar David S. Miller

Merge branch 'net-ch9200-code-cleanup'

Chen Zhou says:

====================
net: ch9200: code cleanup

patch 1 introduce __func__ in debug message.
patch 2 remove unnecessary return.
====================
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parents 58cf542a 195234b8
...@@ -111,8 +111,8 @@ static int control_read(struct usbnet *dev, ...@@ -111,8 +111,8 @@ static int control_read(struct usbnet *dev,
request_type = (USB_DIR_IN | USB_TYPE_VENDOR | request_type = (USB_DIR_IN | USB_TYPE_VENDOR |
USB_RECIP_DEVICE); USB_RECIP_DEVICE);
netdev_dbg(dev->net, "Control_read() index=0x%02x size=%d\n", netdev_dbg(dev->net, "%s() index=0x%02x size=%d\n",
index, size); __func__, index, size);
buf = kmalloc(size, GFP_KERNEL); buf = kmalloc(size, GFP_KERNEL);
if (!buf) { if (!buf) {
...@@ -130,8 +130,6 @@ static int control_read(struct usbnet *dev, ...@@ -130,8 +130,6 @@ static int control_read(struct usbnet *dev,
err = -EINVAL; err = -EINVAL;
kfree(buf); kfree(buf);
return err;
err_out: err_out:
return err; return err;
} }
...@@ -151,8 +149,8 @@ static int control_write(struct usbnet *dev, unsigned char request, ...@@ -151,8 +149,8 @@ static int control_write(struct usbnet *dev, unsigned char request,
request_type = (USB_DIR_OUT | USB_TYPE_VENDOR | request_type = (USB_DIR_OUT | USB_TYPE_VENDOR |
USB_RECIP_DEVICE); USB_RECIP_DEVICE);
netdev_dbg(dev->net, "Control_write() index=0x%02x size=%d\n", netdev_dbg(dev->net, "%s() index=0x%02x size=%d\n",
index, size); __func__, index, size);
if (data) { if (data) {
buf = kmemdup(data, size, GFP_KERNEL); buf = kmemdup(data, size, GFP_KERNEL);
...@@ -181,8 +179,8 @@ static int ch9200_mdio_read(struct net_device *netdev, int phy_id, int loc) ...@@ -181,8 +179,8 @@ static int ch9200_mdio_read(struct net_device *netdev, int phy_id, int loc)
struct usbnet *dev = netdev_priv(netdev); struct usbnet *dev = netdev_priv(netdev);
unsigned char buff[2]; unsigned char buff[2];
netdev_dbg(netdev, "ch9200_mdio_read phy_id:%02x loc:%02x\n", netdev_dbg(netdev, "%s phy_id:%02x loc:%02x\n",
phy_id, loc); __func__, phy_id, loc);
if (phy_id != 0) if (phy_id != 0)
return -ENODEV; return -ENODEV;
...@@ -199,8 +197,8 @@ static void ch9200_mdio_write(struct net_device *netdev, ...@@ -199,8 +197,8 @@ static void ch9200_mdio_write(struct net_device *netdev,
struct usbnet *dev = netdev_priv(netdev); struct usbnet *dev = netdev_priv(netdev);
unsigned char buff[2]; unsigned char buff[2];
netdev_dbg(netdev, "ch9200_mdio_write() phy_id=%02x loc:%02x\n", netdev_dbg(netdev, "%s() phy_id=%02x loc:%02x\n",
phy_id, loc); __func__, phy_id, loc);
if (phy_id != 0) if (phy_id != 0)
return; return;
...@@ -219,8 +217,8 @@ static int ch9200_link_reset(struct usbnet *dev) ...@@ -219,8 +217,8 @@ static int ch9200_link_reset(struct usbnet *dev)
mii_check_media(&dev->mii, 1, 1); mii_check_media(&dev->mii, 1, 1);
mii_ethtool_gset(&dev->mii, &ecmd); mii_ethtool_gset(&dev->mii, &ecmd);
netdev_dbg(dev->net, "link_reset() speed:%d duplex:%d\n", netdev_dbg(dev->net, "%s() speed:%d duplex:%d\n",
ecmd.speed, ecmd.duplex); __func__, ecmd.speed, ecmd.duplex);
return 0; return 0;
} }
...@@ -309,7 +307,7 @@ static int get_mac_address(struct usbnet *dev, unsigned char *data) ...@@ -309,7 +307,7 @@ static int get_mac_address(struct usbnet *dev, unsigned char *data)
unsigned char mac_addr[0x06]; unsigned char mac_addr[0x06];
int rd_mac_len = 0; int rd_mac_len = 0;
netdev_dbg(dev->net, "get_mac_address:\n\tusbnet VID:%0x PID:%0x\n", netdev_dbg(dev->net, "%s:\n\tusbnet VID:%0x PID:%0x\n", __func__,
le16_to_cpu(dev->udev->descriptor.idVendor), le16_to_cpu(dev->udev->descriptor.idVendor),
le16_to_cpu(dev->udev->descriptor.idProduct)); le16_to_cpu(dev->udev->descriptor.idProduct));
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment