diff options
author | Chinh T Cao <chinh.t.cao@intel.com> | 2018-11-05 15:18:35 -0500 |
---|---|---|
committer | Jeff Kirsher <jeffrey.t.kirsher@intel.com> | 2018-11-07 12:37:28 -0500 |
commit | ffe498237b36ee42624e139b21efa05da4ff1f48 (patch) | |
tree | 4eaba0f799a3617fa386ccf03ad6c8d485b80f27 | |
parent | 4c9b658eeaefedd402a59e858d8ac3bfdf6153e3 (diff) |
ice: Change req_speeds to be u16
Since the req_speeds field in struct ice_link_status is a u8,
req_speeds & ICE_AQ_LINK_SPEED_40GB always returns 0. This was caught
by a coverity scan.
Fix this by changing req_speeds to be u16.
Reported-by: Bruce Allan <bruce.w.allan@intel.com>
Signed-off-by: Chinh T Cao <chinh.t.cao@intel.com>
Signed-off-by: Anirudh Venkataramanan <anirudh.venkataramanan@intel.com>
Tested-by: Andrew Bowers <andrewx.bowers@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
-rw-r--r-- | drivers/net/ethernet/intel/ice/ice_type.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/intel/ice/ice_type.h b/drivers/net/ethernet/intel/ice/ice_type.h index 12f9432abf11..f4dbc81c1988 100644 --- a/drivers/net/ethernet/intel/ice/ice_type.h +++ b/drivers/net/ethernet/intel/ice/ice_type.h | |||
@@ -92,12 +92,12 @@ struct ice_link_status { | |||
92 | u64 phy_type_low; | 92 | u64 phy_type_low; |
93 | u16 max_frame_size; | 93 | u16 max_frame_size; |
94 | u16 link_speed; | 94 | u16 link_speed; |
95 | u16 req_speeds; | ||
95 | u8 lse_ena; /* Link Status Event notification */ | 96 | u8 lse_ena; /* Link Status Event notification */ |
96 | u8 link_info; | 97 | u8 link_info; |
97 | u8 an_info; | 98 | u8 an_info; |
98 | u8 ext_info; | 99 | u8 ext_info; |
99 | u8 pacing; | 100 | u8 pacing; |
100 | u8 req_speeds; | ||
101 | /* Refer to #define from module_type[ICE_MODULE_TYPE_TOTAL_BYTE] of | 101 | /* Refer to #define from module_type[ICE_MODULE_TYPE_TOTAL_BYTE] of |
102 | * ice_aqc_get_phy_caps structure | 102 | * ice_aqc_get_phy_caps structure |
103 | */ | 103 | */ |