On Tue, Jan 08, 2013 at 07:28:59PM +0800, Shawn Guo wrote: > > > + cpu_dev->of_node = np; > > > + > > > + arm_clk = clk_get(cpu_dev, "arm"); > > > + pll1_sys_clk = clk_get(cpu_dev, "pll1_sys"); > > > + pll1_sw_clk = clk_get(cpu_dev, "pll1_sw"); > > > + step_clk = clk_get(cpu_dev, "step"); > > > + pll2_pfd2_396m_clk = clk_get(cpu_dev, "pll2_pfd2_396m"); > > > > devm_*? > > > That what I tried to do at the first time. But the dev of cpu is > special, and managed functions do not work as we expect with it. > Inspired by the comment, I think I should attach /cpus/cpu@0 node to pdev->dev instead of the one retrieved from get_cpu_device(0), so that both managed functions and dev_* print can work. Shawn -- To unsubscribe from this list: send the line "unsubscribe cpufreq" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html