Skip to content

Conversation

@roma-jam
Copy link

@roma-jam roma-jam commented Oct 17, 2025

Increase the verification during tinyusb development by adding the esp32p4 as a target to run test application from ESP-USB and USB Device examples from ESP-IDF.

ESP-USB Test Apps - Build & Run

  • Added esp32p4 target to Run step

ESP-IDF USB Device examples - Build & Run

  • Added esp32p4 target to Run step

@roma-jam roma-jam self-assigned this Oct 17, 2025
@roma-jam roma-jam changed the title feature(ci): Adding esp32p4 target to Run step feature(ci): Adding esp32p4 target to Run step [v0.18] Oct 17, 2025
@roma-jam roma-jam marked this pull request as ready for review October 17, 2025 15:52
Copy link
Collaborator

@peter-marcisovsky peter-marcisovsky left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@igi540 igi540 left a comment

Choose a reason for hiding this comment

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

LGTM.

@roma-jam roma-jam requested a review from Copilot October 21, 2025 20:28
Copy link

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 adds esp32p4 target support to CI workflows for testing TinyUSB with ESP-IDF, expanding hardware validation coverage while ensuring version compatibility through appropriate exclusion rules.

Key Changes:

  • Added esp32p4 to the test matrix for both ESP-USB test apps and ESP-IDF USB device examples
  • Introduced version exclusions to prevent esp32p4 testing on IDF versions prior to 5.3 (where esp32p4 support begins)
  • Extended IDF version coverage to include release-v5.1 and release-v5.2 in the examples workflow

Reviewed Changes

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

File Description
.github/workflows/build_and_run_idf_examples.yml Adds esp32p4 target to matrix and exclusion rules for pre-5.3 IDF versions; expands IDF version coverage to 5.1 and 5.2
.github/workflows/build_and_run_esp_usb_test_apps.yml Adds esp32p4 target to matrix with exclusion rules for IDF versions before 5.3

@roma-jam roma-jam force-pushed the fix/override_tinyusb_in_network_example branch from 4a85b18 to 8486e5c Compare October 22, 2025 11:13
@roma-jam roma-jam force-pushed the feat/ci_add_run_on_esp32p4 branch from 7631a42 to 4e7bb78 Compare October 22, 2025 13:09
@roma-jam roma-jam merged commit d38e814 into fix/override_tinyusb_in_network_example Oct 23, 2025
37 checks passed
@roma-jam roma-jam deleted the feat/ci_add_run_on_esp32p4 branch October 23, 2025 18:17
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