i40e: remove second check of VLAN_N_VID in i40e_vlan_rx_add_vid
authorJacob Keller <jacob.e.keller@intel.com>
Tue, 25 Oct 2016 23:08:49 +0000 (16:08 -0700)
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>
Sat, 3 Dec 2016 06:38:47 +0000 (22:38 -0800)
Replace a check of magic number 4095 with VLAN_N_VID. This
makes it obvious that a later check against VLAN_N_VID is
always true and can be removed.

Change-ID: I28998f127a61a529480ce63d8a07e266f6c63b7b
Signed-off-by: Jacob Keller <jacob.e.keller@intel.com>
Tested-by: Andrew Bowers <andrewx.bowers@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
drivers/net/ethernet/intel/i40e/i40e_main.c

index 2d91274f9f62c633ea03ee685f38e2022eb43bf5..fe5939a3536febdb4353178cdb812661fa2f926d 100644 (file)
@@ -2549,7 +2549,7 @@ static int i40e_vlan_rx_add_vid(struct net_device *netdev,
        struct i40e_vsi *vsi = np->vsi;
        int ret = 0;
 
-       if (vid > 4095)
+       if (vid >= VLAN_N_VID)
                return -EINVAL;
 
        /* If the network stack called us with vid = 0 then
@@ -2561,7 +2561,7 @@ static int i40e_vlan_rx_add_vid(struct net_device *netdev,
        if (vid)
                ret = i40e_vsi_add_vlan(vsi, vid);
 
-       if (!ret && (vid < VLAN_N_VID))
+       if (!ret)
                set_bit(vid, vsi->active_vlans);
 
        return ret;