[nrf noup] samples: smp_svr: Add nrf54h20dk ext flash sample #2846
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
nordic-mik7
force-pushed
the
dev/ext_flash_push
branch
2 times, most recently
from
f04196b to
0366e35
Compare
|
Set as DNM until SecureBoot is supported for 'iron' board variant. |
nordic-mik7
force-pushed
the
dev/ext_flash_push
branch
from
0366e35 to
d35297e
Compare
tomchy
approved these changes
May 13, 2025
ahasztag
reviewed
May 13, 2025
| - nrf5340dk/nrf5340/cpuapp | ||
| integration_platforms: | ||
| - nrf5340dk/nrf5340/cpuapp | ||
| sample.mcumgr.smp_svr.serial.nrf54h20dk.ext_flash.pure_dts: |
There was a problem hiding this comment.
I think static_dts should be used rather than pure_dts
There was a problem hiding this comment.
It is following already existing sample naming, but I can change this anyway if needed.
There was a problem hiding this comment.
In this case leave it as it is
ahasztag
approved these changes
May 13, 2025
…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 6577b70)
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 bca4c04)
Remove the closed loop mode implementation for the fll16m clock. Closed loop causes a hardware bug resulting in increased current consumption if SoC experiences high, but within spec, temperatures. Upstream PR #: 86339 Signed-off-by: Bjarki Arge Andreasen <[email protected]> (cherry picked from commit 3b35ead)
…te fll16m Update the nrf clock control test suite to not attempt to set the removed closed loop mode of the fll16m clock. Upstream PR #: 86339 Signed-off-by: Bjarki Arge Andreasen <[email protected]> (cherry picked from commit 71ebf71)
Add `IPC_SERVICE_BACKEND_ICBMSG_EP_BOUND_WORK_Q_STACK_SIZE` config and parametrize its value depending on the optimization level. Upstream PR #: 89730 Signed-off-by: Adam Kondraciuk <[email protected]> (cherry picked from commit 7d94914)
This reverts commit 28bd478. This commit can be reverted as the cause of test failure has been removed in ipc service by increasing workqueue stack size. Upstream PR #: 89730 Signed-off-by: Adam Kondraciuk <[email protected]> (cherry picked from commit a4fc463)
When a LD sends an ic-msg to SCFW - it happens that sometimes that SCFW does not handle the request. For the moment the problem is solved by sending a second vevif event shortly after the initial request. Upstream PR #: 86455 Signed-off-by: Maciej Meijer <[email protected]> (cherry picked from commit ab82bbb)
…ting When left-shifting '1' by 31, the result is undefined. This is something ASAN detects. Solve this by explicitly defining that the integer is unsigned. Upstream PR #: 86624 Signed-off-by: Rubin Gerritsen <[email protected]> (cherry picked from commit 7792aae)
Add Kconfig options for TDM130 and TDM131. Upstream PR #: 82144 Signed-off-by: Adam Kondraciuk <[email protected]> (cherry picked from commit 2f8af7b)
Add support for configuring pins of the nRF TDM peripheral. Upstream PR #: 82144 Signed-off-by: Adam Kondraciuk <[email protected]> (cherry picked from commit b21905b)
Add TDM support for nRF54H20 and nRF54L20 Upstream PR #: 82144 Signed-off-by: Adam Kondraciuk <[email protected]>
Support for list of images in build info was added with commit 4061311 and is used by sysbuild. Zephyr itself also uses CMake's External Project feature when including TF-M or TF-A in a Zephyr build. Populate build info with TF-M / TF-A information when said image is included in the build. Upstream PR #: 86881 Signed-off-by: Torsten Rasmussen <[email protected]> (cherry picked from commit 92dddd4)
…nfiguration On nrf54h20 there are additional analog pins (AIN8+). When differential mode is used they must not be mixed with AIN0-AIN7. Add build time validation which detects if configuration is invalid. Signed-off-by: Krzysztof Chruściński <[email protected]> (cherry picked from commit 2f7d799)
…vref_mv Sample is reading vref-mv property to get reference voltage and then if reference source is set to ADC_REF_INTERNAL this value is overwritten. If vref-mv property is provided then it should not be overwritten. Signed-off-by: Krzysztof Chruściński <[email protected]> (cherry picked from commit 2174e16)
…nfiguration AIN9 source needs special handling (all from AIN8-AIN14). Conversion result needs to be scaled up by 3.6 additionally to gain and reference voltage. In order to achieve correct value in the sample vref-mv is used as reference voltage instead of predefined internal reference source. Signed-off-by: Krzysztof Chruściński <[email protected]> (cherry picked from commit 7a16284)
Add number of channels implemented by the local DPPIC instances. Upstream PR #: 85853 Signed-off-by: Adam Kondraciuk <[email protected]> (cherry picked from commit b212602)
Added ncs-specific modules to nrfx_config_reserved_resources. The modules are: - mpsl - nrfe Signed-off-by: Rafał Kuźnia <[email protected]> (cherry picked from commit 6c2c9fb)
As nrfx_config is included before MDK, MDK-specific device symbols cannot be used. Use Kconfig symbols instead. Signed-off-by: Nikodem Kastelik <[email protected]> (cherry picked from commit 665d441)
…targets. Mesh currently works with trusted storage on real targets. Until secure storage is supported by default disable it. Signed-off-by: Aleksandr Khromykh <[email protected]> (cherry picked from commit e7c02bf)
…Cracen driver on nRF54H20 A larger stack is needed to accomodate the Cracen driver. Signed-off-by: Gordon Klaus <[email protected]> (cherry picked from commit 7c8d474)
This adds temporary entropy driver simulation for nRF54h20 device since final entropy source is not available yet. TODO: Remove this commit when proper solution will be available. Jira: NCSDK-25947 Signed-off-by: Kamil Gawor <[email protected]> Signed-off-by: Robert Lubos <[email protected]> Signed-off-by: Andreas Moltumyr <[email protected]> Signed-off-by: Karol Lasończyk <[email protected]> (cherry picked from commit 956e404)
This commit brings back modifications from these reverted commits: - f68b2ed - e606246 slightly adjusted so that the EXMIF peripheral is still by default handled by the mspi_dw driver, and in cases where this driver cannot be used because something still does not work correctly, one can switch to the old solution based on the tweaked spi_dw driver. Signed-off-by: Andrzej Głąbek <[email protected]> (cherry picked from commit 9554ebf)
Configure SDP MSPI pins to switch their control to VPR core Signed-off-by: Jakub Zymelka <[email protected]> Signed-off-by: Andrzej Głąbek <[email protected]> Signed-off-by: Magdalena Pastula <[email protected]> (cherry picked from commit 1784754)
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]> Signed-off-by: Théo Battrel <[email protected]> (cherry picked from commit 54d0589)
noup since this option does not exist upstream. The Oberon PSA core has an option to manually set the buffer size of the PSA key slots in bytes. This option here: MBEDTLS_PSA_STATIC_KEY_SLOT_BUFFER_SIZE This option has a default value of 16 since it directly affects the memory usage. The crypto tests use a secp256r1 key and thus since the public key for this key is 65 bytes long we need to set the option in the sample. Signed-off-by: Georgios Vasilakis <[email protected]> (cherry picked from commit e626fbb)
This is needed for privacy tests. Specifically, this fixes GAP/PRIV/CONN/BV-10-C and GAP/CONN/UCON/BV-06-C on nRF54H20. Signed-off-by: Pavel Vasilyev <[email protected]> (cherry picked from commit 989dc5a)
Non-secure variants for nRF7002 DK were removed from upstream in commit 10d4973. Revert these changes downstream, so that the NS variants are still available. Signed-off-by: Andrzej Głąbek <[email protected]> Signed-off-by: Johann Fischer <[email protected]> (cherry picked from commit 4f0fc15)
Fixes an issue whereby just enabling hooks would enable the nrf5340 network core hook despite lacking other requirements Signed-off-by: Jamie McCrae <[email protected]> (cherry picked from commit 0613ebd)
Use PPI resources information from SoftDevice when it is present. Co-authored-by: Krzysztof Chruściński <[email protected]> Signed-off-by: Eivind Jølsgard <[email protected]> (cherry picked from commit 319958a)
The PSA is a cryptographically secure random number generator. It will be enabled by default, eventually, For now, enable it manually. Signed-off-by: Gordon Klaus <[email protected]> (cherry picked from commit cf0e50d)
Same as commit 6ec9d10 but with the REQUIRED keyword on its own line to attempt to avoid a merge conflict when reverting/reapplying this patch. Signed-off-by: Jonathan Nilsen <[email protected]> (cherry picked from commit 058e3fb)
Initial implementation of nRF54L quirks necessary for nRF54LM20A. To be upstreamed after TODO comments are resolved. Signed-off-by: Tomasz Moń <[email protected]> (cherry picked from commit e54edad)
Applied as noup because of conflicts. Upstream PR #: 89333 Add an IPC service API for triggering updates of the Nordic IRONside SE firmware using the IRONside call module. Co-authored-by: Håkon Amundsen <[email protected]> Signed-off-by: Jonathan Nilsen <[email protected]> (cherry picked from commit 205f697)
Necessary changes for NCS CI. - Add a Jenkinsfile - Add a commit-tags workflow: This enables sauce tag checking in sdk-zephyr - compliance.yml: Disable check for merge commits, since we have upmerges downstream. Also, since in the code we refer to Kconfig symbols that are defined in the sdk-nrf repository, the Kconfig checks ((SysBuild)Kconfig, (SysBuild)KconfigBasic and (SysBuild)KconfigBasicNoModules) will not pass so exclude them. Also, disable any maintainers-related checks - scripts/gitlint: Extend the max commit line lengths for Gitlint to account for sauce tags - Adapt to the changes in: nrfconnect/action-commit-tags#4 Signed-off-by: Carles Cufi <[email protected]> Signed-off-by: Dominik Ermel <[email protected]> Signed-off-by: Martí Bolívar <[email protected]> Signed-off-by: Vinayak Kariappa Chettimada <[email protected]> Signed-off-by: Krishna T <[email protected]> Signed-off-by: Dominik Ermel <[email protected]> (cherry picked from commit 47adf55)
Adds symbols used in NCS to the sysbuild Kconfig allow list Signed-off-by: Jamie McCrae <[email protected]> (cherry picked from commit cb627ba)
GCC and Clang support the undefined behavior sanitizer in any configuration, the only restriction is that if you want to get nice messages printed, then you need the ubsan library routines which are only present for posix architecture or when using picolibc. This patch adds three new compiler properties: * sanitizer_undefined. Enables the undefined behavior sanitizer. * sanitizer_undefined_library. Calls ubsan library routines on fault. * sanitizer_undefined_trap. Invokes __builtin_trap() on fault. Overhead for using the trapping sanitizer is fairly low and should be considered for use in CI once all of the undefined behavior faults in Zephyr are fixed. Signed-off-by: Keith Packard <[email protected]> (cherry picked from commit 2d64237)
The Renesas RX support flashing .mot file for binary image This commit target to add the .mot file output for build and flash script Signed-off-by: Duy Nguyen <[email protected]> (cherry picked from commit 5fabd36)
cmake/kobj.cmake now provides nice wrappers around the script itself and common uses. Signed-off-by: Armin Brauns <[email protected]> (cherry picked from commit d6efbc8)
Discard the eh_frame section when C++ exceptions are disabled. In principle the eh_frame may be used for other purposes such as backtracing when linking C programs, then Zephyr compiles each source file with '-fno-asynchronous-unwind-tables', thus keeping the eh_frame in the elf output just takes up space. When using gcc/ld, then the eh_frame is generally generally discarded per default, however for clang/lld it will be included but give a warning that the section is auto-placed. Some platforms already discards the eh_frame, so unify this for all targets. As eh_frame is now discarded in linker scripts then post processing step of removing the section during hex or bin conversion can also be removed. Signed-off-by: Torsten Rasmussen <[email protected]> (cherry picked from commit 8f2560c)
Replace _parse_syscalls_target_ custom target with explicit dependency management for syscall depending file generation. Signed-off-by: David Schneider <[email protected]> (cherry picked from commit d85ed32)
nordic-mik7
force-pushed
the
dev/ext_flash_push
branch
from
d35297e to
1037111
Compare
Adds a configuration to this sample for external flash on the nRF54H20DK. It uses 'pure dts' approach, with no Partition Manager. Signed-off-by: Michal Kozikowski <[email protected]>
nordic-mik7
force-pushed
the
dev/ext_flash_push
branch
from
1037111 to
bffc859
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.
Adds a configuration to this sample for external flash on the nRF54H20DK. It uses 'pure dts' approach, with no Partition Manager.