fluidd icon indicating copy to clipboard operation
fluidd copied to clipboard

TMC Drivers: Report OTPW

Open NyxCode opened this issue 2 years ago • 2 comments
trafficstars

Most (or all?) TMC drivers report OTPW (over-temperature pre warning) conditions. This happens before actual over-temperature occurs & the driver shut down. This information can be queried through TMC registers. All drivers I'm aware of report the otpw flag in DRV_STATUS, which can be read through DUMP_TMC STEPPER={stepper} REGISTER=DRV_STATUS. The result of this will look something like this if an overtemperature pre-warning is raised:

DRV_STATUS: 84004008 sg_result=8 stealth=1 otpw=1(OvertempWarning!) stst=1

Currently, users are unaware of these warnings, unless the pre-warning turns into an overtemperature condition, and their printer shuts down.

Proposed solution

It'd be great if this information could be shown in the UI.
I think it'd probably fit best in the notifications, where e.g a Raspberry Pi overtemperature warning would be shown.

In certain cases, it might be desirable to ignore these warnings - for example, if the user made sure that their drivers don't heat up past the overtemperature pre-warning.

NyxCode avatar May 28 '23 01:05 NyxCode

In certain cases, it might be desirable to ignore these warnings - for example, if the user made sure that their drivers don't heat up past the overtemperature pre-warning.

This might be out-of-scope for fluidd, since all drivers I'm aware of allow users to configure the threshold for these values.
For example, from the TMC2209 datasheet: image

NyxCode avatar May 28 '23 01:05 NyxCode

Hi @NyxCode, thanks for opening this Feature Request.

I've taken a look at this and though we could manually query the driver, that is not the correct way; ideally, Klipper should provide this information as part of the TMCxxxx driver status object, which it currently does.

If and when that situation changes, we are willing to revisit this and consider adding the requested support.

pedrolamas avatar Aug 05 '23 09:08 pedrolamas