Skip to content

Conversation

@SergioGasquez
Copy link
Member

@SergioGasquez SergioGasquez commented Jan 8, 2026

This PR does a few things:

  • Migrates espflash HIL runners from the current VMs to esp-hal runners (RPis)
  • Adds P4 support
    • This required adding an elf file for HIL tests (created with esp-idf)
      • This required updating the P4 bootloader (I only updated this one, maybe I should update all the targets?)
  • Adds C5 support
    • This required adding an elf file for HIL tests (created with esp-idf)
  • Adapted the test_list_ports to also accept USB Serial/Jtag ports
  • HIL tests are now run in USB and UART ports when possible
  • Runners now have an udev rule that symlinks devices to /dev/serial_ports/uart and /dev/serial_ports/usb

HIL CI Run: https://github.com/esp-rs/espflash/actions/runs/21246847880

Pending things:

  • Connect S2
  • Connect C6 uart port
  • Test uart and usb ports when possible
  • Fix C5 errors
  • Fix P4 errors
  • Edit required jobs for merging a PR

@SergioGasquez SergioGasquez force-pushed the ci/esp-hal-hil branch 2 times, most recently from 98374ea to 7605974 Compare January 8, 2026 10:20
@SergioGasquez SergioGasquez force-pushed the ci/esp-hal-hil branch 6 times, most recently from 7c1ab00 to 49b7347 Compare January 8, 2026 11:02
@SergioGasquez SergioGasquez added the skip-changelog Skips the changelog CI check label Jan 12, 2026
@SergioGasquez SergioGasquez force-pushed the ci/esp-hal-hil branch 6 times, most recently from 70e3b0e to 2462b3b Compare January 14, 2026 15:49
@SergioGasquez SergioGasquez changed the title Test esp-hal runners for c6 Use esp-hal runners for HIL Jan 14, 2026
@SergioGasquez SergioGasquez linked an issue Jan 14, 2026 that may be closed by this pull request
@SergioGasquez SergioGasquez force-pushed the ci/esp-hal-hil branch 2 times, most recently from 60adc41 to 4c545a5 Compare January 15, 2026 09:01
]

name: HIL | ${{ matrix.target.soc }}
needs: build-packages
Copy link
Member Author

Choose a reason for hiding this comment

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

We could duplicate a bit of code to speed things up, and only wait for the packages of their arch to be finished, not sure if its worth though

Copy link
Member Author

@SergioGasquez SergioGasquez Jan 15, 2026

Choose a reason for hiding this comment

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

Not sure if this is the best way, and I guess this could supersede probe-rs udev rules in the runner (https://github.com/esp-rs/esp-hal/tree/main/hil-test#rpi-setup), but having the symlink to /dev/serial_ports/usb and /dev/serial_ports/uart is quite useful I think.

I only tested this locally, I haven't yet added this file to the runners, I'll wait for some input before

@SergioGasquez SergioGasquez marked this pull request as draft January 15, 2026 11:43
@SergioGasquez SergioGasquez force-pushed the ci/esp-hal-hil branch 3 times, most recently from d7b3fdf to 2441b23 Compare January 22, 2026 11:06
This was linked to issues Jan 22, 2026
@SergioGasquez SergioGasquez marked this pull request as ready for review January 22, 2026 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog Skips the changelog CI check

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use esp-hal runners for espflash HIL Add ESP32P4 HIL support Add ESP32C5 HIL support

2 participants