> +#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