Skip to content

Add CI coverage for FIT_RAMDISK - #858

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

Add CI coverage for FIT_RAMDISK#858
dgarske wants to merge 1 commit into
wolfSSL:masterfrom
dgarske:ci_fit_ramdisk

Conversation

@dgarske

@dgarske dgarske commented Aug 14, 2026

Copy link
Copy Markdown
Member

Add zynqmp with FIT_RAMDISK to cover the src/fdt.c initrd paths

@dgarske dgarske self-assigned this Aug 14, 2026
Copilot AI lite review requested due to automatic review settings August 14, 2026 17:24

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

Warning

Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.

Adds CI coverage for the WOLFBOOT_FIT_RAMDISK build paths by introducing an additional ZynqMP build variant that enables FIT_RAMDISK and sets a RAM disk load address.

Changes:

  • Add a new GitHub Actions job to build zynqmp_sdcard with FIT_RAMDISK=1.
  • Set WOLFBOOT_LOAD_RAMDISK_ADDRESS in CI to exercise initrd-related code paths in src/fdt.c.

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

Comment thread .github/workflows/test-configs.yml Outdated
@dgarske dgarske assigned danielinux and wolfSSL-Bot and unassigned dgarske Aug 14, 2026
@dgarske
dgarske requested a review from danielinux August 14, 2026 22:08
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