cpufreq: allow cpufreq_generic_suspend() to work without suspend frequency

bzolnier · rafaeljw · commit 201f37165757 · 2015-09-09T02:22:55.000+02:00
Some cpufreq drivers may set suspend frequency only for
selected setups but still would like to use the generic
suspend handler.  Thus don't treat !policy-&gt;suspend_freq
condition as an incorrect one.

Acked-by: Viresh Kumar &lt;viresh.kumar@linaro.org&gt;
Signed-off-by: Bartlomiej Zolnierkiewicz &lt;b.zolnierkie@samsung.com&gt;
Signed-off-by: Rafael J. Wysocki &lt;rafael.j.wysocki@intel.com&gt;
diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c
@@ -1628,8 +1628,8 @@ int cpufreq_generic_suspend(struct cpufreq_policy *policy)
 	int ret;
 
 	if (!policy->suspend_freq) {
-		pr_err("%s: suspend_freq can't be zero\n", __func__);
-		return -EINVAL;
+		pr_debug("%s: suspend_freq not defined\n", __func__);
+		return 0;
 	}
 
 	pr_debug("%s: Setting suspend-freq: %u\n", __func__,

Original file line number	Diff line number	Diff line change
`@@ -1628,8 +1628,8 @@ int cpufreq_generic_suspend(struct cpufreq_policy *policy)`
`1628`	`1628`	`int ret;`
`1629`	`1629`
`1630`	`1630`	`if (!policy->suspend_freq) {`
`1631`		`- pr_err("%s: suspend_freq can't be zero\n", __func__);`
`1632`		`- return -EINVAL;`
	`1631`	`+ pr_debug("%s: suspend_freq not defined\n", __func__);`
	`1632`	`+ return 0;`
`1633`	`1633`	`}`
`1634`	`1634`
`1635`	`1635`	`pr_debug("%s: Setting suspend-freq: %u\n", __func__,`