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

Add PICO_MINIMAL_VECTOR_TABLE compile option #2233

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

will-v-pi
Copy link
Contributor

This option will create a vector table which only contains NMI and HardFault vectors, and a single bkpt instruction for them to point to. I believe that this is the smallest size vector table that can be used safely with RP2350, as the NMI and HardFault interrupts cannot be disabled, and by default BusFaults etc all escalate to HardFaults so don't require separate vectors.

When this is enabled you cannot use any interrupts - I haven't disabled them as I wasn't sure of the best way to do that, but I've mentioned that in the PICO_CONFIG comment.

This supercedes #2200

Creates vector tables which only contains NMI and HardFault vectors, and a single bkpt instruction for them to point to
@lurch
Copy link
Contributor

lurch commented Feb 20, 2025

When this is enabled you cannot use any interrupts - I haven't disabled them as I wasn't sure of the best way to do that

Would the disable_interrupts() that got added in #2276 be useful here?

@will-v-pi
Copy link
Contributor Author

When this is enabled you cannot use any interrupts - I haven't disabled them as I wasn't sure of the best way to do that

Would the disable_interrupts() that got added in #2276 be useful here?

Yes, although I think @kilograham would need to suggest the best place to put a call to that function?

@will-v-pi will-v-pi requested a review from kilograham February 24, 2025 14:31
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.

2 participants