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;
Received-SPF: SoftFail (protection.outlook.com: domain of transitioning
 lenovo.com discourages use of 104.232.225.6 as permitted sender)
Message-ID: <8c7cb257-3e88-e299-5e92-a44cea06d771@lenovo.com>
Date:   Tue, 11 Oct 2022 14:30:33 -0400
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.3.0
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
Content-Language: en-US
To:     Mario Limonciello <mario.limonciello@amd.com>,
        <xazrael@hotmail.com>, <lgirdwood@gmail.com>, <broonie@kernel.org>,
        <perex@perex.cz>, <tiwai@suse.com>, <Syed.SabaKareem@amd.com>,
        <Vijendar.Mukunda@amd.com>, <renjith.pananchikkal@amd.com>
References: <MEYPR01MB8397A3C27DE6206FA3EF834DB6239@MEYPR01MB8397.ausprd01.prod.outlook.com>
 <MN0PR12MB61013CCC5DD10347C3FA3EECE2239@MN0PR12MB6101.namprd12.prod.outlook.com>
 <TYZPR03MB5994C7FD8582F831A69055DEBD239@TYZPR03MB5994.apcprd03.prod.outlook.com>
From:   Mark Pearson <markpearson@lenovo.com>
CC:     <alsa-devel@alsa-project.org>,
        open list <linux-kernel@vger.kernel.org>
In-Reply-To: <TYZPR03MB5994C7FD8582F831A69055DEBD239@TYZPR03MB5994.apcprd03.prod.outlook.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Oct 2022 18:30:36.6995
 (UTC)
X-MS-Exchange-CrossTenant-Network-Message-Id: 447c8877-2a64-4267-8236-08daabb6b127
X-MS-Exchange-CrossTenant-Id: 5c7d0b28-bdf8-410c-aa93-4df372b16203
X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=5c7d0b28-bdf8-410c-aa93-4df372b16203;Ip=[104.232.225.6];Helo=[mail.lenovo.com]
X-MS-Exchange-CrossTenant-AuthSource: TYZAPC01FT052.eop-APC01.prod.protection.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Anonymous
X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PUZPR03MB5824
X-Spam-Status: No, score=-5.0 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,
        RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,SPF_NONE
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

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

Thanks Mario

On 2022-10-11 13:14, Mark Pearson wrote:
> 
> --------------------------------------------------------------------------------
> *From:* Limonciello, Mario <Mario.Limonciello@amd.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@hotmail.com>
>> Sent: Monday, October 10, 2022 21:52
>> To: lgirdwood@gmail.com; broonie@kernel.org; perex@perex.cz;
>> tiwai@suse.com; Limonciello, Mario <Mario.Limonciello@amd.com>
>> Cc: alsa-devel@alsa-project.org; linux-kernel@vger.kernel.org; linkt
>> <xazrael@hotmail.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@hotmail.com>
> 
> This patch alone makes sense to me.
> 
> Cc: stable@vger.kernel.org
> Reviewed-by: Mario Limonciello <mario.limonciello@amd.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?

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

Mark