bigtreetech / BIGTREETECH-SKR-mini-E3

BIGTREETECH SKR-mini-E3 motherboard is a ultra-quiet, low-power, high-quality 3D printing machine control board. It is launched by the 3D printing team of Shenzhen BIGTREE technology co., LTD. This board is specially tailored for Ender 3 printer, perfectly replacing the original Ender3 printer motherboard.
2k stars 1.98k forks source link

[BUG] Klipper SKR Mini 3 V3.0 supplied default printer.cfg file has error in the BL touch section #773

Open misterkaplan opened 3 months ago

misterkaplan commented 3 months ago

BigTreeTech supplies an example printer.cfg file for the SKR Mini 3 v3.0 for Klipper here: https://github.com/bigtreetech/BIGTREETECH-SKR-mini-E3/blob/master/firmware/V3.0/Klipper/SKR-mini-E3-V3.0-klipper.cfg

But there is an error in it.

Here is the section copied and pasted from the file: [bltouch] sensor_pin: PC14 control_pin: PA1 x_offset: -40 y_offset: -10 z_offset: 3.0

speed:

The error is in the _sensorpin: line. It's supposed to be:

sensor_pin: ^PC14

The ^ is missing in the original. According to the official Klipper documentation, "Most BL-Touch devices require a pullup on the sensor pin (prefix the pin name with ^)." https://github.com/Klipper3d/klipper/blob/master/docs/BLTouch.md

Once I added the ^, it worked correctly. Without the ^, I got lots of errors like "BLTouch failed to verify sensor state". I also noticed that the pin on the BL touch would not retract back up when it touched the bed (or even my finger).

Please fix the default file.

767