于 2013年05月23日 18:19, Russell King - ARM Linux 写道:
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...

i will add the error check in next version.

thanks
Huang Shijie

_______________________________________________
devicetree-discuss mailing list
[email protected]
https://lists.ozlabs.org/listinfo/devicetree-discuss

Reply via email to