Skip to content

Commit 20918cc

Browse files
dlezcanorafaeljw
authored andcommitted
thermal/drivers/acerhdf: Remove pointless governor test
The thermal zone parameter specifies the bang-bang governor. The Kconfig selects the bang-bang governor. So it is pointless to test if the governor was set for the thermal zone assuming it may not have been compiled-in. Remove the test and prevent another access into the thermal internals. Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org> Acked-by: Peter Kaestle <peter@piie.net> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
1 parent cad450d commit 20918cc

1 file changed

Lines changed: 0 additions & 7 deletions

File tree

drivers/platform/x86/acerhdf.c

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -691,13 +691,6 @@ static int __init acerhdf_register_thermal(void)
691691
if (ret)
692692
return ret;
693693

694-
if (strcmp(thz_dev->governor->name,
695-
acerhdf_zone_params.governor_name)) {
696-
pr_err("Didn't get thermal governor %s, perhaps not compiled into thermal subsystem.\n",
697-
acerhdf_zone_params.governor_name);
698-
return -EINVAL;
699-
}
700-
701694
return 0;
702695
}
703696

0 commit comments

Comments
 (0)