[PATCH v2] ALSA: hda/senary: Ensure EAPD is enabled during init

wangdich9700@163.com posted 1 patch 1 month, 1 week ago
sound/hda/codecs/senarytech.c | 5 +++++
1 file changed, 5 insertions(+)
[PATCH v2] ALSA: hda/senary: Ensure EAPD is enabled during init
Posted by wangdich9700@163.com 1 month, 1 week ago
From: wangdicheng <wangdicheng@kylinos.cn>

The driver sets spec->gen.own_eapd_ctl to take manual control of the
EAPD (External Amplifier). However, senary_init does not turn on the
EAPD, while senary_shutdown turns it off.

Since the generic driver skips EAPD handling when own_eapd_ctl is set,
the EAPD remains off after initialization (e.g., after resume), leaving
the codec in a non-functional state.

Explicitly call senary_auto_turn_eapd in senary_init to ensure the EAPD
is enabled and the codec is functional.

Signed-off-by: wangdicheng <wangdicheng@kylinos.cn>
---

v1->v2:
Change the email address to be consistent with the Signed-off-by line.

 sound/hda/codecs/senarytech.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/sound/hda/codecs/senarytech.c b/sound/hda/codecs/senarytech.c
index 8822d4dc5e36..6239a25bb8f3 100644
--- a/sound/hda/codecs/senarytech.c
+++ b/sound/hda/codecs/senarytech.c
@@ -25,6 +25,7 @@ struct senary_spec {
 	/* extra EAPD pins */
 	unsigned int num_eapds;
 	hda_nid_t eapds[4];
+	bool dynamic_eapd;
 	hda_nid_t mute_led_eapd;
 
 	unsigned int parse_flags; /* flag for snd_hda_parse_pin_defcfg() */
@@ -131,8 +132,12 @@ static void senary_init_gpio_led(struct hda_codec *codec)
 
 static int senary_init(struct hda_codec *codec)
 {
+	struct senary_spec *spec = codec->spec;
+
 	snd_hda_gen_init(codec);
 	senary_init_gpio_led(codec);
+	if (!spec->dynamic_eapd)
+		senary_auto_turn_eapd(codec, spec->num_eapds, spec->eapds, true);
 	snd_hda_apply_fixup(codec, HDA_FIXUP_ACT_INIT);
 
 	return 0;
-- 
2.25.1
Re: [PATCH v2] ALSA: hda/senary: Ensure EAPD is enabled during init
Posted by Takashi Iwai 1 month, 1 week ago
On Tue, 03 Mar 2026 09:15:16 +0100,
wangdich9700@163.com wrote:
> 
> From: wangdicheng <wangdicheng@kylinos.cn>
> 
> The driver sets spec->gen.own_eapd_ctl to take manual control of the
> EAPD (External Amplifier). However, senary_init does not turn on the
> EAPD, while senary_shutdown turns it off.
> 
> Since the generic driver skips EAPD handling when own_eapd_ctl is set,
> the EAPD remains off after initialization (e.g., after resume), leaving
> the codec in a non-functional state.
> 
> Explicitly call senary_auto_turn_eapd in senary_init to ensure the EAPD
> is enabled and the codec is functional.
> 
> Signed-off-by: wangdicheng <wangdicheng@kylinos.cn>
> ---
> 
> v1->v2:
> Change the email address to be consistent with the Signed-off-by line.

Thanks, applied now.


Takashi