Ncs nrf53 #239
Merged
Ncs nrf53 #239
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 code was not properly taking into account CONFIG_BT_HCI_RESERVE, which would cause buffer underruns for any HCI driver where this value defaults to non-zero. Also, all the allocation functions use the same pool, so we can map them simply to bt_buf_get_rx() instead of repeating the same code. Signed-off-by: Johan Hedberg <[email protected]> Signed-off-by: Ioannis Glaropoulos <[email protected]>
Since the nRF5340 DK contains the required DC/DC circuitry, enable it by default while keeping it configurable. This reduces power consumption in application core , network core and high voltage use. Signed-off-by: Vinayak Kariappa Chettimada <[email protected]> Signed-off-by: Ioannis Glaropoulos <[email protected]>
…usage We need to fix the formato of the macros for nRF CACHE and NVMC peripherals in soc.c, so the _S and _NS suffixes are skipped. Signed-off-by: Ioannis Glaropoulos <[email protected]>
…tions relative A partition start offsets should be expressed relative to the flash device base address. For cpunet flash partitions start offsets were improperly expressed as absolute flash address. This patch fixes partitions start addresses for nRF5340 cpunet. Signed-off-by: Andrzej Puzdrowski <[email protected]> Signed-off-by: Ioannis Glaropoulos <[email protected]>
Contributor
|
Some checks failed. Please fix and resubmit. checkpatch (informational only, not a failure)Gitlint issuesCommit 7898118: Tip: The bot edits this comment instead of posting a new one, so you can check the comment's history to see earlier messages. |
Contributor
Author
|
@kapi-no can you test that the setup still works now that we moved this to NCS? |
Contributor
|
@ioannisg, yes, sure. Could you open matching PR to the nrf repository (with west manifest pointing to this PR)? |
This commit squashes into one the commits of the upstream project pull request 20467, which brings support of BLE HCI controller in nRF5340 over openAMP transport. Signed-off-by: Ioannis Glaropoulos <[email protected]>
This reverts commit 22e7108. Signed-off-by: Ioannis Glaropoulos <[email protected]>
kapi-no
approved these changes
Nov 19, 2019
Default the BLE Controller to the SPLIT architecture. Signed-off-by: Ioannis Glaropoulos <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 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.
Contains the following patches