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

manifest: sdk-zephyr: Pull Wi-Fi shell improvements #13504

Merged
merged 6 commits into from
Jan 4, 2024

Conversation

krish2718
Copy link
Contributor

@krish2718 krish2718 commented Dec 22, 2023

Mainly in help and argument count fixes.

test-sdk-nrf: sdk-nrf-pr-13504

@github-actions github-actions bot added changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. manifest labels Dec 22, 2023
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Dec 22, 2023

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@4262130 nrfconnect/sdk-zephyr@72453d6 (main) nrfconnect/[email protected]

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Dec 22, 2023

Test specification

CI/Jenkins/NRF

  • Integration Platforms

CI/Jenkins/integration

Test Module File based changes Manually selected West overwrite
desktop52_verification X
test-fw-nrfconnect-chip X
test-fw-nrfconnect-fem X
test-fw-nrfconnect-nrf-iot_cloud X
test-fw-nrfconnect-nrf-iot_mosh X
test-fw-nrfconnect-nrf-iot_positioning X
test-fw-nrfconnect-nrf-iot_samples X
test-fw-nrfconnect-nrf-iot_thingy91 X
test-fw-nrfconnect-nrf-iot_zephyr_lwm2m X
test-fw-nrfconnect-rpc X
test-fw-nrfconnect-rs X
test-fw-nrfconnect-thread X
test-sdk-find-my X
test-sdk-wifi X

Detailed information of selected test modules

Note: This message is automatically posted and updated by the CI

@NordicBuilder
Copy link
Contributor

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

Note: This comment is automatically posted by the Documentation Publishing GitHub Action.

@github-actions github-actions bot added the doc-required PR must not be merged without tech writer approval. label Dec 22, 2023
krish2718 and others added 6 commits January 4, 2024 15:29
Mainly in help and argument count fixes.

Signed-off-by: Chaitanya Tata <[email protected]>
Write 0s to scan_info after allocation to prevent reading of stale
data.

Signed-off-by: Ravi Dondaputi <[email protected]>
Based on the modifications done to channel field in scan params,
change the way channel info is read into scan_info.
For reading SSIDs from scan params, add a NULL check along with the
existing strlen check, since strlen may return a non-zero value
even for NULL pointer.

Signed-off-by: Ravi Dondaputi <[email protected]>
List out the Kconfig options related to scan operations, that were
removed from zephyr upstream.

Signed-off-by: Ravi Dondaputi <[email protected]>
WIFI_MGMT_SCAN_MAX_BSS_CNT has been removed from upstream zephyr.
Moving the definition to Wi-Fi driver to fix build issues seen with
the sample using this Kconfig option.

Signed-off-by: Ravi Dondaputi <[email protected]>
As part of upstream zephyr changes, a couple of Konfig options have
been removed and need to be defined locally by the sample.

Signed-off-by: Ravi Dondaputi <[email protected]>
@rlubos rlubos merged commit 60f3b98 into nrfconnect:main Jan 4, 2024
27 checks passed
@krish2718 krish2718 deleted the wifi_shell_improvs branch January 4, 2024 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. doc-required PR must not be merged without tech writer approval. manifest manifest-zephyr
Projects
None yet
Development

Successfully merging this pull request may close these issues.