[BUG] Babystepping does not work after "Clean up stepper and babystep"

[thisiskeithb]

Bug Description

After a recent hotend rebuild on an Ender-3/SKR 1.3/LPC1768/TMC5160 build, I had to redo my BLTouch/probe offsets & I noticed that babystepping was no longer working with the latest bugfix-2.0.x. The values on the "Probe Z Offset" screen increase and decrease as they should using the encoder, but moving through -5 to 5, the hotend doesn't raise or lower.

I tried enabling various options like BABYSTEP_ALWAYS_AVAILABLE, INTEGRATED_BABYSTEPPING as well as disabling software endstops completely (already disabled on Z) with no success.

Reverting to 189c101 with the same config, restores babystepping capability.

My Configurations

broken-babystepping.zip

Steps to Reproduce

1. Compile Marlin based on 75e3aa1 with the above config
2. Observe that babystepping has no effect
3. Use the same config on top of 189c101 & observe that babytepping works

Expected behavior: Babystepping to work

Actual behavior: Babystepping does not work

[thisiskeithb]

I just rebased my MK3/BTT002 (STM32F407VET6)/TMC2209 config on 75e3aa1 and babystepping does not work anymore, so this is likely not an SKR 1.3/LPC1768-specific bug.

[rado79]

Confirmed babystep not working with/without INTEGRATED_BABYSTEPPING. SKR1.3 TMC2208/9

[rado79]

#16617 (comment)

[Scope666]

Also confirmed babystep not working with/without INTEGRATED_BABYSTEPPING. SKR1.3 TMC2209

EDIT: Checked another machine, SKR 1.4 w/ 2209's, latest bugfix, also not working. (changing Z offset does nothing)

[hamster65]

My Arduino Mega with Ramps setup is also affected. Z Babystepping is not possible, nothing happens. Enabling integrated baby stepping makes no difference.

[thisiskeithb]

Since 2.0.4 was released today, I tried the latest bugfix-2.0.x (5d16497), but babystepping is still broken.

[Scope666]

Shouldn't 2.0.4 be held back until Live Z is fixed? That's kind of a big deal. Pretty much can't use a probe without live Z changes.

[thisiskeithb]

Shouldn't 2.0.4 be held back until Live Z is fixed? That's kind of a big deal. Pretty much can't use a probe without live Z changes.

It'll get fixed soon, I'm sure. Scott's been busy improving Auto Build Marlin this week.

[thinkyhead]

Thanks for keeping on this. I have been otherwise engaged. The issue should be patched now. Please confirm that it also works for you. Note that there is a known issue with babystepping on SKR boards with 2209 steppers where the stepper drivers can turn off. We're still experimenting to find a decent solution to that problem.