On Sun, 2017-02-19 at 11:50 +0000, Jonathan Cameron wrote:
> On 13/02/17 09:54, Song Hongyan wrote:
> > Add HID_USAGE_SENSOR_HUMAN_PRESENCE support for Human Presence
> > sensitivity
> > parsing, in case some implementation uses it as sensitivity
> > modifier.
> >
> > Signed-off-by: Song Hongyan <hongyan.song@xxxxxxxxx>
Acked-by: Srinivas Pandruvada <srinivas.pandruvada@xxxxxxxxxxxxxxx>
>
> Do we want to perhaps indicate to userspace in some way that this is
> what is being
> reported? (not that I have any idea how a proximity sensor could
> potentially tell
> a human from a passing cat for example ;)
>
> Otherwise, I'm fine with this but would like an Ack from Srinivas.
> > ---
> > drivers/iio/light/hid-sensor-prox.c | 7 +++++++
> > 1 file changed, 7 insertions(+)
> >
> > diff --git a/drivers/iio/light/hid-sensor-prox.c
> > b/drivers/iio/light/hid-sensor-prox.c
> > index 45ca056..73fced8 100644
> > --- a/drivers/iio/light/hid-sensor-prox.c
> > +++ b/drivers/iio/light/hid-sensor-prox.c
> > @@ -240,6 +240,13 @@ static int prox_parse_report(struct
> > platform_device *pdev,
> > st->common_attributes.sensitivity.index,
> > st-
> > >common_attributes.sensitivity.report_id);
> > }
> > + if (st->common_attributes.sensitivity.index < 0)
> > + sensor_hub_input_get_attribute_info(hsdev,
> > + HID_FEATURE_REPORT, usage_id,
> > + HID_USAGE_SENSOR_DATA_MOD_CHANGE_SENSITIVI
> > TY_ABS |
> > + HID_USAGE_SENSOR_HUMAN_PRESENCE,
> > + &st->common_attributes.sensitivity);
> > +
> > return ret;
> > }
> >
> >
>
> ��.n��������+%������w��{.n�����{��(��)�� jg��� �����ݢj����G���� ���j:+v���w�m������w�������h�����٥
