diff options
| author | Jesse Brandeburg <[email protected]> | 2021-03-31 21:17:03 +0000 |
|---|---|---|
| committer | Tony Nguyen <[email protected]> | 2021-04-15 00:00:06 +0000 |
| commit | 1cdea9a7eae3a976adc2735bc7ce62ac07cafcdb (patch) | |
| tree | fbf898d5992a3320a0ae51092c9e839cbf7d235a /drivers/net/ethernet/intel/ice/ice_controlq.h | |
| parent | ice: use local for consistency (diff) | |
| download | kernel-1cdea9a7eae3a976adc2735bc7ce62ac07cafcdb.tar.gz kernel-1cdea9a7eae3a976adc2735bc7ce62ac07cafcdb.zip | |
ice: remove unused struct member
The only time you can ever have a rq_last_status is if
a firmware event was somehow reporting a status on the receive
queue, which are generally firmware initiated events or
mailbox messages from a VF. Mostly this struct member was unused.
Fix this problem by still printing the value of the field in a debug
print, but don't store the value forever in a struct, potentially
creating opportunities for callers to use the wrong struct member.
Signed-off-by: Jesse Brandeburg <[email protected]>
Tested-by: Tony Brelinski <[email protected]>
Signed-off-by: Tony Nguyen <[email protected]>
Diffstat (limited to 'drivers/net/ethernet/intel/ice/ice_controlq.h')
| -rw-r--r-- | drivers/net/ethernet/intel/ice/ice_controlq.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/net/ethernet/intel/ice/ice_controlq.h b/drivers/net/ethernet/intel/ice/ice_controlq.h index 77c2307d4fb8..fe75871e48ca 100644 --- a/drivers/net/ethernet/intel/ice/ice_controlq.h +++ b/drivers/net/ethernet/intel/ice/ice_controlq.h @@ -83,7 +83,6 @@ struct ice_rq_event_info { /* Control Queue information */ struct ice_ctl_q_info { enum ice_ctl_q qtype; - enum ice_aq_err rq_last_status; /* last status on receive queue */ struct ice_ctl_q_ring rq; /* receive queue */ struct ice_ctl_q_ring sq; /* send queue */ u32 sq_cmd_timeout; /* send queue cmd write back timeout */ |