Skip to content

Commit

Permalink
ath5k: Simplify loop when setting up channels
Browse files Browse the repository at this point in the history
Simplify confusing code and get rid of an unnecessary variable.

Signed-off-by: Bruno Randolf <br1@einfach.org>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
  • Loading branch information
Bruno Randolf authored and John W. Linville committed Jan 21, 2011
1 parent d7e86c3 commit 2b1351a
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions drivers/net/wireless/ath/ath5k/base.c
Original file line number Diff line number Diff line change
Expand Up @@ -267,7 +267,7 @@ ath5k_setup_channels(struct ath5k_hw *ah,
unsigned int mode,
unsigned int max)
{
unsigned int i, count, size, chfreq, freq, ch;
unsigned int count, size, chfreq, freq, ch;
enum ieee80211_band band;

if (!test_bit(mode, ah->ah_modes))
Expand All @@ -291,8 +291,8 @@ ath5k_setup_channels(struct ath5k_hw *ah,
return 0;
}

for (i = 0, count = 0; i < size && max > 0; i++) {
ch = i + 1 ;
count = 0;
for (ch = 1; ch <= size && count < max; ch++) {
freq = ieee80211_channel_to_frequency(ch, band);

if (freq == 0) /* mapping failed - not a standard channel */
Expand All @@ -319,7 +319,6 @@ ath5k_setup_channels(struct ath5k_hw *ah,
}

count++;
max--;
}

return count;
Expand Down

0 comments on commit 2b1351a

Please sign in to comment.