Skip to content

Commit

Permalink
modules: hostap: Update POSIX_MAX_FDS default
Browse files Browse the repository at this point in the history
Since hostap Kconfig move to sdk-nrf repository, the default for
POSIX_MAX_FDS set in the supplicant's Kconfig takes precedence over the
default set in sdk-nrf originally. This causes issues with Zephyr
samples running with default setting for this config, as supplicant sets
this config very tight.

I propose to increase the default value set by supplicant, to leave a
few FDS entries for the applications to use. Otherwise, every single
socket-based sample will need to override this config in its prj.conf
file.

Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
  • Loading branch information
rlubos committed Aug 7, 2023
1 parent 112f0bb commit 7a215cc
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
3 changes: 0 additions & 3 deletions drivers/wifi/nrf700x/Kconfig.nrf700x
Original file line number Diff line number Diff line change
Expand Up @@ -311,9 +311,6 @@ config HEAP_MEM_POOL_SIZE
config SYSTEM_WORKQUEUE_STACK_SIZE
default 4096

config POSIX_MAX_FDS
default 16

config NET_TX_STACK_SIZE
default 4096

Expand Down
3 changes: 2 additions & 1 deletion modules/hostap/Kconfig
Original file line number Diff line number Diff line change
Expand Up @@ -121,7 +121,8 @@ config POSIX_MAX_FDS
# l2_packet - 1
# ctrl_iface - 2 * socketpairs = 4(local and global)
# z_wpa_event_sock - 1 socketpair = 2
default 7 if !POSIX_API
# Remaining left for the applications running in default configuration
default 16 if !POSIX_API

config BSS_MAX_IDLE_TIME
int "BSS max idle timeout in seconds"
Expand Down

0 comments on commit 7a215cc

Please sign in to comment.