Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated BL Touch PIN for CR touch #718

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Chaosfelix4451
Copy link

Here is a updated version of the config with the bl touch pin so the CR touch work again. [Issue]:
Z axis crashing into bed and cr touch didnt worked .

[Solution]
Adding a little ^ to the Pin make the cr touch working (could be different for other abl models

Requirements

  • Filling out this template is required. Pull Requests without a clear description may be closed at the maintainers' discretion.

Description

Benefits

Related Issues

Here is a updated version of the config with the bl touch pin so the CR touch work again.
[Issue]:
Z axis crashing into bed and cr touch didnt worked .

[Solution]
Adding a little ^ to the Pin
@andersevenrud
Copy link

andersevenrud commented Jul 15, 2023

This really should get merged ASAP.

My toolhead crashed into the bed when I set up Klipper for the first time :(

I probably should have used my finger to check the trigger before trying to probe the bed, so that's on me... but quite frustrating having to debug and finally figure out it was related to official documentation/configuration.


From the Klipper docs 1:

Most BL-Touch devices require a pullup on the sensor pin

Footnotes

  1. https://www.klipper3d.org/BLTouch.html#connecting-bl-touch

@Stoutscientist
Copy link

Same here.... It should really just work, I mean they have had multiple years to fix this, I'm starting to regret my purchase..

@do3cc
Copy link

do3cc commented Oct 29, 2024

Hi, this is also true for BLTouch. The default config works in Open Drain mode, which requires the Pullup resistor to avoid a floating signal.
This is true for at least BLTouch 3.0, older BLTouch would be from before 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants