Re: [PATCH] platform/x86: ideapad-laptop: Ignore VPC event bit 10

[Hans de Goede <hdegoede@redhat.com>]

Hi,

On 5/23/21 7:23 PM, Hans de Goede wrote:
> 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>

I've merged this into my review-hans (and soon also for-next) branch
now.

Regards,

Hans


> ---
>  drivers/platform/x86/ideapad-laptop.c | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 
> diff --git a/drivers/platform/x86/ideapad-laptop.c b/drivers/platform/x86/ideapad-laptop.c
> index 387817290921..784326bd72f0 100644
> --- a/drivers/platform/x86/ideapad-laptop.c
> +++ b/drivers/platform/x86/ideapad-laptop.c
> @@ -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;
>