On Tue, Mar 01, 2022 at 08:44:12PM +0800, Mark Brown wrote: >> +err: >> + clk_disable(clock->xck); >> + clk_disable(clock->ick); >> + clk_disable(clock->div); > > You need separate labels for each enable so that we don't end up > disabling clocks we didn't enable, that would also be a bug. Thanks, I have submitted a v2 to fix it. Jiang