Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <24882ec3fed5c9772e1b04088cecece1b2477b23.camel@icenowy.me>
Subject: Re: [PATCH 2/2] ASoC: Intel: Skylake: try to get NHLT blob with PCM
 params as fallback
From:   Icenowy Zheng <uwu@icenowy.me>
To:     Cezary Rojewski <cezary.rojewski@intel.com>
Cc:     alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org,
        Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>,
        Liam Girdwood <liam.r.girdwood@linux.intel.com>,
        Peter Ujfalusi <peter.ujfalusi@linux.intel.com>,
        Bard Liao <yung-chuan.liao@linux.intel.com>,
        Ranjani Sridharan <ranjani.sridharan@linux.intel.com>,
        Kai Vehmanen <kai.vehmanen@linux.intel.com>,
        Mark Brown <broonie@kernel.org>,
        Jaroslav Kysela <perex@perex.cz>, Takashi Iwai <tiwai@suse.com>
Date:   Thu, 18 Aug 2022 17:25:09 +0800
In-Reply-To: <c3386d45-b643-b4aa-c868-5c113cd2955f@intel.com>
References: <20220725111002.143765-1-uwu@icenowy.me>
         <20220725111002.143765-2-uwu@icenowy.me>
         <4208aece-0aea-ba86-9a00-9e217546e1dd@intel.com>
         <3346b62c3cd7e07766457c9140849a31d6d8775c.camel@icenowy.me>
         <85648922-27df-051d-8ae8-d0e4e810198a@intel.com>
         <1891bc72834cb6e25d479c3968c04bcef94c3ad5.camel@icenowy.me>
         <c3386d45-b643-b4aa-c868-5c113cd2955f@intel.com>
Organization: Anthon Open-Source Community
Content-Type: text/plain; charset="UTF-8"
User-Agent: Evolution 3.40.4 
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

=E5=9C=A8 2022-08-17=E6=98=9F=E6=9C=9F=E4=B8=89=E7=9A=84 15:19 +0200=EF=BC=
=8CCezary Rojewski=E5=86=99=E9=81=93=EF=BC=9A
> On 2022-08-17 2:48 PM, Icenowy Zheng wrote:
> > =E5=9C=A8 2022-08-16=E6=98=9F=E6=9C=9F=E4=BA=8C=E7=9A=84 21:08 +0200=EF=
=BC=8CCezary Rojewski=E5=86=99=E9=81=93=EF=BC=9A
>=20
> ...
>=20
> >=20
> > Yes, it's Chell.
> >=20
> > BTW do you need other ACPI tables?
>=20
> At this point, no. Thanks for confirming the platform's name.
>=20
> > >=20
> > > ..and at this point I probably know more than enough. We have
> > > tested
> > > basically all of the KBL and AML configurations when fixing
> > > regressions
> > > during recent skylake-driver up-revs. But Chell (and Lars for
> > > that
> > > matter) families were not among them as these are based on SKL.
> > > I'll
> > > follow up on this with our partners and come back here. I'm
> > > almost
> > > certain topology files for the two families mentioned were not
> > > updated
> > > along the way.
> >=20
> > Could this be an issue of Coreboot, which generates the NHLT table?
>=20
> NHLT was left alone across all the updates. Updating it is one way of
> fixing problems but I don't believe it's necessary in your case.=20
> Topology update is more desirable approach.

BTW how could I fix the topology?

I now use topology files from GalliumOS (which, I assume, is extracted
from ChromeOS).

>=20
> > BTW I think Google pinned the official OS of this hardware to a
> > much
> > lower kernel version (but I don't want to use the official OS
> > because
> > of limited storage of Chell and lack of VM ability of the OS on
> > Chell).
>=20
> That clarifies things out. Guess the kernel version used there is
> v4.4=20
> (plus a ton of un-upstreamed patches). Again, will propagate the=20
> information up the chain. Perhaps one of the solutions for end-users=20
> would be providing working UCM files to alsa-topology-conf repo so
> users=20
> are not powerless in situations such as this one.
>=20
>=20
> Regards,
> Czarek