dts: bindings: nrf-vpr: Allow specifying pins and not using launcher #88168
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 possibility to use pinctrl to configure pins that should be assigned to nRF VPR coprocessors and also provide a way of preventing activation of the nordic_vpr_launcher driver for an enabled VPR so that it can be supplied with the code to execute and launched in a custom way. Signed-off-by: Andrzej Głąbek <andrzej.glabek@nordicsemi.no>
github-actions
bot
added
platform: nRF
github-actions
bot
requested review from
carlocaione,
edersondisouza,
fkokosinski,
jaz1-nordic,
katsuster,
kgugala,
kl-cruz,
magp-nordic,
masz-nordic,
mgielda,
nika-nordic,
npitre and
tgorochowik
gmarull
reviewed
Apr 4, 2025
gmarull
approved these changes
Apr 4, 2025
masz-nordic
approved these changes
Apr 4, 2025
57300
reviewed
Apr 4, 2025
Comment on lines
+20
to
+22
| by the nordic_vpr_launcher driver when the node is enabled. In such case, | ||
| the launching is supposed to be done explicitly by the user code, in some | ||
| custom way presumably. |
There was a problem hiding this comment.
I'm curious about this comment. I assume that a custom way of launching VPR boils down to allowing the App core to determine at runtime when and/or where it boots the VPR. Should this really be covered by zephyr,deferred-init and maybe an extra driver API for setting the execution address before calling device_init()?
There was a problem hiding this comment.
This is also about a custom way of supplying the code to be executed by the VPR. The application my want to copy the code to RAM by itself and then start the VPR, so in fact do everything that is provided by the nordic_vpr_launcher driver but in its own way (this is in fact an actual use case for which I need these changes).
carlescufi
approved these changes
Apr 4, 2025
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.
Add possibility to use pinctrl to configure pins that should be assigned to nRF VPR coprocessors and also provide a way of preventing activation of the nordic_vpr_launcher driver for an enabled VPR so that it can be supplied with the code to execute and launched in a custom way.