ACPI: thermal: Adjust critical.flags.valid check

rafaeljw · rafaeljw · commit 57336224da83 · 2022-12-07T18:06:00.000+01:00
It is not necessary to compare critical.flags.valid to 1 in
acpi_thermal_trips_update() and doing so is also inconsistent with
other similar checks in that code, so simply check if the flag is
not 0 instead.

No expected functional impact.

Signed-off-by: Rafael J. Wysocki &lt;rafael.j.wysocki@intel.com&gt;
diff --git a/drivers/acpi/thermal.c b/drivers/acpi/thermal.c
@@ -291,7 +291,7 @@ static int acpi_thermal_trips_update(struct acpi_thermal *tz, int flag)
 					  "Found critical threshold [%lu]\n",
 					  tz->trips.critical.temperature);
 		}
-		if (tz->trips.critical.flags.valid == 1) {
+		if (tz->trips.critical.flags.valid) {
 			if (crt == -1) {
 				tz->trips.critical.flags.valid = 0;
 			} else if (crt > 0) {

Original file line number	Diff line number	Diff line change
`@@ -291,7 +291,7 @@ static int acpi_thermal_trips_update(struct acpi_thermal *tz, int flag)`
`291`	`291`	`"Found critical threshold [%lu]\n",`
`292`	`292`	`tz->trips.critical.temperature);`
`293`	`293`	`}`
`294`		`- if (tz->trips.critical.flags.valid == 1) {`
	`294`	`+ if (tz->trips.critical.flags.valid) {`
`295`	`295`	`if (crt == -1) {`
`296`	`296`	`tz->trips.critical.flags.valid = 0;`
`297`	`297`	`} else if (crt > 0) {`