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.

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

Add imx9352 mu1 driver to build #448

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

Conversation

alxlastur
Copy link

PR details:

  • include the mu1 driver when building the the project for MIMX9352.
  • add MU_GetRxStatusFlags() function in the fsl_mu.h of the mu and mu1 drivers.

The fsl_mu.h driver need a function
(static inline uint32_t MU_GetRxStatusFlags(MU_Type *base))
that get the MU RX status flags, because ipm_imx.c driver
from zephyr/drivers/ipm get the RX status flags directly
from MU register instead of using a function.
As ipm_imx.c driver is used by multiple imx platforms that have
different MU regiter map layout, each platform need to implement
a 'static inline uint32_t MU_GetRxStatusFlags(MU_Type *base)'
function that will be used by the ipm_imx.c in imx_mu_isr().
As imx_mu_isr() parse the RX status flags in reverse numerical
order, the function need to return the RX status flags
the same way.

Signed-off-by: Alexandru Lastur <[email protected]>
Add mu1 driver in the build recipe, when
the target is MIMX9352.

Signed-off-by: Alexandru Lastur <[email protected]>
/*!
* brief Return the RX status flags in reverse order.
*
* This function return the RX status flags in reverse order.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update function description

@dbaluta
Copy link
Contributor

dbaluta commented Oct 10, 2024

@alxlastur Please merge your code first into https://github.com/nxp-mcuxpresso/mcux-sdk and then come back to this repo.

Please move this PR to draft or close it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants