[PATCH 1/5] imx-drm: ipu-di: Check the return value from clk_prepare_enable()

Sascha Hauer s.hauer at pengutronix.de
Mon Aug 19 08:52:42 UTC 2013


On Sun, Aug 18, 2013 at 09:40:02PM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam at 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.estevam at freescale.com>

For the series:

Acked-by: Sascha Hauer <s.hauer at 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 |


More information about the devel mailing list