2018-03-08 05:54:22

by Xinming Hu

[permalink] [raw]
Subject: [PATCH] mwifiex: correct antenna number with high bits reserved

High bits of antenna number are reserved in hardware spec,
using low 4 bits represent supported antenna.

Signed-off-by: Cathy Luo <[email protected]>
Signed-off-by: Xinming Hu <[email protected]>
---
drivers/net/wireless/marvell/mwifiex/cmdevt.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/net/wireless/marvell/mwifiex/cmdevt.c b/drivers/net/wireless/marvell/mwifiex/cmdevt.c
index dcc529e..78a4a6a 100644
--- a/drivers/net/wireless/marvell/mwifiex/cmdevt.c
+++ b/drivers/net/wireless/marvell/mwifiex/cmdevt.c
@@ -1526,7 +1526,8 @@ int mwifiex_ret_get_hw_spec(struct mwifiex_private *priv,

adapter->fw_release_number = le32_to_cpu(hw_spec->fw_release_number);
adapter->fw_api_ver = (adapter->fw_release_number >> 16) & 0xff;
- adapter->number_of_antenna = le16_to_cpu(hw_spec->number_of_antenna);
+ adapter->number_of_antenna =
+ le16_to_cpu(hw_spec->number_of_antenna) & 0xf;

if (le32_to_cpu(hw_spec->dot_11ac_dev_cap)) {
adapter->is_hw_11ac_capable = true;
--
1.9.1


2018-03-13 16:53:28

by Kalle Valo

[permalink] [raw]
Subject: Re: mwifiex: correct antenna number with high bits reserved

Xinming Hu <[email protected]> wrote:

> High bits of antenna number are reserved in hardware spec,
> using low 4 bits represent supported antenna.
>
> Signed-off-by: Cathy Luo <[email protected]>
> Signed-off-by: Xinming Hu <[email protected]>

Patch applied to wireless-drivers-next.git, thanks.

eaab43e505d0 mwifiex: correct antenna number with high bits reserved

--
https://patchwork.kernel.org/patch/10266469/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches