hv_netvsc: make netvsc_destroy_buf void
authorStephen Hemminger <sthemmin@microsoft.com>
Tue, 23 Aug 2016 19:17:54 +0000 (12:17 -0700)
committerDavid S. Miller <davem@davemloft.net>
Tue, 23 Aug 2016 19:05:37 +0000 (12:05 -0700)
No caller checks the return value.

Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/hyperv/netvsc.c

index 830aae2d9510bc26ad96e049103388c73ddc8b66..471c476e72534acdae09fbc7fa2a1e57ce2bc43f 100644 (file)
@@ -206,12 +206,12 @@ get_in_err:
        return net_device;
 }
 
-static int netvsc_destroy_buf(struct hv_device *device)
+static void netvsc_destroy_buf(struct hv_device *device)
 {
        struct nvsp_message *revoke_packet;
-       int ret = 0;
        struct net_device *ndev = hv_get_drvdata(device);
        struct netvsc_device *net_device = net_device_to_netvsc_device(ndev);
+       int ret;
 
        /*
         * If we got a section count, it means we received a
@@ -241,7 +241,7 @@ static int netvsc_destroy_buf(struct hv_device *device)
                if (ret != 0) {
                        netdev_err(ndev, "unable to send "
                                "revoke receive buffer to netvsp\n");
-                       return ret;
+                       return;
                }
        }
 
@@ -256,7 +256,7 @@ static int netvsc_destroy_buf(struct hv_device *device)
                if (ret != 0) {
                        netdev_err(ndev,
                                   "unable to teardown receive buffer's gpadl\n");
-                       return ret;
+                       return;
                }
                net_device->recv_buf_gpadl_handle = 0;
        }
@@ -300,7 +300,7 @@ static int netvsc_destroy_buf(struct hv_device *device)
                if (ret != 0) {
                        netdev_err(ndev, "unable to send "
                                   "revoke send buffer to netvsp\n");
-                       return ret;
+                       return;
                }
        }
        /* Teardown the gpadl on the vsp end */
@@ -314,7 +314,7 @@ static int netvsc_destroy_buf(struct hv_device *device)
                if (ret != 0) {
                        netdev_err(ndev,
                                   "unable to teardown send buffer's gpadl\n");
-                       return ret;
+                       return;
                }
                net_device->send_buf_gpadl_handle = 0;
        }
@@ -324,8 +324,6 @@ static int netvsc_destroy_buf(struct hv_device *device)
                net_device->send_buf = NULL;
        }
        kfree(net_device->send_section_map);
-
-       return ret;
 }
 
 static int netvsc_init_buf(struct hv_device *device)