linux-kernel - Re: [PATCH] HID: Add quirk for Lenovo Yoga 910 with ITE Chips

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: <5c0d5b49-509e-a24d-a45e-3435d71f9561@gmail.com>
Date:   Mon, 17 Jul 2017 08:26:37 +0200
From:   Arek Burdach <arek.burdach@...il.com>
To:     Brian Masney <masneyb@...tation.org>
Cc:     Patrick Pedersen <ctx.xda@...il.com>, jikos@...nel.org,
        benjamin.tissoires@...hat.com, jic23@...nel.org,
        srinivas.pandruvada@...ux.intel.com, linux-input@...r.kernel.org,
        linux-kernel@...r.kernel.org, linux-iio@...r.kernel.org
Subject: Re: [PATCH] HID: Add quirk for Lenovo Yoga 910 with ITE Chips



On 16.07.2017 12:23, Brian Masney wrote:
> On Sun, Jul 16, 2017 at 09:39:19AM +0200, Arek Burdach wrote:
>> Hi Patrick,
>>
>> On 15.07.2017 14:27, Patrick Pedersen wrote:
>>> It shall additionally be noted that the i2c-hid 'sleep' bug (present since kernel ver. 4.3)
>>> still affects the driver. This means that the sensor hub will not report any movement, until
>>> the device is suspended and resumed.
>>>
>> Do you have workaround for that? In my case suspending and resuming doesn't
>> help. Sensor reporting is backing to work in unpredictable way. What I've
>> tested:
>>
>> - kernel v4.13-rc1 with your patch applied
>> - watch -n1 cat /sys/bus/iio/devices/iio\:device*/*_raw <- value not
>> changing
>> - suspend
>> - resume
>> - watch -n1 cat /sys/bus/iio/devices/iio\:device*/*_raw <- still not
>> changing
>>
>> If you have some workaround scenario please add it to bug reported be me:
>> https://bugzilla.kernel.org/show_bug.cgi?id=195681
> I have a Lenovo Yoga 910-13IKB
So do I
>   and verified that the patch works
> correctly for me on an upstream 4.12 kernel. The ALS sensor and
> accelerometer function correctly without suspending the laptop with
> this patch. The difference may be due the following kernel boot options
> that I use:
>
> 	i915.enable_rc6=0 pci=noaer intel_pstate=disable
>
> I have to disable power management for the CPU/GPU (i915.enable_rc6=0)
> since I can only suspend and resume once on this laptop. Suspending a
> second time will cause the laptop to hang before the suspend finishes.
>
> The pci=noaer is likely not needed for UEFI firmware revisions 2JCN36WW
> and newer. I'm stuck on UEFI firmware revision 2JCN28WW, which is what
> came preinstalled on the system. Lenovo's UEFI update tool requires
> Windows, which I no longer have.
>
> I need to get on the latest UEFI revision and go through
> basic-pm-debugging.txt if the power management issues are still present.
I have installed 2JCN36WW and: pci=noaer intel_pstate=disable flags are 
never need - suspending works ok every time.

I investigated that having iio-sensor-proxy installed may cause sensors 
stopping to work. Looks like only one option can work in the same time - 
either iio-sensor-proxy service or sensor reporting of raw data.

Arek