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

Enable NETC support for i.MX95 EKV M7 #81113

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Enable NETC support for i.MX95 EKV M7 #81113

wants to merge 14 commits into from
+445 −31

Conversation

yangbolu1991
Copy link
Contributor

This PR is to enable NETC support for i.MX95 EKV M7.
There are dependencies of this PR.

Current PR already cherry-pick scmi changes and updated west.yml to address the dependency for CI.
And will drop them once they are merged.

Thanks.

@zephyrbot
Copy link
Collaborator

zephyrbot commented Nov 8, 2024 •
edited
Loading

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
hal_nxp zephyrproject-rtos/hal_nxp@9dc7449 (master) zephyrproject-rtos/hal_nxp#499 zephyrproject-rtos/hal_nxp#499/files

⛔ DNM label due to: 1 project with PR revision

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@zephyrbot zephyrbot added manifest manifest-hal_nxp DNM This PR should not be merged (Do Not Merge) labels Nov 8, 2024
@LaurentiuM1234 LaurentiuM1234 mentioned this pull request Dec 2, 2024
Merged
@yangbolu1991
Copy link
Contributor Author

Updated to v2. Changes,

  • Rebased on latest scmi patches. And addressed west.yml conflict.

Thanks.

@decsny decsny requested review from LaurentiuM1234 and removed request for yvanderv December 17, 2024 17:33
@mmahadevan108
Copy link
Collaborator

Please help address the CI failures.

@yangbolu1991 yangbolu1991 dismissed stale reviews from dleach02 and mmahadevan108 via 70104e8 January 17, 2025 04:47
@zephyrbot zephyrbot added area: ARM ARM (32-bit) Architecture area: Architectures area: Interrupt Controller DNM This PR should not be merged (Do Not Merge) labels Jan 17, 2025
@yangbolu1991
Copy link
Contributor Author

yangbolu1991 commented Jan 17, 2025 •
edited
Loading

Sory, I have an update on the PR.

Updated to v3. Changes,

Already ported #84017 here for CI.
Thank you very much.

@yangbolu1991
Copy link
Contributor Author

Updated to v4. Changes include,

  • Rebased.
  • Added one more patch to exclude tests not suit for multilevel interrupts.

Dependency (ported here for CI):

Thank you very much.

@fabiobaltieri fabiobaltieri added DNM (manifest) This PR should not be merged (controlled by action-manifest) and removed DNM This PR should not be merged (Do Not Merge) labels Feb 4, 2025
@yangbolu1991 yangbolu1991 mentioned this pull request Feb 28, 2025
Open
@yangbolu1991
west.yml: update hal_nxp to support i.MX95 NETC
e7be546 
Updated hal_nxp to support i.MX95 NETC.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@yangbolu1991
arch: arm: irq: allow custom irq control for multi-level interrupts
c3109ce 
Custom irq control for multi-level interrupts should be allowed.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@yangbolu1991
arch: arm: cortex_m: handle multi-level interrupts in irq init
8cdbf2f 
Multi-level interrupts should be handled in irq init. Only level1
interrupts going directly to interrupt controller need to be
initialized.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@yangbolu1991
drivers: interrupt_controller: intc_nxp_irqsteer: support ARM Cortex-M
71268c7 
Added ARM Cortex-M support for intc_nxp_irqsteer driver.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@yangbolu1991
drivers: interrupt_controller: intc_nxp_irqsteer: fix system INTID
00071ae 
Current system INTID calcualtion only worked for SoCs whose extended
interrupts started from IRQ 0.
Otherwise, FSL_FEATURE_IRQSTEER_IRQ_START_INDEX should be added for
system INTID.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@yangbolu1991
drivers: ethernet: eth_nxp_imx_netc: get irq from dts node
3f96f7e 
Added support to get irq from dts node, because some platforms may
use multi-level interrupts, and the system INTID may not be connected
directly to interrupt controller.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@yangbolu1991
dts: arm: nxp_imx95_m7: add power domain node and definitions
8abc9c1 
Added power domain node and i.MX95 power domain header file.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@yangbolu1991
dts: arm: nxp_imx95_m7: add NETC node and IRQSTEER node
993bba5 
Added NETC device tree node, and IRQSTEER node.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@yangbolu1991
soc: nxp: imx95: enable multi-level interrupts for m7
57a0929 
Enabled multi-level interrupts for m7 since IRQSTEER is used.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@yangbolu1991
tests: arm: exclude i.mx95 m7 for some interrupt tests
9a04784 
i.MX95 M7 is using multi-level interrupts with custom
interrupt handling. So, arm_custom_interrupt and
arm_irq_vector_table tests should exclude it.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@yangbolu1991
soc: nxp: imx95: support M7 soc init for NETC
63f4bd1 
Added support for soc init for NETC.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@yangbolu1991
boards: nxp: imx95_evk: add board init for NETC
bc96a7a 
Added board init for NETC.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@yangbolu1991
boards: nxp: imx95_evk: enable NETC PSI0 for M7
f7b6d1d 
Enabled NETC PSI0 for M7.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@yangbolu1991
boards: nxp: imx95_evk: update doc and yml for NETC support on M7
10bee1d 
Updated imx95_evk doc and yml for NETC support on M7.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@yangbolu1991
Copy link
Contributor Author

Updated to v5. Changes include,

  • Rebased as some changes already merged, and the dependency PR had update.

Dependency:

Thank you very much.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mmahadevan108 mmahadevan108 mmahadevan108 left review comments

@dleach02 dleach02 dleach02 left review comments

@ceolin ceolin Awaiting requested review from ceolin

@danieldegrasse danieldegrasse Awaiting requested review from danieldegrasse

@dbaluta dbaluta Awaiting requested review from dbaluta

@decsny decsny Awaiting requested review from decsny

@iuliana-prodan iuliana-prodan Awaiting requested review from iuliana-prodan

@JiafeiPan JiafeiPan Awaiting requested review from JiafeiPan

@lmajewski lmajewski Awaiting requested review from lmajewski

@manuargue manuargue Awaiting requested review from manuargue

@MarkWangChinese MarkWangChinese Awaiting requested review from MarkWangChinese

@nashif nashif Awaiting requested review from nashif

@pdgendt pdgendt Awaiting requested review from pdgendt

@teburd teburd Awaiting requested review from teburd

@LaurentiuM1234 LaurentiuM1234 Awaiting requested review from LaurentiuM1234

At least 2 approving reviews are required to merge this pull request.

Assignees

@mmahadevan108 mmahadevan108

@dleach02 dleach02

Labels
area: Architectures area: ARM ARM (32-bit) Architecture area: Ethernet area: Interrupt Controller area: Power Management DNM (manifest) This PR should not be merged (controlled by action-manifest) manifest manifest-hal_nxp platform: NXP Drivers NXP Semiconductors, drivers platform: NXP MPU
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@yangbolu1991 @zephyrbot @mmahadevan108 @dleach02 @fabiobaltieri