Hi,
On 1/20/22 14:43, Stefan Binding wrote:
From: Lucas Tanure tanureal@opensource.cirrus.com
The ACPI device with CSC3551 or CLSA0100 are sound cards with multiple instances of CS35L41 connectec by I2C or SPI to the main CPU.
We add an ID to the ignore_serial_bus_ids list to enumerate all I2C or SPI devices correctly.
Signed-off-by: Lucas Tanure tanureal@opensource.cirrus.com Signed-off-by: Stefan Binding sbinding@opensource.cirrus.com
Thanks, patch looks good to me:
Reviewed-by: Hans de Goede hdegoede@redhat.com
Regards,
Hans
drivers/acpi/scan.c | 3 +++ drivers/platform/x86/bus-multi-instantiate.c | 14 ++++++++++++++ 2 files changed, 17 insertions(+)
diff --git a/drivers/acpi/scan.c b/drivers/acpi/scan.c index 4fd1346d6e1f..2f2210c969c8 100644 --- a/drivers/acpi/scan.c +++ b/drivers/acpi/scan.c @@ -1744,8 +1744,11 @@ static bool acpi_device_enumeration_by_parent(struct acpi_device *device) */ {"BSG1160", }, {"BSG2150", },
{"CSC3551", },- /* Non-conforming _HID for Cirrus Logic already released */
{"CLSA0100", },
- HIDs of device with an UartSerialBusV2 resource for which userspace
- expects a regular tty cdev to be created (instead of the in kernel
diff --git a/drivers/platform/x86/bus-multi-instantiate.c b/drivers/platform/x86/bus-multi-instantiate.c index 05bc941a38fd..7a394b671f5a 100644 --- a/drivers/platform/x86/bus-multi-instantiate.c +++ b/drivers/platform/x86/bus-multi-instantiate.c @@ -328,6 +328,17 @@ static const struct bmi_node int3515_data = { .bus_type = BMI_I2C, };
+static const struct bmi_node cs35l41_hda = {
- .instances = {
{ "cs35l41-hda", IRQ_RESOURCE_GPIO, 0 },{ "cs35l41-hda", IRQ_RESOURCE_GPIO, 0 },{ "cs35l41-hda", IRQ_RESOURCE_GPIO, 0 },{ "cs35l41-hda", IRQ_RESOURCE_GPIO, 0 },{}- },
- .bus_type = BMI_AUTO_DETECT,
+};
/*
- Note new device-ids must also be added to bus_multi_instantiate_ids in
- drivers/acpi/scan.c: acpi_device_enumeration_by_parent().
@@ -336,6 +347,9 @@ static const struct acpi_device_id bmi_acpi_ids[] = { { "BSG1160", (unsigned long)&bsg1160_data }, { "BSG2150", (unsigned long)&bsg2150_data }, { "INT3515", (unsigned long)&int3515_data },
- { "CSC3551", (unsigned long)&cs35l41_hda },
- /* Non-conforming _HID for Cirrus Logic already released */
- { "CLSA0100", (unsigned long)&cs35l41_hda }, { }
}; MODULE_DEVICE_TABLE(acpi, bmi_acpi_ids);