Skip to content

Enable UART_QSPI_PROGRAM by default in polarfire_mpfs250_hss_l2lim.config - #869

Open
dgarske wants to merge 1 commit into
wolfSSL:masterfrom
dgarske:mpfs_uart_qspi_default
Open

Enable UART_QSPI_PROGRAM by default in polarfire_mpfs250_hss_l2lim.config#869
dgarske wants to merge 1 commit into
wolfSSL:masterfrom
dgarske:mpfs_uart_qspi_default

Conversation

@dgarske

@dgarske dgarske commented Aug 21, 2026

Copy link
Copy Markdown
Member

@dgarske dgarske self-assigned this Aug 21, 2026
Copilot AI lite review requested due to automatic review settings August 21, 2026 11:48

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 updates the PolarFire MPFS250 HSS/L2-LIM example configuration to enable the UART-based QSPI programming flow by default, and aligns the target documentation to describe the UART programmer requirements and its boot-time behavior.

Changes:

  • Enable UART_QSPI_PROGRAM by default in polarfire_mpfs250_hss_l2lim.config for the documented bring-up flow.
  • Update docs/Targets.md to document that the UART QSPI programmer is enabled by default for this target, requires pyserial, and introduces a 3-second boot pause unless disabled.

Reviewed changes

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

File Description
docs/Targets.md Documents UART-based QSPI flashing prerequisites and notes the default-enabled 3s boot pause for this flow.
config/examples/polarfire_mpfs250_hss_l2lim.config Enables UART_QSPI_PROGRAM?=1 by default and documents how/why to disable it once flash contents are stable.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@dgarske dgarske assigned danielinux and wolfSSL-Bot and unassigned dgarske Aug 21, 2026
@dgarske
dgarske requested a review from danielinux August 21, 2026 15:40
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.

4 participants