diff options
| author |   Bob Copeland <me@bobcopeland.com> | 2016-06-02 08:43:50 -0400 |
|---|---|---|
| committer |   Kalle Valo <kvalo@qca.qualcomm.com> | 2016-06-14 15:18:09 +0300 |
| commit | deb85bb1dadf8b4aaa5b33497766dc3f3e4b530e (patch) | |
| tree | 72600486cb576a3a55824ff2c8a554bae1099571 /drivers/net/wireless/ath/ath5k/pcu.c | |
| parent | ath10k: fix cycle counter wraparound handling for QCA4019 (diff) | |
| download | linux-dev-deb85bb1dadf8b4aaa5b33497766dc3f3e4b530e.tar.xz linux-dev-deb85bb1dadf8b4aaa5b33497766dc3f3e4b530e.zip | |
ath5k: fix misplaced default label in sifs switch
In this switch statement, the default case does not always assign
sifs. In practice, ah->ah_bwmode cannot take values besides the
other labels, so this is not an actual problem, but it looks odd
and smatch complains thus:
ath5k_hw_get_default_sifs() warn: missing break? reassigning 'sifs'
Silence the warning by moving default label up a line.
Signed-off-by: Bob Copeland <me@bobcopeland.com>
Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
Diffstat (limited to '')
| -rw-r--r-- | drivers/net/wireless/ath/ath5k/pcu.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wireless/ath/ath5k/pcu.c b/drivers/net/wireless/ath/ath5k/pcu.c index fc47b70988b1..f23c851765df 100644 --- a/drivers/net/wireless/ath/ath5k/pcu.c +++ b/drivers/net/wireless/ath/ath5k/pcu.c @@ -219,8 +219,8 @@ ath5k_hw_get_default_sifs(struct ath5k_hw *ah) sifs = AR5K_INIT_SIFS_QUARTER_RATE; break; case AR5K_BWMODE_DEFAULT: - sifs = AR5K_INIT_SIFS_DEFAULT_BG; default: + sifs = AR5K_INIT_SIFS_DEFAULT_BG; if (channel->band == NL80211_BAND_5GHZ) sifs = AR5K_INIT_SIFS_DEFAULT_A; break; |