On 6/30/21 4:35 PM, David Rhodes wrote: > On 6/29/21 6:51 PM, Pierre-Louis Bossart wrote: >> >>> +#ifdef CONFIG_ACPI >>> +static const struct acpi_device_id cs35l41_acpi_match[] = { >>> + { "CSC3541", 0 }, /* Cirrus Logic PCI ID + part ID */ >> >> Wrong comment or wrong ID, "CSC" is clearly not a PCI ID? >> > > CSC is a PnP ID, not a PCI ID. I will change the comment to reflect this. Funny to see how Crystal Semiconductor still shows in PNP IDs, 30 years after being acquired by Cirrus Logic...