patrick/zephyr
1
0
Fork 0
Code Issues Pull requests Actions 8 Packages Projects Releases Wiki Activity

drivers: eswifi: Add option for maximum data size

Browse source 
Default MAX_DATA_SIZE might be too small depending
on application (e.g. handling the scan result with 12 APs detected)
Added config option for MAX_DATA_SIZE and warning for the user

Signed-off-by: Samy Francelet <samy.francelet@ik.me>
This commit is contained in:
Samy Francelet 2024-03-13 21:01:55 +01:00 committed by David Leach
parent cf362a018c
commit 4163ea0207
3 changed files with 16 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
drivers/wifi/eswifi/Kconfig.eswifi
View file

@ -30,6 +30,14 @@ config WIFI_ESWIFI_BUS_UART
endchoice
config WIFI_ESWIFI_MAX_DATA_SIZE
int "esWiFi message size"
default 1600
range 500 4000
help
This option sets the size of the esWiFi message buffer.
It can be increased to handle larger messages, like scan results.
config WIFI_ESWIFI_THREAD_PRIO
int "esWiFi threads priority"
default 2

4
drivers/wifi/eswifi/eswifi.h
View file

@ -16,8 +16,6 @@
#include "eswifi_offload.h"
#define MAX_DATA_SIZE 1600
#define AT_OK_STR "\r\nOK\r\n> "
#define AT_OK_STR_LEN 8
#define AT_RSP_DELIMITER "\r\n"
@ -71,7 +69,7 @@ struct eswifi_dev {
enum eswifi_request req;
enum eswifi_role role;
uint8_t mac[6];
char buf[MAX_DATA_SIZE];
char buf[CONFIG_WIFI_ESWIFI_MAX_DATA_SIZE];
struct k_mutex mutex;
atomic_val_t mutex_owner;
unsigned int mutex_depth;

7
drivers/wifi/eswifi/eswifi_core.c
View file

@ -141,6 +141,13 @@ int eswifi_at_cmd_rsp(struct eswifi_dev *eswifi, char *cmd, char **rsp)
return -EIO;
}
if (len >= CONFIG_WIFI_ESWIFI_MAX_DATA_SIZE) {
LOG_WRN("Buffer might be too small for response!");
LOG_WRN("Data length %d", len);
LOG_WRN("See CONFIG_WIFI_ESWIFI_MAX_DATA_SIZE (in build: %d)",
CONFIG_WIFI_ESWIFI_MAX_DATA_SIZE);
}
/*
* Check response, format should be "\r\n[DATA]\r\nOK\r\n>"
* Data is in arbitrary format (not only ASCII)