Skip to content

arch/arm/src/imxrt: keep the boot ROM's XIP FlexSPI clock setting - #20117

Merged
acassis merged 2 commits into
apache:masterfrom
PetervdPerk-NXP:pr-imxrt117x-flexspi-clk
Sep 11, 2026
Merged

acassis merged 2 commits into
apache:masterfrom
PetervdPerk-NXP:pr-imxrt117x-flexspi-clk

Conversation

@PetervdPerk-NXP

Copy link
Copy Markdown
Contributor

Summary

When executing in place from flash, the XIP FlexSPI clock must not be reconfigured during the initial clock setup. The boot ROM configures the clock for its flash read sequence, and changing it before the board installs a suitable high-speed read sequence can break instruction fetch. The board's flash setup may reconfigure the clock afterward.

Impact

IMXRT117X

Testing

PX4 FMU-V6XRT board

@github-actions github-actions Bot added Arch: arm Issues related to ARM (32-bit) architecture Size: S The size of the change in this PR is small labels Sep 11, 2026
When executing in place from flash, the XIP FlexSPI clock must not be
reconfigured during the initial clock setup. The boot ROM configures the
clock for its flash read sequence, and changing it before the board installs
a suitable high-speed read sequence can break instruction fetch. The board's
flash setup may reconfigure the clock afterward.

Signed-off-by: Peter van der Perk <peter.vanderperk@nxp.com>
@github-actions github-actions Bot added Size: M The size of the change in this PR is medium and removed Size: S The size of the change in this PR is small labels Sep 11, 2026
Fix nxstyle issues

Signed-off-by: Peter van der Perk <peter.vanderperk@nxp.com>
@github-actions

Copy link
Copy Markdown

MemBrowse Memory Report

No memory changes detected for:

@acassis
acassis merged commit ad176ac into apache:master Sep 11, 2026
38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Arch: arm Issues related to ARM (32-bit) architecture Size: M The size of the change in this PR is medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants