-
-
Notifications
You must be signed in to change notification settings - Fork 448
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
feat: adds TMC2240 support #1133
feat: adds TMC2240 support #1133
Conversation
Signed-off-by: Pedro Lamas <[email protected]>
Signed-off-by: Pedro Lamas <[email protected]>
217379e
to
1b50f6e
Compare
@pedrolamas , may I ask why the temperature reporting is so infrequent? |
@brotherdust Temperature report from TMC2240 is only available when the driver is enabled, the other times Klipper returns |
@pedrolamas , you, sir, are a paragon of kickassery! Thanks for the clarification! |
@pedrolamas , If I can manage to get stepper temperature estimation implemented, it might call for a differentiated label. Perhaps the driver temp should be something like "Stepper [axis] Driver" and the stepper motor could be something like "Stepper [x] Motor (estimated)"? What say you? |
Sounds reasonable to me! I guess it will also depend on how the data will be reported by Klipper... |
Yeah I was reading over the datasheet a couple of days ago in preparation of this and... the implementation details of this function are pretty vague. I'm not sure if it's actually a hardware or software thing yet. Datasheet would seem to indicate that the nerd knobs are available, but it's something that has to be calculated. I'll have to ask TMC, I guess. |
@eldeeb91 I assume you also upgraded Klipper and Moonraker to the latest versions? If so, can you upload what you get from these 2 urls:
Note: replace |
@eldeeb91 there's 2 spaces on your config between Instead of |
@Mosher23 I don't have my TMC2240 in production use (I currently only use them for testing purposes with Fluidd) so I don't have a benchmark here... my advice is that you jump onto Klipper Discord and ask around on |
@eldeeb91 What StepSticks do you use? |
@Mosher23 I also have BTT, check my configuration on my comment up #1133 (comment). |
I have BTT and mine run about 100-110C if the fans aren't on. |
Adds support for TMC2240 temperature monitoring.
Stepper disabled
Stepper enabled
Resolves #1089