On 2 July 2014 11:23, Shawn Guo shawn.guo@linaro.org wrote:
On Tue, Jul 01, 2014 at 10:02:35PM +0530, Viresh Kumar wrote:
diff --git a/drivers/cpufreq/cpufreq-cpu0.c b/drivers/cpufreq/cpufreq-cpu0.c index 4273a5f..b5b8e1c 100644 --- a/drivers/cpufreq/cpufreq-cpu0.c +++ b/drivers/cpufreq/cpufreq-cpu0.c @@ -150,8 +150,17 @@ static int cpu0_cpufreq_probe(struct platform_device *pdev)
cpu_clk = clk_get(cpu_dev, NULL); if (IS_ERR(cpu_clk)) {
ret = PTR_ERR(cpu_clk);
If you keep this ...
pr_err("failed to get cpu0 clock: %d\n", ret);
/*
* If cpu's clk node is present, but clock is not yet
* registered, we should try defering probe.
*/
if (PTR_ERR(cpu_clk) == -EPROBE_DEFER) {
... you can use 'ret' here ...
dev_err(cpu_dev, "cpu0 clock not ready, retry\n");
ret = -EPROBE_DEFER;
... this can be saved ...
} else {
ret = PTR_ERR(cpu_clk);
... and this as well.
All accepted. Thanks.