EAPD trun on time
diff mbox series

Message ID 6FAB7C47BCF00940BB0999A99BE3547A18DBBD24@RTITMBSVM07.realtek.com.tw
State New
Headers show
Series
  • EAPD trun on time
Related show

Commit Message

Kailang April 17, 2019, 3:15 a.m. UTC
Hi Takashi,

Our AE told me, that EAPD turn on timing which need to set pin out first.
It's the safe application. It could avoid pop noise via speaker.

So, could I do the modify as below patch.

BR,
Kailang

Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

Comments

Takashi Iwai April 17, 2019, 5:13 a.m. UTC | #1
On Wed, 17 Apr 2019 05:15:54 +0200,
Kailang wrote:
> 
> Hi Takashi,
> 
> Our AE told me, that EAPD turn on timing which need to set pin out first.
> It's the safe application. It could avoid pop noise via speaker.
> 
> So, could I do the modify as below patch.

OK, that makes sense.  alc_auto_init_amp() should be only for EAPD or
other amp setups, so from the verb consistency POV, there should be no
difference by shuffling the call order, but the timing matters for
click noise.

thanks,

Takashi

> 
> BR,
> Kailang
> 
> diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
> index 810479766090..efcd21b8e0d7 100644
> --- a/sound/pci/hda/patch_realtek.c
> +++ b/sound/pci/hda/patch_realtek.c
> @@ -804,9 +804,8 @@ static int alc_init(struct hda_codec *codec)
>  		spec->init_hook(codec);
>  
>  	alc_fix_pll(codec);
> -	alc_auto_init_amp(codec, spec->init_amp);
> -
>  	snd_hda_gen_init(codec);
> +	alc_auto_init_amp(codec, spec->init_amp);
>  
>  	snd_hda_apply_fixup(codec, HDA_FIXUP_ACT_INIT);
>

Patch
diff mbox series

diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
index 810479766090..efcd21b8e0d7 100644
--- a/sound/pci/hda/patch_realtek.c
+++ b/sound/pci/hda/patch_realtek.c
@@ -804,9 +804,8 @@  static int alc_init(struct hda_codec *codec)
 		spec->init_hook(codec);
 
 	alc_fix_pll(codec);
-	alc_auto_init_amp(codec, spec->init_amp);
-
 	snd_hda_gen_init(codec);
+	alc_auto_init_amp(codec, spec->init_amp);
 
 	snd_hda_apply_fixup(codec, HDA_FIXUP_ACT_INIT);
_______________________________________________