2011/1/19 Bruno Randolf <br1@xxxxxxxxxxx>: > Remove useless test_bit - it's not going to happen because of the way this > function is called only when that bit is set. > > And fix some whitespace. > > Signed-off-by: Bruno Randolf <br1@xxxxxxxxxxx> > --- > Âdrivers/net/wireless/ath/ath5k/base.c | Â 11 +++-------- > Â1 files changed, 3 insertions(+), 8 deletions(-) > > diff --git a/drivers/net/wireless/ath/ath5k/base.c b/drivers/net/wireless/ath/ath5k/base.c > index 9e8b1f4..a28ad58 100644 > --- a/drivers/net/wireless/ath/ath5k/base.c > +++ b/drivers/net/wireless/ath/ath5k/base.c > @@ -256,21 +256,16 @@ static bool ath5k_is_standard_channel(short chan) > Â} > > Âstatic unsigned int > -ath5k_setup_channels(struct ath5k_hw *ah, > - Â Â Â Â Â Â Â struct ieee80211_channel *channels, > - Â Â Â Â Â Â Â unsigned int mode, > - Â Â Â Â Â Â Â unsigned int max) > +ath5k_setup_channels(struct ath5k_hw *ah, struct ieee80211_channel *channels, > + Â Â Â Â Â Â Â unsigned int mode, unsigned int max) > Â{ > Â Â Â Âunsigned int count, size, chfreq, freq, ch; > Â Â Â Âenum ieee80211_band band; > > - Â Â Â if (!test_bit(mode, ah->ah_modes)) > - Â Â Â Â Â Â Â return 0; > - > Â Â Â Âswitch (mode) { > Â Â Â Âcase AR5K_MODE_11A: > Â Â Â Â Â Â Â Â/* 1..220, but 2GHz frequencies are filtered by check_channel */ > - Â Â Â Â Â Â Â size = 220 ; > + Â Â Â Â Â Â Â size = 220; > Â Â Â Â Â Â Â Âchfreq = CHANNEL_5GHZ; > Â Â Â Â Â Â Â Âband = IEEE80211_BAND_5GHZ; > Â Â Â Â Â Â Â Âbreak; > Acked-by: Nick Kossifidis <mickflemm@xxxxxxxxx> -- GPG ID: 0xD21DB2DB As you read this post global entropy rises. Have Fun ;-) Nick -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html