Update for embedded-hal to 1.0.0-rc.2

[purplethunder]

embedded-hal 1.0.0-rc.2 introduced backwards incompatible changes that break riscv builds. This change pins the crate to 1.0.0-rc.1.

error[E0432]: unresolved import `embedded_hal::delay::DelayUs`
 --> riscv/src/delay.rs:3:5
  |
3 | use embedded_hal::delay::DelayUs;
  |     ^^^^^^^^^^^^^^^^^^^^^-------
  |     |                    |
  |     |                    help: a similar name exists in the module: `DelayNs`
  |     no `DelayUs` in `delay`

For more information about this error, try `rustc --explain E0432`.

[MabezDev]

I don't think we should accept this PR, instead we should upgrade riscv to the newest release candidate. Would you mind PRing that instead?

[romancardenas]

Looks good to me!! I left you a couple of comments, please address them before accepting the PR

[romancardenas]

Hmm... maybe we need to bump riscv-rt to 1.60?

[romancardenas]

You will probably also need to update the CI to make it pass:

riscv/.github/workflows/riscv-rt.yaml

Line 14 in f8c3923

toolchain: [ stable, nightly, 1.59.0 ]

[purplethunder]

1.60.0 appears to be problematic as well.

[romancardenas]

Seems that the issue comes with riscv-target, used for building the riscv-rt crate and check the different RISC-V extensions...

I'll try to develop a quick solution to stop using this dependency and allow us to leave the MSRV to 1.60

[romancardenas]

@purplethunder check #160 and use my commit to get rid of the riscv-target build dependency. I think with that you should be able to pass the CI

[romancardenas]

Closing as #160 now solves this