hv_netvsc: use ARRAY_SIZE() for NDIS versions
authorStephen Hemminger <sthemmin@microsoft.com>
Tue, 23 Aug 2016 19:17:49 +0000 (12:17 -0700)
committerDavid S. Miller <davem@davemloft.net>
Tue, 23 Aug 2016 19:05:36 +0000 (12:05 -0700)
Don't hard code size of array of NDIS versions.

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

index 4ef2af63419f9ca0c3584a21adfd17b850c2f7f9..110bbb8a0baac50187d9d77c6cbc48bd8fd9557a 100644 (file)
@@ -577,9 +577,10 @@ static int netvsc_connect_vsp(struct hv_device *device)
        struct netvsc_device *net_device;
        struct nvsp_message *init_packet;
        int ndis_version;
-       u32 ver_list[] = { NVSP_PROTOCOL_VERSION_1, NVSP_PROTOCOL_VERSION_2,
+       const u32 ver_list[] = {
+               NVSP_PROTOCOL_VERSION_1, NVSP_PROTOCOL_VERSION_2,
                NVSP_PROTOCOL_VERSION_4, NVSP_PROTOCOL_VERSION_5 };
-       int i, num_ver = 4; /* number of different NVSP versions */
+       int i;
 
        net_device = get_outbound_net_device(device);
        if (!net_device)
@@ -588,7 +589,7 @@ static int netvsc_connect_vsp(struct hv_device *device)
        init_packet = &net_device->channel_init_pkt;
 
        /* Negotiate the latest NVSP protocol supported */
-       for (i = num_ver - 1; i >= 0; i--)
+       for (i = ARRAY_SIZE(ver_list) - 1; i >= 0; i--)
                if (negotiate_nvsp_ver(device, net_device, init_packet,
                                       ver_list[i])  == 0) {
                        net_device->nvsp_version = ver_list[i];