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

v2.4 YOPAN-165 and YOPAN-168 #16912

Merged

Conversation

ankuns
Copy link
Contributor

@ankuns ankuns commented Aug 19, 2024

This PR brings in:

  • workarounds against anomaly 165 and 168 for the nRF5430 SoC (KRKNWK-19361)
  • network core monitor feature (NCSDK-28820)

@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 Aug 19, 2024
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Aug 19, 2024

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

Name Old Revision New Revision Diff
homekit nrfconnect/sdk-homekit@v2.4.3 https://github.com/nrfconnect/sdk-homekit/commit/b5d8e3ec3ad42b5789b94dbc4caf850c146df068 (v2.4-branch) nrfconnect/sdk-homekit@v2.4.3..b5d8e3ec
zephyr nrfconnect/sdk-zephyr@v3.3.99-ncs1-2 nrfconnect/sdk-zephyr@c0aa4d2 (v3.3.99-ncs1-branch) nrfconnect/sdk-zephyr@v3.3.99-ncs1-2..c0aa4d27

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

@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.

@ankuns
Copy link
Contributor Author

ankuns commented Aug 19, 2024

Note: the commit ppi_trace: add ppi_trace_dppi_ch_trace function necessary for verification

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Aug 19, 2024

Test specification

CI/Jenkins/NRF

  • Integration Platforms

CI/Jenkins/integration

Test Module File based changes Manually selected West overwrite
test-fw-nrfconnect-ble_samples X
test-fw-nrfconnect-chip X
test-fw-nrfconnect-nfc X
test-fw-nrfconnect-rs X
test-fw-nrfconnect-thread X
test-fw-nrfconnect-zigbee X
test-sdk-homekit X

test-sdk-homekit: added because there was no .github/test-spec.yml in 'homekit'

Detailed information of selected test modules

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

ahasztag and others added 2 commits August 20, 2024 07:21
After newly introduced changes the amount of RTC channels
used by the 802.15.4 driver is reduced to 2. This commit
updates this in samples configuration.

Signed-off-by: Artur Hadasz <artur.hadasz@nordicsemi.no>
(cherry picked from commit 308e731)
Add hook needed to RTC pretick.

Signed-off-by: Krzysztof Chruściński <krzysztof.chruscinski@nordicsemi.no>
ankuns and others added 6 commits August 20, 2024 14:30
This commit brings in Zephyr with workarounds for nRF5340 anomalies
165 and 168.

Signed-off-by: Andrzej Kuros <andrzej.kuros@nordicsemi.no>
For current ppi_trace_config it is impossible to trace events for
which PUBLISH register is overwritten in run-time after
ppi_trace_config is called. This function allows also to trace just
one event and does not allow to trace many-to-one topologies
which are possible with use of DPPIs.

To overcome above limitations ppi_trace_dppi_ch_trace is added.
In contrast to ppi_trace_config the function ppi_trace_dppi_ch_trace
focuses on tracing rather the DPPI channel leaving the preparation
of event network publishing to it to a user.

Signed-off-by: Andrzej Kuros <andrzej.kuros@nordicsemi.no>
(cherry picked from commit 503f8e4)
The function `z_nrf_rtc_timer_exact_set` is used in
`nrf_802154_platform_sl_lptimer_hw_task_prepare` to ensure
that moment of hardware timer task trigger is exactly as requested,
without possibility of CC channel adjustment due to late operation.

Signed-off-by: Andrzej Kuros <andrzej.kuros@nordicsemi.no>
(cherry picked from commit 0bb6241)
Instead of using an RTC1 channel for hardware tasks triggering
used by 802.15.4, an RTC0 channel is used. This frees an RTC1
CC channel, allowing it to be used by Zephyr for other purposes.

Signed-off-by: Artur Hadasz <artur.hadasz@nordicsemi.no>
(cherry picked from commit 9b18cb3)
The cherry-picked workaround against nRF53 anomaly 165, the "pretick"
is based on nrfx 3.0.0. The added code introduces missing definitions
to the soc.c file which are missing in nrfx 2.11.0.

Signed-off-by: Andrzej Kuros <andrzej.kuros@nordicsemi.no>
This commit brings in the homekit with build size optimizations.

Signed-off-by: Andrzej Kuros <andrzej.kuros@nordicsemi.no>
@ankuns ankuns force-pushed the KRKNWK-19361_v2.4-branch_yoda_fixes branch from 3861c55 to 54f02cd Compare August 20, 2024 12:32
@NordicBuilder NordicBuilder removed the DNM label Aug 20, 2024
The Network Core Monitor module provides network core status monitoring
by application core for the nRF5340 processor.
It allows detection of network processor suspensions and resets.
After a reset occurs, it is possible to read the cause of the reset.

Ref: NCSDK-22746

Signed-off-by: Marcin Jelinski <marcin.jelinski@nordicsemi.no>
(cherry picked from commit 30c05e3)
maje-emb and others added 3 commits August 21, 2024 11:19
The `__weak ncm_net_core_event_handler` function is used to notify
the application about the network core staus.
This repleace the periodic call of the
`ncm_net_status_check` function.
This makes it easier to use and configure.

Ref: NCSDK-23996

Signed-off-by: Marcin Jelinski <marcin.jelinski@nordicsemi.no>
(cherry picked from commit 9939469)
* Make sure all C++ C includes have an end.
* Correct order of ifdefs and doxygen so that they are in the correct
  order. The order is not aligned from file to file, though it is
  corrected per file.
* Added space after license in some headers and source files.

Signed-off-by: Eivind Jølsgard <eivind.jolsgard@nordicsemi.no>
(cherry picked from commit 6d8b9bb)
Let it have its own page in the menu.

Signed-off-by: Emanuele Di Santo <emdi@nordicsemi.no>
@maje-emb maje-emb requested a review from rlubos as a code owner August 21, 2024 11:00
@github-actions github-actions bot added the doc-required PR must not be merged without tech writer approval. label Aug 21, 2024
@gmarull gmarull merged commit 03e9aeb into nrfconnect:v2.4-branch Aug 22, 2024
19 checks passed
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-homekit manifest-zephyr
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants