2023-05-09 17:33:25

by Amisha.Patel

Subject: [PATCH v2] wifi: wilc1000: Increase ASSOC response buffer

From: Amisha Patel <[email protected]>

In recent access points, information element is longer as they include
additional data which exceeds 256 bytes. To accommodate longer
association response, increase the ASSOC response buffer.

Signed-off-by: Amisha Patel <[email protected]>
---

v1 -> v2: Fixed From field and Signed-off by field.

---
drivers/net/wireless/microchip/wilc1000/hif.h | 2 --
drivers/net/wireless/microchip/wilc1000/wlan_cfg.h | 2 +-
drivers/net/wireless/microchip/wilc1000/wlan_if.h | 2 ++
3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/net/wireless/microchip/wilc1000/hif.h b/drivers/net/wireless/microchip/wilc1000/hif.h
index baa2881f4465..8e386db72e45 100644
--- a/drivers/net/wireless/microchip/wilc1000/hif.h
+++ b/drivers/net/wireless/microchip/wilc1000/hif.h
@@ -30,8 +30,6 @@ enum {
WILC_GET_CFG
};

-#define WILC_MAX_ASSOC_RESP_FRAME_SIZE 256
-
struct rf_info {
u8 link_speed;
s8 rssi;
diff --git a/drivers/net/wireless/microchip/wilc1000/wlan_cfg.h b/drivers/net/wireless/microchip/wilc1000/wlan_cfg.h
index 614c5673f232..7038b74f8e8f 100644
--- a/drivers/net/wireless/microchip/wilc1000/wlan_cfg.h
+++ b/drivers/net/wireless/microchip/wilc1000/wlan_cfg.h
@@ -30,7 +30,7 @@ struct wilc_cfg_str {
struct wilc_cfg_str_vals {
u8 mac_address[7];
u8 firmware_version[129];
- u8 assoc_rsp[256];
+ u8 assoc_rsp[WILC_MAX_ASSOC_RESP_FRAME_SIZE];
};

struct wilc_cfg {
diff --git a/drivers/net/wireless/microchip/wilc1000/wlan_if.h b/drivers/net/wireless/microchip/wilc1000/wlan_if.h
index df2f5a63bdf6..254a046e3b1b 100644
--- a/drivers/net/wireless/microchip/wilc1000/wlan_if.h
+++ b/drivers/net/wireless/microchip/wilc1000/wlan_if.h
@@ -10,6 +10,8 @@
#include <linux/netdevice.h>
#include "fw.h"

+#define WILC_MAX_ASSOC_RESP_FRAME_SIZE 512
+
/********************************************
*
* Wlan Configuration ID
--
2.25.1

2023-05-11 12:57:26

by Kalle Valo

Subject: Re: [PATCH v2] wifi: wilc1000: Increase ASSOC response buffer

<[email protected]> wrote:

> From: Amisha Patel <[email protected]>
>
> In recent access points, information element is longer as they include
> additional data which exceeds 256 bytes. To accommodate longer
> association response, increase the ASSOC response buffer.
>
> Signed-off-by: Amisha Patel <[email protected]>

Patch applied to wireless-next.git, thanks.

7acd69507088 wifi: wilc1000: Increase ASSOC response buffer

--
https://patchwork.kernel.org/project/linux-wireless/patch/[email protected]/

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