On 08/06/2021 15:38, Wei Yongjun wrote: > Fix the missing clk_disable_unprepare() before return > from panfrost_clk_init() in the error handling case. > > Fixes: b681af0bc1cc ("drm: panfrost: add optional bus_clock") > Reported-by: Hulk Robot <hulkci@xxxxxxxxxx> > Signed-off-by: Wei Yongjun <weiyongjun1@xxxxxxxxxx> Reviewed-by: Steven Price <steven.price@xxxxxxx> I'll push this to drm-misc-next. Thanks, Steve > --- > drivers/gpu/drm/panfrost/panfrost_device.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/panfrost/panfrost_device.c b/drivers/gpu/drm/panfrost/panfrost_device.c > index 125ed973feaa..a2a09c51eed7 100644 > --- a/drivers/gpu/drm/panfrost/panfrost_device.c > +++ b/drivers/gpu/drm/panfrost/panfrost_device.c > @@ -54,7 +54,8 @@ static int panfrost_clk_init(struct panfrost_device *pfdev) > if (IS_ERR(pfdev->bus_clock)) { > dev_err(pfdev->dev, "get bus_clock failed %ld\n", > PTR_ERR(pfdev->bus_clock)); > - return PTR_ERR(pfdev->bus_clock); > + err = PTR_ERR(pfdev->bus_clock); > + goto disable_clock; > } > > if (pfdev->bus_clock) { >