ath5k: fix misplaced default label in sifs switch
authorBob Copeland <me@bobcopeland.com>
Thu, 2 Jun 2016 12:43:50 +0000 (08:43 -0400)
committerKalle Valo <kvalo@qca.qualcomm.com>
Tue, 14 Jun 2016 12:18:09 +0000 (15:18 +0300)
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>
drivers/net/wireless/ath/ath5k/pcu.c

index fc47b70988b1692938300e29c3dd20293efaff21..f23c851765df49dfe35c650249b687abb2ebe965 100644 (file)
@@ -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;