linux-kernel - Re: [PATCH 0/3] Adjust behavior for HPD 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 for Android: free password hash cracker in your pocket

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

Message-ID: <rno803rs-8o49-r788-npq8-7n879sr47977@xreary.bet>
Date: Tue, 4 Mar 2025 21:50:08 +0100 (CET)
From: Jiri Kosina <jikos@...nel.org>
To: Mario Limonciello <superm1@...nel.org>
cc: Basavaraj Natikar <basavaraj.natikar@....com>, 
    Benjamin Tissoires <bentiss@...nel.org>, 
    open list <linux-kernel@...r.kernel.org>, 
    "open list:AMD SENSOR FUSION HUB DRIVER" <linux-input@...r.kernel.org>, 
    Mario Limonciello <mario.limonciello@....com>, 
    Pratap Nirujogi <pratap.nirujogi@....com>, Anson Tsao <anson.tsao@....com>
Subject: Re: [PATCH 0/3] Adjust behavior for HPD sensor

On Fri, 28 Feb 2025, Mario Limonciello wrote:

> From: Mario Limonciello <mario.limonciello@....com>
> 
> Some platforms include a human presence detection (HPD) sensor. When
> enabled and a user is detected a wake event will be emitted from the
> sensor fusion hub that software can react to.
> 
> Example use cases are "wake from suspend on approach" or to "lock
> when leaving".
> 
> This is currently enabled by default on supported systems, but users
> can't control it. This essentially means that wake on approach is
> enabled which is a really surprising behavior to users that don't
> expect it.
> 
> Instead of defaulting to enabled add a sysfs knob that users can
> use to enable the feature if desirable and set it to disabled by
> default.
> 
> Cc: Pratap Nirujogi <pratap.nirujogi@....com>
> Cc: Anson Tsao  <anson.tsao@....com>
> 
> Mario Limonciello (3):
>   HID: amd_sfh: Allow configuring whether HPD is enabled or disabled
>   HID: amd_sfh: Default to HPD disabled
>   HID: amd_sfh: Don't show wrong status for amd_sfh_hpd_info()
> 
>  .../ABI/testing/sysfs-driver-amd-sfh          | 13 +++++
>  drivers/hid/amd-sfh-hid/amd_sfh_common.h      |  1 +
>  drivers/hid/amd-sfh-hid/amd_sfh_pcie.c        | 58 +++++++++++++++++++
>  drivers/hid/amd-sfh-hid/sfh1_1/amd_sfh_init.c | 50 +++++++++++++++-
>  drivers/hid/amd-sfh-hid/sfh1_1/amd_sfh_init.h |  3 +
>  .../amd-sfh-hid/sfh1_1/amd_sfh_interface.c    |  2 +-
>  6 files changed, 124 insertions(+), 3 deletions(-)
>  create mode 100644 Documentation/ABI/testing/sysfs-driver-amd-sfh

Now applied to hid.git#for-6.14/amd_sfh. Thanks,

-- 
Jiri Kosina
SUSE Labs