Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.35 as permitted sender) client-ip=23.128.96.35;
Message-ID: <0a3feafc-b843-420a-9b04-c835f8210c1a@amd.com>
Date:   Mon, 2 Oct 2023 08:47:21 -0500
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v4] ASoC: amd: yc: Fix non-functional mic on Lenovo 82YM
To:     Mark Brown <broonie@kernel.org>,
        Linux regressions mailing list <regressions@lists.linux.dev>
Cc:     Sven Frotscher <sven.frotscher@gmail.com>, git@augustwikerfors.se,
        alsa-devel@alsa-project.org, lgirdwood@gmail.com,
        linux-kernel@vger.kernel.org, stable@vger.kernel.org,
        Takashi Iwai <tiwai@suse.com>
References: <20230927223758.18870-1-sven.frotscher@gmail.com>
 <46560887-0b6e-42ac-96c3-b4dbc1d7cb61@leemhuis.info>
 <4fa7d39d-dc34-4550-97fa-2b089f364cca@sirena.org.uk>
Content-Language: en-US
From:   Mario Limonciello <mario.limonciello@amd.com>
In-Reply-To: <4fa7d39d-dc34-4550-97fa-2b089f364cca@sirena.org.uk>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?QTFHR000cVRsa0crOE4ySjVybzhkVW9aenZYQnJTTjBoK05IUnNva01IblhG?=
 =?utf-8?B?VVNTcDV2bkNVUStRbGREMW9lRFNVaVlwakFtaGxXUHBuRHB3VDBDeFRFMkor?=
 =?utf-8?B?ZmdpVzJUZnZMM0U1cWMxbW15M3hkYytaYTFCTjhHMDNadnBCOEtVS1JNcjN6?=
 =?utf-8?B?Z09sQmtFajRmUWxDMW1rWHJIR2FyaHRxaTV2OGtocWZiZFk3dE9EY0hTM1Z1?=
 =?utf-8?B?SStNTzhzbmVDbWF4M3RYMTVmMWpuT1FIQ2hmT1lZMCs3WVQzRGJRU1lETlQz?=
 =?utf-8?B?S2IyR0FwRjA2N0R3QnpvbW9aeUZGVDNiRXZRY3RUNHpHbEVhSDEzWEl2VnJu?=
 =?utf-8?B?blpCeVBGc0NyVHM4clF6ckJ3eFJrYUI3bzF4bVF6NDhjazMyODRZVnI1U0ta?=
 =?utf-8?B?NzJZUU12L3J1WG1BTnlSR2czb09wSU1UMWYvd3N2M1g2NVY2UXdSdEk5MFNC?=
 =?utf-8?B?QklCZ0Yyd05rQzNqYUlMT3BKZGgrZ3V5NDN2Q2lsemNIbGR3eTVNRGJXN3Yy?=
 =?utf-8?B?ejJFRW1zdjFBK2NHVkw0cVhrV2VHd2VuUEx2SlZOYmFONVRMd3pvK3EwaGtn?=
 =?utf-8?B?d3JGQ2xscUJZZUdIdjhBMkoxaG92NWh5YmlJOENDR2YwNkFMYVdZd2hZYlpG?=
 =?utf-8?B?b3k4UVZnRXlaeDlBL25qcDRuU1RGdVdHdmhpblhoK3B6eUtvbTNtWE9lakRu?=
 =?utf-8?B?aTlobzVWRWZQVzAvUmhFNm05RlZDR0xXMzdxeXRrSFhLdnp6ekY0cjFTZE9r?=
 =?utf-8?B?QVBLbkNKaSt2VUJQaC81bktGdlpXRk81RVA4bEN6M2NCN2RLR2liTDhDZnIy?=
 =?utf-8?B?TWUvWUVjZlRBTExNQlVGRThNWHQzWTZlTDdxbjVXZGs0Y2FOZ3hzbTJVcUM2?=
 =?utf-8?B?N3NkdUxwRlRGTDZ0QlZqUE92d3VoQjhPUDF4RWszZjRGUTUzejlwdUw0REdU?=
 =?utf-8?B?R3J0Tkp3c3duZGV2NWFISFpUcEU2Wmh4NkxUbDY4MFkvaHJLSmsyNGtzUWpn?=
 =?utf-8?B?Q0UvdnMyMmNJSTlQTVMxUGhFQzRaRmllZ2tZbEV1WDFrYVlQQmVPUml6bStk?=
 =?utf-8?B?aVF0YnQxWjR2d0M3UzlmZGZWaU5WRkw2aFpVSCtGR25wbjZSbnFZSmRVQ1JN?=
 =?utf-8?B?bjdnWWVpTHNYRStUa0pqUkxYdVQ4QmFxTTM1ZnFEVmhmTC9BTThZb0I1a2Qz?=
 =?utf-8?B?OXg1Mi90bkN1SmpFaDVwbnQ5bnlwK3RtdlZlZm9idUhQa2FYaG5yaSs2aHBW?=
 =?utf-8?B?MGNtZnlnZ0QwZDNvcU1tWlIxU2pKUFBNb0dPV05uM2p4bVVsejJQRHVCTHJk?=
 =?utf-8?B?WVUwRG9VWXVYeGVTS0tLblFURHF2SWprSjFSaE9GMDNMdUFJbGtYS2FqUXda?=
 =?utf-8?B?QU5yQ2NQUVlub0RnRGxlNkN3K1BCVXk3aXlHSit6VjdVOEtNSGpVQnlsd2Zu?=
 =?utf-8?B?LzZONm9Nbm9JVnNwMEVwRVV0KzVQTjdLVjYrL0huMVpFS1N3aWFmc0Q5M0k2?=
 =?utf-8?B?ZkFZQ05uL01ZUDErUjhlUWppaGVRL2lEUEZLbk1HekNTcUNQTjdsbkEyckhx?=
 =?utf-8?B?ZWowMzlseUNxL1Y3ZUwyWm1xcDJ3emFFR0o3dUNqbTNsMXpwR1kra3VKL1dW?=
 =?utf-8?B?T3MxQ0tEVlZTdWY4UWt0cXRuL01aeFlUaXVyYUQzR1IyVHM3a0gySlVvb3VD?=
 =?utf-8?B?TVZSMWF1Wml5QzhEbGRVNU5ndk5CdzZUYmhHa0U4UlZpUVpmNjgrQ2I3eWxi?=
 =?utf-8?B?U2dweU1hekN6Z1hoanB4SGVNYkJzQUt6WGhnRVlZSFVyajBkbW1ZTWl1aXNL?=
 =?utf-8?B?aDhOaTdnZ1NZZmVBemhGdDZKZkJMNTNkdjJmZTY2V2dONkhwZHFvTDYzbFNQ?=
 =?utf-8?B?Mzdzakt1RGhJZDdqVVNtUXc3T2FVTTIrcDg1c0R5K2xIcW01SnMrVjhBWG16?=
 =?utf-8?B?bXdYVzVpQmVpSVVDdW54d1cwY0NPWFh5WExyVUZlSmFKK3IzNVpsR3NEWXRO?=
 =?utf-8?B?b1Q3UUgxdDNKNGRDaGtjV2crVStOSTVRd0VuVnI3QzZER3BXU1JvR2JLdDlM?=
 =?utf-8?B?Z3VWSTBqRlRSV0xTREtsZFEycXZHZlFmWHIraXg0OFJJbis2dTlMOHc0c1pk?=
 =?utf-8?Q?9rcOxaFsR7KULQKWRp8sxrmJB?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: f550d5fa-b0d1-4619-2cd6-08dbc34e1bda
X-MS-Exchange-CrossTenant-AuthSource: MN0PR12MB6101.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Oct 2023 13:47:24.7079
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: LGfhbfOQWwZKQBPe9Owzk0ayCPbQ7tTH/GBnV1suRgSV9TKxIzwpTw6gHcknjA2x4PSIfND5E/JQw6MFo23cKQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW4PR12MB5667
X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=unavailable
	autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on groat.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (groat.vger.email [0.0.0.0]); Mon, 02 Oct 2023 06:47:50 -0700 (PDT)

On 10/2/2023 06:52, Mark Brown wrote:
> On Mon, Oct 02, 2023 at 11:32:48AM +0200, Linux regression tracking (Thorsten Leemhuis) wrote:
> 
>> Makes me wonder: How many more such quirk entries will be needed? Will
>> we have all machines listed soon, or do we expect that future Lenovo
>> hardware will need entries as well? If it's the latter: are quirks
>> really the right solution here, or do they just hide some bug or then
>> need for code that automatically handles things?
> 
> x86 firmware descriptions are terrible, it's just an endless procession
> of quirks.  The model for ACPI is not to describe key information in the
> kernel and instead on Windows load device specific information from
> separately supplied tables.  On Linux that translates into these endless
> quirks, on Windows it's platform specific drivers for otherwise generic
> audio hardware.

I knew there was a TON of "82" prefix systems from Lenovo so it was an 
educated guess that all of them needed DMIC support.  This was incorrect 
because one of them didn't have DMIC and that caused a no mic support 
problem on that system.

So in the case of this seemingly endless list of systems being added to 
enable DMIC support Mark is right, Windows does it differently.

With the "next" generation of hardware (Phoenix) both Windows and Linux 
*should* be using the same _DSD, so hopefully we don't need more quirks 
like this for those.