Re: [PATCH] platform/x86: ideapad-laptop: Add sysfs interface for touchpad state

From: Dmitry Torokhov
Date: Fri Feb 17 2017 - 03:48:19 EST


On Fri, Feb 17, 2017 at 09:39:20AM +0200, Andy Shevchenko wrote:
> On Fri, Feb 17, 2017 at 5:33 AM, Darren Hart <dvhart@xxxxxxxxxxxxx> wrote:
> > On Tue, Feb 14, 2017 at 07:46:12PM +0530, Ritesh Raj Sarraf wrote:
> >> Lenovo Yoga (many variants: Yoga, Yoga2 Pro, Yoga2 13, Yoga3 Pro, Yoga 3
> >> 14 etc) has multiple modles that are a hybrid laptop, working in laptop
> >> mode as well as tablet mode.
>
> > That said, we need to make these systems usable, and as there appears not to be
> > an accepted standard way of doing this, I don't object to the approach. However,
> > you mentioned in the bug comments (#64) on 2/12:
> >
> > "I have attached the final patch that I had proposed, but for whatever
> > reasons, the maintainer isn't convinced that that interface is needed."
> >
> > This is the only version of this patch I have seen.
>
> This patch is actually a v2 that includes maintainers as I had asked
> and additional paragraph to explain why we need such interface.
>
> > Who objected to the patch?
>
> I was trying to understand why /dev/input/eventX can not be used for a such.
>
> > I would like to hear from Rafael and Dmitry, for their opinion on an ACPI or INPUT
> > interface for indicating TABLET_MODE to userspace. Even if we accept this patch
> > as is, we should be thinking about how to do this in a standard way.

Kernel should send EV_SW/SW_TABLET_MODE to communicate tablet mode to
userspace; clients should be listening to these events.

It looks like the patch is trying to provide a way to disable the
touchpad when transitioning in tablet mode. In ChromeOS we have a notion
of "inhibiting" input devices, where userspace policy daemon tells the
kernel that it is not interested in events from a given device and input
core will suppress events from such device (and driver may optionally
put device into low power mode, if it chooses to do so). But in this
case the control seems to be totally outside of the input driver (which
I suspect is PS/2 device), so the policy daemon would have to have
specific knowledge of this new knob.

BTW, I am not sure if this is actually reliable: if system goes to S3
with lid open and user changes it into tablet form, nobody will tell the
EC that touchpad should be ignored and it will wake up the tablet.

Thanks.

--
Dmitry