Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

drivers: wifi: nrfwifi: Enable variable network configurations #78852

Closed

Conversation

D-Triveni
Copy link

Add a name to the existing Kconfig choice for better clarity. Add variable network configuration control into the driver, removing the need for per-sample settings.

sachinthegreen
sachinthegreen previously approved these changes Sep 25, 2024
jukkar
jukkar previously approved these changes Sep 26, 2024
@rlubos
Copy link
Contributor

rlubos commented Sep 26, 2024

It seems that nucleo_h723zg + shield configuration was already on the edge when it comes to RAM (99.53% on main), with the massive net buf space it now overflows. Not sure if it's better to keep the defaults smaller, or choose different platform for testing.

sachinthegreen
sachinthegreen previously approved these changes Oct 8, 2024
sachinthegreen
sachinthegreen previously approved these changes Oct 17, 2024
jukkar
jukkar previously approved these changes Oct 17, 2024
krish2718 pushed a commit to krish2718/sdk-zephyr that referenced this pull request Oct 22, 2024
…leo_h723zg target

Enable fixed data buffers for nucleo_h723zg target to prevent RAM overflows
with variable network buffers usage.

Upstream PR: zephyrproject-rtos/zephyr#78852

Signed-off-by: Triveni Danda <[email protected]>
Update kconfig choice name to enable overriding
from other parts of the code.

Signed-off-by: Triveni Danda <[email protected]>
@krish2718 krish2718 dismissed stale reviews from sachinthegreen and themself via 87bef88 October 23, 2024 13:55
@krish2718
Copy link
Collaborator

Remove adding another abstraction over networking stack, and also removed overrides for highest throughput as they impact all samples/applications RAM usage.

Add variable network configuration control into the driver to optimize
RAM usage by default. The pool sizes are default, and tuning is left to
the applications or samples.

Signed-off-by: Triveni Danda <[email protected]>
krish2718 pushed a commit to krish2718/sdk-zephyr that referenced this pull request Oct 23, 2024
Update kconfig choice name to enable overriding
from other parts of the code.

Upstream PR: zephyrproject-rtos/zephyr#78852

Signed-off-by: Triveni Danda <[email protected]>
krish2718 pushed a commit to krish2718/sdk-zephyr that referenced this pull request Oct 23, 2024
…urations

Add variable network configuration control into the driver to optimize
RAM usage by default. The pool sizes are default, and tuning is left to
the applications or samples.

Upstream PR: zephyrproject-rtos/zephyr#78852

Signed-off-by: Triveni Danda <[email protected]>
krish2718 pushed a commit to krish2718/sdk-zephyr that referenced this pull request Oct 23, 2024
Update kconfig choice name to enable overriding
from other parts of the code.

Upstream PR: zephyrproject-rtos/zephyr#78852

Signed-off-by: Triveni Danda <[email protected]>
krish2718 pushed a commit to krish2718/sdk-zephyr that referenced this pull request Oct 23, 2024
…urations

Add variable network configuration control into the driver to optimize
RAM usage by default. The pool sizes are default, and tuning is left to
the applications or samples.

Upstream PR: zephyrproject-rtos/zephyr#78852

Signed-off-by: Triveni Danda <[email protected]>
krish2718 pushed a commit to krish2718/sdk-zephyr that referenced this pull request Oct 23, 2024
Update kconfig choice name to enable overriding
from other parts of the code.

Upstream PR: zephyrproject-rtos/zephyr#78852

Signed-off-by: Triveni Danda <[email protected]>
krish2718 pushed a commit to krish2718/sdk-zephyr that referenced this pull request Oct 23, 2024
…urations

Add variable network configuration control into the driver to optimize
RAM usage by default. The pool sizes are default, and tuning is left to
the applications or samples.

Upstream PR: zephyrproject-rtos/zephyr#78852

Signed-off-by: Triveni Danda <[email protected]>
krish2718 pushed a commit to krish2718/sdk-zephyr that referenced this pull request Oct 24, 2024
Update kconfig choice name to enable overriding
from other parts of the code.

Upstream PR: zephyrproject-rtos/zephyr#78852

Signed-off-by: Triveni Danda <[email protected]>
krish2718 pushed a commit to krish2718/sdk-zephyr that referenced this pull request Oct 24, 2024
…urations

Add variable network configuration control into the driver to optimize
RAM usage by default. The pool sizes are default, and tuning is left to
the applications or samples.

Upstream PR: zephyrproject-rtos/zephyr#78852

Signed-off-by: Triveni Danda <[email protected]>
krish2718 pushed a commit to krish2718/sdk-zephyr that referenced this pull request Oct 24, 2024
Update kconfig choice name to enable overriding
from other parts of the code.

Upstream PR: zephyrproject-rtos/zephyr#78852

Signed-off-by: Triveni Danda <[email protected]>
krish2718 pushed a commit to krish2718/sdk-zephyr that referenced this pull request Oct 24, 2024
…urations

Add variable network configuration control into the driver to optimize
RAM usage by default. The pool sizes are default, and tuning is left to
the applications or samples.

Upstream PR: zephyrproject-rtos/zephyr#78852

Signed-off-by: Triveni Danda <[email protected]>
rlubos pushed a commit to krish2718/sdk-zephyr that referenced this pull request Oct 25, 2024
Update kconfig choice name to enable overriding
from other parts of the code.

Upstream PR: zephyrproject-rtos/zephyr#78852

Signed-off-by: Triveni Danda <[email protected]>
rlubos pushed a commit to krish2718/sdk-zephyr that referenced this pull request Oct 25, 2024
…urations

Add variable network configuration control into the driver to optimize
RAM usage by default. The pool sizes are default, and tuning is left to
the applications or samples.

Upstream PR: zephyrproject-rtos/zephyr#78852

Signed-off-by: Triveni Danda <[email protected]>
rlubos pushed a commit to nrfconnect/sdk-zephyr that referenced this pull request Oct 25, 2024
Update kconfig choice name to enable overriding
from other parts of the code.

Upstream PR: zephyrproject-rtos/zephyr#78852

Signed-off-by: Triveni Danda <[email protected]>
rlubos pushed a commit to nrfconnect/sdk-zephyr that referenced this pull request Oct 25, 2024
…urations

Add variable network configuration control into the driver to optimize
RAM usage by default. The pool sizes are default, and tuning is left to
the applications or samples.

Upstream PR: zephyrproject-rtos/zephyr#78852

Signed-off-by: Triveni Danda <[email protected]>
NordicBuilder pushed a commit to nrfconnect/sdk-zephyr that referenced this pull request Oct 25, 2024
Update kconfig choice name to enable overriding
from other parts of the code.

Upstream PR: zephyrproject-rtos/zephyr#78852

Signed-off-by: Triveni Danda <[email protected]>
(cherry picked from commit cbcb2db)
NordicBuilder pushed a commit to nrfconnect/sdk-zephyr that referenced this pull request Oct 25, 2024
…urations

Add variable network configuration control into the driver to optimize
RAM usage by default. The pool sizes are default, and tuning is left to
the applications or samples.

Upstream PR: zephyrproject-rtos/zephyr#78852

Signed-off-by: Triveni Danda <[email protected]>
(cherry picked from commit b58ca22)
rlubos pushed a commit to nrfconnect/sdk-zephyr that referenced this pull request Oct 25, 2024
Update kconfig choice name to enable overriding
from other parts of the code.

Upstream PR: zephyrproject-rtos/zephyr#78852

Signed-off-by: Triveni Danda <[email protected]>
(cherry picked from commit cbcb2db)
rlubos pushed a commit to nrfconnect/sdk-zephyr that referenced this pull request Oct 25, 2024
…urations

Add variable network configuration control into the driver to optimize
RAM usage by default. The pool sizes are default, and tuning is left to
the applications or samples.

Upstream PR: zephyrproject-rtos/zephyr#78852

Signed-off-by: Triveni Danda <[email protected]>
(cherry picked from commit b58ca22)
@kartben
Copy link
Collaborator

kartben commented Nov 26, 2024

@D-Triveni this needs to be rebased please

@D-Triveni
Copy link
Author

@kartben These changes were pulled into another PR and merged, so, this one is being closed.

@D-Triveni D-Triveni closed this Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants