[PATCH] soc: imx: gpcv2: Fix clock disabling imbalance in error path

Laurent Pinchart laurent.pinchart at ideasonboard.com
Sun Feb 20 22:29:24 PST 2022


Hello Shawn,

On Mon, Feb 21, 2022 at 01:59:29PM +0800, Shawn Guo wrote:
> On Fri, Feb 18, 2022 at 11:57:20PM +0200, Laurent Pinchart wrote:
> > The imx_pgc_power_down() starts by enabling the domain clocks, and thus
> > disables them in the error path. Commit 18c98573a4cf ("soc: imx: gpcv2:
> > add domain option to keep domain clocks enabled") made the clock enable
> > conditional, but forgot to add the same condition to the error path.
> > This can result in a clock enable/disable imbalance. Fix it.
> > 
> > Fixes: 18c98573a4cf ("soc: imx: gpcv2: add domain option to keep domain clocks enabled")
> > Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> 
> Applied, thanks!

Thank you.

I've noticed that Lucas' reviewed-by tag didn't end up in the commit in
your imx/fixes branch. Depending on your rebase policy with that branch,
you may want to ammend the commit.

-- 
Regards,

Laurent Pinchart



More information about the linux-arm-kernel mailing list