diff options
author | Kalle Valo <kvalo@qca.qualcomm.com> | 2012-03-12 07:22:54 -0400 |
---|---|---|
committer | Kalle Valo <kvalo@qca.qualcomm.com> | 2012-03-13 08:18:34 -0400 |
commit | b0fc7c1a643bb0e689cd2846b0edf7c707a87600 (patch) | |
tree | 3feeab27bd61caab3bc7e0ab292c4da2d212734c | |
parent | 17a7b16df4ba8e71e149fcf31e0b639ff45ddf84 (diff) |
ath6kl: fix regression in ath6kl_upload_board_file()
My patch 24fc32b3 ("ath6kl: add ath6kl_bmi_write_hi32()") caused a regression
in ath6kl_upload_board_file() and the board_address variable was not
properly initialised in some cases:
ath6kl/init.c:1068:6: warning: ‘board_address’ may be used uninitialized
in this function
Most likely this broke ar6004 support but I can't test that right now.
Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
-rw-r--r-- | drivers/net/wireless/ath/ath6kl/init.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/net/wireless/ath/ath6kl/init.c b/drivers/net/wireless/ath/ath6kl/init.c index 231675d4bf70..03cae142f178 100644 --- a/drivers/net/wireless/ath/ath6kl/init.c +++ b/drivers/net/wireless/ath/ath6kl/init.c | |||
@@ -1078,8 +1078,9 @@ static int ath6kl_upload_board_file(struct ath6kl *ar) | |||
1078 | * writing board data. | 1078 | * writing board data. |
1079 | */ | 1079 | */ |
1080 | if (ar->hw.board_addr != 0) { | 1080 | if (ar->hw.board_addr != 0) { |
1081 | board_address = ar->hw.board_addr; | ||
1081 | ath6kl_bmi_write_hi32(ar, hi_board_data, | 1082 | ath6kl_bmi_write_hi32(ar, hi_board_data, |
1082 | ar->hw.board_addr); | 1083 | board_address); |
1083 | } else { | 1084 | } else { |
1084 | ath6kl_bmi_read_hi32(ar, hi_board_data, &board_address); | 1085 | ath6kl_bmi_read_hi32(ar, hi_board_data, &board_address); |
1085 | } | 1086 | } |