Skip to content

v3.0.2: wifi: Fix disconnections with low RSSI #22382

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
May 20, 2025
Merged

v3.0.2: wifi: Fix disconnections with low RSSI #22382

merged 6 commits into from
May 20, 2025

Conversation

krish2718
Copy link
Contributor

Fixes a regression in v3.0.0

@krish2718 krish2718 requested review from a team as code owners May 16, 2025 14:26
@github-actions github-actions bot added manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels May 16, 2025
@krish2718 krish2718 changed the title wifi: Fix low rssi disc 30 v3.0.2: wifi: Fix disconnections with low RSSI May 16, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented May 16, 2025 •
edited
Loading

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
nrfxlib nrfconnect/[email protected] nrfconnect/sdk-nrfxlib@3a14dbc (v3.0-branch) nrfconnect/[email protected]
zephyr nrfconnect/[email protected] nrfconnect/sdk-zephyr@114d1da (v4.0.99-ncs1-branch) nrfconnect/[email protected]

✅ All manifest checks OK

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

@NordicBuilder
Copy link
Contributor

NordicBuilder commented May 16, 2025 •
edited
Loading

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 6

Inputs:

Sources:

sdk-nrf: PR head: 822203e3f855808488e84f77e4110aaad2955923
nrfxlib: PR head: 3a14dbc326c385a0161fc122f72b6d9be308f7d6
zephyr: PR head: 114d1da9fd7ad016ef45e8fee8f1b512b93841ce

more details

sdk-nrf:

PR head: 822203e3f855808488e84f77e4110aaad2955923
merge base: 799a0711807626887eab8b72e31f308ecacd415a
target head (v3.0-branch): 799a0711807626887eab8b72e31f308ecacd415a
Diff

nrfxlib:

PR head: 3a14dbc326c385a0161fc122f72b6d9be308f7d6
merge base: f953fbcc0e7a871815b32a687b21c1062fe62d45
Diff

zephyr:

PR head: 114d1da9fd7ad016ef45e8fee8f1b512b93841ce
merge base: 77f865b8f8d0cb3d19002bfe713e9dd46e6f71b7
Diff

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (14) 
.github
│  │ test-spec.yml
nrfxlib
│  ├── nrf_wifi
│  │  ├── bin
│  │  │  ├── ncs
│  │  │  │  ├── default
│  │  │  │  │  │ nrf70.bin
│  │  │  │  ├── offloaded_raw_tx
│  │  │  │  │  │ nrf70.bin
│  │  │  │  ├── radio_test
│  │  │  │  │  │ nrf70.bin
│  │  │  │  ├── scan_only
│  │  │  │  │  │ nrf70.bin
│  │  │  │  ├── system_with_raw
│  │  │  │  │  │ nrf70.bin
samples
│  ├── net
│  │  ├── aws_iot
│  │  │  ├── boards
│  │  │  │  │ nrf7002dk_nrf5340_cpuapp_ns.conf
│  │  ├── azure_iot_hub
│  │  │  ├── boards
│  │  │  │  │ nrf7002dk_nrf5340_cpuapp_ns.conf
scripts
│  ├── ci
│  │  │ tags.yaml
subsys
│  ├── net
│  │  ├── lib
│  │  │  ├── nrf70_fw_ext
│  │  │  │  │ ext_fw_load.c
west.yml
zephyr
│  ├── boards
│  │  ├── nordic
│  │  │  ├── nrf7002dk
│  │  │  │  │ nrf5340_cpuapp_common_pinctrl.dtsi
│  │  ├── shields
│  │  │  ├── nrf7002ek
│  │  │  │  ├── boards
│  │  │  │  │  │ nrf5340dk_nrf5340_cpuapp.overlay
│  │ west.yml

Outputs:

Toolchain

Version: 7cbc0036f4
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:7cbc0036f4_8bf7ca4353

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 7
    • sdk-zephyr test count: 538
  • ✅ Integration tests
    • ✅ test-fw-nrfconnect-chip
    • ✅ test-fw-nrfconnect-nrf-iot_thingy91
    • ✅ test-fw-nrfconnect-rs
    • ✅ test-fw-nrfconnect-fem
    • ✅ test-sdk-find-my
    • ✅ test-sdk-wifi
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-boot
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_cloud
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_mosh
    • test-fw-nrfconnect-nrf-iot_positioning
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-proprietary_esb
    • test-fw-nrfconnect-ps
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-secdom-samples-public

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

@github-actions GitHub Actions
Copy link

You can find the documentation preview for this PR here.

@krish2718 krish2718 requested review from a team, PerMac, katgiadla and nordic-piks as code owners May 18, 2025 08:04
@krish2718 krish2718 mentioned this pull request May 18, 2025
Closed
@github-actions github-actions bot removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label May 18, 2025
@krish2718 krish2718 mentioned this pull request May 19, 2025
Closed
krish2718 added 6 commits May 20, 2025 18:06
@krish2718
manifest: sdk-zephyr: Pull fix for low RSSI disconnections
c4cc049 
Pull fix issue of disconnections when RSSI is lower.

Signed-off-by: Chaitanya Tata <[email protected]>
@krish2718
manifest: sdk-nrfxlib: Pull latest FW bins
ec1920d 
Pull latest FW bins that fix disconnection issue at low RSSI levels.

Signed-off-by: Chaitanya Tata <[email protected]>
@krish2718
net: lib: nrf70_fw_ext: Fix build error for nRF52
51b0a62 
The global variable was missing the guard causing twister failure for
nRF52.

Signed-off-by: Chaitanya Tata <[email protected]>
@krish2718
ci: wifi: Fix missing test spec entries
38828c7 
Add the downstream alternatives for hostap crypto and FW loading.

Signed-off-by: Chaitanya Tata <[email protected]>
@krish2718
ci: Fix Wi-Fi file list
f29a5de 
Due to re-org over multiple releases Wi-Fi files paths have changed, fix
those entries.

Signed-off-by: Chaitanya Tata <[email protected]>
@krish2718
samples: net: Enable LTO to fix flash overflow
822203e 
Due to recent upmerge we are seeing flash overflow in aws_iot and
azure_iot_hub samples, so, enable LTO to fix the build.

Signed-off-by: Chaitanya Tata <[email protected]>
@krish2718 krish2718 force-pushed the fix_low_rssi_disc_30 branch from 1146ade to 822203e Compare May 20, 2025 12:37
@NordicBuilder NordicBuilder removed the DNM label May 20, 2025
@krish2718
Copy link
Contributor Author

please review @nrfconnect/ncs-cia @katgiadla @PerMac @nrfconnect/ncs-code-owners

@carlescufi carlescufi merged commit e6400a6 into nrfconnect:v3.0-branch May 20, 2025
13 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bama-nordic bama-nordic bama-nordic approved these changes

@eivindj-nordic eivindj-nordic eivindj-nordic approved these changes

@nordic-piks nordic-piks nordic-piks approved these changes

@sachinthegreen sachinthegreen sachinthegreen approved these changes

@karun2796 karun2796 karun2796 approved these changes

@rajb9 rajb9 rajb9 approved these changes

@PerMac PerMac Awaiting requested review from PerMac PerMac is a code owner

@katgiadla katgiadla Awaiting requested review from katgiadla katgiadla is a code owner

Assignees

No one assigned

Labels

manifest manifest-nrfxlib manifest-zephyr

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

9 participants

@krish2718 @NordicBuilder @bama-nordic @eivindj-nordic @nordic-piks @sachinthegreen @karun2796 @rajb9 @carlescufi