diff options
| author | Imre Vadász <ivadasz@FreeBSD.org> | 2016-09-19 22:08:47 +0000 |
|---|---|---|
| committer | Imre Vadász <ivadasz@FreeBSD.org> | 2016-09-19 22:08:47 +0000 |
| commit | ef81578c9f0bd01c8ffd630622dafdba347f1591 (patch) | |
| tree | f97124583b30bef55d37167d1c94b56268080f28 /sys/dev/iwm | |
| parent | b47237ce60365b12bf11dbf16f4e8d775e0d9823 (diff) | |
[iwm] Remove unused field from iwm_rx_data. Use uint32_t instead of enum type.
The wantresp field in struct iwm_rx_data has never been used for anything,
so we can just delete it.
Apparently struct iwm_sf_cfg_cmd was compiled correctly (using a 32bit
value to represent the enum), but it still seems like a very bad idea to use
an enum type in a __packed struct.
Approved by: adrian (mentor)
Differential Revision: https://reviews.freebsd.org/D7930
Notes
Notes:
svn path=/head/; revision=306000
Diffstat (limited to 'sys/dev/iwm')
| -rw-r--r-- | sys/dev/iwm/if_iwmreg.h | 2 | ||||
| -rw-r--r-- | sys/dev/iwm/if_iwmvar.h | 1 |
2 files changed, 1 insertions, 2 deletions
diff --git a/sys/dev/iwm/if_iwmreg.h b/sys/dev/iwm/if_iwmreg.h index 96a9e9eed9c9..7b0daa368168 100644 --- a/sys/dev/iwm/if_iwmreg.h +++ b/sys/dev/iwm/if_iwmreg.h @@ -3251,7 +3251,7 @@ enum iwm_sf_scenario { * @full_on_timeouts: timer values for each scenario in full on state. */ struct iwm_sf_cfg_cmd { - enum iwm_sf_state state; + uint32_t state; uint32_t watermark[IWM_SF_TRANSIENT_STATES_NUMBER]; uint32_t long_delay_timeouts[IWM_SF_NUM_SCENARIO][IWM_SF_NUM_TIMEOUT_TYPES]; uint32_t full_on_timeouts[IWM_SF_NUM_SCENARIO][IWM_SF_NUM_TIMEOUT_TYPES]; diff --git a/sys/dev/iwm/if_iwmvar.h b/sys/dev/iwm/if_iwmvar.h index 15a4655c03db..8ccb60c21a75 100644 --- a/sys/dev/iwm/if_iwmvar.h +++ b/sys/dev/iwm/if_iwmvar.h @@ -275,7 +275,6 @@ struct iwm_tx_ring { struct iwm_rx_data { struct mbuf *m; bus_dmamap_t map; - int wantresp; }; struct iwm_rx_ring { |