[PATCH 3/8] clk: mxs: add clock support for imx28
Turquette, Mike
mturquette at ti.com
Mon Apr 23 19:22:02 EDT 2012
On Sat, Apr 21, 2012 at 8:57 AM, Shawn Guo <shawn.guo at linaro.org> wrote:
> Add imx28 clock support based on common clk framework.
>
> Signed-off-by: Shawn Guo <shawn.guo at linaro.org>
> ---
...
> +int __init mx28_clocks_init(void)
> +{
> + struct clk *clk;
> + int ret;
> +
> + clk_misc_init();
> +
> + mxs_clk_fixed("ref_xtal", 24000000);
Hi Shawn,
My comment here is the same as patch 2/8. There is zero
error handling here. Registration functions _should_ be returning
-EERROR.
Regards,
Mike
More information about the linux-arm-kernel
mailing list