diff options
| author | Daniel Lezcano <daniel.lezcano@linaro.org> | 2023-03-01 21:14:46 +0100 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2023-03-03 20:45:03 +0100 |
| commit | 20918ccacd06eea4a4ff457bd86cae62859461a2 (patch) | |
| tree | 8be50d07aa258e7bf6e0d16e87341e3289aea3cd /drivers | |
| parent | thermal/drivers/acerhdf: Make interval setting only at module load time (diff) | |
| download | linux-20918ccacd06eea4a4ff457bd86cae62859461a2.tar.xz linux-20918ccacd06eea4a4ff457bd86cae62859461a2.zip | |
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>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/platform/x86/acerhdf.c | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/drivers/platform/x86/acerhdf.c b/drivers/platform/x86/acerhdf.c index 50cdf70ade7f..5c9293e7eccb 100644 --- a/drivers/platform/x86/acerhdf.c +++ b/drivers/platform/x86/acerhdf.c @@ -691,13 +691,6 @@ static int __init acerhdf_register_thermal(void) if (ret) return ret; - if (strcmp(thz_dev->governor->name, - acerhdf_zone_params.governor_name)) { - pr_err("Didn't get thermal governor %s, perhaps not compiled into thermal subsystem.\n", - acerhdf_zone_params.governor_name); - return -EINVAL; - } - return 0; } |