feedc0de/esp-idf
1
0
Fork 0
forked from espressif/esp-idf
Code Pull Requests Activity

bugfix(temperature_sensor): Fix issue that get the value is negative

Browse Source
This commit is contained in:
Cao Sen Miao
2023-09-01 11:06:02 +08:00
parent f0ca580c80
commit 8ce60fa50a
4 changed files with 7 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
components/driver/esp32c3/rtc_tempsensor.c
View File

@@ -126,7 +126,7 @@ static void read_delta_t_from_efuse(void)
ESP_LOGD(TAG, "s_deltaT = %f", s_deltaT); ESP_LOGD(TAG, "s_deltaT = %f", s_deltaT);
} }
static float parse_temp_sensor_raw_value(uint32_t tsens_raw) static float parse_temp_sensor_raw_value(int16_t tsens_raw)
{ {
if (isnan(s_deltaT)) { //suggests that the value is not initialized if (isnan(s_deltaT)) { //suggests that the value is not initialized
read_delta_t_from_efuse(); read_delta_t_from_efuse();
@@ -141,7 +141,7 @@ esp_err_t temp_sensor_read_celsius(float *celsius)
temp_sensor_config_t tsens; temp_sensor_config_t tsens;
temp_sensor_get_config(&tsens); temp_sensor_get_config(&tsens);
bool range_changed; bool range_changed;
uint16_t tsens_out = temp_sensor_get_raw_value(&range_changed); int16_t tsens_out = temp_sensor_get_raw_value(&range_changed);
*celsius = parse_temp_sensor_raw_value(tsens_out); *celsius = parse_temp_sensor_raw_value(tsens_out);
if (*celsius < TEMPERATURE_SENSOR_MEASURE_MIN || *celsius > TEMPERATURE_SENSOR_MEASURE_MAX) { if (*celsius < TEMPERATURE_SENSOR_MEASURE_MIN || *celsius > TEMPERATURE_SENSOR_MEASURE_MAX) {
ESP_LOGE(TAG, "Exceeding temperature measure range."); ESP_LOGE(TAG, "Exceeding temperature measure range.");

4
components/driver/esp32s2/rtc_tempsensor.c
View File

@@ -148,7 +148,7 @@ static void read_delta_t_from_efuse(void)
ESP_LOGD(TAG, "s_deltaT = %f\n", s_deltaT); ESP_LOGD(TAG, "s_deltaT = %f\n", s_deltaT);
} }
static float parse_temp_sensor_raw_value(uint32_t tsens_raw) static float parse_temp_sensor_raw_value(int16_t tsens_raw)
{ {
if (isnan(s_deltaT)) { //suggests that the value is not initialized if (isnan(s_deltaT)) { //suggests that the value is not initialized
read_delta_t_from_efuse(); read_delta_t_from_efuse();
@@ -163,7 +163,7 @@ esp_err_t temp_sensor_read_celsius(float *celsius)
temp_sensor_config_t tsens; temp_sensor_config_t tsens;
temp_sensor_get_config(&tsens); temp_sensor_get_config(&tsens);
bool range_changed; bool range_changed;
uint16_t tsens_out = temp_sensor_get_raw_value(&range_changed); int16_t tsens_out = temp_sensor_get_raw_value(&range_changed);
*celsius = parse_temp_sensor_raw_value(tsens_out); *celsius = parse_temp_sensor_raw_value(tsens_out);
if (*celsius < TEMPERATURE_SENSOR_MEASURE_MIN || *celsius > TEMPERATURE_SENSOR_MEASURE_MAX) { if (*celsius < TEMPERATURE_SENSOR_MEASURE_MIN || *celsius > TEMPERATURE_SENSOR_MEASURE_MAX) {
ESP_LOGE(TAG, "Exceeding temperature measure range."); ESP_LOGE(TAG, "Exceeding temperature measure range.");

4
components/driver/esp32s3/rtc_tempsensor.c
View File

@@ -128,7 +128,7 @@ static void read_delta_t_from_efuse(void)
ESP_LOGD(TAG, "s_deltaT = %f", s_deltaT); ESP_LOGD(TAG, "s_deltaT = %f", s_deltaT);
} }
static float parse_temp_sensor_raw_value(uint32_t tsens_raw) static float parse_temp_sensor_raw_value(int16_t tsens_raw)
{ {
if (isnan(s_deltaT)) { //suggests that the value is not initialized if (isnan(s_deltaT)) { //suggests that the value is not initialized
read_delta_t_from_efuse(); read_delta_t_from_efuse();
@@ -143,7 +143,7 @@ esp_err_t temp_sensor_read_celsius(float *celsius)
temp_sensor_config_t tsens; temp_sensor_config_t tsens;
temp_sensor_get_config(&tsens); temp_sensor_get_config(&tsens);
bool range_changed; bool range_changed;
uint16_t tsens_out = temp_sensor_get_raw_value(&range_changed); int16_t tsens_out = temp_sensor_get_raw_value(&range_changed);
*celsius = parse_temp_sensor_raw_value(tsens_out); *celsius = parse_temp_sensor_raw_value(tsens_out);
if (*celsius < TEMPERATURE_SENSOR_MEASURE_MIN || *celsius > TEMPERATURE_SENSOR_MEASURE_MAX) { if (*celsius < TEMPERATURE_SENSOR_MEASURE_MIN || *celsius > TEMPERATURE_SENSOR_MEASURE_MAX) {
ESP_LOGE(TAG, "Exceeding temperature measure range."); ESP_LOGE(TAG, "Exceeding temperature measure range.");

2
components/esp_hw_support/port/esp32s2/sar_periph_ctrl.c
View File

@@ -210,4 +210,4 @@ int16_t temp_sensor_get_raw_value(bool *range_changed)
portEXIT_CRITICAL(&rtc_spinlock); portEXIT_CRITICAL(&rtc_spinlock);
return degree; return degree;
} }