diff mbox series

ath11k: fix multicast rate parameter size

Message ID 20190812091304.27184-1-sven@narfation.org (mailing list archive)
State Accepted
Commit bcd3f5043856af045fda6a763462048942535217
Delegated to: Kalle Valo
Headers show
Series ath11k: fix multicast rate parameter size | expand

Commit Message

Sven Eckelmann Aug. 12, 2019, 9:13 a.m. UTC
From: Sven Eckelmann <seckelmann@datto.com>

The ATH11K_HW_RATE_CODE() macro uses at least 28 bit to encode the
parameters. An u8 cannot store all these bits.

Fixes: c885aaf227ec ("ath11k: support for multicast rate control")
Signed-off-by: Sven Eckelmann <seckelmann@datto.com>
---
 drivers/net/wireless/ath/ath11k/mac.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Kalle Valo Sept. 29, 2019, 2:57 p.m. UTC | #1
Sven Eckelmann <sven@narfation.org> wrote:

> The ATH11K_HW_RATE_CODE() macro uses at least 28 bit to encode the
> parameters. An u8 cannot store all these bits.
> 
> Fixes: c885aaf227ec ("ath11k: support for multicast rate control")
> Signed-off-by: Sven Eckelmann <seckelmann@datto.com>
> Signed-off-by: Kalle Valo <kvalo@codeaurora.org>

Patch applied to ath11k-bringup branch of ath.git, thanks.

bcd3f5043856 ath11k: fix multicast rate parameter size
diff mbox series

Patch

diff --git a/drivers/net/wireless/ath/ath11k/mac.c b/drivers/net/wireless/ath/ath11k/mac.c
index 6a3b1a110738..5a65caa019ed 100644
--- a/drivers/net/wireless/ath/ath11k/mac.c
+++ b/drivers/net/wireless/ath/ath11k/mac.c
@@ -1586,7 +1586,7 @@  static void ath11k_mac_op_bss_info_changed(struct ieee80211_hw *hw,
 	u16 bitrate;
 	int ret = 0;
 	u8 rateidx;
-	u8 rate;
+	u32 rate;
 
 	mutex_lock(&ar->conf_mutex);