atanisoft / ESP32CommandStation

An ESP32 based DCC Command Station with integrated OpenLCB (LCC) --- NOTE: this project is not under active development.
https://atanisoft.github.io/ESP32CommandStation/
GNU General Public License v3.0
90 stars 34 forks source link

Add option to disable current sensing for Arduino motor shield #109

Closed Aleksandr-ru closed 2 years ago

Aleksandr-ru commented 2 years ago

I'm using Wemos D1 R32 board with Arduino L298 motor shield and always getting [ULP-ADC] OPS short detected message.

If I comment out this check in components/ULPADC/UlpAdc.cpp everything runs fine. If I disconnect half of the motor shield pins - fine too.

Tried to add jumpers from A0 and A1 to another A* pins according to available configuration options - no use.

atanisoft commented 2 years ago

If you select ESP32 Command Station Configuration -> ESP32 with Arduino Motor Shield (OPS and PROG) the CS will be automatically configured for the L298 to work with that ESP32 board. You will only need to add a jumper from A0 ->A2 and A1 -> A3.

When selecting the above configuration option it will ignore all other configuration options related to pin assignments.

Aleksandr-ru commented 2 years ago

add a jumper from A0 ->A2 and A1 -> A3.

This is magic, it works now, thank you!

But why, why nowhere no big warning like "to get started with L298 motor shield add jumpers"...

atanisoft commented 2 years ago

The docs cover jumpers in the supported hardware: https://github.com/atanisoft/ESP32CommandStation/blob/master/docs/supported_hardware.md#arduino-motor-shield-required-jumpersv

It should also likely be printed in the configuration section as well.

Aleksandr-ru commented 2 years ago

Thanks again )