clk_prepare_enable() may fail, so we should better check its return value
and propagate it in the case of error.

Signed-off-by: Fabio Estevam <feste...@gmail.com>
Reviewed-by: Philipp Zabel <p.za...@pengutronix.de>
---
Changes since v2:
- None

 drivers/media/platform/imx-pxp.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/drivers/media/platform/imx-pxp.c b/drivers/media/platform/imx-pxp.c
index b76cd0e..27780f1 100644
--- a/drivers/media/platform/imx-pxp.c
+++ b/drivers/media/platform/imx-pxp.c
@@ -1666,7 +1666,10 @@ static int pxp_probe(struct platform_device *pdev)
                return ret;
        }
 
-       clk_prepare_enable(dev->clk);
+       ret = clk_prepare_enable(dev->clk);
+       if (ret < 0)
+               return ret;
+
        pxp_soft_reset(dev);
 
        spin_lock_init(&dev->irqlock);
-- 
2.7.4

Reply via email to