Skip to content

dts: renesas: correct interrupt numbers for ra4w1 #84411

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

Merged
merged 1 commit into from
Jan 29, 2025

Conversation

quytranpzz
Copy link
Contributor

@quytranpzz quytranpzz commented Jan 23, 2025

- Update the overlapping irq number between port_irq4 and spi1
- Remove irq number for sci9 as it exceeds the limit (32 irq numbers)
Now users will define the irq numbers themselves

Signed-off-by: Quy Tran <quy.tran.pz@renesas.com>
@zephyrbot zephyrbot added the platform: Renesas RA Renesas Electronics Corporation, RA label Jan 23, 2025
@quytranpzz quytranpzz changed the title bug: dts: renesas: Fix interrupt numbers bug for ra4w1 dts: renesas: Fix interrupt numbers bug for ra4w1 Jan 23, 2025
@quytranpzz quytranpzz changed the title dts: renesas: Fix interrupt numbers bug for ra4w1 dts: renesas: correct interrupt numbers for ra4w1 Jan 23, 2025
@@ -149,8 +149,6 @@

sci9: sci9@40070120 {
compatible = "renesas,ra-sci";
interrupts = <36 1>, <37 1>, <38 1>, <39 1>;
Copy link
Member

Choose a reason for hiding this comment

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

@KhiemNguyenT Although it is not part of this PR, it is time to consider automatic IRQ numbering, which was previously discussed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@soburi , cc @KhiemNguyenT :
We opened a draft PR to support the interrupt map (for RA2A1 first) at #84427, please take a look. Thank you

@kartben kartben merged commit a9c0fca into zephyrproject-rtos:main Jan 29, 2025
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform: Renesas RA Renesas Electronics Corporation, RA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants