platform/x86: ideapad-laptop: Ignore VPC event bit 10
authorHans de Goede <hdegoede@redhat.com>
Sun, 23 May 2021 17:23:31 +0000 (19:23 +0200)
committerHans de Goede <hdegoede@redhat.com>
Wed, 16 Jun 2021 15:47:50 +0000 (17:47 +0200)
VPC event bit 10 gets set on a Yoga 300-11IBR when the EC believes that the
device has changed between laptop/tent/stand/tablet mode.

The EC relies on getting angle info from 2 accelerometers through a special
windows service calling a DSM on the DUAL250E ACPI-device. Linux does not
do this, making the laptop/tent/stand/tablet mode info unreliable.

Ignore VPC event bit 10 to avoid the warnings triggered by the default case
in ideapad_acpi_notify().

Note that the plan for Linux is to have iio-sensor-proxy read the 2
accelerometers and have it provide info about which mode 360° hinges
2-in-1s to the rest of userspace:
https://gitlab.freedesktop.org/hadess/iio-sensor-proxy/-/issues/216

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Link: https://lore.kernel.org/r/20210523172331.177834-1-hdegoede@redhat.com
drivers/platform/x86/ideapad-laptop.c

index 3878172..784326b 100644 (file)
@@ -1408,6 +1408,18 @@ static void ideapad_acpi_notify(acpi_handle handle, u32 event, void *data)
                case 6:
                        ideapad_input_report(priv, bit);
                        break;
+               case 10:
+                       /*
+                        * This event gets send on a Yoga 300-11IBR when the EC
+                        * believes that the device has changed between laptop/
+                        * tent/stand/tablet mode. The EC relies on getting
+                        * angle info from 2 accelerometers through a special
+                        * windows service calling a DSM on the DUAL250E ACPI-
+                        * device. Linux does not do this, making the laptop/
+                        * tent/stand/tablet mode info unreliable, so we simply
+                        * ignore these events.
+                        */
+                       break;
                case 9:
                        ideapad_sync_rfk_state(priv);
                        break;