-
Notifications
You must be signed in to change notification settings - Fork 645
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
Downmerge a few nrfbsim related changes #2112
Closed
Closed
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
Update the HW models module to: 85944c64f224406e4d781aa382c5f1f71ed307fd Including the following: * 5340: Allocate a buffer for the APP core RAM (2nd version) * 82ca90c Add NCS test-spec.yml to trigger downstream CI job * a05d7a6 docs: top README: Mention nrf54l15 support and minor fixes * cd64524 54 ECB/CCM tests: Remove comment which does not apply Signed-off-by: Alberto Escolar Piedras <[email protected]> (cherry picked from commit 942fa67)
Update the HW models module to: bf8e45bd1f870e49cc15392e045c28d54ea12285 Including the following: * bf8e45b (52) CCM HAL: Support nrfx 3.7.0 with renamed TASK_CRYPT * 14c14d7 HW_models/NHW_GRTC: Support MDK not exposing SYSCOUNTERVALID * 6d66801 RADIO: Improve RSSISTART behaviour * ceb6a80 hal grtc: Add replacements for new int group functions * 838aa38 docs/README_HW_models: Add reference to Bsim HW models descr * 0ff34d7 docs: Several updates and fixes * 2972a93 AES_CCM: Avoid UBSAN pointer align warnwhen reading CNFPTR * f8cd477 UART: Minor bugfix: Set rx status to off during ENABLE Signed-off-by: Alberto Escolar Piedras <[email protected]> (cherry picked from commit 867e84c)
…e libCs ssize_t is not part of the C library ISO standard subset, and as such one cannot expect a C library to expose the type by default. Some libraries like glibc do not expose this type in general, and trying to build using them results in build errors. There is 3 possible options: 1. Continue using ssize_t and: 1.a define _POSIX_C_SOURCE before including any header 1.b include sys/types.h directly (A bit uglier) 3. Not use ssize_t, and instead rely on ISO standard types Let's just go with 1.a assuming we want to keep using this type, as that is the correct way of getting this type defined. Signed-off-by: Alberto Escolar Piedras <[email protected]> (cherry picked from commit b6f1767)
Provide a new function for initializing the Rx side, so users do not need to initialize the pointers by hand if they did not use PBUF_DEFINE(). Let's also rename pbuf_init() to pbuf_tx_init() to clearly signify the previous function was only meant to be used by the Tx side. Signed-off-by: Alberto Escolar Piedras <[email protected]> (cherry picked from commit 5dc810e)
Add a function which can be used to remap embedded device address, into addresses which can be used in the simulated native boards. For the nrf_bsim boards we provide an actual implementation. For other boards, we provide an optional dummy version which does nothing. It is up to each board implementation to decide if they want to provide one or use the dummy. Signed-off-by: Alberto Escolar Piedras <[email protected]> (cherry picked from commit 5efe751)
Add support in this IPC backends for POSIX arch targets in general, and ensure the nrf5340bsim defines the buffer which will be used. Signed-off-by: Alberto Escolar Piedras <[email protected]> (cherry picked from commit b5b91b3)
…bsim Enable this sample for the nrf5340bsim Signed-off-by: Alberto Escolar Piedras <[email protected]> (cherry picked from commit be35bbd)
Enable this sample for the nrf5340bsim Signed-off-by: Alberto Escolar Piedras <[email protected]> (cherry picked from commit ca62ff8)
Enable this sample for the nrf5340bsim Signed-off-by: Alberto Escolar Piedras <[email protected]> (cherry picked from commit 13efb39)
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. |
carlescufi
approved these changes
Oct 16, 2024
All commits got in => closing |
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.
To simplify how many fixes need to go through other PRs, let's merge in advance here a few commits related to the nrf_bsim targets.
All these should be safe to merge as they are backwards compatible.