[nrf fromlist] modules: hal_nordic: error/exception handling in NRFS dispatcher #2118
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This reverts commit 01c1752.
This reverts commit 1351b7f.
… test_plan.py script" This reverts commit 70b2886.
This reverts commit 1404fdc.
This reverts commit 249a7ad.
This reverts commit d49407f.
This reverts commit 80f445f.
This reverts commit eac356d.
…support" This reverts commit 2dfa0f5.
This reverts commit ec3b301.
…r GRTC" This reverts commit d89bc2f.
…mpty gcda files" This reverts commit bc6eb9e.
Signed-off-by: Bjarki Arge Andreasen <[email protected]>
…a files In case of problem with parsing hex data from coverage dump, do not create empty gcda file. Such empty file will break gcovr parsing. Upstream PR: zephyrproject-rtos/zephyr#70297 Signed-off-by: Piotr Kosycarz <[email protected]> (cherry picked from commit 38cf979)
cpuppr is a small core with less than 64k of memory. In order to fit tests into that memory by default disable asserts (which saves few kB). Upstream PR: zephyrproject-rtos/zephyr#74297 Signed-off-by: Krzysztof Chruściński <[email protected]> (cherry picked from commit 68776ed)
Add dedicated interrupt lines for nrf54h20_cpuppr. Similar exception was already added to nrf54l15_flpr: 8742e24. Upstream PR: zephyrproject-rtos/zephyr#74297 Signed-off-by: Krzysztof Chruściński <[email protected]> (cherry picked from commit d074251)
cpuflpr has limited memory. In order to fit tests into that memory by default disable asserts (which saves few kB). Upstream PR: zephyrproject-rtos/zephyr#74297 Signed-off-by: Krzysztof Chruściński <[email protected]> (cherry picked from commit 90f792f)
This is implementation of ICBMsg endpoint deregistration. The underlying ICMsg instance and blocks are not affected by deregistration, so it is possible to reuse the same address during the re-registration of the same endpoint. Upstream PR: zephyrproject-rtos/zephyr#74243 Signed-off-by: Dominik Kilian <[email protected]> (cherry picked from commit 297b00d)
This commit sets `ZEPHYR_<MODULE_NAME>_KCONFIG` variable for each Kconfig file discovered in `nrf/modules/<module>/Kconfig`. This is not meant as a permanent solution; we should do more careful consideration on the optimal approach forward that will allow compliance_check.py to be used downstream with custom module_ext_roots, and at the same time keep current flexibility for module glue code handling intact. Signed-off-by: Torsten Rasmussen <[email protected]> Signed-off-by: Martí Bolívar <[email protected]> (cherry picked from commit 1b2d372)
Add scripts/quarantine.yaml file, which will be used in CI. Signed-off-by: Piotr Golyzniak <[email protected]> Signed-off-by: Andrzej Głąbek <[email protected]> Signed-off-by: Maciej Perkowski <[email protected]> Signed-off-by: Robert Lubos <[email protected]> (cherry picked from commit b9e3996)
This file is used for NCS-specific testing configuration based on modifications to files in this repository. Signed-off-by: Alperen Sener <[email protected]> Signed-off-by: Elisabeth Solheim Klakken <[email protected]> Signed-off-by: Mariusz Poslinski <[email protected]> Signed-off-by: Markus Swarowsky <[email protected]> Signed-off-by: Robert Lubos <[email protected]> Signed-off-by: Sebastian Wezel <[email protected]> Signed-off-by: Tomasz Tyzenhauz <[email protected]> Signed-off-by: Fredrik Ås <[email protected]> Signed-off-by: Michał Szablowski <[email protected]> Signed-off-by: Tony Le <[email protected]> Signed-off-by: Krishna T <[email protected]> Signed-off-by: Dawid Przybylo <[email protected]> Signed-off-by: Rubin Gerritsen <[email protected]> Signed-off-by: Jørgen Kvalvaag <[email protected]> Signed-off-by: Magne Værnes <[email protected]> Signed-off-by: Lang Xie <[email protected]> Signed-off-by: Alexander Svensen <[email protected]> Signed-off-by: Jan Gałda <[email protected]> Signed-off-by: Vladislav Litvinov <[email protected]> Signed-off-by: Guojun Wang <[email protected]> Signed-off-by: Piotr Kosycarz <[email protected]> Signed-off-by: Thomas Stilwell <[email protected]> (cherry picked from commit dc1e8c4) (cherry picked from commit 80d7115) (cherry picked from commit e7aa700) (cherry picked from commit ebd324c)
New Twister schema is not supported yet in sdk-zephyr, drop this patch once Twister is updated. Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit 3176db5)
… map" This reverts commit c37deeb. This is only a temporary change, until we align our CI. To be removed once natsort is avaialble in the NCS CI. Signed-off-by: Robert Lubos <[email protected]> (cherry picked from commit ea4c4bf)
Adds a variable to Kconfig which, when set, will disable showing child/parent Kconfig values in builds (enabled when Sysbuild is used) Signed-off-by: Jamie McCrae <[email protected]> (cherry picked from commit f3e6dd5)
Kconfig search is handled in a separate docset in NCS, so remove the page. This is a long-term noup patch. Signed-off-by: Gerard Marull-Paretas <[email protected]> Signed-off-by: Krishna T <[email protected]> (cherry picked from commit 9b45014)
-This allows configurations enabled by PSA_WANTS_ALG_XXXX to be used to control which TF-M module is enabled -If the TF-M image doesn't support e.g. the MAC APIs, then the MAC interface is not enabled Note: This functionality requires that nrf_security is enabled ref: NCSDK-11689 Make TF-M crypto module depend on PSA_WANT_GENERATE_RANDOM, same as all other crypto modules, which have PSA_HAS to group all PSA features that require the module. This makes TF-M by default exclude the RNG module when not needed. Signed-off-by: Frank Audun Kvamtrø <[email protected]> Signed-off-by: Joakim Andersson <[email protected]> (cherry picked from commit cdd7590)
Out-of-tree crypto subsystems need to deselect MBEDTLS_BUILTIN, but deselection is not supported. It is however supported to select a dependency in a ! expression. Signed-off-by: Sebastian Bøe <[email protected]> (cherry picked from commit 0028d0d)
We moved the header files in sdk-mbedtls from the library folder to the include/library folder. This was done to avoid issues when building MbedTLS with the nrf_security module and the Oberon PSA core. The Oberon PSA core provides a subset of these header files and since they are included with quotes we cannot have them in the same directory. This change make the needed adaptions in CMake for the applications that don't use nrf_security. Signed-off-by: Georgios Vasilakis <[email protected]> Signed-off-by: Markus Swarowsky <[email protected]> (cherry picked from commit 2ab7243)
Update Kconfig options for qualification: - Remove experimental on qualified feature. - Add experimental on unqualified feature. - BT_L2CAP_ECRED is not marked as experimental upstream and we qualify it downstream. Signed-off-by: Joakim Andersson <[email protected]> Signed-off-by: Trond Einar Snekvik <[email protected]> Signed-off-by: Martí Bolívar <[email protected]> Signed-off-by: Robert Lubos <[email protected]> Signed-off-by: Dominik Ermel <[email protected]> Signed-off-by: Ingar Kulbrandstad <[email protected]> Signed-off-by: Torsten Rasmussen <[email protected]> Signed-off-by: Herman Berget <[email protected]> Signed-off-by: Tomasz Moń <[email protected]> (cherry picked from commit 7c54cf3)
Connection manager L2 Wi-Fi calls internally invoke WPA supplicant calls which requires higher stack size. Update the default stack size for such cases. Marking this "nrf noup" since supplicant is not upstream yet. Signed-off-by: Ravi Dondaputi <[email protected]> (cherry picked from commit e46ed86)
- Added `cs-supported` property to nrf-radio devicetree - Added `HAS_HW_NRF_RADIO_CS` Kconfig option which is set if `cs-supported` property is enabled - Enabled `cs-supported` property for nrf54-series devices - Disabled `cs-supported` on nrf54l15bsim because it is not yet supported Signed-off-by: Ivan Iushkov <[email protected]> (cherry picked from commit e357814)
… is enabled Channel sounding frequently produces very large events, so the default of 68 bytes can lead to buffer overruns / memory corruption. Signed-off-by: Olivier Lesage <[email protected]> (cherry picked from commit 7538a98)
Adds following HCI support for CS configuration: - LE CS Create Config command - LE CS Remove Config command - LE CS Config Complete event Two callbacks have been added to notify the application when a new CS configuration is created or an existing CS configuration is removed. Signed-off-by: Burak Gorduk <[email protected]> (cherry picked from commit 7b82214)
This change adds bt_le_cs prefix to public CS API as well as adding le_cs prefix to CS related bt_conn callbacks. This is to ensure consistency within CS API while maintaining the separation between the Bluetooth LE and Classic features. Signed-off-by: Burak Gorduk <[email protected]> (cherry picked from commit 6e3c3f6)
…t results Adds support for: - LE CS Subevent Result event - LE CS Test End Complete event For now, recombination of subevent results with more steps than could fit within a single HCI event is not supported, and such events are discarded. Signed-off-by: Olivier Lesage <[email protected]> (cherry picked from commit a12b869)
Makes it easier to make sure the CS test code is removed if desired. It should hopefully allow for a clean split in general. Signed-off-by: Olivier Lesage <[email protected]> (cherry picked from commit 9e3943d)
…config Include EXPERIMENTAL tag in the kconfig text and remove the word "support" to align with other options Signed-off-by: Olivier Lesage <[email protected]> (cherry picked from commit 9ae2e23)
Due to the possibility of simultaneous accesess to LRCCONF registers, additional management is required. Upstream PR: zephyrproject-rtos/zephyr#79067 Signed-off-by: Adam Kondraciuk <[email protected]>
Add `idle` and `s2ram` power states for nRF54H20 cpuapp and cpurad. Also the substate `idle_cache_disable` added. Upstream PR: zephyrproject-rtos/zephyr#79067 Signed-off-by: Adam Kondraciuk <[email protected]>
Added the SMP service and characteristic UUIDs to the smp_bt module header and used those definitions. Signed-off-by: Mateusz Kapala <[email protected]> (cherry picked from commit 7755589)
Add node with STMESP registers. Signed-off-by: Krzysztof Chruściński <[email protected]> Signed-off-by: Sebastian Głąb <[email protected]> (cherry picked from commit 8e22222)
…not needed Currently we can assume that flpr and ppr cores are always owned by cpuapp. In that case, cpuapp (which processes ETR buffer content) has access to the memory where logging strings from ppr and flpr are located. Those strings do not need to be appended to the message which is written to the STM (in case of standalone configuration). This approach reduced logging time, e.g. logging a message with one argument takes 1.8us on cpuapp and 6.3us on cpurad. This setting applies only to the case when cpuapp is the owner of those co-oprocessors and shall be updated if that becomes configurable but currently there are no means to detect core owner. Signed-off-by: Krzysztof Chruściński <[email protected]> Signed-off-by: Sebastian Głąb <[email protected]> (cherry picked from commit c2454f2)
Add interrupt locking during test to ensure that main thread is not interrupts which would impact the test. Add longer sleep time between tests to ensure that all logs are processed on time and not dropped. Signed-off-by: Krzysztof Chruściński <[email protected]> (cherry picked from commit c61e92a)
…pr support Add flpr and ppr core to the sample. Since there are only 2 UARTs available on DK, local UART configuration is using only cpuapp and cpurad. Signed-off-by: Krzysztof Chruściński <[email protected]> (cherry picked from commit d11997d)
Asserts are by default enabled for tests but flpr and ppr are small cores (<64k) and many tests does not fit in memory with asserts enabled. Signed-off-by: Krzysztof Chruściński <[email protected]> Signed-off-by: Sebastian Głąb <[email protected]> (cherry picked from commit 3786b61)
…ous API UART asynchronous API will ensure highest throughput but it is not necessary as polling API is also supported as a fallback. Do not enforce use of asynchronous API as there might be cases when polling might be used (e.g. to reduce memory footprint). Signed-off-by: Krzysztof Chruściński <[email protected]> Signed-off-by: Sebastian Głąb <[email protected]> (cherry picked from commit c6f4e71)
…nous API Asynchronous API is recommended to be used by ETR buffer handler but polling is also supported. Imply UART_ASYNC_API so that it is possible to disable it in project config and fall back to polling if needed. For example to reduce memory footprint. Signed-off-by: Krzysztof Chruściński <[email protected]> Signed-off-by: Sebastian Głąb <[email protected]> (cherry picked from commit 2423312)
|
The following west manifest projects have been modified in this Pull Request:
Note: This message is automatically posted and updated by the Manifest GitHub Action. |
rlubos
requested review from
anangl,
carlescufi,
de-nordic,
nordic-krch and
a team
as code owners
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
zephyrproject-rtos/zephyr#79922