linux-kernel - Re: [PATCH 0/1] HID: amd_sfh: Add support for tablet mode

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: <19norq05-rp74-9qrr-382r-40q9r59p1pnn@xreary.bet>
Date: Thu, 24 Apr 2025 11:27:04 +0200 (CEST)
From: Jiri Kosina <jikos@...nel.org>
To: Denis Benato <benato.denis96@...il.com>, 
    Basavaraj Natikar <basavaraj.natikar@....com>
cc: Benjamin Tissoires <bentiss@...nel.org>, "Luke D. Jones" <luke@...nes.dev>, 
    Ivan Dovgal <iv.dovg@...il.com>, Adrian Freund <adrian@...und.io>, 
    linux-input@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 0/1] HID: amd_sfh: Add support for tablet mode

On Sun, 9 Mar 2025, Denis Benato wrote:

> Recently there has been a renewed interest in this patch: ASUS has launched a new z13 model
> and many more users requested tablet mode support for previous models.
> 
> I have made required adjustments to apply cleanly on top of linux-next:
> nothing substantial, a macro has been changed from 5 to 6 upstream as
> the previous patch also did and a few line changed their position.
> 
> Given there were no functional changes at all I took
> the liberty to retain previous tags.
> 
> Denis Benato (1):
>   HID: amd_sfh: Add support for tablet mode switch sensors
> 
>  drivers/hid/amd-sfh-hid/amd_sfh_client.c      |  2 ++
>  drivers/hid/amd-sfh-hid/amd_sfh_pcie.c        |  4 +++
>  drivers/hid/amd-sfh-hid/amd_sfh_pcie.h        |  1 +
>  .../hid_descriptor/amd_sfh_hid_desc.c         | 27 +++++++++++++++++++
>  .../hid_descriptor/amd_sfh_hid_desc.h         |  8 ++++++
>  .../hid_descriptor/amd_sfh_hid_report_desc.h  | 20 ++++++++++++++
>  6 files changed, 62 insertions(+)

Basavaraj, could you please provide your Acked-by/Reviewed-by: for this 
(or any other sort of feedback)?

Thanks,

-- 
Jiri Kosina
SUSE Labs