cpufreq: Fix checkpatch errors and warnings

Fix 2 checkpatch errors about using assignment in if condition,
1 checkpatch error about a required space after comma
and 3 warnings about line over 80 characters.

Signed-off-by: Stratos Karafotis <stratosk@semaphore.gr>
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>

1 files changed, 14 insertions, 11 deletions

diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c
index e3aa9deb40d9..77a99bfea9a9 100644
--- a/drivers/cpufreq/cpufreq.c
+++ b/drivers/cpufreq/cpufreq.c
@@ -939,8 +939,11 @@ static int cpufreq_add_policy_cpu(struct cpufreq_policy *policy,
         up_write(&policy->rwsem);
 
         if (has_target()) {
-                if ((ret = __cpufreq_governor(policy, CPUFREQ_GOV_START)) ||
-                        (ret = __cpufreq_governor(policy, CPUFREQ_GOV_LIMITS))) {
+                ret = __cpufreq_governor(policy, CPUFREQ_GOV_START);
+                if (!ret)
+                        ret = __cpufreq_governor(policy, CPUFREQ_GOV_LIMITS);
+
+                if (ret) {
                         pr_err("%s: Failed to start governor\n", __func__);
                         return ret;
                 }
@@ -1394,14 +1397,14 @@ static int __cpufreq_remove_dev_finish(struct device *dev,
 
                 if (!cpufreq_suspended)
                         cpufreq_policy_free(policy);
-        } else {
-                if (has_target()) {
-                        if ((ret = __cpufreq_governor(policy, CPUFREQ_GOV_START)) ||
-                                        (ret = __cpufreq_governor(policy, CPUFREQ_GOV_LIMITS))) {
-                                pr_err("%s: Failed to start governor\n",
-                                       __func__);
-                                return ret;
-                        }
+        } else if (has_target()) {
+                ret = __cpufreq_governor(policy, CPUFREQ_GOV_START);
+                if (!ret)
+                        ret = __cpufreq_governor(policy, CPUFREQ_GOV_LIMITS);
+
+                if (ret) {
+                        pr_err("%s: Failed to start governor\n", __func__);
+                        return ret;
                 }
         }
 
@@ -2086,7 +2089,7 @@ static int cpufreq_set_policy(struct cpufreq_policy *policy,
         if (old_gov) {
                 __cpufreq_governor(policy, CPUFREQ_GOV_STOP);
                 up_write(&policy->rwsem);
-                __cpufreq_governor(policy,CPUFREQ_GOV_POLICY_EXIT);
+                __cpufreq_governor(policy, CPUFREQ_GOV_POLICY_EXIT);
                 down_write(&policy->rwsem);
         }