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