search

ClemensElflein / OpenMower

Let's upgrade cheap off-the-shelf robotic mowers to modern, smart RTK GPS based lawn mowing robots!
Other
4.64k stars 274 forks source link

Bugfix/wt901 via SerialPIO #96

Closed Apehaenger closed 1 month ago

Apehaenger commented 3 months ago

Got WT901 via SerialPIO now working reliable. Also tested if WT901_INSTEAD_OF_SOUND still works = yes. It's still draft as I wait for Deyan to test it also (he's on the road for a week), but like to comment before forget ;-)

Neopixel: For easier communication-error identification (as well as helper during my debugging), I added a magenta color to Neopixel which flashes whenever an overflow or packet error occurs (Pico's LED was a bad idea by me, as it's hidden by RTK2B). If accepted and merged, we probably should also add other SerialPIO devices to check if they also get affected by the SerialPIO timing issue. "Sound" probably also, but the sound packets are much smaller. CoverUI?! For this I had to slow down the normal Emergency flash rate by one more bit. Without this, visual identification wasn't possible and you quickly got blind :-/

This PR also contain a tiny nv_config fix for a compiler warning, as long as #80 isn't merged.

Apehaenger commented 2 months ago

Deyan reported back yesterday, that this patch also works for his wt901, and that he's connected his WT901 back to j20 now.