Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/81197?usp=email )
Change subject: drivers/acpi/thermal_zone: Correct Kelvin constant used for conversion ......................................................................
drivers/acpi/thermal_zone: Correct Kelvin constant used for conversion
As 0C is 273.15K you could argue that 2731 and 2732 are both correct. However, 2732 is deemed as correct both throughout the codebase and in the ACPI specification[1].
[1]: https://uefi.org/htmlspecs/ACPI_Spec_6_4_html/11_Thermal_Management/thermal-...
Change-Id: I845bc750681c7ae6f2d1342b32983b990ce6d296 Signed-off-by: Nicholas Sudsgaard devel+coreboot@nsudsgaard.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/81197 Reviewed-by: Paul Menzel paulepanter@mailbox.org Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Tim Wawrzynczak inforichland@gmail.com --- M src/drivers/acpi/thermal_zone/thermal_zone.c 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Paul Menzel: Looks good to me, but someone else must approve Tim Wawrzynczak: Looks good to me, approved
diff --git a/src/drivers/acpi/thermal_zone/thermal_zone.c b/src/drivers/acpi/thermal_zone/thermal_zone.c index 34fa1b8..ef95bed 100644 --- a/src/drivers/acpi/thermal_zone/thermal_zone.c +++ b/src/drivers/acpi/thermal_zone/thermal_zone.c @@ -16,7 +16,7 @@ #define DEFAULT_TC2 5 #define DEFAULT_TSP 10
-#define CELSIUS_TO_DECI_KELVIN(temp_c) ((temp_c) * 10 + 2731) +#define CELSIUS_TO_DECI_KELVIN(temp_c) ((temp_c) * 10 + 2732) #define SECONDS_TO_DECI_SECONDS(s) ((s) * 10)
static const char *thermal_zone_acpi_name(const struct device *dev)