On Fri, Sep 11, 2015 at 12:02 AM, WEN Pingbo <pingbo.wen@xxxxxxxxxx> wrote: > - clk_enable(psif->pclk); > + retval = clk_enable(psif->pclk); > + if (retval < 0) > + dev_err(&psif->pdev->dev, > + "could not enable pclk, ret %d\n", retval); Shouldn't you add a 'return' here to prevent the code to proceed with the failed clock? -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html