Skip to content

CoreXY sensorless homing#17972

Merged
thinkyhead merged 2 commits intoMarlinFirmware:bugfix-2.0.xfrom
thinkyhead:bf2_core_sensorless_PR
May 14, 2020
Merged

CoreXY sensorless homing#17972
thinkyhead merged 2 commits intoMarlinFirmware:bugfix-2.0.xfrom
thinkyhead:bf2_core_sensorless_PR

Conversation

@thinkyhead
Copy link
Copy Markdown
Member

@thinkyhead thinkyhead commented May 12, 2020

Suggested by @deltaford in #17838

Background: Sensorless endstop states are crudely gathered from driver DIAG pins or SPI. Later the bits are processed according to whether axes are moving. The Core direction checks are good, and sensorless homing is enabled correctly for Core kinematic drivers, but in the endstop processing logic only one of the sensorless endstops that could have been triggered gets tested.

Solution: This PR patches the issue by checking all the endstop bits that could have gotten triggered during sensorless homing on different Core arrangements.

@thinkyhead thinkyhead force-pushed the bf2_core_sensorless_PR branch 3 times, most recently from 26d8051 to 11c6682 Compare May 12, 2020 09:57
@GMagician
Copy link
Copy Markdown
Contributor

GMagician commented May 12, 2020

May this be useful also for #17687? (multiendstop check)?

@guruathwal
Copy link
Copy Markdown
Contributor

Tested on COREXY with TMC2130 SPI with SPI_ENDSTOPS enabled on BTT SKR 1.4 Board.
not much difference noticed in sound/vibration when homing.
With SPI_ENDSTOPS enabled no noticeable difference while homing with or without changes in this PR.
Although these changes are only for homing with SPI_ENDSTOP. SPI_ENDSTOP have a little bit slow response in comparison to sensorless homing without it and without changes in this PR. This may be due to the Software SPI.

Driver settings:
Sensorless homing on X & Y
TMC_DEBUG Enabled
HOLD_MULTIPLIER 0.6
INTERPOLATE true

X_CURRENT       600
X_CURRENT_HOME  200
X_MICROSTEPS     16
X_RSENSE          0.11

Y_CURRENT       600
Y_CURRENT_HOME  200
Y_MICROSTEPS     16
Y_RSENSE          0.11

X_STALL_SENSITIVITY  8
Y_STALL_SENSITIVITY  8

@thinkyhead thinkyhead force-pushed the bf2_core_sensorless_PR branch from 11c6682 to 251ccb9 Compare May 13, 2020 19:30
@thinkyhead
Copy link
Copy Markdown
Member Author

Oops, I didn't quite have the logic right. Please test once more.

@thinkyhead thinkyhead merged commit 6ae7a40 into MarlinFirmware:bugfix-2.0.x May 14, 2020
@thinkyhead thinkyhead deleted the bf2_core_sensorless_PR branch May 14, 2020 18:00
@guruathwal
Copy link
Copy Markdown
Contributor

@thinkyhead with 251ccb9 there is some noticeable reduction in stepper sound while homing.👍

vgadreau pushed a commit to vgadreau/Marlin that referenced this pull request May 29, 2020
jmp0x0000 pushed a commit to jmp0x0000/Marlin that referenced this pull request Aug 7, 2020
HairingX pushed a commit to HairingX/Marlin that referenced this pull request Jun 16, 2021
@fiveorders
Copy link
Copy Markdown

Hello, I searched the issues and couldn't find a solution for this particular problem. I am using Octopus Board with UART 2226 drivers. My X Y currents are 800 and my homing currents are 600. Stall sensibility is 25 for X and 43 for Y.

Whatever combination I tried last homing axis regardless Y or X, makes the other axis jump to a random distance from point 0. If X homes last, Y axis jumps between 0.5-4mm away from point 0. The only way to overcome, or let's say reduce the jump distance is reducing the homing speed. Reducing currents results in false triggers, and also reducing the Bump sensitivity also creates false triggers.

If I make the homing speed as 10x60 and 10x60, it is acceptable. but such speeds are killing on a 300x300 build plate.

So I think only one motor is monitored. I am on MArlin 2.9.1 right now, I tried nightly builds as well.

Any recommendation will be great. Many thank you in advance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants