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