* update ALC222 depop optimize
@ 2025-03-05 6:32 Kailang
2025-03-05 7:53 ` Takashi Iwai
0 siblings, 1 reply; 9+ messages in thread
From: Kailang @ 2025-03-05 6:32 UTC (permalink / raw)
To: Takashi Iwai (tiwai@suse.de)
Cc: (alsa-devel@alsa-project.org), linux-sound@vger.kernel.org
[-- Attachment #1: Type: text/plain, Size: 72 bytes --]
Hi Takashi,
Update ALC222 depop optimize as attach.
BR,
Kailang
[-- Attachment #2: 0000-alc222-dual-hp-depop.patch --]
[-- Type: application/octet-stream, Size: 2897 bytes --]
From b818b6a4f8bbe031c5a85d325c89fc0bbcbd5e5c Mon Sep 17 00:00:00 2001
From: Kailang Yang <kailang@realtek.com>
Date: Wed, 5 Mar 2025 13:54:34 +0800
Subject: [PATCH] ALSA: hda/realtek: update ALC222 depop optimize
Add ALC222 its own depop functions for alc_init and alc_shutup.
Signed-off-by: Kailang Yang <kailang@realtek.com>
diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
index ca10b9e9e3a4..706a35509518 100644
--- a/sound/pci/hda/patch_realtek.c
+++ b/sound/pci/hda/patch_realtek.c
@@ -3844,6 +3844,79 @@ static void alc225_shutup(struct hda_codec *codec)
}
}
+static void alc222_init(struct hda_codec *codec)
+{
+ struct alc_spec *spec = codec->spec;
+ hda_nid_t hp_pin = alc_get_hp_pin(spec);
+ bool hp1_pin_sense, hp2_pin_sense;
+
+ if (!hp_pin)
+ return;
+
+ msleep(30);
+
+ hp1_pin_sense = snd_hda_jack_detect(codec, hp_pin);
+ hp2_pin_sense = snd_hda_jack_detect(codec, 0x14);
+
+ if (hp1_pin_sense || hp2_pin_sense) {
+ msleep(2);
+
+ if (hp1_pin_sense)
+ snd_hda_codec_write(codec, hp_pin, 0,
+ AC_VERB_SET_PIN_WIDGET_CONTROL, PIN_OUT);
+ if (hp2_pin_sense)
+ snd_hda_codec_write(codec, 0x14, 0,
+ AC_VERB_SET_PIN_WIDGET_CONTROL, PIN_OUT);
+ msleep(75);
+
+ if (hp1_pin_sense)
+ snd_hda_codec_write(codec, hp_pin, 0,
+ AC_VERB_SET_AMP_GAIN_MUTE, AMP_OUT_UNMUTE);
+ if (hp2_pin_sense)
+ snd_hda_codec_write(codec, 0x14, 0,
+ AC_VERB_SET_AMP_GAIN_MUTE, AMP_OUT_UNMUTE);
+
+ msleep(75);
+ }
+}
+
+static void alc222_shutup(struct hda_codec *codec)
+{
+ struct alc_spec *spec = codec->spec;
+ hda_nid_t hp_pin = alc_get_hp_pin(spec);
+ bool hp1_pin_sense, hp2_pin_sense;
+
+ if (!hp_pin)
+ hp_pin = 0x21;
+
+ hp1_pin_sense = snd_hda_jack_detect(codec, hp_pin);
+ hp2_pin_sense = snd_hda_jack_detect(codec, 0x14);
+
+ if (hp1_pin_sense || hp2_pin_sense) {
+ msleep(2);
+
+ if (hp1_pin_sense)
+ snd_hda_codec_write(codec, hp_pin, 0,
+ AC_VERB_SET_AMP_GAIN_MUTE, AMP_OUT_MUTE);
+ if (hp2_pin_sense)
+ snd_hda_codec_write(codec, 0x14, 0,
+ AC_VERB_SET_AMP_GAIN_MUTE, AMP_OUT_MUTE);
+
+ msleep(75);
+
+ if (hp1_pin_sense)
+ snd_hda_codec_write(codec, hp_pin, 0,
+ AC_VERB_SET_PIN_WIDGET_CONTROL, 0x0);
+ if (hp2_pin_sense)
+ snd_hda_codec_write(codec, 0x14, 0,
+ AC_VERB_SET_PIN_WIDGET_CONTROL, 0x0);
+
+ msleep(75);
+ }
+ alc_auto_setup_eapd(codec, false);
+ alc_shutup_pins(codec);
+}
+
static void alc_default_init(struct hda_codec *codec)
{
struct alc_spec *spec = codec->spec;
@@ -11939,8 +12012,11 @@ static int patch_alc269(struct hda_codec *codec)
spec->codec_variant = ALC269_TYPE_ALC300;
spec->gen.mixer_nid = 0; /* no loopback on ALC300 */
break;
+ case 0x10ec0222:
case 0x10ec0623:
spec->codec_variant = ALC269_TYPE_ALC623;
+ spec->shutup = alc222_shutup;
+ spec->init_hook = alc222_init;
break;
case 0x10ec0700:
case 0x10ec0701:
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: update ALC222 depop optimize
2025-03-05 6:32 update ALC222 depop optimize Kailang
@ 2025-03-05 7:53 ` Takashi Iwai
2025-03-05 7:56 ` Kailang
0 siblings, 1 reply; 9+ messages in thread
From: Takashi Iwai @ 2025-03-05 7:53 UTC (permalink / raw)
To: Kailang; +Cc: (alsa-devel@alsa-project.org), linux-sound@vger.kernel.org
On Wed, 05 Mar 2025 07:32:00 +0100,
Kailang wrote:
>
> Hi Takashi,
>
> Update ALC222 depop optimize as attach.
Could you give a bit more background info why this change is needed?
Is it a mandatory change to fix something, or it improves something?
thanks,
Takashi
^ permalink raw reply [flat|nested] 9+ messages in thread
* RE: update ALC222 depop optimize
2025-03-05 7:53 ` Takashi Iwai
@ 2025-03-05 7:56 ` Kailang
2025-03-05 8:16 ` Takashi Iwai
0 siblings, 1 reply; 9+ messages in thread
From: Kailang @ 2025-03-05 7:56 UTC (permalink / raw)
To: Takashi Iwai; +Cc: (alsa-devel@alsa-project.org), linux-sound@vger.kernel.org
> -----Original Message-----
> From: Takashi Iwai <tiwai@suse.de>
> Sent: Wednesday, March 5, 2025 3:54 PM
> To: Kailang <kailang@realtek.com>
> Cc: (alsa-devel@alsa-project.org) <alsa-devel@alsa-project.org>;
> linux-sound@vger.kernel.org
> Subject: Re: update ALC222 depop optimize
>
>
> External mail : This email originated from outside the organization. Do not
> reply, click links, or open attachments unless you recognize the sender and
> know the content is safe.
>
>
>
> On Wed, 05 Mar 2025 07:32:00 +0100,
> Kailang wrote:
> >
> > Hi Takashi,
> >
> > Update ALC222 depop optimize as attach.
>
> Could you give a bit more background info why this change is needed?
> Is it a mandatory change to fix something, or it improves something?
>
This codec has two headphone design.
The HP2 was nid 0x14.
>
> thanks,
>
> Takashi
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: update ALC222 depop optimize
2025-03-05 7:56 ` Kailang
@ 2025-03-05 8:16 ` Takashi Iwai
2025-03-05 8:22 ` Kailang
0 siblings, 1 reply; 9+ messages in thread
From: Takashi Iwai @ 2025-03-05 8:16 UTC (permalink / raw)
To: Kailang; +Cc: (alsa-devel@alsa-project.org), linux-sound@vger.kernel.org
On Wed, 05 Mar 2025 08:56:41 +0100,
Kailang wrote:
>
>
> > -----Original Message-----
> > From: Takashi Iwai <tiwai@suse.de>
> > Sent: Wednesday, March 5, 2025 3:54 PM
> > To: Kailang <kailang@realtek.com>
> > Cc: (alsa-devel@alsa-project.org) <alsa-devel@alsa-project.org>;
> > linux-sound@vger.kernel.org
> > Subject: Re: update ALC222 depop optimize
> >
> >
> > External mail : This email originated from outside the organization. Do not
> > reply, click links, or open attachments unless you recognize the sender and
> > know the content is safe.
> >
> >
> >
> > On Wed, 05 Mar 2025 07:32:00 +0100,
> > Kailang wrote:
> > >
> > > Hi Takashi,
> > >
> > > Update ALC222 depop optimize as attach.
> >
> > Could you give a bit more background info why this change is needed?
> > Is it a mandatory change to fix something, or it improves something?
> >
> This codec has two headphone design.
> The HP2 was nid 0x14.
Sorry not clear enough: do you mean the patch is to fix the depop
problems on the models with two headphone pins with ALC222?
thanks,
Takashi
^ permalink raw reply [flat|nested] 9+ messages in thread
* RE: update ALC222 depop optimize
2025-03-05 8:16 ` Takashi Iwai
@ 2025-03-05 8:22 ` Kailang
2025-03-05 8:50 ` Takashi Iwai
0 siblings, 1 reply; 9+ messages in thread
From: Kailang @ 2025-03-05 8:22 UTC (permalink / raw)
To: Takashi Iwai; +Cc: (alsa-devel@alsa-project.org), linux-sound@vger.kernel.org
> -----Original Message-----
> From: Takashi Iwai <tiwai@suse.de>
> Sent: Wednesday, March 5, 2025 4:17 PM
> To: Kailang <kailang@realtek.com>
> Cc: (alsa-devel@alsa-project.org) <alsa-devel@alsa-project.org>;
> linux-sound@vger.kernel.org
> Subject: Re: update ALC222 depop optimize
>
>
> External mail : This email originated from outside the organization. Do not
> reply, click links, or open attachments unless you recognize the sender and
> know the content is safe.
>
>
>
> On Wed, 05 Mar 2025 08:56:41 +0100,
> Kailang wrote:
> >
> >
> > > -----Original Message-----
> > > From: Takashi Iwai <tiwai@suse.de>
> > > Sent: Wednesday, March 5, 2025 3:54 PM
> > > To: Kailang <kailang@realtek.com>
> > > Cc: (alsa-devel@alsa-project.org) <alsa-devel@alsa-project.org>;
> > > linux-sound@vger.kernel.org
> > > Subject: Re: update ALC222 depop optimize
> > >
> > >
> > > External mail : This email originated from outside the organization.
> > > Do not reply, click links, or open attachments unless you recognize
> > > the sender and know the content is safe.
> > >
> > >
> > >
> > > On Wed, 05 Mar 2025 07:32:00 +0100,
> > > Kailang wrote:
> > > >
> > > > Hi Takashi,
> > > >
> > > > Update ALC222 depop optimize as attach.
> > >
> > > Could you give a bit more background info why this change is needed?
> > > Is it a mandatory change to fix something, or it improves something?
> > >
> > This codec has two headphone design.
> > The HP2 was nid 0x14.
>
> Sorry not clear enough: do you mean the patch is to fix the depop problems on
> the models with two headphone pins with ALC222?
Yes, the original depop was only supported one headphone.
>
>
> thanks,
>
> Takashi
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: update ALC222 depop optimize
2025-03-05 8:22 ` Kailang
@ 2025-03-05 8:50 ` Takashi Iwai
2025-03-05 9:22 ` Kailang
0 siblings, 1 reply; 9+ messages in thread
From: Takashi Iwai @ 2025-03-05 8:50 UTC (permalink / raw)
To: Kailang; +Cc: (alsa-devel@alsa-project.org), linux-sound@vger.kernel.org
On Wed, 05 Mar 2025 09:22:20 +0100,
Kailang wrote:
>
>
>
> > -----Original Message-----
> > From: Takashi Iwai <tiwai@suse.de>
> > Sent: Wednesday, March 5, 2025 4:17 PM
> > To: Kailang <kailang@realtek.com>
> > Cc: (alsa-devel@alsa-project.org) <alsa-devel@alsa-project.org>;
> > linux-sound@vger.kernel.org
> > Subject: Re: update ALC222 depop optimize
> >
> >
> > External mail : This email originated from outside the organization. Do not
> > reply, click links, or open attachments unless you recognize the sender and
> > know the content is safe.
> >
> >
> >
> > On Wed, 05 Mar 2025 08:56:41 +0100,
> > Kailang wrote:
> > >
> > >
> > > > -----Original Message-----
> > > > From: Takashi Iwai <tiwai@suse.de>
> > > > Sent: Wednesday, March 5, 2025 3:54 PM
> > > > To: Kailang <kailang@realtek.com>
> > > > Cc: (alsa-devel@alsa-project.org) <alsa-devel@alsa-project.org>;
> > > > linux-sound@vger.kernel.org
> > > > Subject: Re: update ALC222 depop optimize
> > > >
> > > >
> > > > External mail : This email originated from outside the organization.
> > > > Do not reply, click links, or open attachments unless you recognize
> > > > the sender and know the content is safe.
> > > >
> > > >
> > > >
> > > > On Wed, 05 Mar 2025 07:32:00 +0100,
> > > > Kailang wrote:
> > > > >
> > > > > Hi Takashi,
> > > > >
> > > > > Update ALC222 depop optimize as attach.
> > > >
> > > > Could you give a bit more background info why this change is needed?
> > > > Is it a mandatory change to fix something, or it improves something?
> > > >
> > > This codec has two headphone design.
> > > The HP2 was nid 0x14.
> >
> > Sorry not clear enough: do you mean the patch is to fix the depop problems on
> > the models with two headphone pins with ALC222?
>
> Yes, the original depop was only supported one headphone.
OK, and then would it be a bit risky to blindly assume the second
headphone pin 0x14? Isn't it set up via pin config?
thanks,
Takashi
^ permalink raw reply [flat|nested] 9+ messages in thread
* RE: update ALC222 depop optimize
2025-03-05 8:50 ` Takashi Iwai
@ 2025-03-05 9:22 ` Kailang
2025-03-05 9:29 ` Takashi Iwai
0 siblings, 1 reply; 9+ messages in thread
From: Kailang @ 2025-03-05 9:22 UTC (permalink / raw)
To: Takashi Iwai; +Cc: (alsa-devel@alsa-project.org), linux-sound@vger.kernel.org
> -----Original Message-----
> From: Takashi Iwai <tiwai@suse.de>
> Sent: Wednesday, March 5, 2025 4:50 PM
> To: Kailang <kailang@realtek.com>
> Cc: (alsa-devel@alsa-project.org) <alsa-devel@alsa-project.org>;
> linux-sound@vger.kernel.org
> Subject: Re: update ALC222 depop optimize
>
>
> External mail : This email originated from outside the organization. Do not
> reply, click links, or open attachments unless you recognize the sender and
> know the content is safe.
>
>
>
> On Wed, 05 Mar 2025 09:22:20 +0100,
> Kailang wrote:
> >
> >
> >
> > > -----Original Message-----
> > > From: Takashi Iwai <tiwai@suse.de>
> > > Sent: Wednesday, March 5, 2025 4:17 PM
> > > To: Kailang <kailang@realtek.com>
> > > Cc: (alsa-devel@alsa-project.org) <alsa-devel@alsa-project.org>;
> > > linux-sound@vger.kernel.org
> > > Subject: Re: update ALC222 depop optimize
> > >
> > >
> > > External mail : This email originated from outside the organization.
> > > Do not reply, click links, or open attachments unless you recognize
> > > the sender and know the content is safe.
> > >
> > >
> > >
> > > On Wed, 05 Mar 2025 08:56:41 +0100,
> > > Kailang wrote:
> > > >
> > > >
> > > > > -----Original Message-----
> > > > > From: Takashi Iwai <tiwai@suse.de>
> > > > > Sent: Wednesday, March 5, 2025 3:54 PM
> > > > > To: Kailang <kailang@realtek.com>
> > > > > Cc: (alsa-devel@alsa-project.org) <alsa-devel@alsa-project.org>;
> > > > > linux-sound@vger.kernel.org
> > > > > Subject: Re: update ALC222 depop optimize
> > > > >
> > > > >
> > > > > External mail : This email originated from outside the organization.
> > > > > Do not reply, click links, or open attachments unless you
> > > > > recognize the sender and know the content is safe.
> > > > >
> > > > >
> > > > >
> > > > > On Wed, 05 Mar 2025 07:32:00 +0100, Kailang wrote:
> > > > > >
> > > > > > Hi Takashi,
> > > > > >
> > > > > > Update ALC222 depop optimize as attach.
> > > > >
> > > > > Could you give a bit more background info why this change is needed?
> > > > > Is it a mandatory change to fix something, or it improves something?
> > > > >
> > > > This codec has two headphone design.
> > > > The HP2 was nid 0x14.
> > >
> > > Sorry not clear enough: do you mean the patch is to fix the depop
> > > problems on the models with two headphone pins with ALC222?
> >
> > Yes, the original depop was only supported one headphone.
>
> OK, and then would it be a bit risky to blindly assume the second headphone
> pin 0x14? Isn't it set up via pin config?
>
It doesn't have risk to set this pin as headphone or line out.
This pin (0x14) can be a line out or headphone. It has JD which driver will get it.
Line out and Headhpone were need to do depop procedure.
If pin 0x14 set to speaker, it wouldn't have JD.
hp2_pin_sense = snd_hda_jack_detect(codec, 0x14);
This just got JD to do depop.
>
> thanks,
>
> Takashi
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: update ALC222 depop optimize
2025-03-05 9:22 ` Kailang
@ 2025-03-05 9:29 ` Takashi Iwai
2025-03-05 9:33 ` Kailang
0 siblings, 1 reply; 9+ messages in thread
From: Takashi Iwai @ 2025-03-05 9:29 UTC (permalink / raw)
To: Kailang; +Cc: (alsa-devel@alsa-project.org), linux-sound@vger.kernel.org
On Wed, 05 Mar 2025 10:22:04 +0100,
Kailang wrote:
>
>
>
> > -----Original Message-----
> > From: Takashi Iwai <tiwai@suse.de>
> > Sent: Wednesday, March 5, 2025 4:50 PM
> > To: Kailang <kailang@realtek.com>
> > Cc: (alsa-devel@alsa-project.org) <alsa-devel@alsa-project.org>;
> > linux-sound@vger.kernel.org
> > Subject: Re: update ALC222 depop optimize
> >
> >
> > External mail : This email originated from outside the organization. Do not
> > reply, click links, or open attachments unless you recognize the sender and
> > know the content is safe.
> >
> >
> >
> > On Wed, 05 Mar 2025 09:22:20 +0100,
> > Kailang wrote:
> > >
> > >
> > >
> > > > -----Original Message-----
> > > > From: Takashi Iwai <tiwai@suse.de>
> > > > Sent: Wednesday, March 5, 2025 4:17 PM
> > > > To: Kailang <kailang@realtek.com>
> > > > Cc: (alsa-devel@alsa-project.org) <alsa-devel@alsa-project.org>;
> > > > linux-sound@vger.kernel.org
> > > > Subject: Re: update ALC222 depop optimize
> > > >
> > > >
> > > > External mail : This email originated from outside the organization.
> > > > Do not reply, click links, or open attachments unless you recognize
> > > > the sender and know the content is safe.
> > > >
> > > >
> > > >
> > > > On Wed, 05 Mar 2025 08:56:41 +0100,
> > > > Kailang wrote:
> > > > >
> > > > >
> > > > > > -----Original Message-----
> > > > > > From: Takashi Iwai <tiwai@suse.de>
> > > > > > Sent: Wednesday, March 5, 2025 3:54 PM
> > > > > > To: Kailang <kailang@realtek.com>
> > > > > > Cc: (alsa-devel@alsa-project.org) <alsa-devel@alsa-project.org>;
> > > > > > linux-sound@vger.kernel.org
> > > > > > Subject: Re: update ALC222 depop optimize
> > > > > >
> > > > > >
> > > > > > External mail : This email originated from outside the organization.
> > > > > > Do not reply, click links, or open attachments unless you
> > > > > > recognize the sender and know the content is safe.
> > > > > >
> > > > > >
> > > > > >
> > > > > > On Wed, 05 Mar 2025 07:32:00 +0100, Kailang wrote:
> > > > > > >
> > > > > > > Hi Takashi,
> > > > > > >
> > > > > > > Update ALC222 depop optimize as attach.
> > > > > >
> > > > > > Could you give a bit more background info why this change is needed?
> > > > > > Is it a mandatory change to fix something, or it improves something?
> > > > > >
> > > > > This codec has two headphone design.
> > > > > The HP2 was nid 0x14.
> > > >
> > > > Sorry not clear enough: do you mean the patch is to fix the depop
> > > > problems on the models with two headphone pins with ALC222?
> > >
> > > Yes, the original depop was only supported one headphone.
> >
> > OK, and then would it be a bit risky to blindly assume the second headphone
> > pin 0x14? Isn't it set up via pin config?
> >
> It doesn't have risk to set this pin as headphone or line out.
> This pin (0x14) can be a line out or headphone. It has JD which driver will get it.
> Line out and Headhpone were need to do depop procedure.
> If pin 0x14 set to speaker, it wouldn't have JD.
>
> hp2_pin_sense = snd_hda_jack_detect(codec, 0x14);
>
> This just got JD to do depop.
Fair enough, I took the patch now. Thanks.
Takashi
^ permalink raw reply [flat|nested] 9+ messages in thread
* RE: update ALC222 depop optimize
2025-03-05 9:29 ` Takashi Iwai
@ 2025-03-05 9:33 ` Kailang
0 siblings, 0 replies; 9+ messages in thread
From: Kailang @ 2025-03-05 9:33 UTC (permalink / raw)
To: Takashi Iwai; +Cc: (alsa-devel@alsa-project.org), linux-sound@vger.kernel.org
> -----Original Message-----
> From: Takashi Iwai <tiwai@suse.de>
> Sent: Wednesday, March 5, 2025 5:30 PM
> To: Kailang <kailang@realtek.com>
> Cc: (alsa-devel@alsa-project.org) <alsa-devel@alsa-project.org>;
> linux-sound@vger.kernel.org
> Subject: Re: update ALC222 depop optimize
>
>
> External mail : This email originated from outside the organization. Do not
> reply, click links, or open attachments unless you recognize the sender and
> know the content is safe.
>
>
>
> On Wed, 05 Mar 2025 10:22:04 +0100,
> Kailang wrote:
> >
> >
> >
> > > -----Original Message-----
> > > From: Takashi Iwai <tiwai@suse.de>
> > > Sent: Wednesday, March 5, 2025 4:50 PM
> > > To: Kailang <kailang@realtek.com>
> > > Cc: (alsa-devel@alsa-project.org) <alsa-devel@alsa-project.org>;
> > > linux-sound@vger.kernel.org
> > > Subject: Re: update ALC222 depop optimize
> > >
> > >
> > > External mail : This email originated from outside the organization.
> > > Do not reply, click links, or open attachments unless you recognize
> > > the sender and know the content is safe.
> > >
> > >
> > >
> > > On Wed, 05 Mar 2025 09:22:20 +0100,
> > > Kailang wrote:
> > > >
> > > >
> > > >
> > > > > -----Original Message-----
> > > > > From: Takashi Iwai <tiwai@suse.de>
> > > > > Sent: Wednesday, March 5, 2025 4:17 PM
> > > > > To: Kailang <kailang@realtek.com>
> > > > > Cc: (alsa-devel@alsa-project.org) <alsa-devel@alsa-project.org>;
> > > > > linux-sound@vger.kernel.org
> > > > > Subject: Re: update ALC222 depop optimize
> > > > >
> > > > >
> > > > > External mail : This email originated from outside the organization.
> > > > > Do not reply, click links, or open attachments unless you
> > > > > recognize the sender and know the content is safe.
> > > > >
> > > > >
> > > > >
> > > > > On Wed, 05 Mar 2025 08:56:41 +0100, Kailang wrote:
> > > > > >
> > > > > >
> > > > > > > -----Original Message-----
> > > > > > > From: Takashi Iwai <tiwai@suse.de>
> > > > > > > Sent: Wednesday, March 5, 2025 3:54 PM
> > > > > > > To: Kailang <kailang@realtek.com>
> > > > > > > Cc: (alsa-devel@alsa-project.org)
> > > > > > > <alsa-devel@alsa-project.org>; linux-sound@vger.kernel.org
> > > > > > > Subject: Re: update ALC222 depop optimize
> > > > > > >
> > > > > > >
> > > > > > > External mail : This email originated from outside the organization.
> > > > > > > Do not reply, click links, or open attachments unless you
> > > > > > > recognize the sender and know the content is safe.
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > On Wed, 05 Mar 2025 07:32:00 +0100, Kailang wrote:
> > > > > > > >
> > > > > > > > Hi Takashi,
> > > > > > > >
> > > > > > > > Update ALC222 depop optimize as attach.
> > > > > > >
> > > > > > > Could you give a bit more background info why this change is
> needed?
> > > > > > > Is it a mandatory change to fix something, or it improves
> something?
> > > > > > >
> > > > > > This codec has two headphone design.
> > > > > > The HP2 was nid 0x14.
> > > > >
> > > > > Sorry not clear enough: do you mean the patch is to fix the
> > > > > depop problems on the models with two headphone pins with ALC222?
> > > >
> > > > Yes, the original depop was only supported one headphone.
> > >
> > > OK, and then would it be a bit risky to blindly assume the second
> > > headphone pin 0x14? Isn't it set up via pin config?
> > >
> > It doesn't have risk to set this pin as headphone or line out.
> > This pin (0x14) can be a line out or headphone. It has JD which driver will get
> it.
> > Line out and Headhpone were need to do depop procedure.
> > If pin 0x14 set to speaker, it wouldn't have JD.
> >
> > hp2_pin_sense = snd_hda_jack_detect(codec, 0x14);
> >
> > This just got JD to do depop.
>
> Fair enough, I took the patch now. Thanks.
This codec speaker pin will be 0x17. 0x14 pin was not have class D amp. So, 0x14 pin will not assign to speaker pin.
>
>
> Takashi
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2025-03-05 9:33 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-03-05 6:32 update ALC222 depop optimize Kailang
2025-03-05 7:53 ` Takashi Iwai
2025-03-05 7:56 ` Kailang
2025-03-05 8:16 ` Takashi Iwai
2025-03-05 8:22 ` Kailang
2025-03-05 8:50 ` Takashi Iwai
2025-03-05 9:22 ` Kailang
2025-03-05 9:29 ` Takashi Iwai
2025-03-05 9:33 ` Kailang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox