Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1633723

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

Show all headers | View raw

On Sun, Apr 30, 2017 at 03:54:43PM +0300, Andy Shevchenko wrote:
> On Sat, Apr 29, 2017 at 9:52 AM, Ritesh Raj Sarraf <rrs@debian.org> wrote:
> > Hello,
> >
> > On Fri, 2017-04-28 at 22:17 +0300, Andy Shevchenko wrote:
> >> On Wed, Feb 22, 2017 at 12:24 PM, Ritesh Raj Sarraf <rrs@debian.org> wrote:
> >>
> >> > Any further comment on this patch ? Will this be accepted ?
> >> > Please give a N/ACK.
> >>
> >> Sorry for a long delay,

Eeek, likewise. This one fell off my stack. We've improved our patch management
process since this was originally submitted, so hopefully this is more and more
rare.

> >> but I can't go with this without clear
> >> understanding that there is indeed no other ways available.
> >> From what I read in the above link
> >
> > i am assuming you are referring to the launchpad bug report here.
> 
> Correct.
> 
> >> the main issue that driver doesn't
> >> send SW_TABLET_MODE event through input device.
> >
> > Well. Yes. That is one part. If SW_TABLET_MODE is done in the driver, much
> > better. My patch was simply in line with some of the other drivers (hp-wmi and
> > thinkpad_acpi) to get it working for Lenovo Yoga series.
> 
> sysfs ABI for drivers that provide input interface is quite strong for
> my opinion. It means I'm not totally objecting, but I would accept it
> if and only if there is nothing else could be done.

Agreed, we've recently wanted to remove certain sysfs attributes from another
driver as they were obsolete and better implemented in other ways, but once they
are there, are hands are tied.

That said, we will support getting these systems functional. From what I see in
the patch you are implementing a polling sysfs interface. Have you verified that
there is no event we can capture and send the SW_TABLET_MODE along to the input
system?

Finally, I just want to point out that while it may seem capricious for us to
resist accepting a new sysfs attribute that exists in other drivers, please note
that the thinkpad driver (for example) is ancient and implements multiple
mechanisms for several things and should not be considered as a reference for
new code. hp_wmi is another story.

Andy, we may also need to consult on a set of common laptop attributes that we
will accept in general. This would ease review and also lead to some more
uniformity in the sysfs attributes of the most common features (like tablet
mode, touchpad, etc.). That said, the value are likely to be platform specific
and it could prove difficult to map all that variety to a common set of values.

-- 
Darren Hart
VMware Open Source Technology Center

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

Re: [PATCH v4] platform/x86: ideapad-laptop: Add sysfs interface for  touchpad state Andy Shevchenko <andy.shevchenko@gmail.com> - 2017-04-28 21:20 +0200
  Re: [PATCH v4] platform/x86: ideapad-laptop: Add sysfs interface  for touchpad state Ritesh Raj Sarraf <rrs@debian.org> - 2017-04-29 09:00 +0200
    Re: [PATCH v4] platform/x86: ideapad-laptop: Add sysfs interface for  touchpad state Andy Shevchenko <andy.shevchenko@gmail.com> - 2017-04-30 15:00 +0200
      Re: [PATCH v4] platform/x86: ideapad-laptop: Add sysfs interface for  touchpad state Darren Hart <dvhart@infradead.org> - 2017-05-01 18:10 +0200
        Re: [PATCH v4] platform/x86: ideapad-laptop: Add sysfs interface  for touchpad state Ritesh Raj Sarraf <rrs@debian.org> - 2017-05-01 20:30 +0200
          Re: [PATCH v4] platform/x86: ideapad-laptop: Add sysfs interface for  touchpad state Darren Hart <dvhart@infradead.org> - 2017-05-01 21:20 +0200
      Re: [PATCH v4] platform/x86: ideapad-laptop: Add sysfs interface  for touchpad state Ritesh Raj Sarraf <rrs@debian.org> - 2017-05-03 16:40 +0200
        Re: [PATCH v4] platform/x86: ideapad-laptop: Add sysfs interface for  touchpad state Andy Shevchenko <andy.shevchenko@gmail.com> - 2017-05-03 16:50 +0200
          Re: [PATCH v4] platform/x86: ideapad-laptop: Add sysfs interface for  touchpad state Darren Hart <dvhart@infradead.org> - 2017-05-03 18:20 +0200
            Re: [PATCH v4] platform/x86: ideapad-laptop: Add sysfs interface for  touchpad state Andy Shevchenko <andy.shevchenko@gmail.com> - 2017-05-08 01:10 +0200

csiph-web