Enable C3, H2, S2 and S3 HIL

[SergioGasquez]

Thank you for your contribution!

We appreciate the time and effort you've put into this pull request.
To help us review it efficiently, please ensure you've gone through the following checklist:

Submission Checklist 📝

• I have updated existing examples or added new ones (if applicable).
• My changes were added to the CHANGELOG.md in the proper section.

Extra:

• I have read the CONTRIBUTING.md guide and followed its instructions.

Pull Request Details 📖

Description

• Enables C3, H2, S2 and S3 HIL.
• Updates the version of probe-rs being used.

Limitations

Some test are disabled for some targets, see: #1524 and #1509

Important

Once merged we need to add the HIL Test | esp32c3, HIL Test | esp32h2, HIL Test | esp32s2 and HIL Test | esp32s3 test to the required checks

Testing

Manually triggered the HIL tests workflow: https://github.com/esp-rs/esp-hal/actions/runs/8891148274

[bjoernQ]

LGTM - however I'm a noob in regards to CI

[SergioGasquez]

Converted to draft as uart is now failing for C3 after the rebase, will investigate the issue. Seems like, when using the RcFast clock source, the code just hangs in https://github.com/esp-rs/esp-hal/blob/main/esp-hal/src/uart.rs#L1108 and then, the test times out.

[SergioGasquez]

Disabled the following test for the following targets:

• delay: H2 and S2
• spi_full_duplex_dma: S2

All the issues are covered in #1509 and #1524

HIL are now running green: https://github.com/esp-rs/esp-hal/actions/runs/8891148274 so this should be ready fore review again!

[jessebraham]

Would really like to see this implemented in the xtask instead, please.

[SergioGasquez]

Me too, this bash code and the one preparing the artifacts looks quite bad to me. Any suggestions on how to integrate this with xtask other than creating a subcommand that points to a folder, and it executes all the elfs on that folder?

[SergioGasquez]

Added a subcommand to run all the elfs in a folder in daf8746. Here is a HIL run: https://github.com/esp-rs/esp-hal/actions/runs/8924001488 with that changes, it increases the time it takes for the run as it needs to build the xtask package.

[SergioGasquez]

I've also tested with cargo build --out-dir... but since the elfs are generated under the target/.../deps and are not the "final" artifacts it does not help. My idea was to modify cargo xtast build-tests to add the --out-dir argument and avoid all this bash code.

[JurajSadel]

LGTM, left one nitpick comment, sorry. Not going to approve so it's not accidentally merged before remaining discussion threads are resolved.

[jessebraham]

LGTM, thanks!