Skip to content

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

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

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

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
@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 added 6 commits May 20, 2025 18:06
Pull fix issue of disconnections when RSSI is lower.

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

Signed-off-by: Chaitanya Tata <[email protected]>
The global variable was missing the guard causing twister failure for
nRF52.

Signed-off-by: Chaitanya Tata <[email protected]>
Add the downstream alternatives for hostap crypto and FW loading.

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

Signed-off-by: Chaitanya Tata <[email protected]>
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants