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

Switch to native Zephyr support for imx8MP #9608

Merged
merged 2 commits into from
Jan 7, 2025

Conversation

dbaluta
Copy link
Collaborator

@dbaluta dbaluta commented Oct 23, 2024

With this series all NXP i.MX8 support is using Zephyr native drivers.

This depends on

Copy link
Member

@lgirdwood lgirdwood left a comment

Choose a reason for hiding this comment

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

I guess you should now be able to followup and start removing old code ?

Copy link
Collaborator

@kv2019i kv2019i left a comment

Choose a reason for hiding this comment

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

Very cool, only minor comments. I guess we need to delay approval until Zephyr side PR is merged.

zephyr/Kconfig Outdated Show resolved Hide resolved
@dbaluta dbaluta force-pushed the native_drivers_imx8mp branch 2 times, most recently from 4a47d32 to 6fd41c7 Compare October 24, 2024 14:51
@dbaluta
Copy link
Collaborator Author

dbaluta commented Oct 24, 2024

Changes since v1:

  • fixed checkpatch.pl issues
  • fixed @kv2019i comments.

@dbaluta dbaluta changed the title Switch to native Zephyr support for imx8MP [DNM] Switch to native Zephyr support for imx8MP Oct 24, 2024
@dbaluta dbaluta self-assigned this Oct 24, 2024
@dbaluta dbaluta force-pushed the native_drivers_imx8mp branch from 6fd41c7 to e042553 Compare October 25, 2024 08:53
@dbaluta dbaluta force-pushed the native_drivers_imx8mp branch from e042553 to 2ba3d02 Compare December 20, 2024 11:16
@lgirdwood
Copy link
Member

@dbaluta I assume you will remove the DNM when you are happy with Zephyr PR and with internal validation ?

@dbaluta
Copy link
Collaborator Author

dbaluta commented Dec 20, 2024

@lgirdwood yes. Zephyr PR is all approved and waiting now to be merged

This includes:
   - switching all topologies from DMA domain to timer domain.
   - interrupt related fixes done via CONFIG_IMX8M.
   - remove SOF standalone drivers from compilation

Signed-off-by: Daniel Baluta <[email protected]>
@dbaluta dbaluta force-pushed the native_drivers_imx8mp branch from 2ba3d02 to f19b7b4 Compare January 6, 2025 13:48
@dbaluta dbaluta changed the title [DNM] Switch to native Zephyr support for imx8MP Switch to native Zephyr support for imx8MP Jan 6, 2025
@dbaluta
Copy link
Collaborator Author

dbaluta commented Jan 6, 2025

@lgirdwood Ready to be merged once #9761 is merged.

@lgirdwood
Copy link
Member

@wszypelt good to merge ? this PR is for not tested by Intel CI.

This enables sai3 and sdma3 nodes and also adds host dma node
used by SOF.

Signed-off-by: Daniel Baluta <[email protected]>
@dbaluta dbaluta force-pushed the native_drivers_imx8mp branch from f19b7b4 to eb567d2 Compare January 6, 2025 19:17
@dbaluta
Copy link
Collaborator Author

dbaluta commented Jan 6, 2025

Retrigger build after west.yml update.

Copy link
Collaborator

@kv2019i kv2019i left a comment

Choose a reason for hiding this comment

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

Looks great!

@wszypelt
Copy link

wszypelt commented Jan 7, 2025

@wszypelt good to merge ? this PR is for not tested by Intel CI.

good to merge

@kv2019i kv2019i merged commit 4094048 into thesofproject:main Jan 7, 2025
44 of 49 checks passed
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.

5 participants