rust-embedded/embedded-hal

Change reference implementation to something more maintained

TheZoq2 opened this issue · 10 comments

While scrolling through the docs tooday, I noticed the line

The stm32f30x-hal crate contains a reference implementation of this HAL.

As far as I can tell, this crate hasn't been updated in two years, and is kind of superseded by the stm32-rs maintained version.

Therefore I think it would make sense to switch the reference implementation to something else, or to remove the line completely.

@TheZoq2 Which docs? Can you do a PR with the fix?

I guess I should have clarified that :P. The main page of the docs on docs.rs https://docs.rs/embedded-hal/0.2.3/embedded_hal/#reference-implementation

I'll gladly do a PR, but i'm not sure what to replace it with. Is there still such a thing as a reference implementation? Perhaps we could link to the HAL section of awesome-embedded-rust instead

Good question. @jamesmunns can we stick this on the agenda for Tuesday? Personally, I think there should be a reference implementation - perhaps for lm3s811evb or lm3s6965evb emulated by QEMU.

@thejpster you probably want to ping @therealprof this week, I will likely not be there this week (same with the other issues you tagged).

@thejpster @jamesmunns

Good question. @jamesmunns can we stick this on the agenda for Tuesday? Personally, I think there should be a reference implementation - perhaps for lm3s811evb or lm3s6965evb emulated by QEMU.

I can rewrite this part of examples and remove not common code. Btw now I creating new clean PR for embedded-hal. Thanks

I noticed this today as well. I went back and found the wg minutes.
This issue was mentioned, but there were no notes on the result of the conversation.
Does anyone remember the result?

https://github.com/rust-embedded/wg/blob/master/minutes/2019-10-29.md

Looks like this is even more wrong in the 1.0.0.alpha version since the mentioned f3 crate is stuck at 0.2.0

Linking to #177

Now the stm32f3 is used directly. Closing.

The reference to stm32f30x-hal is still present in the docs. Reopening