mirror of https://gitee.com/openkylin/linux.git
i40e: redfine I40E_PHY_TYPE_MAX
Since I40E_PHY_TYPE_MAX is used as an iterator, usually combined with some sort of bit-shifting, it should only include actual PHY types and not error cases. Move it up in the enum declaration so that loops only iterate across valid PHY types. Signed-off-by: Mitch Williams <mitch.a.williams@intel.com> Tested-by: Andrew Bowers <andrewx.bowers@intel.com> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
This commit is contained in:
parent
c3d26b75c2
commit
60518a0489
|
@ -1771,9 +1771,9 @@ enum i40e_aq_phy_type {
|
||||||
I40E_PHY_TYPE_25GBASE_CR = 0x20,
|
I40E_PHY_TYPE_25GBASE_CR = 0x20,
|
||||||
I40E_PHY_TYPE_25GBASE_SR = 0x21,
|
I40E_PHY_TYPE_25GBASE_SR = 0x21,
|
||||||
I40E_PHY_TYPE_25GBASE_LR = 0x22,
|
I40E_PHY_TYPE_25GBASE_LR = 0x22,
|
||||||
|
I40E_PHY_TYPE_MAX,
|
||||||
I40E_PHY_TYPE_EMPTY = 0xFE,
|
I40E_PHY_TYPE_EMPTY = 0xFE,
|
||||||
I40E_PHY_TYPE_DEFAULT = 0xFF,
|
I40E_PHY_TYPE_DEFAULT = 0xFF,
|
||||||
I40E_PHY_TYPE_MAX
|
|
||||||
};
|
};
|
||||||
|
|
||||||
#define I40E_LINK_SPEED_100MB_SHIFT 0x1
|
#define I40E_LINK_SPEED_100MB_SHIFT 0x1
|
||||||
|
|
|
@ -1767,9 +1767,9 @@ enum i40e_aq_phy_type {
|
||||||
I40E_PHY_TYPE_25GBASE_CR = 0x20,
|
I40E_PHY_TYPE_25GBASE_CR = 0x20,
|
||||||
I40E_PHY_TYPE_25GBASE_SR = 0x21,
|
I40E_PHY_TYPE_25GBASE_SR = 0x21,
|
||||||
I40E_PHY_TYPE_25GBASE_LR = 0x22,
|
I40E_PHY_TYPE_25GBASE_LR = 0x22,
|
||||||
|
I40E_PHY_TYPE_MAX,
|
||||||
I40E_PHY_TYPE_EMPTY = 0xFE,
|
I40E_PHY_TYPE_EMPTY = 0xFE,
|
||||||
I40E_PHY_TYPE_DEFAULT = 0xFF,
|
I40E_PHY_TYPE_DEFAULT = 0xFF,
|
||||||
I40E_PHY_TYPE_MAX
|
|
||||||
};
|
};
|
||||||
|
|
||||||
#define I40E_LINK_SPEED_100MB_SHIFT 0x1
|
#define I40E_LINK_SPEED_100MB_SHIFT 0x1
|
||||||
|
|
Loading…
Reference in New Issue