mirror of
https://github.com/tbnobody/OpenDTU.git
synced 2025-12-11 01:10:20 +01:00
Fix: disable temperature readings for ESP32-S3
reading the temperature is flawed and leads to unexpected reboots, which we observed more and more often recently. since the user experience is trash if the module resets unexpectedly from time to time, it seems to be more important to prevent reboots where ever possible rather than keeping the temperature readings. closes #1708.
This commit is contained in:
committed by
Thomas Basler
parent
3934284a6f
commit
0b798056f3
@@ -19,8 +19,9 @@ CpuTemperatureClass CpuTemperature;
|
||||
|
||||
float CpuTemperatureClass::read()
|
||||
{
|
||||
#ifdef CONFIG_IDF_TARGET_ESP32S2
|
||||
#if defined(CONFIG_IDF_TARGET_ESP32S2) || defined(CONFIG_IDF_TARGET_ESP32S3)
|
||||
// Disabling temperature reading for ESP32-S2 models as it might lead to WDT resets.
|
||||
// Same issue with ESP32-S3, where such WDT resets have been observed multiple times.
|
||||
// See: https://github.com/espressif/esp-idf/issues/8088
|
||||
return NAN;
|
||||
#endif
|
||||
|
||||
Reference in New Issue
Block a user