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: <db1d95be-e92a-4328-b43e-8214a0b520be@amd.com>
Date: Thu, 24 Apr 2025 15:21:30 +0530
From: Basavaraj Natikar <bnatikar@....com>
To: Jiri Kosina <jikos@...nel.org>, 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 4/24/2025 2:57 PM, Jiri Kosina wrote:
> 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)?

Sure, Jiri, I will get back to this patch soon.

We are reviewing similar features and related issues,
and we may need some more time to investigate them internally.

Therefore, I would like to hold off on this patch for a little longer
before providing feedback.

Thanks,
--
Basavaraj

>
> Thanks,
>