Unverified Commit 213c4e51 authored by Tagir Garaev's avatar Tagir Garaev Committed by Mark Brown
Browse files

ASoC: Intel: sof_es8336: fix headphone GPIO logic inversion 



The headphone GPIO should be set to the inverse of speaker_en.
When speakers are enabled, headphones should be disabled and vice versa.

Currently both GPIOs are set to the same value (speaker_en), causing
audio to play through both speakers and headphones simultaneously
when headphones are plugged in.

Tested on Huawei Matebook (BOD-WXX9) with ES8336 codec.

Fixes: 6e1ff145 ("ASoC: Intel: sof_es8336: support a separate gpio to control headphone")
Signed-off-by: default avatarTagir Garaev <tgaraev653@gmail.com>
Link: https://patch.msgid.link/20260121152435.101698-1-tgaraev653@gmail.com


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent 9502b7df

sound/soc/intel/boards/sof_es8336.c

+1 −1
Original line number Diff line number Diff line
@@ -120,7 +120,7 @@ static void pcm_pop_work_events(struct work_struct *work) 
	gpiod_set_value_cansleep(priv->gpio_speakers, priv->speaker_en);



	if (quirk & SOF_ES8336_HEADPHONE_GPIO)

		gpiod_set_value_cansleep(priv->gpio_headphone, priv->speaker_en);

		gpiod_set_value_cansleep(priv->gpio_headphone, !priv->speaker_en);



}