Commit 578fb092 authored by Paul Greenwalt's avatar Paul Greenwalt Committed by Tony Nguyen
Browse files

ice: update ICE_PHY_TYPE_HIGH_MAX_INDEX 



ICE_PHY_TYPE_HIGH_MAX_INDEX should be the maximum index value and not the
length/number of ICE_PHY_TYPE_HIGH. This is not an issue because this
define is only used when calling ice_get_link_speed_based_on_phy_type(),
which will return ICE_AQ_LINK_SPEED_UNKNOWN for any invalid index. The
caller of ice_get_link_speed_based_on_phy_type(), ice_update_phy_type()
checks that the return value is a valid link speed before using it and
ICE_AQ_LINK_SPEED_UNKNOWN is not. However, update the define to reflect
the correct value.

Signed-off-by: default avatarPaul Greenwalt <paul.greenwalt@intel.com>
Tested-by: Pucha Himasekhar Reddy <himasekharx.reddy.pucha@intel.com> (A Contingent worker at Intel)
Signed-off-by: default avatarTony Nguyen <anthony.l.nguyen@intel.com>
parent af2eab1a

drivers/net/ethernet/intel/ice/ice_adminq_cmd.h

+1 −1
Original line number Diff line number Diff line
@@ -1087,7 +1087,7 @@ struct ice_aqc_get_phy_caps { 
#define ICE_PHY_TYPE_HIGH_100G_CAUI2		BIT_ULL(2)

#define ICE_PHY_TYPE_HIGH_100G_AUI2_AOC_ACC	BIT_ULL(3)

#define ICE_PHY_TYPE_HIGH_100G_AUI2		BIT_ULL(4)

#define ICE_PHY_TYPE_HIGH_MAX_INDEX		5

#define ICE_PHY_TYPE_HIGH_MAX_INDEX		4



struct ice_aqc_get_phy_caps_data {

	__le64 phy_type_low; /* Use values from ICE_PHY_TYPE_LOW_* */