Skip to content

Switch to native Zephyr support for imx8MP - #9608

Merged
kv2019i merged 2 commits into
thesofproject:mainfrom
dbaluta:native_drivers_imx8mp
Jan 7, 2025
Merged

Switch to native Zephyr support for imx8MP#9608
kv2019i merged 2 commits into
thesofproject:mainfrom
dbaluta:native_drivers_imx8mp

Conversation

@dbaluta

@dbalutadbaluta commented Oct 23, 2024•
edited by lgirdwood
Loading

Copy link
Copy Markdown
Collaborator

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

This depends on

lgirdwood
lgirdwood approved these changes Oct 23, 2024
View reviewed changes

@lgirdwoodlgirdwood left a comment

Copy link
Copy Markdown
Member

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 ?

Comment threadsrc/platform/imx8m/include/platform/platform.h
kv2019i
kv2019i reviewed Oct 23, 2024
View reviewed changes

@kv2019ikv2019i left a comment

Copy link
Copy Markdown
Collaborator

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.

Comment threadzephyr/Kconfig Outdated
@dbaluta
dbalutaforce-pushed the native_drivers_imx8mp branch 2 times, most recently from 4a47d32 to 6fd41c7CompareOctober 24, 2024 14:51
@dbaluta

dbaluta commented Oct 24, 2024

Copy link
Copy Markdown
CollaboratorAuthor

Changes since v1:

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

@dbalutadbaluta changed the title Switch to native Zephyr support for imx8MP[DNM] Switch to native Zephyr support for imx8MPOct 24, 2024
@dbalutadbaluta self-assigned this Oct 24, 2024
@dbaluta
dbalutaforce-pushed the native_drivers_imx8mp branch from 6fd41c7 to e042553CompareOctober 25, 2024 08:53
@kv2019ikv2019i mentioned this pull request Nov 12, 2024
Merged
This was referenced Dec 13, 2024
Closed
Closed
Merged
@dbaluta
dbalutaforce-pushed the native_drivers_imx8mp branch from e042553 to 2ba3d02CompareDecember 20, 2024 11:16
@lgirdwood

lgirdwood commented Dec 20, 2024

Copy link
Copy Markdown
Member

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

@dbaluta

dbaluta commented Dec 20, 2024

Copy link
Copy Markdown
CollaboratorAuthor

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

@dbaluta
imx8mp: Switch to native drivers and timer domain
4d3e737
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 <daniel.baluta@nxp.com>
@dbaluta
dbalutaforce-pushed the native_drivers_imx8mp branch from 2ba3d02 to f19b7b4CompareJanuary 6, 2025 13:48
@dbalutadbaluta changed the title [DNM] Switch to native Zephyr support for imx8MPSwitch to native Zephyr support for imx8MPJan 6, 2025
@dbaluta

dbaluta commented Jan 6, 2025

Copy link
Copy Markdown
CollaboratorAuthor

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

@lgirdwood

lgirdwood commented Jan 6, 2025

Copy link
Copy Markdown
Member

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

@dbaluta
boards: overlay: Add overlay file for imx8mp
eb567d2
This enables sai3 and sdma3 nodes and also adds host dma node
used by SOF.
Signed-off-by: Daniel Baluta <daniel.baluta@nxp.com>
@dbaluta
dbalutaforce-pushed the native_drivers_imx8mp branch from f19b7b4 to eb567d2CompareJanuary 6, 2025 19:17
@dbaluta

dbaluta commented Jan 6, 2025

Copy link
Copy Markdown
CollaboratorAuthor

Retrigger build after west.yml update.

kv2019i
kv2019i approved these changes Jan 7, 2025
View reviewed changes

@kv2019ikv2019i left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks great!

@wszypelt

wszypelt commented Jan 7, 2025

Copy link
Copy Markdown

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

good to merge

@kv2019i
kv2019i merged commit 4094048 into thesofproject:mainJan 7, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lyakhlyakhlyakh approved these changes
@lgirdwoodlgirdwoodlgirdwood approved these changes
@kv2019ikv2019ikv2019i approved these changes
@iuliana-prodaniuliana-prodanAwaiting requested review from iuliana-prodaniuliana-prodan is a code owner
@abonislawskiabonislawskiAwaiting requested review from abonislawskiabonislawski is a code owner
@ranj063ranj063Awaiting requested review from ranj063ranj063 is a code owner
@plbossartplbossartAwaiting requested review from plbossartplbossart is a code owner
@mmaka1mmaka1Awaiting requested review from mmaka1mmaka1 is a code owner
@lbetlejlbetlejAwaiting requested review from lbetlejlbetlej is a code owner

Assignees

@dbalutadbaluta

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@dbaluta@lgirdwood@wszypelt@lyakh@kv2019i