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> <CAAd53p64w38NCo7c0cnKCyjaswa0_Rns-CjWBwOnph3V5J2taQ@mail.gmail.com>
 <20220117181112.bmbjcofdjjpsfgzo@localhost.localdomain>
In-Reply-To: <20220117181112.bmbjcofdjjpsfgzo@localhost.localdomain>
From:   Kai-Heng Feng <kai.heng.feng@canonical.com>
Date:   Wed, 19 Jan 2022 08:18:52 +0800
Message-ID: <CAAd53p7OWs2ULT10hwC=E7t+KHEidUFMiRmDHGwEM_8+Bib8fA@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 Tue, Jan 18, 2022 at 2:05 AM Alexander Sergeyev <sergeev917@gmail.com> wrote:
>
> On Mon, Jan 17, 2022 at 08:12:08AM +0800, Kai-Heng Feng wrote:
> > If this issue also happens to cold boot it seems to be a BIOS issue.
>
> I've discussed the problem in more detail [1] before sending the patch.
> In short, speakers seems to reliably work in live Ubuntu which differs in
> using initrd with modules vs UEFI stub with built-in drivers in my case
> (I've tried Ubuntu kernel sources). I've tried using modules as well
> (modprobe'ing the list of modules which are load by Ubuntu) with no
> success.
>
> It was suggested that there is a timing issue somewhere. I'm opened for
> ideas to try to debug the problem origins further (to avoid the need in
> the fixup).

Yes it does sound like a timing issue.
Most system firmwares are only test against Windows.

HP has started to enable Ubuntu on some of their systems, but your
case (EFI stub) is uncovered.

>
> > > > 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.
> >
> > If possible, please still give that commit a try.
>
> Well, the first cold boot (with this commit reverted) didn't have any
> sound, the same goes for the subsequent reboot.

Thanks for the testing.

Kai-Heng

>
> [1] https://lkml.org/lkml/2022/1/12/221