linux-kernel - RE: Fw: [External] RE: [PATCH -next] ASoC: amd: yc: Adding Lenovo ThinkBook 14 Gen 4+ ARA and Lenovo ThinkBook 16 Gen 4+ ARA to the Quirks List

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Tue, 11 Oct 2022 19:17:51 +0000
From:   "Limonciello, Mario" <Mario.Limonciello@....com>
To:     Mark Pearson <markpearson@...ovo.com>,
        "xazrael@...mail.com" <xazrael@...mail.com>,
        "lgirdwood@...il.com" <lgirdwood@...il.com>,
        "broonie@...nel.org" <broonie@...nel.org>,
        "perex@...ex.cz" <perex@...ex.cz>,
        "tiwai@...e.com" <tiwai@...e.com>,
        "Saba Kareem, Syed" <Syed.SabaKareem@....com>,
        "Mukunda, Vijendar" <Vijendar.Mukunda@....com>,
        "Pananchikkal, Renjith" <Renjith.Pananchikkal@....com>
CC:     "alsa-devel@...a-project.org" <alsa-devel@...a-project.org>,
        open list <linux-kernel@...r.kernel.org>
Subject: RE: Fw: [External] RE: [PATCH -next] ASoC: amd: yc: Adding Lenovo
 ThinkBook 14 Gen 4+ ARA and Lenovo ThinkBook 16 Gen 4+ ARA to the Quirks List

[Public]


> <moved to my more open-source friendly email account>

Whoops sorry; I mix them up constantly.

> 
> Thanks Mario
> 
> On 2022-10-11 13:14, Mark Pearson wrote:
> >
> > --------------------------------------------------------------------------------
> > *From:* Limonciello, Mario <Mario.Limonciello@....com>
> > *Sent:* October 11, 2022 13:08
> > *Subject:* [External] RE: [PATCH -next] ASoC: amd: yc: Adding Lenovo
> ThinkBook
> > 14 Gen 4+ ARA and Lenovo ThinkBook 16 Gen 4+ ARA to the Quirks List
> > [Public]
> >
> > Cc:
> > Mark Pearson
> > Vijendar
> > Syed
> > Renjith
> >
> >> -----Original Message-----
> >> From: linkt <xazrael@...mail.com>
> >> Sent: Monday, October 10, 2022 21:52
> >> To: lgirdwood@...il.com; broonie@...nel.org; perex@...ex.cz;
> >> tiwai@...e.com; Limonciello, Mario <Mario.Limonciello@....com>
> >> Cc: alsa-devel@...a-project.org; linux-kernel@...r.kernel.org; linkt
> >> <xazrael@...mail.com>
> >> Subject: [PATCH -next] ASoC: amd: yc: Adding Lenovo ThinkBook 14 Gen
> 4+
> >> ARA and Lenovo ThinkBook 16 Gen 4+ ARA to the Quirks List
> >>
> >> Lenovo ThinkBook 14 Gen 4+ ARA and ThinkBook 16 Gen 4+ ARA
> >> need to be added to the list of quirks for the microphone to work
> properly.
> >>
> >> Signed-off-by: linkt <xazrael@...mail.com>
> >
> > This patch alone makes sense to me.
> >
> > Cc: stable@...r.kernel.org
> > Reviewed-by: Mario Limonciello <mario.limonciello@....com>
> >
> > I think it should go in as is, but it prompts another idea to me I want to run
> by
> > Mark specifically..
> >
> > Should we just drop this whole file down to a much simpler DMI match
> prefix like
> > this:
> >
> >> +     {
> >> +             .driver_data = &acp6x_card,
> >> +             .matches = {
> >> +                     DMI_MATCH(DMI_BOARD_VENDOR, "LENOVO"),
> >> +                     DMI_MATCH(DMI_PRODUCT_NAME, "21"),
> >> +             }
> >> +     },
> >
> > That would allow it to match all the Lenovo Rembrandt systems presumably
> and
> > can stop touching the file.  This only makes sense if every single Lenovo
> > Rembrandt design
> > uses a DMIC from ACP.  I don't know if that's a valid assumption and I think
> we
> > need Mark
> > to confirm it.
> >
> > The DMI match "technically" would match some of the Intel systems from
> this
> > generation
> > but this driver is only "loaded" on AMD Rembrandt systems, so I don't think
> it's
> > a problem.
> >
> 
> I love the idea but I'm a little hesitant as I don't know if I'll be
> able to confirm it authoritatively - I don't have particularly great
> levers to pull on for platforms outside the Linux program. I'll try -
> but I think chances of being able to reply yes or no with confidence are
> limited.
> 
> I think it's relatively low risk for breaking anything on this years
> platforms - but may get more interesting for next years. If needed can
> we then just quirk any platforms that come up that need the logic
> reversing or does that get really messy.
> 
> I can confirm the "21" match is going to be match a ton of Intel
> platforms. For the AMD platforms - will this be limited to Rembrandt
> only or will it affect any past/future generations?

This driver only loads on Yellow Carp (Rembrandt).  It is not used for
any other generations.

My hope is that with the _DSD supported in future programs that
less of these quirks are needed too even for platforms now in your
Linux program.

> 
> Short answer - I'll see what I can find...but no promises
> 

OK, thanks.