Nrfx none cherry pick latest stm #2098
Merged
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
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)
NordicBuilder
requested review from
57300,
anangl,
jaz1-nordic,
kl-cruz,
magp-nordic,
masz-nordic,
nika-nordic,
nordicjm and
tejlmand
nordic-segl
added a commit
to nordic-segl/sdk-nrf
that referenced
this pull request
Oct 11, 2024
Include latest changes to STM, FLPR and serial port. nrfconnect/sdk-zephyr#2098 Signed-off-by: Sebastian Głąb <[email protected]>
nordic-segl
requested review from
nordic-babu,
nordic-piks,
nordic-pikr and
nordic-bami
nordic-pikr
approved these changes
Oct 11, 2024
nordic-krch
approved these changes
Oct 11, 2024
nordic-piks
approved these changes
Oct 11, 2024
nordic-segl
added a commit
to nordic-segl/sdk-nrf
that referenced
this pull request
Oct 14, 2024
Include latest changes to STM, FLPR and serial port. nrfconnect/sdk-zephyr#2098 Signed-off-by: Sebastian Głąb <[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.
Cherry pick from the upstream Zephyr changes related to STM, FLPR and serial port.
manifest-pr-skip