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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <1493822196.17987.3.camel@debian.org>
Date:   Wed, 03 May 2017 20:06:36 +0530
From:   Ritesh Raj Sarraf <rrs@...ian.org>
To:     Andy Shevchenko <andy.shevchenko@...il.com>
Cc:     Platform Driver <platform-driver-x86@...r.kernel.org>,
        Darren Hart <dvhart@...radead.org>,
        Ike Panhc <ike.pan@...onical.com>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v4] platform/x86: ideapad-laptop: Add sysfs interface
 for touchpad state

Resending again, as Google servers are behaving weird lately.

On Sun, 2017-04-30 at 15:54 +0300, Andy Shevchenko wrote:
> > > 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.

The need we have in the user application is for read-only access. 

I have attached the same old patch with just the minor file attribute change,
which would make the sysfs interface to read-only. If in future, there are good
reasons to expose that interface as read/write, updating that attribute can be
revisited.

Can this now be included in your tree ? This minor patch may be helpful to many
Ideapad users with a Hybrid machine.


PS: Given how minimal the change is, I just hand updated the patch. If you want
it in the proper git send-email flow, please let me know.

-- 
Ritesh Raj Sarraf | http://people.debian.org/~rrs
Debian - The Universal Operating System
View attachment "0001-Add-sysfs-interface-for-touchpad-state.patch" of type "text/x-patch" (3269 bytes)

Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)