-
Notifications
You must be signed in to change notification settings - Fork 722
Workaround for mltpan18 #2670
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
Closed
Closed
Workaround for mltpan18 #2670
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
The buffer used for the TWIM DMA needs to be allocated to the section provided with the memory-regions dt prop. The macros for doing this where malformed. This commit fixes and cleans up the macros. Signed-off-by: Bjarki Arge Andreasen <[email protected]> (cherry picked from commit 62815dd)
…quired The DMA buffer needs to be used both for tx and rx if required. Extend RTIO driver variant to use DMA buffer and copy received data from it to user buffer. Signed-off-by: Bjarki Arge Andreasen <[email protected]> (cherry picked from commit 125752e)
Add nRF boards to the rtio_loopback sample. Signed-off-by: Bjarki Arge Andreasen <[email protected]> (cherry picked from commit 7d5a912)
Add nrf54h20 board support to the i2c rtio loopback sample. Signed-off-by: Bjarki Arge Andreasen <[email protected]> (cherry picked from commit 5c7f47d)
RTIO correctly declares tx buffers as const, however, the existing I2C API and NRFX (SDK) drivers don't. Therefor cast the const buf to non-const when passing the tx buf to the twim driver from the RTIO call to avoid the const warning. The tx buffer is being treated as const data naturally in the NRFX driver, its just not declared as such since we reuse the buffer for both RX and TX data. Alternatively the SDK and "shim" drivers built on top of it need to be updated, which is quite a bit of work :) Signed-off-by: Bjarki Arge Andreasen <[email protected]> (cherry picked from commit 2e00923)
Add i2c_bus_short fixture to the rtio_loopback sample to ensure it is only run on boards with the bus shorted. The i2c_bus_short fixture is also used in the i2c_target_api test suite. Signed-off-by: Bjarki Arge Andreasen <[email protected]> (cherry picked from commit 846b468)
…o console The harness was set to ztest, but the sample requires the console harness to match on the "sample complete" line, its not actually using ztest. Signed-off-by: Bjarki Arge Andreasen <[email protected]> (cherry picked from commit 7ee4011)
Main changes since v2.4: * libRandv2: New API to fill buffers & Wextra warning fixes * libCryptov1: Add new AES-ECB API Note: Like before, bsim remains fully backwards compatible Signed-off-by: Alberto Escolar Piedras <[email protected]> (cherry picked from commit ebbdb10)
The models include enough of the CRACEN to run the same nrfx drivers as in Zephyr. So let's add it to the list. Also let's remove the RTC from the list as the hal does not expose it anymore for this platform as the GRTC is to be used instead. Signed-off-by: Alberto Escolar Piedras <[email protected]> (cherry picked from commit a211999)
…sting int Similar to 923d4fb. Prevent overrunning the irq vector table. This is not happening today in tree, but coverity thinks it does. Checking for it to prevent it is not a bad idea anyhow, so let's do it. Signed-off-by: Krzysztof Chruściński <[email protected]> (cherry picked from commit cd23fdf)
…nch behind Update the HW models module back to the main branch, 2570f4a697ce2d860ff39ec34afd91749bd66d3 leaving behind zephyr_2025_01_22 * 68df9ed Force build of cracen hal even if HAL is not 3.10 As the nordic HAL has been updated to be this version, we do not need this hack in the HW models anymore to be able to build the CRACEN HW models for the nrf54l15. This update includes the following new commit in main: * 2570f4a docs: Do not list the RTC as included for the 54L15 Signed-off-by: Alberto Escolar Piedras <[email protected]> (cherry picked from commit a2bb2bc)
There seems like there is no good reason for having it disabled. For bsim boards CONFIG_SERIAL is disabled by default, there will anyways not be any logging on the UART. Signed-off-by: Rubin Gerritsen <[email protected]> (cherry picked from commit a2d1b28)
This makes the bsim boards more similar to their corresponding DKs. This makes it simpler to run samples and application requiring the use of a console. Signed-off-by: Rubin Gerritsen <[email protected]> (cherry picked from commit 77de593)
… bsim This commit describes how to build and run applications using the console. This documentation is added because it may not be obvious how this is done. That is, sometimes a user may believe that the terminal launching the application is the the console which is not the case. Signed-off-by: Rubin Gerritsen <[email protected]> (cherry picked from commit 0f3e6fd)
Update the HW models module to:
5dc34b26662c6ec91edf1174d775d78590b1a05b
Including the following:
* 5dc34b2 CLOCK (52-54): Silly bugfix
* 4f622b7 CLOCK (52-54): Make TASK_*CLKSTOP instantaneous and clear
LFCLK.STAT
* 07b1bdb RADIO 54: Correct 2 comments related the CCM trigger
Signed-off-by: Alberto Escolar Piedras <[email protected]>
(cherry picked from commit 58b9583)
Update the HW models module to
dcfcffeee5c3ad8718723df4b5297caec33c23e7
Including the following:
* dcfcffe UART: Support diffent clocks, correct UART00 clock and fix
BAUDRATE model
Signed-off-by: Alberto Escolar Piedras <[email protected]>
(cherry picked from commit aa6d483)
Update the HW models module to
73a5d5827a94820be65b7d276d28173ec10bab9f
Including the following:
* 73a5d58 54 CCM: Fix UBSAN reported issue
with zero size VLAs
Signed-off-by: Rubin Gerritsen <[email protected]>
(cherry picked from commit c031e12)
Fast PWM120 prevents GPIO from driving pin with low/high state when PWM duty is 0% or 100%. In such case, regular PWM signal generation needs to be used. Signed-off-by: Michał Stasiak <[email protected]> (cherry picked from commit a71f042)
Added the node representing the POWER peripheral on the nRF54L20. Upstream PR #: 85883 Signed-off-by: Rafał Kuźnia <[email protected]>
To ensure that the accesses to the radio peripheral are done with predictable timings, the constant latency must be enabled. The constant latency mode is enabled alongside the HFXO. Upstream PR #: 85883 Signed-off-by: Rafał Kuźnia <[email protected]>
Fixes report targets where tfm_* and bl2_* targets wrongly generates a footprint file by having a common target which consists of RAM and ROM report targets Upstream PR #: 86067 Signed-off-by: Jamie McCrae <[email protected]>
Fixes the prefix for these files to not have _report in them Upstream PR #: 86067 Signed-off-by: Jamie McCrae <[email protected]>
nrf-squash! [nrf noup] tests: drivers: mspi: api: Add nRF54L15 overlay file Fix device frequency for nRF54L15. SDP MSPI allows only a predefined set of device frequencies. Signed-off-by: Magdalena Pastula <[email protected]>
…t log twice Macro was missing a break when single argument with non 32 bit word argument was detect. Because of that, there were two logging messages created for a single log entry. Signed-off-by: Krzysztof Chruściński <[email protected]> (cherry picked from commit 9b2532a)
…or real targets." This reverts commit 107f9f0. Signed-off-by: Aleksandr Khromykh <[email protected]>
…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]>
…es support Both the TMAP kconfig file and public header file defined which roles were supported. The Kconfig file options were recently added and were more up to date (and correct), and allows for other Kconfig files to use these values, thus allowing for more flexibility when implementing applications. Signed-off-by: Emil Gydesen <[email protected]> (cherry picked from commit 48ac31c)
Extend nrf saadc device driver with pm device runtime support. To preserve previous behavior: * if pm device is disabled, saadc is resumed on sampling start and suspended when sampling done. * if pm device is enabled only, saadc does nothing on sampling start/stop. its resumed on init. * if pm device runtime is enabled, saadc is got on sampling start, and put on sampling stop. Signed-off-by: Bjarki Arge Andreasen <[email protected]> (cherry picked from commit 96c3cd6)
Set pm device runtime runtime auto flag to ensure saadc instances are initialized correctly if pm device runtime is used. Signed-off-by: Bjarki Arge Andreasen <[email protected]> (cherry picked from commit d01c93c)
…peration_t` In `db_hash_setup()` the state object for MAC operations was initialized using `psa_mac_operation_init()`. This function was not always optimized or inlined. A way to reduce stack usage is to use `memset()` and set the object to 0. This is one of the option documented to initialize `psa_mac_operation_t` object. Upstream PR #: 86094 Signed-off-by: Théo Battrel <[email protected]>
This commit adds more detailed description to the "Bluetooth APIs" and "GAP" group within the bluetooth header. Signed-off-by: Kyra Lengfeld <[email protected]> (cherry picked from commit 3570783)
This commit changes log level for already registered callback from warning to debug as this is cause unnecessary noise and doesn't indicate actual issue. Caller gets a certain error code for this case which can be handled properly. Signed-off-by: Pavel Vasilyev <[email protected]> (cherry picked from commit 886faba)
The api has not changed since its introduction some time ago, mark it as stable. Signed-off-by: Herman Berget <[email protected]> (cherry picked from commit 8b909b4)
…ection Subrating The API has not changed since it was introduced so should no longer be considered experimental. Signed-off-by: Timothy Keys <[email protected]> (cherry picked from commit 7ef8116)
Fix the following warning
att.c:734:3: warning: label followed by a declaration is a C23 extension
[-Wc23-extensions]
734 | k_tid_t current_thread = k_current_get();
| ^
By wrapping that code as a compound statement
Signed-off-by: Alberto Escolar Piedras <[email protected]>
(cherry picked from commit 1dab77b)
…_amend() Extend `zephyr_library_amend()` to support an optional directory argument. The current `zephyr_library_amend()` works well when used inside a Zephyr module with same structure, but fails when the macro is called from Zephyr module integration code is located in a Zephyr `MODULE_EXT_ROOT` because in this case the CMake code being executed is not present in the Zephyr module itself, in which case the dir name creation based on relative to module dir give wrong result. For this use-case then support a base directory. This also allows for use-cases in Zephyr modules where the directory structure matching Zephyr's own structure is placed in a sub-folder. Upstream PR #: 87274 Signed-off-by: Torsten Rasmussen <[email protected]>
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]>
…and MODE_QUAD_1_4_4" This reverts commit dd02c86. Signed-off-by: Marcin Szymczyk <[email protected]>
…_QUAD_1_4_4 Extend driver to support single lane and 1-4-4 IO modes. Upstream PR #: 85520 Signed-off-by: Marcin Szymczyk <[email protected]>
Allow configuring MTU for CMUX. Some AT manual and specification define this as a frame size. Linux ldattach default to 127 bytes, 3GPP TS 27.010 defaults to 31. We should limit our CMUX frames to a size that remote end is capable of handling. Linux silently drops oversized frames. Also, remove MODEM_CELLULAR_CMUX_MAX_FRAME_SIZE as this was only limiting a buffer sizes, and resulted CMUX frames to be capped to same value. Use MODEM_CMUX_WORK_BUFFER_SIZE and MODEM_CMUX_MTU instead. Also rename CONFIG_MODEM_CELLULAR_CHAT_BUFFER_SIZES to CONFIG_MODEM_CELLULAR_CHAT_BUFFER_SIZE as it is now only used as a Chat module. DLCI pipes use CONFIG_MODEM_CMUX_WORK_BUFFER_SIZE. Signed-off-by: Seppo Takalo <[email protected]> (cherry picked from commit c78081e)
If we end up writing zero bytes to cmux output, we can return zero instead of -ENOMEM as it would break various modules when using small buffers. For example modem_chat.c does not tolerate -ENOMEM but handles zero OK. Signed-off-by: Seppo Takalo <[email protected]> (cherry picked from commit 15b0f90)
…` folder Relocated `bt_shell_private.c` to the `common` folder to enable independent use between `BT_SHELL` and `BT_MESH_SHELL`. Signed-off-by: Pisit Sawangvonganan <[email protected]> (cherry picked from commit eeda631)
This change aims to eliminate the dependency on `ctx_shell` in the Bluetooth `audio/shell/*`, making the code more maintainable. Replaced `shell_*` functions that depended on `ctx_shell` with the appropriate `bt_shell_*` functions. Signed-off-by: Pisit Sawangvonganan <[email protected]> (cherry picked from commit f1516c9)
…ests Instead of using rs=1,2,3 and 4 we make it 10, 20, 30, 40 as the other values could cause some connection issues. Signed-off-by: Emil Gydesen <[email protected]> (cherry picked from commit 7d24771)
…BAP broadcast sink Workaround by using `-start_offset` to ensure that the ACL established does not overlap the sink being established. The behavior between nrf52_bsim and nrf54l15bsim is different where in the two device are starting to advertising at the same time in nrf54l15bsim where as there is an offset already in nrf52_bsim test. Signed-off-by: Vinayak Kariappa Chettimada <[email protected]> (cherry picked from commit 490da9e)
…te for the BA The broadcast assistant will now validate the content of the received receive states. Some modifications were done to the tests to properly pass these new validations. Signed-off-by: Emil Gydesen <[email protected]> (cherry picked from commit b361c95)
…le adv Most specs require the use of extended advertising, and most tests used legacy advertising. Implement a common function to create and start an extended advertising set to reduce code duplication. Signed-off-by: Emil Gydesen <[email protected]> (cherry picked from commit 0e9a3e0)
…e with BAP Add a selection of interval values that are suitable for BAP, which will allow better coexistence between ISO and ACL, for both broadcast and unicast. Some of these are defined by the BAP spec, and some are defined by Zephyr, since they do have a suggested value from BAP. Samples and tests have been updated to use these new values. Peripheral samples have also been updated with CONFIG_BT_GAP_AUTO_UPDATE_CONN_PARAMS so that the connection parameters from the centrals aren't updated to something else shortly after. The shell has also been updated to use the LE Audio (BAP) values if audio is enabled, and the audio.conf file has disabled automatic updating of the connection parameters as the peripheral, as we rarely (if ever) want to do that. Due to the connection interval change, CI hit an issue with test_bass_broadcast_code in test_main_client_sync, where the reading of the long receive state did not finish before we attempted to do another procedure, so the function was updated to have a retry. Signed-off-by: Emil Gydesen <[email protected]> (cherry picked from commit 9d4cc4b)
Renamed BT_CSIP_SET_MEMBER_NOTIFIABLE to BT_CSIP_SET_MEMBER_SIRK_NOTIFIABLE to be more specific. Signed-off-by: Emil Gydesen <[email protected]> (cherry picked from commit 501b95f)
…et size The set size can now be dynamically set and notified. The rank is added as a argument in the case that changing the set size, also affects the device's rank, as ranks in a coordinated set needs to be continuous. The set coordinator implementation has been updated to support receiving the new set size, and providing this information to the upper layers. This commit adds a babblesim test for the new API, as well as a shell command. Signed-off-by: Emil Gydesen <[email protected]> (cherry picked from commit d19abff)
The commit exports control of MBEDTLS_ASN1_PARSE_C, in mbedTLS module, via Kconfig. This allows applications to use ASN1 parser independently from other functions. Signed-off-by: Dominik Ermel <[email protected]> (cherry picked from commit e8589d2)
…IONS The commit adds Kconfig options that allows users to select - MBEDTLS_PLATFORM_NO_STD_FUNCTIONS - MBEDTLS_PLATFORM_SNPRINTF_ALT allowing Mbed TLS to use alternative definitions of STD functions. Upstream PR #: 87135 Signed-off-by: Dominik Ermel <[email protected]>
`CI-zigbee-test` not to be triggered anymore. nrf-squash! [nrf noup] ci: add .github/test-spec.yml Signed-off-by: Eduardo Montoya <[email protected]>
…config to int pin Add possibility to configure open drain and pull up on nPM2100's interrupt output through DTS flags. Signed-off-by: Sergei Ovchinnikov <[email protected]> (cherry picked from commit 32b79cf)
Signed-off-by: Maciej Meijer <[email protected]>
|
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.



Going to hibernate sleep (sleep mode 0xF) multiple times without powering down (or writing to any of the VPR's APB registers) in between will cause GPR x1 (ra) to not be saved