Skip to content

Conversation

@EmilioCBen
Copy link
Contributor

@EmilioCBen EmilioCBen commented Nov 6, 2024

Fixes #79837
The FRDM_MCXW71 Platform has a reserved IRQ as its last IRQ in the NVIC, this test was using this IRQ to test an interrupt and would not fire. This change
ensures the test does not use the reserved IRQ.

@zephyrbot zephyrbot added area: ARM ARM (32-bit) Architecture size: XS A PR changing only a single line of code labels Nov 6, 2024
@EmilioCBen EmilioCBen added this to the v4.0.0 milestone Nov 6, 2024
@EmilioCBen EmilioCBen force-pushed the fix/mcxw71_dynamic_irq branch from 911dfc0 to 603cd2c Compare November 6, 2024 22:23
@EmilioCBen EmilioCBen added the DNM This PR should not be merged (Do Not Merge) label Nov 6, 2024
The FRDM_MCXW71 Platform has a reserved IRQ as its
last IRQ, this test was using this IRQ to
test an interrupt and would not fire. This change
ensures the test does not use the reserved IRQ.

Signed-off-by: Emilio Benavente <[email protected]>
@EmilioCBen EmilioCBen force-pushed the fix/mcxw71_dynamic_irq branch from 603cd2c to 0db83ab Compare November 6, 2024 22:54
@EmilioCBen EmilioCBen removed the DNM This PR should not be merged (Do Not Merge) label Nov 6, 2024
@decsny decsny added bug The issue is a bug, or the PR is fixing a bug Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc. labels Nov 6, 2024
@decsny decsny assigned mmahadevan108 and unassigned ithinuel Nov 7, 2024
@decsny decsny removed the area: ARM ARM (32-bit) Architecture label Nov 7, 2024
@hakehuang
Copy link
Contributor

board testing pass.

@decsny decsny changed the title test: arch: arm: Update IRQ being selected for Dynamic IRQ test soc: mcxw: Fix CONFIG_NUM_IRQS to not incude last reserved irq Nov 7, 2024
@mmahadevan108 mmahadevan108 merged commit c06ecf9 into zephyrproject-rtos:main Nov 8, 2024
29 of 30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug The issue is a bug, or the PR is fixing a bug platform: NXP NXP size: XS A PR changing only a single line of code Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

tests: frdm_mcxw71: tests/arch/arm/arm_irq_advanced_features fails

8 participants