From: Rafael J. Wysocki Date: Mon, 27 Mar 2017 22:14:08 +0000 (+0200) Subject: cpufreq: intel_pstate: Modify check in intel_pstate_update_status() X-Git-Tag: v4.12-rc6~35^2~7^2~8 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=0042b2c0692ade097ef3a6bbffa491da5dc89273;p=platform%2Fkernel%2Flinux-exynos.git cpufreq: intel_pstate: Modify check in intel_pstate_update_status() 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 --- diff --git a/drivers/cpufreq/intel_pstate.c b/drivers/cpufreq/intel_pstate.c index 73ccddf..185006e 100644 --- a/drivers/cpufreq/intel_pstate.c +++ b/drivers/cpufreq/intel_pstate.c @@ -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();