net: hinic: replace magic numbers with macro
authorZhengchao Shao <shaozhengchao@huawei.com>
Wed, 21 Sep 2022 12:33:57 +0000 (20:33 +0800)
committerJakub Kicinski <kuba@kernel.org>
Fri, 23 Sep 2022 00:41:28 +0000 (17:41 -0700)
Replace magic numbers with macro.

Signed-off-by: Zhengchao Shao <shaozhengchao@huawei.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/huawei/hinic/hinic_sriov.c

index d54ccef..a5f08b9 100644 (file)
@@ -24,6 +24,7 @@ MODULE_PARM_DESC(set_vf_link_state, "Set vf link state, 0 represents link auto,
 #define HINIC_VLAN_PRIORITY_SHIFT 13
 #define HINIC_ADD_VLAN_IN_MAC 0x8000
 #define HINIC_TX_RATE_TABLE_FULL 12
+#define HINIC_MAX_QOS 7
 
 static int hinic_set_mac(struct hinic_hwdev *hwdev, const u8 *mac_addr,
                         u16 vlan_id, u16 func_id)
@@ -774,7 +775,7 @@ int hinic_ndo_set_vf_vlan(struct net_device *netdev, int vf, u16 vlan, u8 qos,
        u16 vlanprio, cur_vlanprio;
 
        sriov_info = &nic_dev->sriov_info;
-       if (vf >= sriov_info->num_vfs || vlan > 4095 || qos > 7)
+       if (vf >= sriov_info->num_vfs || vlan >= VLAN_N_VID || qos > HINIC_MAX_QOS)
                return -EINVAL;
        if (vlan_proto != htons(ETH_P_8021Q))
                return -EPROTONOSUPPORT;