net: cdc_ncm: remove tx_speed and rx_speed fields
authorBjørn Mork <bjorn@mork.no>
Fri, 1 Nov 2013 10:16:44 +0000 (11:16 +0100)
committerDavid S. Miller <davem@davemloft.net>
Sat, 2 Nov 2013 06:02:02 +0000 (02:02 -0400)
These fields are only used to prevent printing the same speeds
multiple times if we receive multiple identical speed notifications.

The value of these printk's is questionable, and even more so when
we filter out some of the notifications sent us by the firmware. If
we are going to print any of these, then we should print them all.

Removing little used fields is a bonus.

Cc: Alexey Orishko <alexey.orishko@gmail.com>
Signed-off-by: Bjørn Mork <bjorn@mork.no>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/usb/cdc_ncm.c
include/linux/usb/cdc_ncm.h

index 9cdd762916dc4b56f6263ad2fbad45d05b124b0c..fc36a99f8183574ea12fc1b0daa3b597f78d4e31 100644 (file)
@@ -510,7 +510,6 @@ advance:
            ctx->tx_max % usb_maxpacket(dev->udev, dev->out, 1) == 0)
                ctx->tx_max++;
 
-       ctx->tx_speed = ctx->rx_speed = 0;
        return 0;
 
 error2:
@@ -1048,7 +1047,6 @@ static void
 cdc_ncm_speed_change(struct usbnet *dev,
                     struct usb_cdc_speed_change *data)
 {
-       struct cdc_ncm_ctx *ctx = (struct cdc_ncm_ctx *)dev->data[0];
        uint32_t rx_speed = le32_to_cpu(data->DLBitRRate);
        uint32_t tx_speed = le32_to_cpu(data->ULBitRate);
 
@@ -1056,25 +1054,20 @@ cdc_ncm_speed_change(struct usbnet *dev,
         * Currently the USB-NET API does not support reporting the actual
         * device speed. Do print it instead.
         */
-       if ((tx_speed != ctx->tx_speed) || (rx_speed != ctx->rx_speed)) {
-               ctx->tx_speed = tx_speed;
-               ctx->rx_speed = rx_speed;
-
-               if ((tx_speed > 1000000) && (rx_speed > 1000000)) {
-                       printk(KERN_INFO KBUILD_MODNAME
-                              ": %s: %u mbit/s downlink "
-                              "%u mbit/s uplink\n",
-                              dev->net->name,
-                              (unsigned int)(rx_speed / 1000000U),
-                              (unsigned int)(tx_speed / 1000000U));
-               } else {
-                       printk(KERN_INFO KBUILD_MODNAME
-                              ": %s: %u kbit/s downlink "
-                              "%u kbit/s uplink\n",
-                              dev->net->name,
-                              (unsigned int)(rx_speed / 1000U),
-                              (unsigned int)(tx_speed / 1000U));
-               }
+       if ((tx_speed > 1000000) && (rx_speed > 1000000)) {
+               printk(KERN_INFO KBUILD_MODNAME
+                      ": %s: %u mbit/s downlink "
+                      "%u mbit/s uplink\n",
+                      dev->net->name,
+                      (unsigned int)(rx_speed / 1000000U),
+                      (unsigned int)(tx_speed / 1000000U));
+       } else {
+               printk(KERN_INFO KBUILD_MODNAME
+                      ": %s: %u kbit/s downlink "
+                      "%u kbit/s uplink\n",
+                      dev->net->name,
+                      (unsigned int)(rx_speed / 1000U),
+                      (unsigned int)(tx_speed / 1000U));
        }
 }
 
@@ -1111,8 +1104,6 @@ static void cdc_ncm_status(struct usbnet *dev, struct urb *urb)
                       dev->net->name, ctx->connected ? "" : "dis");
 
                usbnet_link_change(dev, ctx->connected, 0);
-               if (!ctx->connected)
-                       ctx->tx_speed = ctx->rx_speed = 0;
                break;
 
        case USB_CDC_NOTIFY_SPEED_CHANGE:
index 059dcc93c4d8b697c59de055a4f6aa17d88795d3..f14af3dd0cce43582d03d08d4dbc7003d5182e96 100644 (file)
@@ -110,8 +110,6 @@ struct cdc_ncm_ctx {
 
        u32 tx_timer_pending;
        u32 tx_curr_frame_num;
-       u32 rx_speed;
-       u32 tx_speed;
        u32 rx_max;
        u32 tx_max;
        u32 max_datagram_size;