linux-kernel - Re: [PATCH] HID: quirks: ignore non-functional sensor in HP 5MP Camera

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: <3135248fcbf284e19a2ffd3ab01767709e322980.camel@linux.intel.com>
Date: Wed, 15 Jan 2025 05:15:07 -0800
From: srinivas pandruvada <srinivas.pandruvada@...ux.intel.com>
To: "Chia-Lin Kao (AceLan)" <acelan.kao@...onical.com>, Jonathan Cameron
	 <jic23@...nel.org>, Lars-Peter Clausen <lars@...afoo.de>, Peter Zijlstra
	 <peterz@...radead.org>, Greg KH <gregkh@...uxfoundation.org>, 
	linux-iio@...r.kernel.org, linux-kernel@...r.kernel.org, Jiri Kosina
	 <jikos@...nel.org>, Benjamin Tissoires <bentiss@...nel.org>, 
	linux-input@...r.kernel.org
Subject: Re: [PATCH] HID: quirks: ignore non-functional sensor in HP 5MP
 Camera

One minor comment:
"HID: quirks: ignore non-functional sensor in HP 5MP Camera"
For this
The format is  not same as other commits in this file. There is no
"quirks". But not sure if Jiri cares.

On Wed, 2025-01-15 at 15:00 +0800, Chia-Lin Kao (AceLan) wrote:
> The HP 5MP Camera (USB ID 0408:5473)
>  reports a HID sensor interface that
> is not actually implemented. Attempting to access this non-functional
> sensor via iio_info causes system hangs as runtime PM tries to wake
> up
> an unresponsive sensor.
> 
>   [453] hid-sensor-hub 0003:0408:5473.0003: Report latency
> attributes: ffffffff:ffffffff
>   [453] hid-sensor-hub 0003:0408:5473.0003: common attributes: 5:1,
> 2:1, 3:1 ffffffff:ffffffff
> 
> Add this device to the HID ignore list since the sensor interface is
> non-functional by design and should not be exposed to userspace.
> 
> Signed-off-by: Chia-Lin Kao (AceLan) <acelan.kao@...onical.com>
Acked-by: Srinivas Pandruvada <srinivas.pandruvada@...ux.intel.com>

> ---
>  drivers/hid/hid-ids.h    | 1 +
>  drivers/hid/hid-quirks.c | 1 +
>  2 files changed, 2 insertions(+)
> 
> diff --git a/drivers/hid/hid-ids.h b/drivers/hid/hid-ids.h
> index 1f47fda809b9..c5b57e857e77 100644
> --- a/drivers/hid/hid-ids.h
> +++ b/drivers/hid/hid-ids.h
> @@ -1094,6 +1094,7 @@
>  #define
> USB_DEVICE_ID_QUANTA_OPTICAL_TOUCH_3001		0x3001
>  #define
> USB_DEVICE_ID_QUANTA_OPTICAL_TOUCH_3003		0x3003
>  #define
> USB_DEVICE_ID_QUANTA_OPTICAL_TOUCH_3008		0x3008
> +#define
> USB_DEVICE_ID_QUANTA_HP_5MP_CAMERA_5473		0x5473
>  
>  #define I2C_VENDOR_ID_RAYDIUM		0x2386
>  #define I2C_PRODUCT_ID_RAYDIUM_4B33	0x4b33
> diff --git a/drivers/hid/hid-quirks.c b/drivers/hid/hid-quirks.c
> index e0bbf0c6345d..5d7a418ccdbe 100644
> --- a/drivers/hid/hid-quirks.c
> +++ b/drivers/hid/hid-quirks.c
> @@ -891,6 +891,7 @@ static const struct hid_device_id
> hid_ignore_list[] = {
>  	{ HID_USB_DEVICE(USB_VENDOR_ID_SYNAPTICS,
> USB_DEVICE_ID_SYNAPTICS_DPAD) },
>  #endif
>  	{ HID_USB_DEVICE(USB_VENDOR_ID_YEALINK,
> USB_DEVICE_ID_YEALINK_P1K_P4K_B2K) },
> +	{ HID_USB_DEVICE(USB_VENDOR_ID_QUANTA,
> USB_DEVICE_ID_QUANTA_HP_5MP_CAMERA_5473) },
>  	{ }
>  };
>