-
Notifications
You must be signed in to change notification settings - Fork 6.1k
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
boards: dts: nordic: SysCtrl IPC configuration #71265
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you please add dts/common/nordic to the nrf platforms in the maintainer yml, I think this is why kumar was assigned, but it really should be the nordic maintainer assigned
Thanks for the suggestion. Updated maintainers and reassigned this PR. |
Add a VEVIF node to be used for communicating with SysCtrl (cpusys). This is the only part of the SysCtrl VPR exposed to local domains. Signed-off-by: Grzegorz Swiderski <grzegorz.swiderski@nordicsemi.no>
Add the default `zephyr,ipc-icmsg` nodes for communication with Application and Radiocore. Signed-off-by: Grzegorz Swiderski <grzegorz.swiderski@nordicsemi.no>
Files in `dts/common/nordic` and `dts/riscv/nordic` belong to this area, now that Nordic's devicetrees span multiple architectures. Use a glob to cover all paths. Signed-off-by: Grzegorz Swiderski <grzegorz.swiderski@nordicsemi.no>
Rebased and added nodes for Radiocore <-> SysCtrl. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should there be some proper error message (instead of cryptic compile error) when zephyr,ipc-icmsg
compatible (here: cpuapp_cpusys_ipc
) has status = "okay"; but any of its mboxes (here: cpuapp_bellboard
and cpusys_vevif
) have status = "disabled";?
Sounds reasonable to me. I'm just not familiar enough with Zephyr IPC to know how and where this should be validated, if it isn't already. |
It is not validated. I stumbled across the cryptic error at zephyr/subsys/ipc/ipc_service/backends/ipc_icmsg.c Lines 59 to 60 in b4eac67
Not sure if the |
Upstream PR: zephyrproject-rtos/zephyr#71265 Add a VEVIF node to be used for communicating with SysCtrl (cpusys). This is the only part of the SysCtrl VPR exposed to local domains. Signed-off-by: Grzegorz Swiderski <grzegorz.swiderski@nordicsemi.no> (cherry picked from commit 801a13f8cf5c5b08908757b7e1f3b08d6b8ea667)
…SysCtrl Upstream PR: zephyrproject-rtos/zephyr#71265 Add the default `zephyr,ipc-icmsg` nodes for communication with Application and Radiocore. Signed-off-by: Grzegorz Swiderski <grzegorz.swiderski@nordicsemi.no> (cherry picked from commit f10a45e62c19413e0dcbea881527436c3a87e387)
Upstream PR: zephyrproject-rtos/zephyr#71265 Add a VEVIF node to be used for communicating with SysCtrl (cpusys). This is the only part of the SysCtrl VPR exposed to local domains. Signed-off-by: Grzegorz Swiderski <grzegorz.swiderski@nordicsemi.no> (cherry picked from commit 801a13f8cf5c5b08908757b7e1f3b08d6b8ea667)
…SysCtrl Upstream PR: zephyrproject-rtos/zephyr#71265 Add the default `zephyr,ipc-icmsg` nodes for communication with Application and Radiocore. Signed-off-by: Grzegorz Swiderski <grzegorz.swiderski@nordicsemi.no> (cherry picked from commit f10a45e62c19413e0dcbea881527436c3a87e387)
Upstream PR: zephyrproject-rtos/zephyr#71265 Add a VEVIF node to be used for communicating with SysCtrl (cpusys). This is the only part of the SysCtrl VPR exposed to local domains. Signed-off-by: Grzegorz Swiderski <grzegorz.swiderski@nordicsemi.no> (cherry picked from commit 801a13f8cf5c5b08908757b7e1f3b08d6b8ea667)
…SysCtrl Upstream PR: zephyrproject-rtos/zephyr#71265 Add the default `zephyr,ipc-icmsg` nodes for communication with Application and Radiocore. Signed-off-by: Grzegorz Swiderski <grzegorz.swiderski@nordicsemi.no> (cherry picked from commit f10a45e62c19413e0dcbea881527436c3a87e387)
Upstream PR: zephyrproject-rtos/zephyr#71265 Add a VEVIF node to be used for communicating with SysCtrl (cpusys). This is the only part of the SysCtrl VPR exposed to local domains. Signed-off-by: Grzegorz Swiderski <grzegorz.swiderski@nordicsemi.no> (cherry picked from commit 801a13f8cf5c5b08908757b7e1f3b08d6b8ea667)
…SysCtrl Upstream PR: zephyrproject-rtos/zephyr#71265 Add the default `zephyr,ipc-icmsg` nodes for communication with Application and Radiocore. Signed-off-by: Grzegorz Swiderski <grzegorz.swiderski@nordicsemi.no> (cherry picked from commit f10a45e62c19413e0dcbea881527436c3a87e387)
Add DT nodes to be used for communicating with SysCtrl on nRF54H20.