-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
ipc: Call ipm_set_enabled() also for dual IPM support #37931
ipc: Call ipm_set_enabled() also for dual IPM support #37931
Conversation
tagging @maje-emb as well |
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.
Seems rpmsg_service sample for nRF53 stopped working after this change
After first testing: it seems that the ipc_service and rpmsg_multi_instance samples also stopped working. Tested on nRF5340. |
Fix is on the way: carlocaione#1 |
@hubertmis @maje-emb uhm, on a second thought I'm not convinced anymore that is the correct fix. The API documentation says indeed that |
The IPC drivers rpmsg_service and rpmsg_multi_instance are not explicitly enabling the RX IPM channel when two different devices are used for TX and RX. While this could be redundant for some IPM drivers, in some cases the hardware needs to be enabled before using it. Add the missing calls to ipm_set_enabled() for both the devices. Signed-off-by: Carlo Caione <[email protected]>
e1068c8
to
54f9247
Compare
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.
Verified rpmsg_service sample works fine on nRF5340
ipc_service and rpmsg_multi_instance samples also work fine. |
The IPC drivers rpmsg_service and rpmsg_multi_instance are not
explicitly enabling the IPM channels when two different devices are used
for TX and RX. While this could be redundant for some IPM drivers, in
some cases the hardware needs to be enabled before using it.
Add the missing calls to ipm_set_enabled() for both the devices.
Signed-off-by: Carlo Caione [email protected]