Thread (5 messages) flat view 5 messages, 3 authors, 2021-08-04

Re: [PATCH v2] cpufreq: arm_scmi: Fix error path when allocation failed

From: Lukasz Luba <lukasz.luba@arm.com>
Date: 2021-08-04 10:52:03
Also in: linux-pm, lkml


On 8/4/21 5:02 AM, Viresh Kumar wrote:
On 03-08-21, 10:07, Lukasz Luba wrote:
quoted
Stop the initialization when cpumask allocation failed and return an
error.

Fixes: 80a064dbd556 ("scmi-cpufreq: Get opp_shared_cpus from opp-v2 for EM")
Signed-off-by: Lukasz Luba <lukasz.luba@arm.com>
---
changes:
v2:
- removed dev_warn() since it's not needed in this case
- adjusted the patch description

  drivers/cpufreq/scmi-cpufreq.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/cpufreq/scmi-cpufreq.c b/drivers/cpufreq/scmi-cpufreq.c
index ec9a87ca2dbb..75f818d04b48 100644
--- a/drivers/cpufreq/scmi-cpufreq.c
+++ b/drivers/cpufreq/scmi-cpufreq.c
@@ -134,7 +134,7 @@ static int scmi_cpufreq_init(struct cpufreq_policy *policy)
  	}
  
  	if (!zalloc_cpumask_var(&opp_shared_cpus, GFP_KERNEL))
-		ret = -ENOMEM;
+		return -ENOMEM;
  
  	/* Obtain CPUs that share SCMI performance controls */
  	ret = scmi_get_sharing_cpus(cpu_dev, policy->cpus);
Applied. Thanks.
Thank you for taking it.

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help