Hi, On 22/05/20 10:23 am, Dinghao Liu wrote: > When devm_clk_get() returns an error code, a pairing > runtime PM usage counter decrement is needed to keep > the counter balanced. > > Signed-off-by: Dinghao Liu <dinghao.liu@xxxxxxxxxx> > --- Thanks for the patch! But this fix is incomplete, I have posted a more comprehensive fix at [1].. Please take a look! [1] https://lore.kernel.org/linux-scsi/20200526100340.15032-1-vigneshr@xxxxxx/T/#u > drivers/scsi/ufs/ti-j721e-ufs.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/scsi/ufs/ti-j721e-ufs.c b/drivers/scsi/ufs/ti-j721e-ufs.c > index 5216d228cdd9..f3f212f6f9a9 100644 > --- a/drivers/scsi/ufs/ti-j721e-ufs.c > +++ b/drivers/scsi/ufs/ti-j721e-ufs.c > @@ -39,6 +39,7 @@ static int ti_j721e_ufs_probe(struct platform_device *pdev) > clk = devm_clk_get(dev, NULL); > if (IS_ERR(clk)) { > dev_err(dev, "Cannot claim MPHY clock.\n"); > + pm_runtime_put_sync(dev); > return PTR_ERR(clk); > } > clk_rate = clk_get_rate(clk); > Regards Vignesh
