diff mbox series

[1/2] ACPI: x86: utils: Mark SMO8810 accel on Dell XPS 15 9550 as always present

Message ID 20240406135625.17118-1-hdegoede@redhat.com
State Accepted
Commit bfd1a492b56082034cdf560d66c26b3b636fad18
Headers show
Series [1/2] ACPI: x86: utils: Mark SMO8810 accel on Dell XPS 15 9550 as always present | expand

Commit Message

Hans de Goede April 6, 2024, 1:56 p.m. UTC
The Dell XPS 15 9550 has a SMO8110 accelerometer /
HDD freefall sensor which is wrongly marked as not present.

Mark this as always present so that the dell-smo8800 driver
can bind to it.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 drivers/acpi/x86/utils.c | 9 +++++++++
 1 file changed, 9 insertions(+)

Comments

Rafael J. Wysocki April 8, 2024, 2:52 p.m. UTC | #1
On Sat, Apr 6, 2024 at 3:58 PM Hans de Goede <hdegoede@redhat.com> wrote:
>
> The Dell XPS 15 9550 has a SMO8110 accelerometer /
> HDD freefall sensor which is wrongly marked as not present.
>
> Mark this as always present so that the dell-smo8800 driver
> can bind to it.
>
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> ---
>  drivers/acpi/x86/utils.c | 9 +++++++++
>  1 file changed, 9 insertions(+)
>
> diff --git a/drivers/acpi/x86/utils.c b/drivers/acpi/x86/utils.c
> index 90c3d2eab9e9..c7af2d2986fd 100644
> --- a/drivers/acpi/x86/utils.c
> +++ b/drivers/acpi/x86/utils.c
> @@ -100,6 +100,15 @@ static const struct override_status_id override_status_ids[] = {
>                 DMI_MATCH(DMI_PRODUCT_NAME, "Venue 11 Pro 7139"),
>               }),
>
> +       /*
> +        * The Dell XPS 15 9550 has a SMO8110 accelerometer /
> +        * HDD freefall sensor which is wrongly marked as not present.
> +        */
> +       PRESENT_ENTRY_HID("SMO8810", "1", SKYLAKE, {
> +               DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> +               DMI_MATCH(DMI_PRODUCT_NAME, "XPS 15 9550"),
> +             }),
> +
>         /*
>          * The GPD win BIOS dated 20170221 has disabled the accelerometer, the
>          * drivers sometimes cause crashes under Windows and this is how the
> --

Applied as 6.10 material along with the [2/2], thanks!
diff mbox series

Patch

diff --git a/drivers/acpi/x86/utils.c b/drivers/acpi/x86/utils.c
index 90c3d2eab9e9..c7af2d2986fd 100644
--- a/drivers/acpi/x86/utils.c
+++ b/drivers/acpi/x86/utils.c
@@ -100,6 +100,15 @@  static const struct override_status_id override_status_ids[] = {
 		DMI_MATCH(DMI_PRODUCT_NAME, "Venue 11 Pro 7139"),
 	      }),
 
+	/*
+	 * The Dell XPS 15 9550 has a SMO8110 accelerometer /
+	 * HDD freefall sensor which is wrongly marked as not present.
+	 */
+	PRESENT_ENTRY_HID("SMO8810", "1", SKYLAKE, {
+		DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
+		DMI_MATCH(DMI_PRODUCT_NAME, "XPS 15 9550"),
+	      }),
+
 	/*
 	 * The GPD win BIOS dated 20170221 has disabled the accelerometer, the
 	 * drivers sometimes cause crashes under Windows and this is how the