ACPI: CPPC: Replace ternary operator with umax()

prabhakarpujeri · rafaeljw · commit 86932cd8ccd4 · 2024-07-02T21:01:08.000+02:00
Replace ternary operator with umax() in cppc_find_dmi_mhz(). Signed-off-by: Prabhakar Pujeri <prabhakar.pujeri@gmail.com> [ rjw: Subject and changelog edits ] Link: https://patch.msgid.link/20240626130941.1527127-2-prabhakar.pujeri@gmail.com Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
diff --git a/drivers/acpi/cppc_acpi.c b/drivers/acpi/cppc_acpi.c
@@ -1839,7 +1839,7 @@ static void cppc_find_dmi_mhz(const struct dmi_header *dm, void *private)
 	    dm->length >= DMI_ENTRY_PROCESSOR_MIN_LENGTH) {
 		u16 val = (u16)get_unaligned((const u16 *)
 				(dmi_data + DMI_PROCESSOR_MAX_SPEED));
-		*mhz = val > *mhz ? val : *mhz;
+		*mhz = umax(val, *mhz);
 	}
 }
 

Original file line number	Diff line number	Diff line change
`@@ -1839,7 +1839,7 @@ static void cppc_find_dmi_mhz(const struct dmi_header dm, void private)`
`1839`	`1839`	`dm->length >= DMI_ENTRY_PROCESSOR_MIN_LENGTH) {`
`1840`	`1840`	`u16 val = (u16)get_unaligned((const u16 *)`
`1841`	`1841`	`(dmi_data + DMI_PROCESSOR_MAX_SPEED));`
`1842`		`- mhz = val > mhz ? val : *mhz;`
	`1842`	`+ mhz = umax(val, mhz);`
`1843`	`1843`	`}`
`1844`	`1844`	`}`
`1845`	`1845`