Re: [PATCH] ASoC: cs42l43: leak MCLK when PLL enable fails on PRE_PMU

From: Tang Bin

Date: Thu Nov 05 2026 - 20:02:21 EST


Hi Charles,

Thanks for the review and pointing this out.

On Wed, Nov 04, 2026 at 11:04:26AM +0000, Charles Keepax wrote:
> Could you point to the relevant bits of the core code here? My
> understanding is that DAPM mostly just carrys on in the case of
> an error from an event handler, so I would not expect the
> sequence to play out as you describe.

You are absolutely right. I dug into dapm_seq_check_event() in soc-dapm.c.
As you noted, if the event handler returns an error, DAPM only logs it
(via dev_err) but continues the power-up sequence.

Therefore, the widget is still marked as powered, and the matching POST_PMD
will indeed run and execute clk_disable_unprepare(mclk) as expected. There
is no reference count leak here.

My initial premise was incorrect. Please drop this patch.

Best regards,
Tang Bin