linux-kernel - Re: [PATCH v2] HID: amd_sfh: Add support for tablet-mode-switch sensor

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] [day] [month] [year] [list]

Message-ID: <e2b16837-cc05-4f04-f58e-a8315ab85321@freund.io>
Date:   Wed, 18 Jan 2023 10:13:22 +0100
From:   Adrian Freund <adrian@...und.io>
To:     Jiri Kosina <jikos@...nel.org>
Cc:     linux-input@...r.kernel.org, linux-kernel@...r.kernel.org,
        Basavaraj Natikar <basavaraj.natikar@....com>,
        Benjamin Tissoires <benjamin.tissoires@...hat.com>,
        "Luke D. Jones" <luke@...nes.dev>, Ivan Dovgal <iv.dovg@...il.com>
Subject: Re: [PATCH v2] HID: amd_sfh: Add support for tablet-mode-switch
 sensor

On 1/18/23 09:49, Jiri Kosina wrote:
> On Sun, 18 Dec 2022, Adrian Freund wrote:
>
>> This patch adds support for the tablet mode switch sensors on
>> convertible devices where that sensor is managed by AMD SFH, like the
>> Asus Flow X13 and the Lenovo ThinkPad L13 Yoga Gen2 (AMD).
>>
>> Co-developed-by: Ivan Dovgal <iv.dovg@...il.com>
>> Signed-off-by: Ivan Dovgal <iv.dovg@...il.com>
>> Co-developed-by: Luke D. Jones <luke@...nes.dev>
>> Signed-off-by: Luke D. Jones <luke@...nes.dev>
>> Signed-off-by: Adrian Freund <adrian@...und.io>
> Basavaraj, could you please review this? Thanks,
>
I think this doesn't need a review right now. I recently found some big 
problems with the patch recently and am working on an updated version. I 
think reviewing once I send v3 should be enough.

In short: This patch generates nonsensical HID messages that don't even 
contain the sensor state.
It still makes the tablet-mode-switch work on some but not all devices 
due to a mechanism I don't fully understand yet. Somehow ACPI devices 
that previously did nothing suddenly work with this patch and are then 
getting picked up by various platform drivers depending on the device. 
This driver however doesn't properly handle the actual data directly 
from the sensor.

---
Adrian