Clock monitor HIL test

[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

Adds an HIL test for verifying the estimated clock freq. Not sure how useful this test is, tbh.

Testing

Tested locally on S3, C6 and H2.

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

[jessebraham]

Thanks for the contribution, and sorry to ask this, but does it really make sense to test this? The clock monitor isn't even close to accurate to begin with, and I also don't really see what we gain by testing it.

[SergioGasquez]

but does it really make sense to test this?

Not really sure, tbh. I am happy to close the PR if we think its not worth

[MabezDev]

Thanks for the contribution, and sorry to ask this, but does it really make sense to test this? The clock monitor isn't even close to accurate to begin with, and I also don't really see what we gain by testing it.

I would say yes in this case, be cause we actually use it internally (that's how we removed the xtal features), if this silently starts being wrong we'll have some fun debugging :D.

[bjoernQ]

besides what Clippy says this looks good to me

[JurajSadel]

LGTM, thanks!

[MabezDev]

LGTM, thanks! Just needs a rebase.