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.Thanks for the review. I will fix the license headers, NULL checks, includes, dev_crit, and inits in the next revision as well.
