cpufreq: intel_pstate: Modify check in intel_pstate_update_status()
authorRafael J. Wysocki <rafael.j.wysocki@intel.com>
Mon, 27 Mar 2017 22:14:08 +0000 (00:14 +0200)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Tue, 28 Mar 2017 21:12:15 +0000 (23:12 +0200)
One of the checks in intel_pstate_update_status() implicitly relies
on the information that there are only two struct cpufreq_driver
objects available, but it is better to do it directly against the
value it really is about (to make the code easier to follow if
nothing else).

Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/cpufreq/intel_pstate.c

index 73ccddf94cf48f961ad1fbdd115c1399a70a232f..185006e9fbdb632dcdcb8e745b3609c3105dc395 100644 (file)
@@ -2345,7 +2345,7 @@ static int intel_pstate_update_status(const char *buf, size_t size)
 
        if (size == 7 && !strncmp(buf, "passive", size)) {
                if (intel_pstate_driver) {
-                       if (intel_pstate_driver != &intel_pstate)
+                       if (intel_pstate_driver == &intel_cpufreq)
                                return 0;
 
                        ret = intel_pstate_unregister_driver();