Skip to content

Commit

Permalink
wl12xx/wl18xx: scan all 5ghz channels
Browse files Browse the repository at this point in the history
Due to a typo, the current code copies only sizeof(cmd->channels_2)
bytes, which is smaller than the correct sizeof(cmd->channels_5)
size, resulting in a partial scan (some channels are skipped).

Signed-off-by: Eliad Peller <eliad@wizery.com>
Signed-off-by: Luciano Coelho <coelho@ti.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
  • Loading branch information
Eliad Peller authored and John W. Linville committed Jun 12, 2013
1 parent 0e284c0 commit a805de4
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion drivers/net/wireless/ti/wl12xx/scan.c
Original file line number Diff line number Diff line change
@@ -310,7 +310,7 @@ static void wl12xx_adjust_channels(struct wl1271_cmd_sched_scan_config *cmd,
memcpy(cmd->channels_2, cmd_channels->channels_2,
sizeof(cmd->channels_2));
memcpy(cmd->channels_5, cmd_channels->channels_5,
sizeof(cmd->channels_2));
sizeof(cmd->channels_5));
/* channels_4 are not supported, so no need to copy them */
}

2 changes: 1 addition & 1 deletion drivers/net/wireless/ti/wl18xx/scan.c
Original file line number Diff line number Diff line change
@@ -34,7 +34,7 @@ static void wl18xx_adjust_channels(struct wl18xx_cmd_scan_params *cmd,
memcpy(cmd->channels_2, cmd_channels->channels_2,
sizeof(cmd->channels_2));
memcpy(cmd->channels_5, cmd_channels->channels_5,
sizeof(cmd->channels_2));
sizeof(cmd->channels_5));
/* channels_4 are not supported, so no need to copy them */
}

0 comments on commit a805de4

Please sign in to comment.