thermal/drivers/acerhdf: Remove pointless governor test
authorDaniel Lezcano <daniel.lezcano@linaro.org>
Wed, 1 Mar 2023 20:14:46 +0000 (21:14 +0100)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Fri, 3 Mar 2023 19:45:03 +0000 (20:45 +0100)
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>
drivers/platform/x86/acerhdf.c

index 50cdf70..5c9293e 100644 (file)
@@ -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;
 }