Reimplement battery voltage read and display for Badger 2040 W #29
Conversation
|
Thank you- However I am still very much of the opinion that it's MicroPython / Pico SDK's responsibility to implement the correct way of accomplishing this, taking into account any thread locks and other edge cases that we might otherwise encounter and smoothing over the implementation differences between Pico and Pico W. This issue in particular touches upon that idea - micropython/micropython#11185 And this reference implementation uses specific Right now I don't think we can do this correctly in just Python? |
|
That's a good point. I did end up with some issues with the pin not reading correctly in a previous iteration perhaps due to this. Where would be a good place to insert new code? In a Badger-specific board module, or patching the rp2 module? |
This measures VSYS ADC on Pi Pico W, which should be OK accuracy since the 3.3V reference is provided from the Pico W's buck regulator. Battery display is only enabled for Badger 2040 W because I didn't reimplement the battery measuring logic for Badger 2040.
Nothing special needs to be done to restore pin 29 back to output because the driver in Pico SDK automatically sets it on starting SPI comms.
Resolves #26