Skip to content

Add TMC2240 temperature reading #27903

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

Conversation

dbuezas
Copy link
Contributor

@dbuezas dbuezas commented Jun 1, 2025

M122
[...]
Analog in (v)	1.7799	1.8043	1.7970	1.8019
Supply (v)	24.2132	24.2521	24.2229	24.2813
Temp (°C)	34.8052	36.8831	35.7143	35.3247
OT pre warn (°C)	120.0000	120.0000	120.0000	120.0000
OV theshold (v)	37.7310	37.7310	37.7310	37.7310
Testing X connection... OK
Testing Y connection... OK
Testing Z connection... OK
Testing E connection... OK
ok P63 B15

Description

Requirements

Benefits

Configurations

Related Issues

(cherry picked from commit 13633c3)
@thisiskeithb
Copy link
Member

Needs MarlinFirmware/TMCStepper#7

@dbuezas
Copy link
Contributor Author

dbuezas commented Jun 1, 2025

Forgot to mention: this PR switches to (my branch of the libs)

HAS_TRINAMIC_CONFIG                    = TMCStepper=https://github.com/dbuezas/TMCStepper/archive/refs/heads/dbuezas/tmc-2240-temperature-reading.zip

for testing.
As @thisiskeithb well mentions, it requires MarlinFirmware/TMCStepper#7 before merging (and obviously switching back to Marlin's branch of the driver)

@thinkyhead thinkyhead force-pushed the bugfix-2.1.x branch 3 times, most recently from 3791e7d to 6ea4a16 Compare June 2, 2025 21:51
@thinkyhead
Copy link
Member

@dbuezas @thisiskeithb — Indeed those things are true.

@thinkyhead thinkyhead merged commit 3bb9364 into MarlinFirmware:bugfix-2.1.x Jun 2, 2025
66 checks passed
dbuezas added a commit to dbuezas/Marlin that referenced this pull request Jun 8, 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.

4 participants