Re: [PATCH 10/11] hda: cs35l41: Add support for CS35L41 in HDA systems

[tanureal]

On 11/23/21 4:59 PM, Pierre-Louis Bossart <pierre-louis.bossart@xxxxxxxxxxxxxxx> wrote:
> > +#ifdef CONFIG_ACPI
> > +static const struct acpi_device_id cs35l41_acpi_hda_match[] = {
> > +	{"CLSA0100", 0 },
>
> I could be wrong but this doesn't look like a legit ACPI _HID?
>
> Cirrus Logic can use 'CIR', "CLI", or 'CSC' PNP ID, or an PCI ID.
>
> in the past you used
>
> +#ifdef CONFIG_ACPI
> +static const struct acpi_device_id cs35l41_acpi_match[] = {
> +	{ "CSC3541", 0 }, /* Cirrus Logic PnP ID + part ID */
> +	{},
> +};
> +MODULE_DEVICE_TABLE(acpi, cs35l41_acpi_match);
> +#endif

This ACPI _HID is already released, there is nothing I can do about it.
Future devices will use 'CSC' PnP ID.