Re: [PATCH 1/3] iio: hid-sensors: Add proximity and attention IDs

[Jonathan Cameron <jic23@kernel.org>]

On Thu, 24 Oct 2024 13:29:05 +0000
Ricardo Ribalda <ribalda@chromium.org> wrote:

> The HID Usage Table at https://usb.org/sites/default/files/hut1_5.pdf	
> 
> - 0x4b2 for Human Proximity Range

Add more here from the references the above has.
https://www.usb.org/sites/default/files/hutrr39b_0.pdf

Distance between a human and the computer. Default unit of
measure is meters;



> - 0x4bd for Human Attention Detected
> 
Human-Presence sensors detect the presence of humans in the sensor’s field-of-view using
diverse and evolving technologies. Some presence sensors are implemented with low
resolution video cameras, which can additionally track a subject’s attention (i.e. if the
user is ‘looking’ at the system with the integrated sensor).
A Human-Presence sensor, providing a Host with the user’s attention state, allows the
Host to optimize its behavior. For example, to brighten/dim the system display, based on
the user’s attention to the system (potentially prolonging battery life).

This one is a 0/1 signal if I read the schema right.  Which is also true for
the existing HUMAN_PRESENCE. Not a problem but worth mentioning in this description.
  
https://www.usb.org/sites/default/files/hutrr107-humanpresenceattention_1.pdf



> Signed-off-by: Ricardo Ribalda <ribalda@chromium.org>
> ---
>  include/linux/hid-sensor-ids.h | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/include/linux/hid-sensor-ids.h b/include/linux/hid-sensor-ids.h
> index 6730ee900ee1..8a03d9696b1c 100644
> --- a/include/linux/hid-sensor-ids.h
> +++ b/include/linux/hid-sensor-ids.h
> @@ -30,6 +30,8 @@
>  #define HID_USAGE_SENSOR_PROX                                   0x200011
>  #define HID_USAGE_SENSOR_DATA_PRESENCE                          0x2004b0
>  #define HID_USAGE_SENSOR_HUMAN_PRESENCE                         0x2004b1
> +#define HID_USAGE_SENSOR_HUMAN_PROXIMITY                        0x2004b2
> +#define HID_USAGE_SENSOR_HUMAN_ATTENTION                        0x2004bd
>  
>  /* Pressure (200031) */
>  #define HID_USAGE_SENSOR_PRESSURE                               0x200031
>