On Sun, Aug 18, 2013 at 09:40:02PM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.este...@freescale.com>
> 
> clk_prepare_enable() may fail, so let's check its return value and propagate 
> it
> in the case of error.
> 
> Signed-off-by: Fabio Estevam <fabio.este...@freescale.com>

For the series:

Acked-by: Sascha Hauer <s.ha...@pengutronix.de>

Sascha

> ---
>  drivers/staging/imx-drm/ipu-v3/ipu-di.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/staging/imx-drm/ipu-v3/ipu-di.c 
> b/drivers/staging/imx-drm/ipu-v3/ipu-di.c
> index 0b6806e..948a49b 100644
> --- a/drivers/staging/imx-drm/ipu-v3/ipu-di.c
> +++ b/drivers/staging/imx-drm/ipu-v3/ipu-di.c
> @@ -654,7 +654,9 @@ EXPORT_SYMBOL_GPL(ipu_di_init_sync_panel);
>  
>  int ipu_di_enable(struct ipu_di *di)
>  {
> -     clk_prepare_enable(di->clk_di_pixel);
> +     int ret = clk_prepare_enable(di->clk_di_pixel);
> +     if (ret)
> +             return ret;
>  
>       ipu_module_enable(di->ipu, di->module);
>  
> -- 
> 1.8.1.2
> 
> 

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to