Hi Fabio, On Tue, Apr 19, 2016 at 07:42:17AM -0300, Fabio Estevam wrote: >Hi Peng, > >On Tue, Apr 19, 2016 at 5:33 AM, Peng Fan <van.freenix@xxxxxxxxx> wrote: > >> @@ -46,11 +48,15 @@ static int imx_ocotp_read(void *context, const void *reg, size_t reg_size, >> if (count > (priv->nregs - index)) >> count = priv->nregs - index; >> >> + clk_prepare_enable(priv->clk); > >clk_prepare_enable() may fail, so you should better check its return >value and propagate it in the case of error. Thanks for correcting me. Will fix it in V2. Thanks, Peng -- -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html