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;
Date:   Sat, 15 Jan 2022 19:32:08 +0300
From:   Alexander Sergeyev <sergeev917@gmail.com>
To:     Kai-Heng Feng <kai.heng.feng@canonical.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
Subject: Re: [PATCH v2 1/1] ALSA: hda/realtek: fix speakers and micmute on HP
 855 G8
Message-ID: <20220115163208.xmvum6ehqcadonj7@localhost.localdomain>
User-Agent: mtt
References: <20220114164904.lgj7yimbei6fmloe@localhost.localdomain>
 <20220114165050.ouw2nknuspclynro@localhost.localdomain>
 <CAAd53p6KXD2mEHgkU_TpTrsU-vQ9Vxdip+6sPfDaVoSOkmaz-g@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <CAAd53p6KXD2mEHgkU_TpTrsU-vQ9Vxdip+6sPfDaVoSOkmaz-g@mail.gmail.com>
Precedence: bulk

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 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).