]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
ice: Rename ethtool private flag for lldp
authorDave Ertman <david.m.ertman@intel.com>
Mon, 29 Jul 2019 09:04:50 +0000 (02:04 -0700)
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>
Fri, 23 Aug 2019 17:15:15 +0000 (10:15 -0700)
The current flag name of "enable-fw-lldp" is a bit cumbersome.

Change priv-flag name to "fw-lldp-agent" with a value of on or
off.  This is more straight-forward in meaning.

Signed-off-by: Dave Ertman <david.m.ertman@intel.com>
Tested-by: Andrew Bowers <andrewx.bowers@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
drivers/net/ethernet/intel/ice/ice.h
drivers/net/ethernet/intel/ice/ice_dcb_lib.c
drivers/net/ethernet/intel/ice/ice_ethtool.c
drivers/net/ethernet/intel/ice/ice_lib.c

index 9f9c30d29eb5aebd90d4f763d776353b875a1b43..99e0febd8e5000703898e52964abc8ebfed76e33 100644 (file)
@@ -329,7 +329,7 @@ enum ice_pf_flags {
        ICE_FLAG_DCB_ENA,
        ICE_FLAG_LINK_DOWN_ON_CLOSE_ENA,
        ICE_FLAG_NO_MEDIA,
-       ICE_FLAG_ENABLE_FW_LLDP,
+       ICE_FLAG_FW_LLDP_AGENT,
        ICE_FLAG_ETHTOOL_CTXT,          /* set when ethtool holds RTNL lock */
        ICE_PF_FLAGS_NBITS              /* must be last */
 };
index 4fc9faf5bc715a1115f202a815bdd7c45bc772e8..734cef8eed9ec1a19915dfa23790fd2dc1bb0cdd 100644 (file)
@@ -457,7 +457,7 @@ int ice_init_pf_dcb(struct ice_pf *pf, bool locked)
                dev_info(&pf->pdev->dev,
                         "FW LLDP is disabled, DCBx/LLDP in SW mode.\n");
                port_info->is_sw_lldp = true;
-               clear_bit(ICE_FLAG_ENABLE_FW_LLDP, pf->flags);
+               clear_bit(ICE_FLAG_FW_LLDP_AGENT, pf->flags);
                err = ice_dcb_sw_dflt_cfg(pf, locked);
                if (err) {
                        dev_err(&pf->pdev->dev,
@@ -473,7 +473,7 @@ int ice_init_pf_dcb(struct ice_pf *pf, bool locked)
        }
 
        port_info->is_sw_lldp = false;
-       set_bit(ICE_FLAG_ENABLE_FW_LLDP, pf->flags);
+       set_bit(ICE_FLAG_FW_LLDP_AGENT, pf->flags);
 
        /* DCBX in FW and LLDP enabled in FW */
        pf->dcbx_cap = DCB_CAP_DCBX_LLD_MANAGED | DCB_CAP_DCBX_VER_IEEE;
index 6a97ddbbda76a15f184fbb03140c6aa4888dd634..948a33716290e03b26324788f1192d1e84a402dc 100644 (file)
@@ -155,7 +155,7 @@ struct ice_priv_flag {
 
 static const struct ice_priv_flag ice_gstrings_priv_flags[] = {
        ICE_PRIV_FLAG("link-down-on-close", ICE_FLAG_LINK_DOWN_ON_CLOSE_ENA),
-       ICE_PRIV_FLAG("enable-fw-lldp", ICE_FLAG_ENABLE_FW_LLDP),
+       ICE_PRIV_FLAG("fw-lldp-agent", ICE_FLAG_FW_LLDP_AGENT),
 };
 
 #define ICE_PRIV_FLAG_ARRAY_SIZE       ARRAY_SIZE(ice_gstrings_priv_flags)
@@ -1201,8 +1201,8 @@ static int ice_set_priv_flags(struct net_device *netdev, u32 flags)
 
        bitmap_xor(change_flags, pf->flags, orig_flags, ICE_PF_FLAGS_NBITS);
 
-       if (test_bit(ICE_FLAG_ENABLE_FW_LLDP, change_flags)) {
-               if (!test_bit(ICE_FLAG_ENABLE_FW_LLDP, pf->flags)) {
+       if (test_bit(ICE_FLAG_FW_LLDP_AGENT, change_flags)) {
+               if (!test_bit(ICE_FLAG_FW_LLDP_AGENT, pf->flags)) {
                        enum ice_status status;
 
                        /* Disable FW LLDP engine */
index c067ef6be7f4727ba4dd09313f67e59f96abb69d..343d0c305423bf017ad067faade88f197603b781 100644 (file)
@@ -2541,7 +2541,7 @@ ice_vsi_setup(struct ice_pf *pf, struct ice_port_info *pi,
                ice_cfg_sw_lldp(vsi, true, true);
 
                /* Rx LLDP packets */
-               if (!test_bit(ICE_FLAG_ENABLE_FW_LLDP, pf->flags))
+               if (!test_bit(ICE_FLAG_FW_LLDP_AGENT, pf->flags))
                        ice_cfg_sw_lldp(vsi, false, true);
        }
 
@@ -2888,7 +2888,7 @@ int ice_vsi_release(struct ice_vsi *vsi)
                /* The Rx rule will only exist to remove if the LLDP FW
                 * engine is currently stopped
                 */
-               if (!test_bit(ICE_FLAG_ENABLE_FW_LLDP, pf->flags))
+               if (!test_bit(ICE_FLAG_FW_LLDP_AGENT, pf->flags))
                        ice_cfg_sw_lldp(vsi, false, false);
        }