]> git.ipfire.org Git - thirdparty/kernel/linux.git/commit
ASoC: codecs: wcd-clsh: Always update buck/flyback on transitions on transitions
authorCédric Bellegarde <cedric.bellegarde@adishatz.org>
Wed, 4 Mar 2026 14:10:06 +0000 (15:10 +0100)
committerMark Brown <broonie@kernel.org>
Mon, 16 Mar 2026 18:43:13 +0000 (18:43 +0000)
commitf8d51e903a6c97d8d298f14d9f8b4fff808670e3
tree78e13aa866320e0804c87583fbf991d4beaef6f1
parent804dce6c73fdfa44184ee4e8b09abad7f5da408f
ASoC: codecs: wcd-clsh: Always update buck/flyback on transitions on transitions

The WCD934x audio outputs (earpiece, headphone, speaker) share two power
supply converters, a buck and a flyback, managed by reference counters
(buck_users, flyback_users) in the Class-H controller.

The early return in wcd_clsh_ctrl_set_state() when nstate == ctrl->state
prevented _wcd_clsh_ctrl_set_state() from being called when switching
between outputs sharing the same state value. As a result, the buck and
flyback reference counters were never decremented on disable, leaving the
converters active and their counters out of sync with the actual hardware
state.

This caused audible distortion on the earpiece output and spurious MBHC
over-current protection interrupts on HPHL/HPHR during output switching.

Remove the early return so that CLSH_REQ_ENABLE and CLSH_REQ_DISABLE are
always dispatched, keeping the buck and flyback reference counters
consistent on every state transition.

Signed-off-by: Cédric Bellegarde <cedric.bellegarde@adishatz.org>
Link: https://patch.msgid.link/20260304141006.280894-1-cedric.bellegarde@adishatz.org
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/wcd-clsh-v2.c