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

FYSETC S6 with TMC2209 #131

Closed
terjeio opened this issue Aug 5, 2023 Discussed in #130 · 4 comments
Closed

FYSETC S6 with TMC2209 #131

terjeio opened this issue Aug 5, 2023 Discussed in #130 · 4 comments

Comments

@terjeio
Copy link
Contributor

terjeio commented Aug 5, 2023

Discussed in #130

Originally posted by hartmutgw2 August 2, 2023
Hello,

since weeks I try to get my MPCNC with Fysetc S6 V2.1 and Fysetc TMC2209 V3.1 drivers working.
Without TRINAMIC_ENABLE=2209 it works, however I cannot define a hold current for Z stepper and due to weight of the spindle this configuration doesn't work for me.

When TMC2209 is enabled the board does not show up with lsusb so no communication to controller.
Jumper for UART communication to stepper drivers is set on the board and the controller worked with klipper and tmc2209 support before so I assume it is not hardware related.

If I flash the non-Trinamic version it works again and I'm wondering what I've missed.

Anybody here with a similar configuration or any hint what I'm doing wrong?

Thanks,
Hartmut

terjeio added a commit that referenced this issue Aug 5, 2023
@hartmutgw2
Copy link

hartmutgw2 commented Aug 5, 2023

Many thanks for your help @terjeio !!!
Merged with my pin mapping and I'm able to communicate.
note that PROBE and RESET use the same PORT/PIN in fysetc_s6_map.h

M122

[TRINAMIC]
                      X       Y       Z
Driver          TMC2209 TMC2209 TMC2209
Set current           0       0       0
RMS current        1767    1767    1767
Peak current       2498    2498    2498
Run current       31/31   31/31   31/31
Hold current      15/31   15/31   15/31
CS actual         15/31   15/31   15/31
PWM scale            15      14      16
--- snip ---

I plan to test the new version in the next week.

Hartmut

@terjeio
Copy link
Contributor Author

terjeio commented Aug 6, 2023

Great, I'll assign Reset to THERM1 in the next commit. Thanks for reporting.

@hartmutgw2
Copy link

Do you think THERM1 is a good idea as this is an analog port?

I use 5 steppers (dual ganged X&Y) and 6 endstops (X-,Y-,Z-,X+,Y+,Z+)
so as I already use all of the endstop headers I will use one expansion header for Probe, Reset, Hold and Start

@terjeio
Copy link
Contributor Author

terjeio commented Aug 8, 2023

Do you think THERM1 is a good idea as this is an analog port?

It is a digital port until reconfigured. There is a large capacitor to ground that might cause issues, mainly "slow" restart if connected to a NC switch. It may also cause the switch to wear out a tad faster from the discharge current...

@terjeio terjeio closed this as completed Aug 26, 2023
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

No branches or pull requests

2 participants