Hi,

> +     ret = clk_prepare(dev->clk);
> +     if (ret) {
> +             dev_err(&adev->dev, "can't prepare clock\n");
> +             goto err_prep_clk;
> +     }
> +
>       adap = &dev->adap;
>       adap->dev.parent = &adev->dev;
>       adap->owner     = THIS_MODULE;
> @@ -994,6 +1092,8 @@ static int nmk_i2c_probe(struct amba_device *adev, const 
> struct amba_id *id)
>       return 0;
>  
>   err_add_adap:
> +     clk_unprepare(dev->clk);
> + err_prep_clk:

This is unrelated, right? And it is also unneeded because of Ulf
Hansson's patch?

Regards,

   Wolfram

-- 
Pengutronix e.K.                           | Wolfram Sang                |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

Attachment: signature.asc
Description: Digital signature

Reply via email to