Skip to content

NVIC line count in the Devicetree; athrill_device_target_node_u32 - #85

Merged
tmori merged 1 commit into
mainfrom
feat/62-nvic-from-dt
Nov 8, 2026
Merged

tmori merged 1 commit into
mainfrom
feat/62-nvic-from-dt

Conversation

@tmori

@tmori tmori commented Nov 8, 2026

Copy link
Copy Markdown
Contributor

For hakoniwalab/athrill-target-stm32#62.

What changed

  • boards/pixhawk4/stm32f765-athrill.dtsi and boards/pixhawk6c/stm32h743-athrill.dtsi give the NVIC node athrill,num-irqs: 110 on F7 and 150 on H7, the values the STM32 target hard-coded. Zephyr's DTSI has arm,num-irq-priority-bits but not the line count, which Zephyr takes from Kconfig.
  • New runtime function athrill_device_target_node_u32(). It reads a 32-bit cell of a property of the first enabled node with a given compatible, so a target can read its own nodes, such as the NVIC. It is a function, not an ops-struct field, so plugins are unaffected.
  • The Pixhawk 4 and 6C README hashes are updated; they were already stale before this change.

Checks
ctest passes 566/566. The four STM32 gates with the target change (athrill-target-stm32 PR) all match their references:

  • PX4 bench: 2,135,921,900 instructions and the tick order;
  • PX4 SIH: the published run;
  • 6C armed: every value (5,265,357,395 instructions);
  • ArduPilot bench: 722,036,516 steps, final pc 0x08138a48.

🤖 Generated with Claude Code

The SoC layers give the NVIC node athrill,num-irqs (F7 110, H7 150, the
values the STM32 target hard-coded); the Zephyr DTSI already has
arm,num-irq-priority-bits. athrill_device_target_node_u32 lets a target
read a property of its own nodes. The Pixhawk README hashes are updated
(they were stale before).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@tmori
tmori merged commit a2ccc82 into main Nov 8, 2026
10 checks passed
@tmori
tmori deleted the feat/62-nvic-from-dt branch November 8, 2026 07:21
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.

1 participant