All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stefan Binding <sbinding@opensource.cirrus.com>
To: Mark Brown <broonie@kernel.org>,
	"Rafael J . Wysocki" <rafael@kernel.org>,
	Len Brown <lenb@kernel.org>, Hans de Goede <hdegoede@redhat.com>,
	Mark Gross <markgross@kernel.org>,
	Jaroslav Kysela <perex@perex.cz>, Takashi Iwai <tiwai@suse.com>
Cc: <alsa-devel@alsa-project.org>, <linux-kernel@vger.kernel.org>,
	<linux-spi@vger.kernel.org>, <linux-acpi@vger.kernel.org>,
	<platform-driver-x86@vger.kernel.org>,
	<patches@opensource.cirrus.com>,
	Lucas Tanure <tanureal@opensource.cirrus.com>,
	Stefan Binding <sbinding@opensource.cirrus.com>
Subject: [PATCH v6 9/9] ACPI / scan: Create platform device for CS35L41
Date: Fri, 21 Jan 2022 17:24:31 +0000	[thread overview]
Message-ID: <20220121172431.6876-10-sbinding@opensource.cirrus.com> (raw)
In-Reply-To: <20220121172431.6876-1-sbinding@opensource.cirrus.com>

From: Lucas Tanure <tanureal@opensource.cirrus.com>

The ACPI device with CSC3551 or CLSA0100 are sound cards
with multiple instances of CS35L41 connected 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.

The same IDs are also added into serial-multi-instantiate
so that the driver can correctly enumerate the ACPI.

Signed-off-by: Lucas Tanure <tanureal@opensource.cirrus.com>
Signed-off-by: Stefan Binding <sbinding@opensource.cirrus.com>
---
 drivers/acpi/scan.c                             |  3 +++
 drivers/platform/x86/serial-multi-instantiate.c | 14 ++++++++++++++
 2 files changed, 17 insertions(+)

diff --git a/drivers/acpi/scan.c b/drivers/acpi/scan.c
index 48db5e80c2dc..ebd10af3ff7f 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", },
 		{"INT33FE", },
 		{"INT3515", },
+	/* 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/serial-multi-instantiate.c b/drivers/platform/x86/serial-multi-instantiate.c
index 3f05385ca2cf..d09f11eac4f8 100644
--- a/drivers/platform/x86/serial-multi-instantiate.c
+++ b/drivers/platform/x86/serial-multi-instantiate.c
@@ -308,6 +308,17 @@ static const struct smi_node int3515_data = {
 	.bus_type = SMI_I2C,
 };
 
+static const struct smi_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 = SMI_AUTO_DETECT,
+};
+
 /*
  * Note new device-ids must also be added to serial_multi_instantiate_ids in
  * drivers/acpi/scan.c: acpi_device_enumeration_by_parent().
@@ -316,6 +327,9 @@ static const struct acpi_device_id smi_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, smi_acpi_ids);
-- 
2.25.1


WARNING: multiple messages have this Message-ID (diff)
From: Stefan Binding <sbinding@opensource.cirrus.com>
To: Mark Brown <broonie@kernel.org>,
	"Rafael J . Wysocki" <rafael@kernel.org>,
	 Len Brown <lenb@kernel.org>, Hans de Goede <hdegoede@redhat.com>,
	Mark Gross <markgross@kernel.org>,
	Jaroslav Kysela <perex@perex.cz>, Takashi Iwai <tiwai@suse.com>
Cc: alsa-devel@alsa-project.org,
	Lucas Tanure <tanureal@opensource.cirrus.com>,
	patches@opensource.cirrus.com, linux-kernel@vger.kernel.org,
	platform-driver-x86@vger.kernel.org, linux-acpi@vger.kernel.org,
	Stefan Binding <sbinding@opensource.cirrus.com>,
	linux-spi@vger.kernel.org
Subject: [PATCH v6 9/9] ACPI / scan: Create platform device for CS35L41
Date: Fri, 21 Jan 2022 17:24:31 +0000	[thread overview]
Message-ID: <20220121172431.6876-10-sbinding@opensource.cirrus.com> (raw)
In-Reply-To: <20220121172431.6876-1-sbinding@opensource.cirrus.com>

From: Lucas Tanure <tanureal@opensource.cirrus.com>

The ACPI device with CSC3551 or CLSA0100 are sound cards
with multiple instances of CS35L41 connected 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.

The same IDs are also added into serial-multi-instantiate
so that the driver can correctly enumerate the ACPI.

Signed-off-by: Lucas Tanure <tanureal@opensource.cirrus.com>
Signed-off-by: Stefan Binding <sbinding@opensource.cirrus.com>
---
 drivers/acpi/scan.c                             |  3 +++
 drivers/platform/x86/serial-multi-instantiate.c | 14 ++++++++++++++
 2 files changed, 17 insertions(+)

diff --git a/drivers/acpi/scan.c b/drivers/acpi/scan.c
index 48db5e80c2dc..ebd10af3ff7f 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", },
 		{"INT33FE", },
 		{"INT3515", },
+	/* 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/serial-multi-instantiate.c b/drivers/platform/x86/serial-multi-instantiate.c
index 3f05385ca2cf..d09f11eac4f8 100644
--- a/drivers/platform/x86/serial-multi-instantiate.c
+++ b/drivers/platform/x86/serial-multi-instantiate.c
@@ -308,6 +308,17 @@ static const struct smi_node int3515_data = {
 	.bus_type = SMI_I2C,
 };
 
+static const struct smi_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 = SMI_AUTO_DETECT,
+};
+
 /*
  * Note new device-ids must also be added to serial_multi_instantiate_ids in
  * drivers/acpi/scan.c: acpi_device_enumeration_by_parent().
@@ -316,6 +327,9 @@ static const struct acpi_device_id smi_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, smi_acpi_ids);
-- 
2.25.1


  parent reply	other threads:[~2022-01-21 17:25 UTC|newest]

Thread overview: 58+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-01-21 17:24 [PATCH v6 0/9] Support Spi in i2c-multi-instantiate driver Stefan Binding
2022-01-21 17:24 ` Stefan Binding
2022-01-21 17:24 ` [PATCH v6 1/9] spi: Make spi_alloc_device and spi_add_device public again Stefan Binding
2022-01-21 17:24   ` Stefan Binding
2022-02-01 14:26   ` Hans de Goede
2022-02-01 14:26     ` Hans de Goede
2022-02-01 14:52     ` Rafael J. Wysocki
2022-02-01 14:52       ` Rafael J. Wysocki
2022-02-01 14:54       ` Hans de Goede
2022-02-01 14:54         ` Hans de Goede
2022-01-21 17:24 ` [PATCH v6 2/9] spi: Create helper API to lookup ACPI info for spi device Stefan Binding
2022-01-21 17:24   ` Stefan Binding
2022-02-01 14:28   ` Hans de Goede
2022-02-01 14:28     ` Hans de Goede
2022-02-01 17:28     ` Stefan Binding
2022-02-01 17:28       ` Stefan Binding
2022-02-01 17:37       ` Mark Brown
2022-02-01 17:37         ` Mark Brown
2022-01-21 17:24 ` [PATCH v6 3/9] spi: Support selection of the index of the ACPI Spi Resource before alloc Stefan Binding
2022-01-21 17:24   ` Stefan Binding
2022-02-01 14:29   ` Hans de Goede
2022-02-01 14:29     ` Hans de Goede
2022-01-21 17:24 ` [PATCH v6 4/9] spi: Add API to count spi acpi resources Stefan Binding
2022-01-21 17:24   ` Stefan Binding
2022-02-01 14:42   ` Hans de Goede
2022-02-01 14:42     ` Hans de Goede
2022-01-21 17:24 ` [PATCH v6 5/9] platform/x86: i2c-multi-instantiate: Rename it for a generic serial driver name Stefan Binding
2022-01-21 17:24   ` Stefan Binding
2022-02-01 14:55   ` Hans de Goede
2022-02-01 14:55     ` Hans de Goede
2022-01-21 17:24 ` [PATCH v6 6/9] platform/x86: serial-multi-instantiate: Reorganize I2C functions Stefan Binding
2022-01-21 17:24   ` Stefan Binding
2022-02-01 14:59   ` Hans de Goede
2022-02-01 14:59     ` Hans de Goede
2022-01-21 17:24 ` [PATCH v6 7/9] platform/x86: serial-multi-instantiate: Add SPI support Stefan Binding
2022-01-21 17:24   ` Stefan Binding
2022-02-01 15:02   ` Hans de Goede
2022-02-01 15:02     ` Hans de Goede
2022-02-02  8:55     ` Hans de Goede
2022-02-02  8:55       ` Hans de Goede
2022-01-21 17:24 ` [PATCH v6 8/9] ALSA: hda/realtek: Add support for HP Laptops Stefan Binding
2022-01-21 17:24   ` Stefan Binding
2022-02-02  9:05   ` Takashi Iwai
2022-02-02  9:05     ` Takashi Iwai
2022-02-02 16:58     ` Hans de Goede
2022-02-02 16:58       ` Hans de Goede
2022-02-02 17:19       ` Takashi Iwai
2022-02-02 17:19         ` Takashi Iwai
2022-01-21 17:24 ` Stefan Binding [this message]
2022-01-21 17:24   ` [PATCH v6 9/9] ACPI / scan: Create platform device for CS35L41 Stefan Binding
2022-02-01 15:03   ` Hans de Goede
2022-02-01 15:03     ` Hans de Goede
2022-02-01 14:25 ` [PATCH v6 0/9] Support Spi in i2c-multi-instantiate driver Hans de Goede
2022-02-01 14:25   ` Hans de Goede
2022-02-01 17:18 ` (subset) " Mark Brown
2022-02-01 17:18   ` Mark Brown
2022-02-01 17:41 ` Mark Brown
2022-02-01 17:41   ` Mark Brown

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220121172431.6876-10-sbinding@opensource.cirrus.com \
    --to=sbinding@opensource.cirrus.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=broonie@kernel.org \
    --cc=hdegoede@redhat.com \
    --cc=lenb@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-spi@vger.kernel.org \
    --cc=markgross@kernel.org \
    --cc=patches@opensource.cirrus.com \
    --cc=perex@perex.cz \
    --cc=platform-driver-x86@vger.kernel.org \
    --cc=rafael@kernel.org \
    --cc=tanureal@opensource.cirrus.com \
    --cc=tiwai@suse.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.