Skip to content

Trinamic TMC2240 #25974

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

Merged
merged 2 commits into from
May 24, 2025
Merged

Conversation

thinkyhead
Copy link
Member

Add support for Trinamic TMC2240 stepper drivers. Assuming they have the same capabilities as TMC2209 based on the changes in drivers.h and elsewhere.

@thisiskeithb
Copy link
Member

thisiskeithb commented Jun 12, 2023

Will these be configured via UART under Marlin? They’re setup as SPI drivers under Klipper, but I don’t have a set to test with.

Edit: Alan over at BTT just responded to me and said these are setup/configured via SPI as the factory default.

TMC2240 hardware details have been uploaded: https://github.com/bigtreetech/BIGTREETECH-Stepper-Motor-Driver/tree/master/TMC2240

@thinkyhead thinkyhead force-pushed the bf2_tmc2240_PR branch 6 times, most recently from e79acce to 2dbd4a0 Compare June 19, 2023 05:57
@thisiskeithb
Copy link
Member

I've finally got a set of these drivers in so I can test them out when code is ready.

@discip
Copy link
Contributor

discip commented Oct 7, 2023

@thisiskeithb
Have you by any chance tried what @thinkyhead posted here teemuatlut/TMCStepper#270 (comment)?

I am not quite sure what exactly I am doing wrong, as I can't get it to complie. 🤷🏻‍♂️

@thisiskeithb
Copy link
Member

Have you by any chance tried what @thinkyhead posted here teemuatlut/TMCStepper#270 (comment)?

No. There's still more work that needs to be done before I test things out.

@discip
Copy link
Contributor

discip commented Oct 24, 2023

@thinkyhead
Good morning Sir,
do you still plan to add support for the TMC2240?
Please don't get me wrong, just asking, no intention to be rude.
Hope you understand. 😊

@thisiskeithb
Copy link
Member

do you still plan to add support for the TMC2240?

@discip: This PR is still open & pending, so support will be added in due time.

@discip
Copy link
Contributor

discip commented Oct 26, 2023

@thisiskeithb

support will be added in due time.

Thanks! 👍🏻
That's great news! 😊

@thinkyhead thinkyhead force-pushed the bugfix-2.1.x branch 3 times, most recently from 37d77d6 to aa44542 Compare September 28, 2024 01:10
@thinkyhead thinkyhead force-pushed the bugfix-2.1.x branch 3 times, most recently from 4354891 to efa1758 Compare March 28, 2025 01:57
@dbuezas dbuezas mentioned this pull request May 21, 2025
@thinkyhead thinkyhead force-pushed the bf2_tmc2240_PR branch 7 times, most recently from 3d2bda7 to b5b0237 Compare May 23, 2025 08:06
@thinkyhead thinkyhead marked this pull request as ready for review May 23, 2025 18:58
@dbuezas
Copy link
Contributor

dbuezas commented May 23, 2025

Wondering if slew rate and current range should be configurable at run time so the user can test noise & heat

@thinkyhead thinkyhead merged commit 202ec4b into MarlinFirmware:bugfix-2.1.x May 24, 2025
66 checks passed
@thinkyhead thinkyhead deleted the bf2_tmc2240_PR branch May 24, 2025 02:11
thinkyhead added a commit to MarlinFirmware/Configurations that referenced this pull request May 24, 2025
thinkyhead added a commit to MarlinFirmware/Configurations that referenced this pull request May 24, 2025
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.

6 participants