Klipper3d / klipper

Klipper is a 3d-printer firmware
GNU General Public License v3.0
9.55k stars 5.33k forks source link

Tmc5160 spi issue #2086

Closed Akex2 closed 5 years ago

Akex2 commented 5 years ago

hello I have 2 tmc2208 for X and Y with uart and a tmc 5160 v1.2 from bitreetech in spi on a SKR1.3 Capture2

The config with init command seems not work properly, when i do some init with dump, i have MRES = 14 or 13 ... klippy.log Capture the cs pin is on extruder pin because i want try other place with the driver, for test it printer.cfg.log

KevinOConnor commented 5 years ago

How did you connect the jumpers on the skr v1.3? If there are any other connected devices that use spi then all must be configured in klipper. I've also seen reports of various tmc5160 step stick boards that need modification to be used properly - I don't know if that is needed for "tmc 5160 v1.2 from bigreetech".

-Kevin

Akex2 commented 5 years ago

8625ADF3-0932-4CAC-A74A-6BF850AEF82F

Akex2 commented 5 years ago

at the top is my board, and this is the setup for spi, i don't use sensorless Capture

there aren't other spi device on that board

For the V1.2 of this driver, no need modification for use in spi.

i don,t know where is the error :(

vumario commented 5 years ago

SKR board has SLP and RST pins connected together, but on TMC5160 those are CLK and SDO pins. To get SPI working correctly, you must cut off or otherwise disconnect CLK pin on your TMC5160 drivers

lenne0815 commented 5 years ago

SKR board has SLP and RST pins connected together, but on TMC5160 those are CLK and SDO pins. To get SPI working correctly, you must cut off or otherwise disconnect CLK pin on your TMC5160 drivers

This is the right answer.

KevinOConnor commented 5 years ago

Any further updates on this? It sounds like the issue is thought to be a wiring issue with the stepsticks.

-Kevin

gaggi commented 5 years ago

there aren't other spi device on that board

For the V1.2 of this driver, no need modification for use in spi.

i don,t know where is the error :(

That is a misjudgment, they added a pull-up resistor on v1.2 that pulls the CLK pin low so you don't need to solder a jumper wire from clock to ground, but you still need to remove the clock pin.

Nandox7 commented 5 years ago

I also have the SKR v1.3 with BigTreeTechTMC5160 v1.2.

@gaggi so to confirm this is the pin I need to remove in the drivers? Inkedexplorer_edDI8tHqwC_LI

--Update [Leaving here for future reference] For testing only removed the CLK pin to the Z stepper driver.

10:34:50.357: #DUMP_TMC STEPPER=stepper_z 10:34:50.360: // ========== Write-only registers ========== 10:34:50.361: // TPWMTHRS: 00000008 TPWMTHRS=8 10:34:50.362: // COOLCONF: 00000000 10:34:50.363: // IHOLD_IRUN: 00060704 IHOLD=4 IRUN=7 IHOLDDELAY=6 10:34:50.364: // TPOWERDOWN: 0000000a TPOWERDOWN=10 10:34:50.365: // ========== Queried registers ========== 10:34:50.369: // GCONF: 00000000 10:34:50.373: // CHOPCONF: 08100000 tpfd=1 MRES=8(1usteps) 10:34:50.377: // GSTAT: 10000000 10:34:50.381: // DRV_STATUS: eff00000 CSACTUAL=240 stallGuard=1 ot=1(OvertempError!) otpw=1(OvertempWarning!) s2ga=1(ShortToGND_A!) ola=1(OpenLoad_A!) olb=1(OpenLoad_B!) stst=1 10:34:50.384: // FACTORY_CONF: 10000000 10:34:50.389: // IOIN: 20000000 VERSION=0x20 10:34:50.393: // LOST_STEPS: f3e00000 10:34:50.396: // MSCNT: a0400000 10:34:50.401: // MSCURACT: e81f8000 CUR_B=31 10:34:50.404: // OTP_READ: 03f80000 10:34:50.407: // PWMCONF: f0000000 PWM_LIM=15 10:34:50.416: // PWM_SCALE: e0000000 10:34:50.419: // PWM_AUTO: e0000000 10:34:50.423: // TSTEP: 00000000 10:34:50.423: ok

10:46:28.987: #DUMP_TMC STEPPER=stepper_z 10:46:28.992: // ========== Write-only registers ========== 10:46:28.993: // TPWMTHRS: 00000008 TPWMTHRS=8 10:46:28.995: // COOLCONF: 00000000 10:46:28.997: // IHOLD_IRUN: 00060704 IHOLD=4 IRUN=7 IHOLDDELAY=6 10:46:28.998: // TPOWERDOWN: 0000000a TPOWERDOWN=10 10:46:28.999: // ========== Queried registers ========== 10:46:29.003: // GCONF: 00000004 en_pwm_mode=1 10:46:29.007: // CHOPCONF: 14410153 toff=3 hstrt=5 hend=2 tbl=2 tpfd=4 MRES=4(16usteps) intpol=1 10:46:29.013: // GSTAT: 00000005 reset=1(reset) uv_cp=1(Undervoltage!) 10:46:29.016: // DRV_STATUS: 80044000 stealth=1 CSACTUAL=4 stst=1 10:46:29.020: // FACTORY_CONF: 0000000c FACTORY_CONF=12 10:46:29.024: // IOIN: 30000050 DRV_ENN=1 SD_MODE=1 VERSION=0x30 10:46:29.028: // LOST_STEPS: 00000000 10:46:29.032: // MSCNT: 00000008 MSCNT=8 10:46:29.036: // MSCURACT: 00f7000c CUR_A=12 CUR_B=247 10:46:29.040: // OTP_READ: 0000000c OTP_FCLKTRIM=12 10:46:29.044: // PWMCONF: 00000000 10:46:29.048: // PWM_SCALE: 00000004 PWM_SCALE_SUM=4 10:46:29.051: // PWM_AUTO: 0000001d PWM_OFS_AUTO=29 10:46:29.055: // TSTEP: 000fffff TSTEP=1048575 10:46:29.055: ok

Tried other before/after and they remained the same, the Z clearly changed the settings.

@gaggi thank you

gaggi commented 5 years ago

@Nandox7 yes that is the pin that needs to be removed.

Akex2 commented 5 years ago

i already bended the pin, thanks at all for your help. i think my driver has some hardware issue.