Skip to content

Add NXP QorIQ support using wolfIP to test-app#800

Open
dgarske wants to merge 5 commits into
wolfSSL:masterfrom
dgarske:nxp_t2080_wolfip
Open

Add NXP QorIQ support using wolfIP to test-app#800
dgarske wants to merge 5 commits into
wolfSSL:masterfrom
dgarske:nxp_t2080_wolfip

Conversation

@dgarske

@dgarske dgarske commented Jun 16, 2026

Copy link
Copy Markdown
Member

No description provided.

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

Adds an optional wolfIP-based network test path to the wolfBoot test-app, targeting NXP QorIQ (FMan) and LS1028A (ENETC) platforms to validate link/PHY bring-up, DHCP, and optionally TFTP fetch or a TCP throughput server.

Changes:

  • Introduces wolfip_tftp_test glue (DHCP bring-up + optional TFTP RRQ / speed test) and hooks it into NXP test-app entrypoints when enabled.
  • Extends test-app/Makefile to optionally build/link wolfIP sources (and TFTP client) with target-specific port selection and flags.
  • Documents usage via updated NXP example config files.

Reviewed changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
test-app/wolfip_tftp_test.h Declares the optional wolfIP test entrypoint for the test-app.
test-app/wolfip_tftp_test.c Implements DHCP bring-up plus optional TFTP fetch and TCP speed-test modes for NXP ports.
test-app/Makefile Adds optional wolfIP build integration, object rules, and target-based port selection.
test-app/app_nxp_t2080.c Calls the wolfIP test when ENABLE_WOLFIP is enabled.
test-app/app_nxp_t1040.c Calls the wolfIP test when ENABLE_WOLFIP is enabled.
test-app/app_nxp_t1024.c Calls the wolfIP test when ENABLE_WOLFIP is enabled.
test-app/app_nxp_ls1028a.c Switches to HAL-provided UART and calls the wolfIP test when enabled.
config/examples/nxp-t2080.config Documents how to enable/configure wolfIP testing for T2080.
config/examples/nxp-t1040.config Documents how to enable/configure wolfIP testing for T1040.
config/examples/nxp-t1024.config Documents how to enable/configure wolfIP testing for T1024.
config/examples/nxp-ls1028a.config Documents how to enable/configure wolfIP testing for LS1028A (incl. DDR/partition sizing notes).

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

Comment thread test-app/wolfip_tftp_test.c Outdated
Comment thread test-app/wolfip_tftp_test.c
Comment thread test-app/wolfip_tftp_test.c
Comment thread test-app/wolfip_tftp_test.c
Comment thread test-app/wolfip_tftp_test.c Outdated
Comment thread test-app/Makefile
@dgarske
dgarske force-pushed the nxp_t2080_wolfip branch from 789c91f to a71a549 Compare June 22, 2026 18:32
@dgarske
dgarske force-pushed the nxp_t2080_wolfip branch 3 times, most recently from c0928e6 to ac33896 Compare July 7, 2026 22:01
@dgarske
dgarske requested a review from Copilot July 8, 2026 00:00

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

Copilot reviewed 26 out of 26 changed files in this pull request and generated 5 comments.

Comment thread tools/scripts/nxp_t1040/t1040_flash.cmm Outdated
Comment thread tools/scripts/nxp_t1040/t1040_flash.cmm Outdated
Comment thread src/update_ram.c Outdated
Comment thread src/boot_aarch64_start.S Outdated
Comment thread hal/nxp_ls1028a.c
@dgarske
dgarske force-pushed the nxp_t2080_wolfip branch from ac33896 to 60ee80a Compare July 9, 2026 16:46
@dgarske
dgarske requested a review from Copilot July 9, 2026 21:40
@dgarske
dgarske marked this pull request as ready for review July 9, 2026 21:40

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

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

Comment thread test-app/wolfip_tftp_test.c Outdated
Comment thread test-app/wolfip_tftp_test.c Outdated
@dgarske
dgarske force-pushed the nxp_t2080_wolfip branch from 60ee80a to bdf722f Compare July 10, 2026 22:53
@dgarske dgarske assigned danielinux and wolfSSL-Bot and unassigned dgarske Jul 11, 2026
@dgarske
dgarske requested a review from danielinux July 11, 2026 00:11
Comment thread hal/nxp_t2080.c
Comment thread docs/Targets.md
@danielinux danielinux assigned dgarske and unassigned danielinux Jul 16, 2026
@dgarske
dgarske force-pushed the nxp_t2080_wolfip branch from bdf722f to ea6b3e0 Compare July 20, 2026 20:54
@dgarske
dgarske requested a review from danielinux July 20, 2026 21:40
@dgarske dgarske assigned danielinux and unassigned dgarske Jul 20, 2026
@dgarske
dgarske force-pushed the nxp_t2080_wolfip branch from ea6b3e0 to 888cc46 Compare July 23, 2026 21:37
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