[nrf fromlist] modules: hal_nordic: nrfs: DVFS cleanup #2528
Conversation
This file contains hidden or 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
… ppr tests" This reverts commit 7851352. Signed-off-by: Andrzej Głąbek <[email protected]>
…mpty gcda files" This reverts commit 59cb93d. Signed-off-by: Andrzej Głąbek <[email protected]>
Signed-off-by: Andrzej Głąbek <[email protected]>
…n dts.cmake Instead of hardcoding the name `devicetree_target`, we can update the generated `dts.cmake` to accept `DEVICETREE_TARGET` as input variable. This will become useful in multi-image builds, where we would like to process multiple devicetrees. Upstream PR #: 73903 Signed-off-by: Grzegorz Swiderski <[email protected]> (cherry picked from commit 4cd36a9)
Add new functions to `sysbuild_extensions.cmake`, which will mirror the familiar dt_* API from Zephyr `extensions.cmake`. For example: dt_nodelabel(<var> NODELABEL <label>) gets the following sysbuild counterpart, with one extra argument: sysbuild_dt_nodelabel(<var> IMAGE <image> NODELABEL <label>) This API allows sysbuild to retrieve devicetree information for a given <image>, only after its respective `ExternalZephyrProject_Cmake()` call. This works by importing the generated `dts.cmake` files from each image's build directory, and creating multiple CMake targets to hold the generated properties - much like how the `CMakeCache.txt` and `.config` are also imported to be used by the related `sysbuild_get()` function. The dt_* API itself also has to be updated, in order to read properties from a variable `DEVICETREE_TARGET` set in the parent scope. Upstream PR #: 73903 Signed-off-by: Grzegorz Swiderski <[email protected]> (cherry picked from commit 38fdd95)
…orage) This adds the initial backend support for the ZMS storage system. Upstream PR #: 78632 Signed-off-by: Riadh Ghaddab <[email protected]> (cherry picked from commit 8ed9b80)
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 df94528)
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 980e3ca)
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]> Signed-off-by: Krzysztof Szromek <[email protected]> Signed-off-by: Grzegorz Chwierut <[email protected]> (cherry picked from commit 98762a9)
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 d1ded33)
… 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 374374b)
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 3fe805d)
-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 676d548)
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 5a62fe9)
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 801154a)
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 e92e87d)
The commits adds support for generating flash disks from Partition Manager defined partitions. Signed-off-by: Dominik Ermel <[email protected]> (cherry picked from commit 1601992)
The "zephyr-code-partition" chosen DTS node cannot be used when build uses Partition Manager. In that case, mcumgr must rely on the definitions provided by the Partition Manager. Jira: NCSDK-21381 Signed-off-by: Marek Pieta <[email protected]> Signed-off-by: Dominik Ermel <[email protected]> (cherry picked from commit 763a188)
The commit adds bootutil hook, for nrf5340, to allow it handling the non-accessible image-1/primary slot. Signed-off-by: Andrzej Głąbek <[email protected]> Signed-off-by: Vinayak Kariappa Chettimada <[email protected]> Signed-off-by: Johann Fischer <[email protected]> Signed-off-by: Dominik Ermel <[email protected]> Signed-off-by: Jamie McCrae <[email protected]> (cherry picked from commit 7b33eaf)
Move it inside ifdef. Signed-off-by: Dominik Ermel <[email protected]> (cherry picked from commit 44b6e2c)
Migrates child image configuration for this sample over to sysbuild Signed-off-by: Jamie McCrae <[email protected]> (cherry picked from commit beecaaa)
Migrates child image configuration for this sample over to sysbuild Signed-off-by: Jamie McCrae <[email protected]> (cherry picked from commit 6199743)
This is a follow-up to commit 9dd570f. Since in NCS, unlike in vanilla Zephyr, the nano variant of newlib is the default one, restore entries that disable the nano variant in one sample and one test that require the full newlib variant. This patch is supposed to be removed when picolibc becomes the default. Signed-off-by: Andrzej Głąbek <[email protected]> Signed-off-by: Dominik Ermel <[email protected]> (cherry picked from commit cb05ba5)
Disables partition manager when building some samples and tests which use sysbuild to prevent build issues Signed-off-by: Jamie McCrae <[email protected]> (cherry picked from commit cd344ab)
Add a Kconfig for th TFM_CRYPTO_PAKE_MODULE_ENABLED to support the PAKE APIs. noup as the PAKE support including the PAKE module doesn't exist yet in upstream TF-M as they depend on mbed TLS support for it Ref: NCSDK-22416 Signed-off-by: Markus Swarowsky <[email protected]> (cherry picked from commit eb58994)
Friend's replies on LPN's polls do not assume randomization in advertiser. Zero randomization will help to optimize time when LPN keeps receiving window open and save power. Signed-off-by: Aleksandr Khromykh <[email protected]> (cherry picked from commit 6d82661)
Fixes issue where randomness can be removed for advertising sets that have to handle other adv types than the BT_MESH_FRIEND_ADV tag type. Signed-off-by: Anders Storrø <[email protected]> Signed-off-by: Aleksandr Khromykh <[email protected]> Signed-off-by: Dominik Ermel <[email protected]> (cherry picked from commit 8709818)
…N_MANAGER Active partition ID need to be extracted basing on PARTITION_MANAGER products. ref.:NCSDK-26693 Signed-off-by: Andrzej Puzdrowski <[email protected]> Signed-off-by: Jamie McCrae <[email protected]> (cherry picked from commit a7b53d1) (cherry picked from commit d99f13e)
Using a comment to explain Kconfig options make them invisible to Kconfig search. Use help instead. Signed-off-by: Sigurd Hellesvik <[email protected]> (cherry picked from commit 2bff40a)
Added overlay with m25r64 on-DK memory selecton. This aim to support testing DFU with secondary partition moved to the external flash. Signed-off-by: Andrzej Puzdrowski <[email protected]> (cherry picked from commit f8b30ba)
…YNTH When using synthesized low frequency clock, HFXO should be running to ensure correct frequency. Signed-off-by: Michał Stasiak <[email protected]> (cherry picked from commit 9a47667)
…pported peripherals Enable Twister tests of PWM driver on nrf54l20pdk. Overlays were already added to PWM tests. Signed-off-by: Sebastian Głąb <[email protected]> (cherry picked from commit ecd53f9)
Define PWM LED at nRF54L20pdk board definition. Set status=okay for pwm20 node. Add pincontrol for pwm20:OUT0 at P1.07 (LED1). Add pwm_led1 node. Add alias pwm-led0. Signed-off-by: Sebastian Głąb <[email protected]> (cherry picked from commit 23e6854)
All needed configuration was moved to the board definition. Signed-off-by: Sebastian Głąb <[email protected]> (cherry picked from commit 8f39cea)
… and nRF54L09 PDKs Add support for nRF54L20 and nRF54L09 PDKs in temp_sensor twister tests. Signed-off-by: Michał Stasiak <[email protected]> (cherry picked from commit 2a642f1)
…4L09 PDK. Added further support for nRF54L09 PDK in watchdog twister tests. Signed-off-by: Michał Stasiak <[email protected]> (cherry picked from commit a0303d3)
Added support for nRF54L20 PDK in SPIM twister tests. Signed-off-by: Michał Stasiak <[email protected]> (cherry picked from commit 04df6de)
Added support for nRF54L20 PDK in tests involving multiple instances. Corrected pin assignements in spi_error_cases test. Upstream PR #: 84672 Signed-off-by: Michał Stasiak <[email protected]>
Added support for nRF54L09 PDK in SPIM twister tests. Signed-off-by: Michał Stasiak <[email protected]> (cherry picked from commit ccd2fd6)
Extend current adc gains with new entry 2/7. Signed-off-by: Karol Lasończyk <[email protected]> Signed-off-by: Sebastian Głąb <[email protected]> (cherry picked from commit 057de45)
Added the test configuration for nRF54L20 for the following tests: - adc_api - adc_error_cases Signed-off-by: Rafał Kuźnia <[email protected]> Signed-off-by: Sebastian Głąb <[email protected]> (cherry picked from commit f78742f)
Select Reference voltage of 900mV for ADC on nRF54L20pdk. Add ADC to the list of supported peripherals. Signed-off-by: Sebastian Głąb <[email protected]> (cherry picked from commit 99c3988)
…mple Adds a configuration to this sample that is optimised for testing/usage with the nRF5340 which enables the Bluetooth transport, enables most management groups, has a static partition manager file with secondary partitions on QSPI flash, configures MCUboot in overwrite only mode with network core update support Signed-off-by: Jamie McCrae <[email protected]>
Adds support for prefixing files with the target of bl2 or tfm Signed-off-by: Jamie McCrae <[email protected]> (cherry picked from commit f804469)
Add Nordic platforms and rename the defines to avoid redefining a nrf driver internal symbols Signed-off-by: Piotr Krzyzanowski <[email protected]> (cherry picked from commit 09d5b9d)
Improve test stability in 54l15 case by measuring duty cycle after two periods Signed-off-by: Piotr Krzyzanowski <[email protected]> (cherry picked from commit cb7690a)
…ration for nrf54h20 Extend CONFIG_SKIP_EDGE_NUM. Signed-off-by: Piotr Kosycarz <[email protected]> (cherry picked from commit e3e4548)
…BT_HCI_RAW In zephyrproject-rtos/zephyr#84268 the ability to use the controller for ECDH was removed from the host. This means that BT_CTLR_ECDH is now only useful when using BT_HCI_RAW. Signed-off-by: Sean Madigan <[email protected]> (cherry picked from commit 706938d)
Recently I have had to debug issues with PSA and having the returns values from PSA is very useful in order to find the root cause of the issue. Signed-off-by: Sean Madigan <[email protected]> (cherry picked from commit 8cff70a)
…baudrate enum Add faster baudrates (2M, 4M and 8M). Signed-off-by: Krzysztof Chruściński <[email protected]> (cherry picked from commit e737895)
Rename UARTE_ANY_FAST to UARTE_ANY_FAST_PD. There are 2 types of 'fast' UARTE instances. In nrf54h20 instance uart120 is in fast power domain that requires additional power and clock management of that domain. In nrf54lx fast uart00 instance does not require that. Add _PD to indicate fast power domain. Signed-off-by: Krzysztof Chruściński <[email protected]> (cherry picked from commit 1573be5)
…udrates nrf54x devices have UARTE instance capable of using baudrate higher than 1M. Higher baudrates does not have predefined values for BAUDRATE register. A formula can be used to calculate BAUDRATE value that shall be used for desired baudrate. Add UARTE_ANY_HIGH_SPEED macro which is set when high speed is enabled (uart00 in nrf54lx or uart120 in nrf54h20). For high speed instance use formula for getting value that shall be written to BAUDRATE register. When runtime configuration is not used then same formula is used to calculate fixed BAUDRATE value. Signed-off-by: Krzysztof Chruściński <[email protected]> (cherry picked from commit 972ec82)
Fast instance in nrf54h (uart120) can generate a spurious RXTO event some time after RXTO event that indicates that RX path is disabled. The time when event is generated depends on baudrate and when slower baudrates are used peripheral is disabled on time to not notice it in the test but with higher baudates issue become visible. In order to avoid spurious interrupt, RXTO interrupt is disabled during RXTO event handling and enabled when RX is enabled. This workaround is applied only for fast instance to avoid unnecessary register accesses for slower instances. Signed-off-by: Krzysztof Chruściński <[email protected]> (cherry picked from commit e235e7b)
…abort test Test was staring a TX transfer and aborting it after 300 us from the timer handler. Test was assuming that ongoing transfer will not finish in those 300 us. This might not be true for higher baudrates. Instead of using fixed timeout, a value is calculated from the baudrate and targets to abort the transfer after approx. 20 bytes of 95 byte long transfer. Signed-off-by: Krzysztof Chruściński <[email protected]> (cherry picked from commit 38c129d)
…rf54h20dk uart120 Use high baudrate when testing uart120 on nrf54h20dk/nrf54h20/cpuapp. Signed-off-by: Krzysztof Chruściński <[email protected]> (cherry picked from commit fe6a4d0)
Removing unnecessary logging from DVFS reg operations. Upstream PR #: 86023 Signed-off-by: Paweł Pelikan <[email protected]>
ppelikan-nordic
force-pushed
the
dvfs_cleanup
branch
from
7b789dc to
6000d4b
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
71 participants
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.
Removing unnecessary logging from DVFS reg operations.
Upstream PR #: 86023
zephyrproject-rtos/zephyr#86023