Commit aa502b4a authored by Jacob Keller's avatar Jacob Keller Committed by Jeff Kirsher

fm10k: consistently refer to VLANs and VLAN IDs

Instead of using lowercase vlan, vid, or VID, always use VLAN or VLAN ID
in comments when referring to VLANs. The original driver code was
consistent, but recent patches have not been as consistent with this
naming scheme.
Signed-off-by: default avatarJacob Keller <jacob.e.keller@intel.com>
Reviewed-by: default avatarBruce Allan <bruce.w.allan@intel.com>
Tested-by: default avatarKrishneil Singh <Krishneil.k.singh@intel.com>
Signed-off-by: default avatarJeff Kirsher <jeffrey.t.kirsher@intel.com>
parent b80b1a51
...@@ -138,7 +138,7 @@ struct fm10k_ring { ...@@ -138,7 +138,7 @@ struct fm10k_ring {
* different for DCB and RSS modes * different for DCB and RSS modes
*/ */
u8 qos_pc; /* priority class of queue */ u8 qos_pc; /* priority class of queue */
u16 vid; /* default vlan ID of queue */ u16 vid; /* default VLAN ID of queue */
u16 count; /* amount of descriptors */ u16 count; /* amount of descriptors */
u16 next_to_alloc; u16 next_to_alloc;
...@@ -440,7 +440,7 @@ union fm10k_ftag_info { ...@@ -440,7 +440,7 @@ union fm10k_ftag_info {
struct { struct {
/* dglort and sglort combined into a single 32bit desc read */ /* dglort and sglort combined into a single 32bit desc read */
__le32 glort; __le32 glort;
/* upper 16 bits of vlan are reserved 0 for swpri_type_user */ /* upper 16 bits of VLAN are reserved 0 for swpri_type_user */
__le32 vlan; __le32 vlan;
} d; } d;
struct { struct {
......
...@@ -632,7 +632,7 @@ static netdev_tx_t fm10k_xmit_frame(struct sk_buff *skb, struct net_device *dev) ...@@ -632,7 +632,7 @@ static netdev_tx_t fm10k_xmit_frame(struct sk_buff *skb, struct net_device *dev)
return NETDEV_TX_OK; return NETDEV_TX_OK;
} }
/* locate vlan header */ /* locate VLAN header */
vhdr = (struct vlan_hdr *)(skb->data + ETH_HLEN); vhdr = (struct vlan_hdr *)(skb->data + ETH_HLEN);
/* pull the 2 key pieces of data out of it */ /* pull the 2 key pieces of data out of it */
...@@ -778,7 +778,7 @@ static int fm10k_update_vid(struct net_device *netdev, u16 vid, bool set) ...@@ -778,7 +778,7 @@ static int fm10k_update_vid(struct net_device *netdev, u16 vid, bool set)
if (!set) if (!set)
clear_bit(vid, interface->active_vlans); clear_bit(vid, interface->active_vlans);
/* disable the default VID on ring if we have an active VLAN */ /* disable the default VLAN ID on ring if we have an active VLAN */
for (i = 0; i < interface->num_rx_queues; i++) { for (i = 0; i < interface->num_rx_queues; i++) {
struct fm10k_ring *rx_ring = interface->rx_ring[i]; struct fm10k_ring *rx_ring = interface->rx_ring[i];
u16 rx_vid = rx_ring->vid & (VLAN_N_VID - 1); u16 rx_vid = rx_ring->vid & (VLAN_N_VID - 1);
...@@ -789,7 +789,7 @@ static int fm10k_update_vid(struct net_device *netdev, u16 vid, bool set) ...@@ -789,7 +789,7 @@ static int fm10k_update_vid(struct net_device *netdev, u16 vid, bool set)
rx_ring->vid &= ~FM10K_VLAN_CLEAR; rx_ring->vid &= ~FM10K_VLAN_CLEAR;
} }
/* Do not remove default VID related entries from VLAN and MAC tables */ /* Do not remove default VLAN ID related entries from VLAN and MAC tables */
if (!set && vid == hw->mac.default_vid) if (!set && vid == hw->mac.default_vid)
return 0; return 0;
...@@ -814,7 +814,7 @@ static int fm10k_update_vid(struct net_device *netdev, u16 vid, bool set) ...@@ -814,7 +814,7 @@ static int fm10k_update_vid(struct net_device *netdev, u16 vid, bool set)
if (err) if (err)
goto err_out; goto err_out;
/* set vid prior to syncing/unsyncing the VLAN */ /* set VLAN ID prior to syncing/unsyncing the VLAN */
interface->vid = vid + (set ? VLAN_N_VID : 0); interface->vid = vid + (set ? VLAN_N_VID : 0);
/* Update the unicast and multicast address list to add/drop VLAN */ /* Update the unicast and multicast address list to add/drop VLAN */
......
...@@ -715,7 +715,7 @@ static void fm10k_configure_rx_ring(struct fm10k_intfc *interface, ...@@ -715,7 +715,7 @@ static void fm10k_configure_rx_ring(struct fm10k_intfc *interface,
/* assign default VLAN to queue */ /* assign default VLAN to queue */
ring->vid = hw->mac.default_vid; ring->vid = hw->mac.default_vid;
/* if we have an active VLAN, disable default VID */ /* if we have an active VLAN, disable default VLAN ID */
if (test_bit(hw->mac.default_vid, interface->active_vlans)) if (test_bit(hw->mac.default_vid, interface->active_vlans))
ring->vid |= FM10K_VLAN_CLEAR; ring->vid |= FM10K_VLAN_CLEAR;
...@@ -1292,7 +1292,7 @@ static s32 fm10k_update_pvid(struct fm10k_hw *hw, u32 **results, ...@@ -1292,7 +1292,7 @@ static s32 fm10k_update_pvid(struct fm10k_hw *hw, u32 **results,
if (!fm10k_glort_valid_pf(hw, glort)) if (!fm10k_glort_valid_pf(hw, glort))
return FM10K_ERR_PARAM; return FM10K_ERR_PARAM;
/* verify VID is valid */ /* verify VLAN ID is valid */
if (pvid >= FM10K_VLAN_TABLE_VID_MAX) if (pvid >= FM10K_VLAN_TABLE_VID_MAX)
return FM10K_ERR_PARAM; return FM10K_ERR_PARAM;
......
...@@ -329,7 +329,7 @@ static s32 fm10k_update_xc_addr_pf(struct fm10k_hw *hw, u16 glort, ...@@ -329,7 +329,7 @@ static s32 fm10k_update_xc_addr_pf(struct fm10k_hw *hw, u16 glort,
/* clear set bit from VLAN ID */ /* clear set bit from VLAN ID */
vid &= ~FM10K_VLAN_CLEAR; vid &= ~FM10K_VLAN_CLEAR;
/* if glort or vlan are not valid return error */ /* if glort or VLAN are not valid return error */
if (!fm10k_glort_valid_pf(hw, glort) || vid >= FM10K_VLAN_TABLE_VID_MAX) if (!fm10k_glort_valid_pf(hw, glort) || vid >= FM10K_VLAN_TABLE_VID_MAX)
return FM10K_ERR_PARAM; return FM10K_ERR_PARAM;
...@@ -921,7 +921,7 @@ static s32 fm10k_iov_assign_default_mac_vlan_pf(struct fm10k_hw *hw, ...@@ -921,7 +921,7 @@ static s32 fm10k_iov_assign_default_mac_vlan_pf(struct fm10k_hw *hw,
txqctl |= (vf_idx << FM10K_TXQCTL_TC_SHIFT) | txqctl |= (vf_idx << FM10K_TXQCTL_TC_SHIFT) |
FM10K_TXQCTL_VF | vf_idx; FM10K_TXQCTL_VF | vf_idx;
/* assign VID */ /* assign VLAN ID */
for (i = 0; i < queues_per_pool; i++) for (i = 0; i < queues_per_pool; i++)
fm10k_write_reg(hw, FM10K_TXQCTL(vf_q_idx + i), txqctl); fm10k_write_reg(hw, FM10K_TXQCTL(vf_q_idx + i), txqctl);
...@@ -1172,12 +1172,12 @@ s32 fm10k_iov_msg_msix_pf(struct fm10k_hw *hw, u32 **results, ...@@ -1172,12 +1172,12 @@ s32 fm10k_iov_msg_msix_pf(struct fm10k_hw *hw, u32 **results,
} }
/** /**
* fm10k_iov_select_vid - Select correct default VID * fm10k_iov_select_vid - Select correct default VLAN ID
* @hw: Pointer to hardware structure * @hw: Pointer to hardware structure
* @vid: VID to correct * @vid: VLAN ID to correct
* *
* Will report an error if VID is out of range. For VID = 0, it will return * Will report an error if the VLAN ID is out of range. For VID = 0, it will
* either the pf_vid or sw_vid depending on which one is set. * return either the pf_vid or sw_vid depending on which one is set.
*/ */
static inline s32 fm10k_iov_select_vid(struct fm10k_vf_info *vf_info, u16 vid) static inline s32 fm10k_iov_select_vid(struct fm10k_vf_info *vf_info, u16 vid)
{ {
...@@ -1707,7 +1707,7 @@ s32 fm10k_msg_update_pvid_pf(struct fm10k_hw *hw, u32 **results, ...@@ -1707,7 +1707,7 @@ s32 fm10k_msg_update_pvid_pf(struct fm10k_hw *hw, u32 **results,
if (!fm10k_glort_valid_pf(hw, glort)) if (!fm10k_glort_valid_pf(hw, glort))
return FM10K_ERR_PARAM; return FM10K_ERR_PARAM;
/* verify VID is valid */ /* verify VLAN ID is valid */
if (pvid >= FM10K_VLAN_TABLE_VID_MAX) if (pvid >= FM10K_VLAN_TABLE_VID_MAX)
return FM10K_ERR_PARAM; return FM10K_ERR_PARAM;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment