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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox