Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
References: <20220114164904.lgj7yimbei6fmloe@localhost.localdomain>
 <20220114165050.ouw2nknuspclynro@localhost.localdomain> <CAAd53p6KXD2mEHgkU_TpTrsU-vQ9Vxdip+6sPfDaVoSOkmaz-g@mail.gmail.com>
 <20220115163208.xmvum6ehqcadonj7@localhost.localdomain>
In-Reply-To: <20220115163208.xmvum6ehqcadonj7@localhost.localdomain>
From:   Kai-Heng Feng <kai.heng.feng@canonical.com>
Date:   Mon, 17 Jan 2022 08:12:08 +0800
Message-ID: <CAAd53p64w38NCo7c0cnKCyjaswa0_Rns-CjWBwOnph3V5J2taQ@mail.gmail.com>
Subject: Re: [PATCH v2 1/1] ALSA: hda/realtek: fix speakers and micmute on HP
 855 G8
To:     Alexander Sergeyev <sergeev917@gmail.com>
Cc:     Jaroslav Kysela <perex@perex.cz>, Takashi Iwai <tiwai@suse.com>,
        Jeremy Szu <jeremy.szu@canonical.com>,
        Werner Sembach <wse@tuxedocomputers.com>,
        Hui Wang <hui.wang@canonical.com>,
        Cameron Berkenpas <cam@neo-zeon.de>,
        Kailang Yang <kailang@realtek.com>, Sami Loone <sami@loone.fi>,
        Elia Devito <eliadevito@gmail.com>,
        alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

On Sun, Jan 16, 2022 at 12:26 AM Alexander Sergeyev
<sergeev917@gmail.com> wrote:
>
> On Sat, Jan 15, 2022 at 11:47:40PM +0800, Kai-Heng Feng wrote:
> > > Note that the device can boot up with working speakers and micmute LED
> > > without this patch, but the success rate would be quite low (order of
> > > 16 working boots across 709 boots) at least for the built-in drivers
> > > scenario. This also means that there are some timing issues during early
> > > boot and this patch is a workaround.
> >
> > Does this issue happen to warm boot (reboot) or cold boot?
>
> The issue is definitely not limited to warm boots only. I can confirm cold
> boots with no sound and also warm boots with functioning sound. With this in
> mind, the provided numbers (16 good boots across 709 boots) are largely coming
> from reboots since the testing was scripted and driven by SSH.

If this issue also happens to cold boot it seems to be a BIOS issue.

>
> > If the issue only happen to warm boot, please try reverting commit
> > 9d3fcb28f9b9 "Revert "PM: ACPI: reboot: Use S5 for reboot"".
> > Many HP systems requires it to have a functional reboot.
>
> Interesting. I haven't noticed anything breaking with reboots on this
> particular system (at least so far).

If possible, please still give that commit a try.

Kai-Heng