From: Ritesh Raj Sarraf <rrs@debian.org>
To: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: Andy Shevchenko <andy.shevchenko@gmail.com>,
Darren Hart <dvhart@infradead.org>,
Rafael Wysocki <rjw@rjwysocki.net>,
Platform Driver <platform-driver-x86@vger.kernel.org>,
Ike Panhc <ike.pan@canonical.com>,
Andy Shevchenko <andy@infradead.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] platform/x86: ideapad-laptop: Add sysfs interface for touchpad state
Date: Sat, 18 Feb 2017 15:21:35 +0530 [thread overview]
Message-ID: <1487411495.13283.3.camel@debian.org> (raw)
In-Reply-To: <20170217195237.GD13050@dtor-ws>
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512
Hello Dmitry,
On Fri, 2017-02-17 at 11:52 -0800, Dmitry Torokhov wrote:
> OK, if firmware disables the touchpad automatically, then it is fine.
>
> I am wondering about the utility of the new interface though. Are there
> already major users of it?
Not yet. I proposed this interface because we want something like this for
Onboard. For detailed discussion, please see:
https://bugs.launchpad.net/onboard/+bug/1366421/comments/43
hp-wmi and thinkpad-acpi do provide a similar interface, but for tablet.
> If there are no users, then there is no point
> in adding it, it will stay there unused and forgotten by anyone.
> Teaching ideapad to emit SW_TABLET_MODE is more useful IMO.
If accepted, Onboard will be a consumer.
Yes. SW_TABLET_MODE would be ideal. But looking at the drive's code, I don't
know if it has an EC command for it.
- --
Ritesh Raj Sarraf | http://people.debian.org/~rrs
Debian - The Universal Operating System
-----BEGIN PGP SIGNATURE-----
iQIzBAEBCgAdFiEEQCVDstmIVAB/Yn02pjpYo/LhdWkFAlioGScACgkQpjpYo/Lh
dWnEvA/5AfQQYaBiOsunZuR6OcM1BrFy1vpSGAToRNdWkaV18p7IfZGSkDUbZ3U+
rLdL5URnD5/CsHhcOPpUp4IhM4wdce+Rxc4T4LdEyjqZ29IlGrA7k21i2fJXfEws
5Mjd+pN01PitXGGBjc/BcX0ol37pcXyLfMpl3UKvE44LU2CwU5sahnSDyHgM+G77
Gnoy5Po0eLAQoOPAJIsLwSaxJlzoZAknZWuAwqQfNXxXdxVFfTr5OmV9pcTx2vgw
cPdg+8lrU8M7w2SNrK91AvccnXLV5T/rNOolk3VHoH/AQJdgxbiZKdkv0jyg/WKp
TyGIkgwNSFjYeph/JZ/IqXkOnkuB/TcKQq7uVKN5b71U2VouV6gEe/2RcOQ+lSYg
au3NX4Wq6Ot9MSMfqwIZ0+TgazGGjbUlFLhf1TFRZ9o8ERsZrtaXsC5c1U3egAS/
XZWuiI0CI5c3k1WTxWpy76pX963gh+3d9kejanWug2LIriIALmD5/ryN56Jml41S
3I3JnnuiLuwnZ/t/ehA3c9JVhsbTJg4dBbZE9tKgnAjIV/MmmbE9kCtnCRcBuFUa
W2vViMxv2eVBfiHP3Z8TC7E0UUqe69htDEomFfwDpQ1oSqDLzm6gafH8LStbYML8
j+/qGUTyg8d3yP/jrUL5UDO6dMkj+FV5bNi4qlue4uVLV7bx4o8=
=flDx
-----END PGP SIGNATURE-----
next prev parent reply other threads:[~2017-02-18 9:51 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-14 14:16 [PATCH] platform/x86: ideapad-laptop: Add sysfs interface for touchpad state Ritesh Raj Sarraf
2017-02-17 3:33 ` Darren Hart
2017-02-17 7:39 ` Andy Shevchenko
2017-02-17 8:48 ` Dmitry Torokhov
2017-02-17 9:19 ` Ritesh Raj Sarraf
2017-02-17 19:52 ` Dmitry Torokhov
2017-02-18 9:51 ` Ritesh Raj Sarraf [this message]
2017-02-17 9:05 ` Ritesh Raj Sarraf
2017-02-17 14:53 ` Ritesh Raj Sarraf
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=1487411495.13283.3.camel@debian.org \
--to=rrs@debian.org \
--cc=andy.shevchenko@gmail.com \
--cc=andy@infradead.org \
--cc=dmitry.torokhov@gmail.com \
--cc=dvhart@infradead.org \
--cc=ike.pan@canonical.com \
--cc=linux-kernel@vger.kernel.org \
--cc=platform-driver-x86@vger.kernel.org \
--cc=rjw@rjwysocki.net \
/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.