embassy-rs / embassy

Modern embedded framework, using Rust and async.
https://embassy.dev
Apache License 2.0
4.79k stars 650 forks source link

wiznet: add version check to initialization sequence #3132

Open Qix- opened 2 days ago

Qix- commented 2 days ago

This performs a read from the Wiznet chips to check their respective version registers, returning an error if they mismatch.

Do note that this is largely untested because I'm trying to debug an issue with a chip at the moment, but it was asked that this gets PR'd in anyway since currently the initialization sequence only performs writes and no reads.

Qix- commented 13 hours ago

Seems to be working! @Dirbaio does the CI have both the W5500 and the W5100s on it?

Dirbaio commented 13 hours ago

only W5100s.