Skip to content
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.

Sign up for GitHub

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

Jump to bottom

drivers: ipm: fix in nrfx multi instance enabling procedure #1

Open
wants to merge 2 commits into
base: add_ipm_set_enabled
Choose a base branch
from
Open

drivers: ipm: fix in nrfx multi instance enabling procedure #1

wants to merge 2 commits into from

Commits on Aug 25, 2021

  1. ipc: Call ipm_set_enabled() also for dual IPM support 

    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]>
    @carlocaione
    carlocaione committed Aug 25, 2021
    Configuration menu
    Copy the full SHA
    e1068c8 View commit details
    Browse the repository at this point in the history

  2. drivers: ipm: fix in nrfx multi instance enabling procedure 

    Enabling procedure of IPM in nrfx multi instance implementation verified
if given instance is configure as RX. If not it returned an error.
Configuring an instance as TX is correct and enabling procedure should
work for such configuration.

This patch allows enabling IPM channels configured as TX.

Signed-off-by: Hubert Miś <[email protected]>
    @hubertmis
    hubertmis committed Aug 25, 2021
    Configuration menu
    Copy the full SHA
    cb803e8 View commit details
    Browse the repository at this point in the history