Skip to content

Vorago VA416x0 port improvements for flash layout and preparing for boot defaults#679

Merged
danielinux merged 2 commits intowolfSSL:masterfrom
dgarske:vorago_va416x0_20260203
Feb 5, 2026
Merged

Vorago VA416x0 port improvements for flash layout and preparing for boot defaults#679
danielinux merged 2 commits intowolfSSL:masterfrom
dgarske:vorago_va416x0_20260203

Conversation

@dgarske
Copy link
Contributor

@dgarske dgarske commented Feb 3, 2026

Fix issue with debug flash layout and improve hal_prepare_boot to restore more hardware settings
ZD 21128

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR tightens the VA416x0 port by fixing the debug FRAM/flash layout and ensuring hal_prepare_boot() leaves the application in a cleaner reset-like hardware state.

Changes:

  • Removed an unused variable in test_ext_flash() and enhanced hal_prepare_boot() on VA416x0 to flush and fully disable the debug UART, stop SysTick, clear its pending interrupt, and disable EDAC-related NVIC IRQs and SYSCONFIG IRQ enables.
  • Adjusted the example VA416x0 FRAM layout options (ML-DSA and Debug profiles) so boot/update/swap regions no longer overrun the 256KB FRAM and better align with documented sizes.
  • Changed the example config default to WOLFBOOT_RESTORE_CLOCK?=1 and added optional debug flags for external flash testing.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
hal/va416x0.c Cleans up unused variable and extends hal_prepare_boot() to restore more hardware state (UART, SysTick, EDAC IRQs, SYSCONFIG IRQs) before jumping to the application.
config/examples/vorago_va416x0.config Corrects commented ML-DSA and Debug partition layouts to fit within 256KB FRAM and updates the default to restore the clock on boot; also adds commented debug flags for FRAM/ext flash testing.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@danielinux danielinux merged commit 7258081 into wolfSSL:master Feb 5, 2026
413 of 415 checks passed
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.

3 participants