rndis_host: cleanup: byteswap data from device instead of RNDIS_* defines
authorJussi Kivilinna <jussi.kivilinna@mbnet.fi>
Fri, 11 May 2012 22:17:34 +0000 (22:17 +0000)
committerDavid S. Miller <davem@davemloft.net>
Sat, 12 May 2012 19:16:13 +0000 (15:16 -0400)
All other values from device provided buffer are byteswapped, so it seems
more logical to do same for these.

Signed-off-by: Jussi Kivilinna <jussi.kivilinna@mbnet.fi>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/usb/rndis_host.c

index dd342dca9d74584ea580499fca30dd31f0aac188..74315fdd4adac684259f18a1d468da9afec654e9 100644 (file)
@@ -77,16 +77,17 @@ static void rndis_msg_indicate(struct usbnet *dev, struct rndis_indicate *msg,
        if (dev->driver_info->indication) {
                dev->driver_info->indication(dev, msg, buflen);
        } else {
-               switch (msg->status) {
-               case cpu_to_le32(RNDIS_STATUS_MEDIA_CONNECT):
+               u32 status = le32_to_cpu(msg->status);
+
+               switch (status) {
+               case RNDIS_STATUS_MEDIA_CONNECT:
                        dev_info(udev, "rndis media connect\n");
                        break;
-               case cpu_to_le32(RNDIS_STATUS_MEDIA_DISCONNECT):
+               case RNDIS_STATUS_MEDIA_DISCONNECT:
                        dev_info(udev, "rndis media disconnect\n");
                        break;
                default:
-                       dev_info(udev, "rndis indication: 0x%08x\n",
-                                       le32_to_cpu(msg->status));
+                       dev_info(udev, "rndis indication: 0x%08x\n", status);
                }
        }
 }
@@ -109,16 +110,17 @@ int rndis_command(struct usbnet *dev, struct rndis_msg_hdr *buf, int buflen)
        int                     retval;
        int                     partial;
        unsigned                count;
-       __le32                  rsp;
-       u32                     xid = 0, msg_len, request_id;
+       u32                     xid = 0, msg_len, request_id, msg_type, rsp,
+                               status;
 
        /* REVISIT when this gets called from contexts other than probe() or
         * disconnect(): either serialize, or dispatch responses on xid
         */
 
+       msg_type = le32_to_cpu(buf->msg_type);
+
        /* Issue the request; xid is unique, don't bother byteswapping it */
-       if (likely(buf->msg_type != cpu_to_le32(RNDIS_MSG_HALT) &&
-                  buf->msg_type != cpu_to_le32(RNDIS_MSG_RESET))) {
+       if (likely(msg_type != RNDIS_MSG_HALT && msg_type != RNDIS_MSG_RESET)) {
                xid = dev->xid++;
                if (!xid)
                        xid = dev->xid++;
@@ -149,7 +151,7 @@ int rndis_command(struct usbnet *dev, struct rndis_msg_hdr *buf, int buflen)
        }
 
        /* Poll the control channel; the request probably completed immediately */
-       rsp = buf->msg_type | cpu_to_le32(RNDIS_MSG_COMPLETION);
+       rsp = le32_to_cpu(buf->msg_type) | RNDIS_MSG_COMPLETION;
        for (count = 0; count < 10; count++) {
                memset(buf, 0, CONTROL_BUFFER_SIZE);
                retval = usb_control_msg(dev->udev,
@@ -160,30 +162,31 @@ int rndis_command(struct usbnet *dev, struct rndis_msg_hdr *buf, int buflen)
                        buf, buflen,
                        RNDIS_CONTROL_TIMEOUT_MS);
                if (likely(retval >= 8)) {
+                       msg_type = le32_to_cpu(buf->msg_type);
                        msg_len = le32_to_cpu(buf->msg_len);
+                       status = le32_to_cpu(buf->status);
                        request_id = (__force u32) buf->request_id;
-                       if (likely(buf->msg_type == rsp)) {
+                       if (likely(msg_type == rsp)) {
                                if (likely(request_id == xid)) {
-                                       if (unlikely(rsp ==
-                                           cpu_to_le32(RNDIS_MSG_RESET_C)))
+                                       if (unlikely(rsp == RNDIS_MSG_RESET_C))
                                                return 0;
-                                       if (likely(cpu_to_le32(RNDIS_STATUS_SUCCESS)
-                                                       == buf->status))
+                                       if (likely(RNDIS_STATUS_SUCCESS ==
+                                                       status))
                                                return 0;
                                        dev_dbg(&info->control->dev,
                                                "rndis reply status %08x\n",
-                                               le32_to_cpu(buf->status));
+                                               status);
                                        return -EL3RST;
                                }
                                dev_dbg(&info->control->dev,
                                        "rndis reply id %d expected %d\n",
                                        request_id, xid);
                                /* then likely retry */
-                       } else switch (buf->msg_type) {
-                       case cpu_to_le32(RNDIS_MSG_INDICATE): /* fault/event */
+                       } else switch (msg_type) {
+                       case RNDIS_MSG_INDICATE: /* fault/event */
                                rndis_msg_indicate(dev, (void *)buf, buflen);
                                break;
-                       case cpu_to_le32(RNDIS_MSG_KEEPALIVE): { /* ping */
+                       case RNDIS_MSG_KEEPALIVE: { /* ping */
                                struct rndis_keepalive_c *msg = (void *)buf;
 
                                msg->msg_type = cpu_to_le32(RNDIS_MSG_KEEPALIVE_C);
@@ -404,14 +407,14 @@ generic_rndis_bind(struct usbnet *dev, struct usb_interface *intf, int flags)
                phym = &phym_unspec;
        }
        if ((flags & FLAG_RNDIS_PHYM_WIRELESS) &&
-           *phym != cpu_to_le32(RNDIS_PHYSICAL_MEDIUM_WIRELESS_LAN)) {
+           le32_to_cpup(phym) != RNDIS_PHYSICAL_MEDIUM_WIRELESS_LAN) {
                netif_dbg(dev, probe, dev->net,
                          "driver requires wireless physical medium, but device is not\n");
                retval = -ENODEV;
                goto halt_fail_and_release;
        }
        if ((flags & FLAG_RNDIS_PHYM_NOT_WIRELESS) &&
-           *phym == cpu_to_le32(RNDIS_PHYSICAL_MEDIUM_WIRELESS_LAN)) {
+           le32_to_cpup(phym) == RNDIS_PHYSICAL_MEDIUM_WIRELESS_LAN) {
                netif_dbg(dev, probe, dev->net,
                          "driver requires non-wireless physical medium, but device is wireless.\n");
                retval = -ENODEV;
@@ -496,16 +499,16 @@ int rndis_rx_fixup(struct usbnet *dev, struct sk_buff *skb)
        while (likely(skb->len)) {
                struct rndis_data_hdr   *hdr = (void *)skb->data;
                struct sk_buff          *skb2;
-               u32                     msg_len, data_offset, data_len;
+               u32                     msg_type, msg_len, data_offset, data_len;
 
+               msg_type = le32_to_cpu(hdr->msg_type);
                msg_len = le32_to_cpu(hdr->msg_len);
                data_offset = le32_to_cpu(hdr->data_offset);
                data_len = le32_to_cpu(hdr->data_len);
 
                /* don't choke if we see oob, per-packet data, etc */
-               if (unlikely(hdr->msg_type != cpu_to_le32(RNDIS_MSG_PACKET) ||
-                            skb->len < msg_len ||
-                            (data_offset + data_len + 8) > msg_len)) {
+               if (unlikely(msg_type != RNDIS_MSG_PACKET || skb->len < msg_len
+                               || (data_offset + data_len + 8) > msg_len)) {
                        dev->net->stats.rx_frame_errors++;
                        netdev_dbg(dev->net, "bad rndis message %d/%d/%d/%d, len %d\n",
                                   le32_to_cpu(hdr->msg_type),