teemuatlut / TMCStepper

MIT License
514 stars 202 forks source link

TMC5160 PWMCONF #43

Closed kAdonis closed 5 years ago

kAdonis commented 5 years ago

The TMC2160Stepper::PWMCONF() Function is trying to read from a write only register, and overwrites the correct value of PWMCONF_register.sr with zero I changed it to:

uint32_t TMC2160Stepper::PWMCONF() { return PWMCONF_register.sr; }

In marlin M122 V now shows the correct value for PWMCONF (PWMCONF is write only for TMC2160, TMC5160, and TMC5161)

kAdonis commented 5 years ago

Wow, that was a fast response! :muscle: