Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
Date:   Fri, 11 Mar 2022 09:30:49 +0100
Message-ID: <s5ho82cevra.wl-tiwai@suse.de>
From:   Takashi Iwai <tiwai@suse.de>
To:     "huangwenhui" <huangwenhuia@uniontech.com>
Cc:     "perex" <perex@perex.cz>, "tiwai" <tiwai@suse.com>,
        "jeremy.szu" <jeremy.szu@canonical.com>,
        "hui.wang" <hui.wang@canonical.com>,
        "wse" <wse@tuxedocomputers.com>, "cam" <cam@neo-zeon.de>,
        "kailang" <kailang@realtek.com>,
        "tanureal" <tanureal@opensource.cirrus.com>,
        "sami" <sami@loone.fi>, "alsa-devel" <alsa-devel@alsa-project.org>,
        "linux-kernel" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] ALSA: hda/realtek - Fix headset mic problem for a HP machine with alc671
In-Reply-To: <tencent_080B040F1CE2D7D83D6FD0C8@qq.com>
References: <tencent_080B040F1CE2D7D83D6FD0C8@qq.com>
User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka)
 FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.8 Emacs/25.3
 (x86_64-suse-linux-gnu) MULE/6.0 (HANACHIRUSATO)
MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka")
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Precedence: bulk

On Fri, 11 Mar 2022 02:20:42 +0100,
huangwenhui wrote:
> 
> Hi  Takashi,
> 
> if adjusting the mixer element or replugging the headset, the headphone would
> be unmuted.
> But most users hope that the default state is unmuted.
> Should this unconditional unmute be required?

It's the standard behavior for most of ALSA drivers.

The initial mute state is rather controlled via alsactl, restoring the
previous state.  Also, the sound server like PulseAudio or pipewire
does unmute automatically.  If this doesn't work as expected, that's a
more problem to diagnose.


Takashi

> 
> Thanks.
> 
> ------------------ Original ------------------
> From:  "Takashi Iwai"<tiwai@suse.de>;
> Date:  Thu, Mar 10, 2022 10:03 PM
> To:  "黄文辉"<huangwenhuia@uniontech.com>;
> Cc:  "perex"<perex@perex.cz>; "tiwai"<tiwai@suse.com>; "jeremy.szu"
> <jeremy.szu@canonical.com>; "hui.wang"<hui.wang@canonical.com>; "wse"
> <wse@tuxedocomputers.com>; "cam"<cam@neo-zeon.de>; "kailang"
> <kailang@realtek.com>; "tanureal"<tanureal@opensource.cirrus.com>; "sami"
> <sami@loone.fi>; "alsa-devel"<alsa-devel@alsa-project.org>; "linux-kernel"
> <linux-kernel@vger.kernel.org>;
> Subject:  Re: [PATCH] ALSA: hda/realtek - Fix headset mic problem for a HP
> machine with alc671
>  
> On Thu, 10 Mar 2022 14:58:36 +0100,
> huangwenhui wrote:
> >
> > Hi  Takashi,
> >
> > Thank you for your reply.
> >
> > When booting with plugged headset, the headphone will be muted.
> 
> The muted state is the default behavior.  Wouldn't it be unmuted if
> you adjust the corresponding mixer element?
> 
> Takashi
> 
> >
> > Thanks.
> > 
> > ------------------ Original ------------------
> > From:  "Takashi Iwai"<tiwai@suse.de>;
> > Date:  Thu, Mar 10, 2022 09:29 PM
> > To:  "huangwenhui"<huangwenhuia@uniontech.com>;
> > Cc:  "perex"<perex@perex.cz>; "tiwai"<tiwai@suse.com>; "jeremy.szu"
> > <jeremy.szu@canonical.com>; "hui.wang"<hui.wang@canonical.com>; "wse"
> > <wse@tuxedocomputers.com>; "cam"<cam@neo-zeon.de>; "kailang"
> > <kailang@realtek.com>; "tanureal"<tanureal@opensource.cirrus.com>; "sami"
> > <sami@loone.fi>; "alsa-devel"<alsa-devel@alsa-project.org>; "linux-kernel"
> > <linux-kernel@vger.kernel.org>;
> > Subject:  Re: [PATCH] ALSA: hda/realtek - Fix headset mic problem for a HP
> > machine with alc671
> > 
> > On Thu, 10 Mar 2022 14:03:01 +0100,
> > huangwenhui wrote:
> > >
> > > On a HP 288 Pro G8, the front Mic could not be detected.
> > >
> > > Signed-off-by: huangwenhui <huangwenhuia@uniontech.com>
> >
> > Thanks for the patch.  Most of the changes look OK, but one thing I
> > still don't get:
> >
> > > + case HDA_FIXUP_ACT_INIT:
> > > + alc_write_coef_idx(codec, 0x19, 0xa054);
> > > + msleep(80);
> > > + snd_hda_codec_write(codec, hp_pin, 0,
> > > +     AC_VERB_SET_AMP_GAIN_MUTE, AMP_OUT_UNMUTE);
> >
> > Why this unconditional unmute is required for fixing the mic problem?
> >
> > Takashi
> >
> >
> 
>