Skip to content

Conversation

lurch
Copy link
Contributor

@lurch lurch commented Jul 25, 2025

extract_configs.py now supports reading min, max or default values from a #define value (which reduces false-positives)

This means for an entry like:

// PICO_CONFIG: PICO_NUM_VTABLE_IRQS, Number of IRQ handlers in the vector table - can be lowered to save space if you aren't using some higher IRQs, type=int, default=NUM_IRQS, group=hardware_irq

the tools/exract_configs.py script is now able to (internally) resolve the default value to 32 (because src/rp2040/hardware_regs/include/hardware/platform_defs.h says #define NUM_IRQS _u(32)), and so you no longer get a warning like:

INFO:__main__:PICO_NUM_VTABLE_IRQS at ./src/rp2_common/pico_platform_common/include/pico/platform/common.h:33 has non-integer default value "NUM_IRQS"

(the contents of the TSV files that get created are unchanged though)

extract_configs.py now supports reading min, max or default values
from a #define value (which reduces false-positives)
@kilograham kilograham added this to the 2.2.0 milestone Jul 25, 2025
@kilograham kilograham merged commit 4f5ebb8 into develop Jul 25, 2025
11 checks passed
@kilograham kilograham deleted the extract_refactor branch July 25, 2025 14:14
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