]> git.proxmox.com Git - mirror_ubuntu-bionic-kernel.git/blobdiff - drivers/net/ieee802154/atusb.c
Merge git://git.kernel.org/pub/scm/linux/kernel/git/davem/ide
[mirror_ubuntu-bionic-kernel.git] / drivers / net / ieee802154 / atusb.c
index d5584063049ff9237d0d88e997906249b058e2a3..9fb9b565a002ff7c2e9f6a06c4ef6b9c4d640266 100644 (file)
@@ -61,7 +61,7 @@ struct atusb {
        struct usb_ctrlrequest tx_dr;
        struct urb *tx_urb;
        struct sk_buff *tx_skb;
-       uint8_t tx_ack_seq;             /* current TX ACK sequence number */
+       u8 tx_ack_seq;          /* current TX ACK sequence number */
 
        /* Firmware variable */
        unsigned char fw_ver_maj;       /* Firmware major version number */
@@ -99,44 +99,43 @@ static int atusb_control_msg(struct atusb *atusb, unsigned int pipe,
        if (ret < 0) {
                atusb->err = ret;
                dev_err(&usb_dev->dev,
-                       "atusb_control_msg: req 0x%02x val 0x%x idx 0x%x, error %d\n",
-                       request, value, index, ret);
+                       "%s: req 0x%02x val 0x%x idx 0x%x, error %d\n",
+                       __func__, request, value, index, ret);
        }
        return ret;
 }
 
-static int atusb_command(struct atusb *atusb, uint8_t cmd, uint8_t arg)
+static int atusb_command(struct atusb *atusb, u8 cmd, u8 arg)
 {
        struct usb_device *usb_dev = atusb->usb_dev;
 
-       dev_dbg(&usb_dev->dev, "atusb_command: cmd = 0x%x\n", cmd);
+       dev_dbg(&usb_dev->dev, "%s: cmd = 0x%x\n", __func__, cmd);
        return atusb_control_msg(atusb, usb_sndctrlpipe(usb_dev, 0),
                                 cmd, ATUSB_REQ_TO_DEV, arg, 0, NULL, 0, 1000);
 }
 
-static int atusb_write_reg(struct atusb *atusb, uint8_t reg, uint8_t value)
+static int atusb_write_reg(struct atusb *atusb, u8 reg, u8 value)
 {
        struct usb_device *usb_dev = atusb->usb_dev;
 
-       dev_dbg(&usb_dev->dev, "atusb_write_reg: 0x%02x <- 0x%02x\n",
-               reg, value);
+       dev_dbg(&usb_dev->dev, "%s: 0x%02x <- 0x%02x\n", __func__, reg, value);
        return atusb_control_msg(atusb, usb_sndctrlpipe(usb_dev, 0),
                                 ATUSB_REG_WRITE, ATUSB_REQ_TO_DEV,
                                 value, reg, NULL, 0, 1000);
 }
 
-static int atusb_read_reg(struct atusb *atusb, uint8_t reg)
+static int atusb_read_reg(struct atusb *atusb, u8 reg)
 {
        struct usb_device *usb_dev = atusb->usb_dev;
        int ret;
-       uint8_t *buffer;
-       uint8_t value;
+       u8 *buffer;
+       u8 value;
 
        buffer = kmalloc(1, GFP_KERNEL);
        if (!buffer)
                return -ENOMEM;
 
-       dev_dbg(&usb_dev->dev, "atusb: reg = 0x%x\n", reg);
+       dev_dbg(&usb_dev->dev, "%s: reg = 0x%x\n", __func__, reg);
        ret = atusb_control_msg(atusb, usb_rcvctrlpipe(usb_dev, 0),
                                ATUSB_REG_READ, ATUSB_REQ_FROM_DEV,
                                0, reg, buffer, 1, 1000);
@@ -151,15 +150,14 @@ static int atusb_read_reg(struct atusb *atusb, uint8_t reg)
        }
 }
 
-static int atusb_write_subreg(struct atusb *atusb, uint8_t reg, uint8_t mask,
-                             uint8_t shift, uint8_t value)
+static int atusb_write_subreg(struct atusb *atusb, u8 reg, u8 mask,
+                             u8 shift, u8 value)
 {
        struct usb_device *usb_dev = atusb->usb_dev;
-       uint8_t orig, tmp;
+       u8 orig, tmp;
        int ret = 0;
 
-       dev_dbg(&usb_dev->dev, "atusb_write_subreg: 0x%02x <- 0x%02x\n",
-               reg, value);
+       dev_dbg(&usb_dev->dev, "%s: 0x%02x <- 0x%02x\n", __func__, reg, value);
 
        orig = atusb_read_reg(atusb, reg);
 
@@ -261,12 +259,12 @@ static void atusb_work_urbs(struct work_struct *work)
 
 /* ----- Asynchronous USB -------------------------------------------------- */
 
-static void atusb_tx_done(struct atusb *atusb, uint8_t seq)
+static void atusb_tx_done(struct atusb *atusb, u8 seq)
 {
        struct usb_device *usb_dev = atusb->usb_dev;
-       uint8_t expect = atusb->tx_ack_seq;
+       u8 expect = atusb->tx_ack_seq;
 
-       dev_dbg(&usb_dev->dev, "atusb_tx_done (0x%02x/0x%02x)\n", seq, expect);
+       dev_dbg(&usb_dev->dev, "%s (0x%02x/0x%02x)\n", __func__, seq, expect);
        if (seq == expect) {
                /* TODO check for ifs handling in firmware */
                ieee802154_xmit_complete(atusb->hw, atusb->tx_skb, false);
@@ -287,7 +285,7 @@ static void atusb_in_good(struct urb *urb)
        struct usb_device *usb_dev = urb->dev;
        struct sk_buff *skb = urb->context;
        struct atusb *atusb = SKB_ATUSB(skb);
-       uint8_t len, lqi;
+       u8 len, lqi;
 
        if (!urb->actual_length) {
                dev_dbg(&usb_dev->dev, "atusb_in: zero-sized URB ?\n");
@@ -326,7 +324,7 @@ static void atusb_in(struct urb *urb)
        struct sk_buff *skb = urb->context;
        struct atusb *atusb = SKB_ATUSB(skb);
 
-       dev_dbg(&usb_dev->dev, "atusb_in: status %d len %d\n",
+       dev_dbg(&usb_dev->dev, "%s: status %d len %d\n", __func__,
                urb->status, urb->actual_length);
        if (urb->status) {
                if (urb->status == -ENOENT) { /* being killed */
@@ -334,7 +332,7 @@ static void atusb_in(struct urb *urb)
                        urb->context = NULL;
                        return;
                }
-               dev_dbg(&usb_dev->dev, "atusb_in: URB error %d\n", urb->status);
+               dev_dbg(&usb_dev->dev, "%s: URB error %d\n", __func__, urb->status);
        } else {
                atusb_in_good(urb);
        }
@@ -388,7 +386,7 @@ static int atusb_xmit(struct ieee802154_hw *hw, struct sk_buff *skb)
        struct usb_device *usb_dev = atusb->usb_dev;
        int ret;
 
-       dev_dbg(&usb_dev->dev, "atusb_xmit (%d)\n", skb->len);
+       dev_dbg(&usb_dev->dev, "%s (%d)\n", __func__, skb->len);
        atusb->tx_skb = skb;
        atusb->tx_ack_seq++;
        atusb->tx_dr.wIndex = cpu_to_le16(atusb->tx_ack_seq);
@@ -399,13 +397,13 @@ static int atusb_xmit(struct ieee802154_hw *hw, struct sk_buff *skb)
                             (unsigned char *)&atusb->tx_dr, skb->data,
                             skb->len, atusb_xmit_complete, NULL);
        ret = usb_submit_urb(atusb->tx_urb, GFP_ATOMIC);
-       dev_dbg(&usb_dev->dev, "atusb_xmit done (%d)\n", ret);
+       dev_dbg(&usb_dev->dev, "%s done (%d)\n", __func__, ret);
        return ret;
 }
 
 static int atusb_ed(struct ieee802154_hw *hw, u8 *level)
 {
-       BUG_ON(!level);
+       WARN_ON(!level);
        *level = 0xbe;
        return 0;
 }
@@ -420,7 +418,7 @@ static int atusb_set_hw_addr_filt(struct ieee802154_hw *hw,
        if (changed & IEEE802154_AFILT_SADDR_CHANGED) {
                u16 addr = le16_to_cpu(filt->short_addr);
 
-               dev_vdbg(dev, "atusb_set_hw_addr_filt called for saddr\n");
+               dev_vdbg(dev, "%s called for saddr\n", __func__);
                atusb_write_reg(atusb, RG_SHORT_ADDR_0, addr);
                atusb_write_reg(atusb, RG_SHORT_ADDR_1, addr >> 8);
        }
@@ -428,7 +426,7 @@ static int atusb_set_hw_addr_filt(struct ieee802154_hw *hw,
        if (changed & IEEE802154_AFILT_PANID_CHANGED) {
                u16 pan = le16_to_cpu(filt->pan_id);
 
-               dev_vdbg(dev, "atusb_set_hw_addr_filt called for pan id\n");
+               dev_vdbg(dev, "%s called for pan id\n", __func__);
                atusb_write_reg(atusb, RG_PAN_ID_0, pan);
                atusb_write_reg(atusb, RG_PAN_ID_1, pan >> 8);
        }
@@ -437,14 +435,13 @@ static int atusb_set_hw_addr_filt(struct ieee802154_hw *hw,
                u8 i, addr[IEEE802154_EXTENDED_ADDR_LEN];
 
                memcpy(addr, &filt->ieee_addr, IEEE802154_EXTENDED_ADDR_LEN);
-               dev_vdbg(dev, "atusb_set_hw_addr_filt called for IEEE addr\n");
+               dev_vdbg(dev, "%s called for IEEE addr\n", __func__);
                for (i = 0; i < 8; i++)
                        atusb_write_reg(atusb, RG_IEEE_ADDR_0 + i, addr[i]);
        }
 
        if (changed & IEEE802154_AFILT_PANC_CHANGED) {
-               dev_vdbg(dev,
-                        "atusb_set_hw_addr_filt called for panc change\n");
+               dev_vdbg(dev, "%s called for panc change\n", __func__);
                if (filt->pan_coord)
                        atusb_write_subreg(atusb, SR_AACK_I_AM_COORD, 1);
                else
@@ -460,7 +457,7 @@ static int atusb_start(struct ieee802154_hw *hw)
        struct usb_device *usb_dev = atusb->usb_dev;
        int ret;
 
-       dev_dbg(&usb_dev->dev, "atusb_start\n");
+       dev_dbg(&usb_dev->dev, "%s\n", __func__);
        schedule_delayed_work(&atusb->work, 0);
        atusb_command(atusb, ATUSB_RX_MODE, 1);
        ret = atusb_get_and_clear_error(atusb);
@@ -474,7 +471,7 @@ static void atusb_stop(struct ieee802154_hw *hw)
        struct atusb *atusb = hw->priv;
        struct usb_device *usb_dev = atusb->usb_dev;
 
-       dev_dbg(&usb_dev->dev, "atusb_stop\n");
+       dev_dbg(&usb_dev->dev, "%s\n", __func__);
        usb_kill_anchored_urbs(&atusb->idle_urbs);
        atusb_command(atusb, ATUSB_RX_MODE, 0);
        atusb_get_and_clear_error(atusb);
@@ -842,7 +839,8 @@ static int atusb_get_and_show_revision(struct atusb *atusb)
 
                dev_info(&usb_dev->dev,
                         "Firmware: major: %u, minor: %u, hardware type: %s (%d)\n",
-                        atusb->fw_ver_maj, atusb->fw_ver_min, hw_name, atusb->fw_hw_type);
+                        atusb->fw_ver_maj, atusb->fw_ver_min, hw_name,
+                        atusb->fw_hw_type);
        }
        if (atusb->fw_ver_maj == 0 && atusb->fw_ver_min < 2) {
                dev_info(&usb_dev->dev,
@@ -880,7 +878,7 @@ static int atusb_get_and_show_build(struct atusb *atusb)
 static int atusb_get_and_conf_chip(struct atusb *atusb)
 {
        struct usb_device *usb_dev = atusb->usb_dev;
-       uint8_t man_id_0, man_id_1, part_num, version_num;
+       u8 man_id_0, man_id_1, part_num, version_num;
        const char *chip;
        struct ieee802154_hw *hw = atusb->hw;
 
@@ -977,7 +975,8 @@ static int atusb_set_extended_addr(struct atusb *atusb)
        int ret;
 
        /* Firmware versions before 0.3 do not support the EUI64_READ command.
-        * Just use a random address and be done */
+        * Just use a random address and be done.
+        */
        if (atusb->fw_ver_maj == 0 && atusb->fw_ver_min < 3) {
                ieee802154_random_extended_addr(&atusb->hw->phy->perm_extended_addr);
                return 0;
@@ -1007,7 +1006,7 @@ static int atusb_set_extended_addr(struct atusb *atusb)
                atusb->hw->phy->perm_extended_addr = extended_addr;
                addr = swab64((__force u64)atusb->hw->phy->perm_extended_addr);
                dev_info(&usb_dev->dev, "Read permanent extended address %8phC from device\n",
-                       &addr);
+                        &addr);
        }
 
        kfree(buffer);
@@ -1129,7 +1128,7 @@ static void atusb_disconnect(struct usb_interface *interface)
 {
        struct atusb *atusb = usb_get_intfdata(interface);
 
-       dev_dbg(&atusb->usb_dev->dev, "atusb_disconnect\n");
+       dev_dbg(&atusb->usb_dev->dev, "%s\n", __func__);
 
        atusb->shutdown = 1;
        cancel_delayed_work_sync(&atusb->work);
@@ -1146,7 +1145,7 @@ static void atusb_disconnect(struct usb_interface *interface)
        usb_set_intfdata(interface, NULL);
        usb_put_dev(atusb->usb_dev);
 
-       pr_debug("atusb_disconnect done\n");
+       pr_debug("%s done\n", __func__);
 }
 
 /* The devices we work with */