On Thu, May 23, 2013 at 04:16:13PM +0800, Huang Shijie wrote: > + /* get the clock */ > + weim->clk = devm_clk_get(&pdev->dev, NULL); > + if (IS_ERR(weim->clk)) > + goto weim_err; > + > + clk_prepare_enable(weim->clk); I notice people are getting lazy about this. clk_prepare_enable() can return an error...