* [PATCH] drivers: iio: Add more data field for iio driver hysteresis parsing @ 2019-04-09 1:31 hongyan.song 2019-04-14 12:36 ` Jonathan Cameron 0 siblings, 1 reply; 5+ messages in thread From: hongyan.song @ 2019-04-09 1:31 UTC (permalink / raw) To: jikos, jic23, linux-iio, linux-input; +Cc: srinivas.pandruvada, hongyan.song From: Song Hongyan <hongyan.song@intel.com> Add more data field support for als, incli_3d, rotation, press hysteresis parsing. In some implementation may uses them as sensitivity modifier, without this change hysteresis may not be able to read and write. Signed-off-by: Song Hongyan <hongyan.song@intel.com> --- drivers/iio/light/hid-sensor-als.c | 8 ++++++++ drivers/iio/orientation/hid-sensor-incl-3d.c | 8 ++++++++ drivers/iio/orientation/hid-sensor-rotation.c | 7 +++++++ drivers/iio/pressure/hid-sensor-press.c | 8 ++++++++ include/linux/hid-sensor-ids.h | 1 + 5 files changed, 32 insertions(+) diff --git a/drivers/iio/light/hid-sensor-als.c b/drivers/iio/light/hid-sensor-als.c index 94f3325..bb19178 100644 --- a/drivers/iio/light/hid-sensor-als.c +++ b/drivers/iio/light/hid-sensor-als.c @@ -271,6 +271,14 @@ static int als_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_SENSITIVITY_REL_PCT | + HID_USAGE_SENSOR_LIGHT_ILLUM, + &st->common_attributes.sensitivity); + } + return ret; } diff --git a/drivers/iio/orientation/hid-sensor-incl-3d.c b/drivers/iio/orientation/hid-sensor-incl-3d.c index bdc5e45..7da1e3f 100644 --- a/drivers/iio/orientation/hid-sensor-incl-3d.c +++ b/drivers/iio/orientation/hid-sensor-incl-3d.c @@ -305,6 +305,14 @@ static int incl_3d_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_SENSITIVITY_ABS | + HID_USAGE_SENSOR_ORIENT_TILT, + &st->common_attributes.sensitivity); + } + return ret; } diff --git a/drivers/iio/orientation/hid-sensor-rotation.c b/drivers/iio/orientation/hid-sensor-rotation.c index a69db20..201e320 100644 --- a/drivers/iio/orientation/hid-sensor-rotation.c +++ b/drivers/iio/orientation/hid-sensor-rotation.c @@ -229,6 +229,13 @@ static int dev_rot_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_SENSITIVITY_ABS | + HID_USAGE_SENSOR_ORIENT_QUATERNION, + &st->common_attributes.sensitivity); + } return 0; } diff --git a/drivers/iio/pressure/hid-sensor-press.c b/drivers/iio/pressure/hid-sensor-press.c index d7b1c00..bf5d2ac 100644 --- a/drivers/iio/pressure/hid-sensor-press.c +++ b/drivers/iio/pressure/hid-sensor-press.c @@ -250,6 +250,14 @@ static int press_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_SENSITIVITY_ABS | + HID_USAGE_SENSOR_ATMOSPHERIC_PRESSURE, + &st->common_attributes.sensitivity); + } + return ret; } diff --git a/include/linux/hid-sensor-ids.h b/include/linux/hid-sensor-ids.h index 76033e0..a875b9be 100644 --- a/include/linux/hid-sensor-ids.h +++ b/include/linux/hid-sensor-ids.h @@ -158,6 +158,7 @@ /* Per data field properties */ #define HID_USAGE_SENSOR_DATA_MOD_NONE 0x00 #define HID_USAGE_SENSOR_DATA_MOD_CHANGE_SENSITIVITY_ABS 0x1000 +#define HID_USAGE_SENSOR_DATA_MOD_CHANGE_SENSITIVITY_REL_PCT 0xE000 /* Power state enumerations */ #define HID_USAGE_SENSOR_PROP_POWER_STATE_UNDEFINED_ENUM 0x200850 -- 2.7.4 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] drivers: iio: Add more data field for iio driver hysteresis parsing 2019-04-09 1:31 [PATCH] drivers: iio: Add more data field for iio driver hysteresis parsing hongyan.song @ 2019-04-14 12:36 ` Jonathan Cameron 2019-04-16 1:29 ` Pandruvada, Srinivas 0 siblings, 1 reply; 5+ messages in thread From: Jonathan Cameron @ 2019-04-14 12:36 UTC (permalink / raw) To: hongyan.song; +Cc: jikos, linux-iio, linux-input, srinivas.pandruvada On Tue, 9 Apr 2019 09:31:04 +0800 hongyan.song@intel.com wrote: > From: Song Hongyan <hongyan.song@intel.com> > > Add more data field support for als, incli_3d, rotation, press > hysteresis parsing. In some implementation may uses them as sensitivity > modifier, without this change hysteresis may not be able to read and > write. > > Signed-off-by: Song Hongyan <hongyan.song@intel.com> Thanks for the patch. I'll be honest I'm lost in these descriptors sometimes! Srinivas, could you give some input on this one? I will note however, that from a quick look we tend to air these queries with dev_dbg entries. We should be consistent with these new ones as well. Thanks, Jonathan > --- > drivers/iio/light/hid-sensor-als.c | 8 ++++++++ > drivers/iio/orientation/hid-sensor-incl-3d.c | 8 ++++++++ > drivers/iio/orientation/hid-sensor-rotation.c | 7 +++++++ > drivers/iio/pressure/hid-sensor-press.c | 8 ++++++++ > include/linux/hid-sensor-ids.h | 1 + > 5 files changed, 32 insertions(+) > > diff --git a/drivers/iio/light/hid-sensor-als.c b/drivers/iio/light/hid-sensor-als.c > index 94f3325..bb19178 100644 > --- a/drivers/iio/light/hid-sensor-als.c > +++ b/drivers/iio/light/hid-sensor-als.c > @@ -271,6 +271,14 @@ static int als_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_SENSITIVITY_REL_PCT | > + HID_USAGE_SENSOR_LIGHT_ILLUM, > + &st->common_attributes.sensitivity); > + } > + > return ret; > } > > diff --git a/drivers/iio/orientation/hid-sensor-incl-3d.c b/drivers/iio/orientation/hid-sensor-incl-3d.c > index bdc5e45..7da1e3f 100644 > --- a/drivers/iio/orientation/hid-sensor-incl-3d.c > +++ b/drivers/iio/orientation/hid-sensor-incl-3d.c > @@ -305,6 +305,14 @@ static int incl_3d_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_SENSITIVITY_ABS | > + HID_USAGE_SENSOR_ORIENT_TILT, > + &st->common_attributes.sensitivity); > + } > + > return ret; > } > > diff --git a/drivers/iio/orientation/hid-sensor-rotation.c b/drivers/iio/orientation/hid-sensor-rotation.c > index a69db20..201e320 100644 > --- a/drivers/iio/orientation/hid-sensor-rotation.c > +++ b/drivers/iio/orientation/hid-sensor-rotation.c > @@ -229,6 +229,13 @@ static int dev_rot_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_SENSITIVITY_ABS | > + HID_USAGE_SENSOR_ORIENT_QUATERNION, > + &st->common_attributes.sensitivity); > + } > > return 0; > } > diff --git a/drivers/iio/pressure/hid-sensor-press.c b/drivers/iio/pressure/hid-sensor-press.c > index d7b1c00..bf5d2ac 100644 > --- a/drivers/iio/pressure/hid-sensor-press.c > +++ b/drivers/iio/pressure/hid-sensor-press.c > @@ -250,6 +250,14 @@ static int press_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_SENSITIVITY_ABS | > + HID_USAGE_SENSOR_ATMOSPHERIC_PRESSURE, > + &st->common_attributes.sensitivity); > + } > + > return ret; > } > > diff --git a/include/linux/hid-sensor-ids.h b/include/linux/hid-sensor-ids.h > index 76033e0..a875b9be 100644 > --- a/include/linux/hid-sensor-ids.h > +++ b/include/linux/hid-sensor-ids.h > @@ -158,6 +158,7 @@ > /* Per data field properties */ > #define HID_USAGE_SENSOR_DATA_MOD_NONE 0x00 > #define HID_USAGE_SENSOR_DATA_MOD_CHANGE_SENSITIVITY_ABS 0x1000 > +#define HID_USAGE_SENSOR_DATA_MOD_CHANGE_SENSITIVITY_REL_PCT 0xE000 > > /* Power state enumerations */ > #define HID_USAGE_SENSOR_PROP_POWER_STATE_UNDEFINED_ENUM 0x200850 ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] drivers: iio: Add more data field for iio driver hysteresis parsing 2019-04-14 12:36 ` Jonathan Cameron @ 2019-04-16 1:29 ` Pandruvada, Srinivas 2019-04-22 8:51 ` Jonathan Cameron 0 siblings, 1 reply; 5+ messages in thread From: Pandruvada, Srinivas @ 2019-04-16 1:29 UTC (permalink / raw) To: Song, Hongyan, jic23; +Cc: jikos, linux-input, linux-iio [-- Attachment #1: Type: text/plain, Size: 5635 bytes --] On Sun, 2019-04-14 at 13:36 +0100, Jonathan Cameron wrote: > On Tue, 9 Apr 2019 09:31:04 +0800 > hongyan.song@intel.com wrote: > > > From: Song Hongyan <hongyan.song@intel.com> > > > > Add more data field support for als, incli_3d, rotation, press > > hysteresis parsing. In some implementation may uses them as > > sensitivity > > modifier, without this change hysteresis may not be able to read > > and > > write. > > > > Signed-off-by: Song Hongyan <hongyan.song@intel.com> > > Thanks for the patch. I'll be honest I'm lost in these descriptors > sometimes! > > Srinivas, could you give some input on this one? I don't see we have defined hysteresis attribute in "Documentation/ABI/testing/sysfs-bus-iio" for non events. The current hysteresis is taken a absolute raw value in relation to the other axes in the same unit. But it is better if we add different hysteresis attributes for non absolute hysteresis. For example in_xxx_hysteresis_pct_range for "Modifier: Change Sensitivity Percent of Range" usage id: 310 in_xxx_hysteresis_pct_relative for "Modifier: Change Sensitivity Percent Relative" usage id: 311 Thanks, Srinivas > > I will note however, that from a quick look we tend to air these > queries with dev_dbg entries. We should be consistent with these new > ones > as well.attaributes > > Thanks, > > Jonathan > > --- > > drivers/iio/light/hid-sensor-als.c | 8 ++++++++ > > drivers/iio/orientation/hid-sensor-incl-3d.c | 8 ++++++++ > > drivers/iio/orientation/hid-sensor-rotation.c | 7 +++++++ > > drivers/iio/pressure/hid-sensor-press.c | 8 ++++++++ > > include/linux/hid-sensor-ids.h | 1 + > > 5 files changed, 32 insertions(+)attaributes > > > > diff --git a/drivers/iio/light/hid-sensor-als.c > > b/drivers/iio/light/hid-sensor-als.c > > index 94f3325..bb19178 100644 > > --- a/drivers/iio/light/hid-sensor-als.c > > +++ b/drivers/iio/light/hid-sensor-als.c > > @@ -271,6 +271,14 @@ static int als_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_SENSITIVITY_RE > > L_PCT | > > + HID_USAGE_SENSOR_LIGHT_ILLUM, > > + &st->common_attributes.sensitivity); > > + } > > + > > return ret; > > } > > > > diff --git a/drivers/iio/orientation/hid-sensor-incl-3d.c > > b/drivers/iio/orientation/hid-sensor-incl-3d.c > > index bdc5e45..7da1e3f 100644 > > --- a/drivers/iio/orientation/hid-sensor-incl-3d.c > > +++ b/drivers/iio/orientation/hid-sensor-incl-3d.c > > @@ -305,6 +305,14 @@ static int incl_3d_parse_report(struct > > platform_device *pdev, > > st->commpcton_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_SENSITIVITY_AB > > S | > > + HID_USAGE_SENSOR_ORIENT_TILT, > > + &st->common_attributes.sensitivity); > > + } > > + > > return ret; > > } > > > > diff --git a/drivers/iio/orientation/hid-sensor-rotation.c > > b/drivers/iio/orientation/hid-sensor-rotation.c > > index a69db20..201e320 100644 > > --- a/drivers/iio/orientation/hid-sensor-rotation.c > > +++ b/drivers/iio/orientation/hid-sensor-rotation.c > > @@ -229,6 +229,13 @@ static int dev_rot_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_SENSITIVITY_AB > > S | > > + HID_USAGE_SENSOR_ORIENT_QUATERNION, > > + &st->common_attributes.sensitivity); > > + } > > > > return 0; > > } > > diff --git a/drivers/iio/pressure/hid-sensor-press.c > > b/drivers/iio/pressure/hid-sensor-press.c > > index d7b1c00..bf5d2ac 10064pct4 > > --- a/drivers/iio/pressure/hid-sensor-press.c > > +++ b/drivers/iio/pressure/hid-sensor-press.c > > @@ -250,6 +250,14 @@ static int press_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_SENSITIVITY_AB > > S | > > + HID_USAGE_SENSOR_ATMOSPHERIC_PRESSURE, > > + &st->common_attributes.sensitivity); > > + } > > + > > return ret; > > } > > > > diff --git a/include/linux/hid-sensor-ids.h b/include/linux/hid- > > sensor-ids.h > > index 76033e0..a875b9be 100644 > > --- a/include/linux/hid-sensor-ids.h > > +++ b/include/linux/hid-sensor-ids.h > > @@ -158,6 +158,7 @@ > > /* Per data field properties */ > > #define HID_USAGE_SENSOR_DATA_MOD_NONE > > 0x00 > > #define HID_USAGE_SENSOR_DATA_MOD_CHANGE_SENSITIVITY_ABS > > 0x1000 > > +#define > > HID_USAGE_SENSOR_DATA_MOD_CHANGE_SENSITIVITY_REL_PCT 0xE > > 000 > > > > /* Power state enumerations */ > > #define HID_USAGE_SENSOR_PROP_POWER_STATE_UNDEFINED_ENUM 0x20085 > > 0 > > [-- Attachment #2: smime.p7s --] [-- Type: application/x-pkcs7-signature, Size: 3290 bytes --] ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] drivers: iio: Add more data field for iio driver hysteresis parsing 2019-04-16 1:29 ` Pandruvada, Srinivas @ 2019-04-22 8:51 ` Jonathan Cameron 2019-04-22 14:21 ` Pandruvada, Srinivas 0 siblings, 1 reply; 5+ messages in thread From: Jonathan Cameron @ 2019-04-22 8:51 UTC (permalink / raw) To: Pandruvada, Srinivas; +Cc: Song, Hongyan, jikos, linux-input, linux-iio On Tue, 16 Apr 2019 01:29:28 +0000 "Pandruvada, Srinivas" <srinivas.pandruvada@intel.com> wrote: > On Sun, 2019-04-14 at 13:36 +0100, Jonathan Cameron wrote: > > On Tue, 9 Apr 2019 09:31:04 +0800 > > hongyan.song@intel.com wrote: > > > > > From: Song Hongyan <hongyan.song@intel.com> > > > > > > Add more data field support for als, incli_3d, rotation, press > > > hysteresis parsing. In some implementation may uses them as > > > sensitivity > > > modifier, without this change hysteresis may not be able to read > > > and > > > write. > > > > > > Signed-off-by: Song Hongyan <hongyan.song@intel.com> > > > > Thanks for the patch. I'll be honest I'm lost in these descriptors > > sometimes! > > > > Srinivas, could you give some input on this one? > I don't see we have defined hysteresis attribute in > "Documentation/ABI/testing/sysfs-bus-iio" for non events. > The current hysteresis is taken a absolute raw value in relation to the > other axes in the same unit. > > But it is better if we add different hysteresis attributes for non > absolute hysteresis. > > For example > in_xxx_hysteresis_pct_range for "Modifier: Change Sensitivity Percent > of Range" usage id: 310 > in_xxx_hysteresis_pct_relative for "Modifier: Change Sensitivity > Percent Relative" usage id: 311 Hmm. This is indeed a corner we don't often hit. Basically we are looking at a 'dead zone'. For most IIO devices we don't do this as it is removing information some usecases might want. If people want to do this on sensors, it's normally done using a motion detect type event and then reading the sensor value to find out what the motion was. I'm not sure hysteresis is the right term to use here though. We aren't dealing with offsets in a threshold depending on direction, but rather at minimum change before reporting from the previous value (in either direction). Mind you I'm not quite sure what the right term would be! It may be that sensitivity is the term to use. In some ways this is very like reducing the bit depth of the sensor, which is what that term brings to mind for me. Thanks, Jonathan > > > Thanks, > Srinivas > > > > > > > I will note however, that from a quick look we tend to air these > > queries with dev_dbg entries. We should be consistent with these new > > ones > > as well.attaributes > > > > Thanks, > > > > Jonathan > > > --- > > > drivers/iio/light/hid-sensor-als.c | 8 ++++++++ > > > drivers/iio/orientation/hid-sensor-incl-3d.c | 8 ++++++++ > > > drivers/iio/orientation/hid-sensor-rotation.c | 7 +++++++ > > > drivers/iio/pressure/hid-sensor-press.c | 8 ++++++++ > > > include/linux/hid-sensor-ids.h | 1 + > > > 5 files changed, 32 insertions(+)attaributes > > > > > > diff --git a/drivers/iio/light/hid-sensor-als.c > > > b/drivers/iio/light/hid-sensor-als.c > > > index 94f3325..bb19178 100644 > > > --- a/drivers/iio/light/hid-sensor-als.c > > > +++ b/drivers/iio/light/hid-sensor-als.c > > > @@ -271,6 +271,14 @@ static int als_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_SENSITIVITY_RE > > > L_PCT | > > > + HID_USAGE_SENSOR_LIGHT_ILLUM, > > > + &st->common_attributes.sensitivity); > > > + } > > > + > > > return ret; > > > } > > > > > > diff --git a/drivers/iio/orientation/hid-sensor-incl-3d.c > > > b/drivers/iio/orientation/hid-sensor-incl-3d.c > > > index bdc5e45..7da1e3f 100644 > > > --- a/drivers/iio/orientation/hid-sensor-incl-3d.c > > > +++ b/drivers/iio/orientation/hid-sensor-incl-3d.c > > > @@ -305,6 +305,14 @@ static int incl_3d_parse_report(struct > > > platform_device *pdev, > > > st->commpcton_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_SENSITIVITY_AB > > > S | > > > + HID_USAGE_SENSOR_ORIENT_TILT, > > > + &st->common_attributes.sensitivity); > > > + } > > > + > > > return ret; > > > } > > > > > > diff --git a/drivers/iio/orientation/hid-sensor-rotation.c > > > b/drivers/iio/orientation/hid-sensor-rotation.c > > > index a69db20..201e320 100644 > > > --- a/drivers/iio/orientation/hid-sensor-rotation.c > > > +++ b/drivers/iio/orientation/hid-sensor-rotation.c > > > @@ -229,6 +229,13 @@ static int dev_rot_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_SENSITIVITY_AB > > > S | > > > + HID_USAGE_SENSOR_ORIENT_QUATERNION, > > > + &st->common_attributes.sensitivity); > > > + } > > > > > > return 0; > > > } > > > diff --git a/drivers/iio/pressure/hid-sensor-press.c > > > b/drivers/iio/pressure/hid-sensor-press.c > > > index d7b1c00..bf5d2ac 10064pct4 > > > --- a/drivers/iio/pressure/hid-sensor-press.c > > > +++ b/drivers/iio/pressure/hid-sensor-press.c > > > @@ -250,6 +250,14 @@ static int press_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_SENSITIVITY_AB > > > S | > > > + HID_USAGE_SENSOR_ATMOSPHERIC_PRESSURE, > > > + &st->common_attributes.sensitivity); > > > + } > > > + > > > return ret; > > > } > > > > > > diff --git a/include/linux/hid-sensor-ids.h b/include/linux/hid- > > > sensor-ids.h > > > index 76033e0..a875b9be 100644 > > > --- a/include/linux/hid-sensor-ids.h > > > +++ b/include/linux/hid-sensor-ids.h > > > @@ -158,6 +158,7 @@ > > > /* Per data field properties */ > > > #define HID_USAGE_SENSOR_DATA_MOD_NONE > > > 0x00 > > > #define HID_USAGE_SENSOR_DATA_MOD_CHANGE_SENSITIVITY_ABS > > > 0x1000 > > > +#define > > > HID_USAGE_SENSOR_DATA_MOD_CHANGE_SENSITIVITY_REL_PCT 0xE > > > 000 > > > > > > /* Power state enumerations */ > > > #define HID_USAGE_SENSOR_PROP_POWER_STATE_UNDEFINED_ENUM 0x20085 > > > 0 > > > > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] drivers: iio: Add more data field for iio driver hysteresis parsing 2019-04-22 8:51 ` Jonathan Cameron @ 2019-04-22 14:21 ` Pandruvada, Srinivas 0 siblings, 0 replies; 5+ messages in thread From: Pandruvada, Srinivas @ 2019-04-22 14:21 UTC (permalink / raw) To: jic23; +Cc: jikos, linux-input, Song, Hongyan, linux-iio [-- Attachment #1: Type: text/plain, Size: 7669 bytes --] On Mon, 2019-04-22 at 09:51 +0100, Jonathan Cameron wrote: > On Tue, 16 Apr 2019 01:29:28 +0000 > "Pandruvada, Srinivas" <srinivas.pandruvada@intel.com> wrote: > > > On Sun, 2019-04-14 at 13:36 +0100, Jonathan Cameron wrote: > > > On Tue, 9 Apr 2019 09:31:04 +0800 > > > hongyan.song@intel.com wrote: > > > > > > > From: Song Hongyan <hongyan.song@intel.com> > > > > > > > > Add more data field support for als, incli_3d, rotation, press > > > > hysteresis parsing. In some implementation may uses them as > > > > sensitivity > > > > modifier, without this change hysteresis may not be able to > > > > read > > > > and > > > > write. > > > > > > > > Signed-off-by: Song Hongyan <hongyan.song@intel.com> > > > > > > Thanks for the patch. I'll be honest I'm lost in these > > > descriptors > > > sometimes! > > > > > > Srinivas, could you give some input on this one? > > > > I don't see we have defined hysteresis attribute in > > "Documentation/ABI/testing/sysfs-bus-iio" for non events. > > The current hysteresis is taken a absolute raw value in relation to > > the > > other axes in the same unit. > > > > But it is better if we add different hysteresis attributes for non > > absolute hysteresis. > > > > For example > > in_xxx_hysteresis_pct_range for "Modifier: Change Sensitivity > > Percent > > of Range" usage id: 310 > > in_xxx_hysteresis_pct_relative for "Modifier: Change Sensitivity > > Percent Relative" usage id: 311 > > Hmm. This is indeed a corner we don't often hit. Basically we are > looking > at a 'dead zone'. For most IIO devices we don't do this as it is > removing information some usecases might want. If people want to > do this on sensors, it's normally done using a motion detect type > event and then reading the sensor value to find out what the motion > was. > > I'm not sure hysteresis is the right term to use here though. > We aren't dealing with offsets in a threshold depending on direction, > but rather at minimum change before reporting from the previous value > (in either direction). Mind you I'm not quite sure what the right > term would be! It may be that sensitivity is the term to use. In > some ways this is very like reducing the bit depth of the sensor, > which is what that term brings to mind for me. We can use in_xxx_sensitivity_pct_range in_xxx_sensitivity_pct_rel in_xxx_sensitivity_abs Thanks, Srinivas > > Thanks, > > Jonathan > > > > > > > Thanks, > > Srinivas > > > > > > > > > > > > I will note however, that from a quick look we tend to air these > > > queries with dev_dbg entries. We should be consistent with these > > > new > > > ones > > > as well.attaributes > > > > > > Thanks, > > > > > > Jonathan > > > > --- > > > > drivers/iio/light/hid-sensor-als.c | 8 ++++++++ > > > > drivers/iio/orientation/hid-sensor-incl-3d.c | 8 ++++++++ > > > > drivers/iio/orientation/hid-sensor-rotation.c | 7 +++++++ > > > > drivers/iio/pressure/hid-sensor-press.c | 8 ++++++++ > > > > include/linux/hid-sensor-ids.h | 1 + > > > > 5 files changed, 32 insertions(+)attaributes > > > > > > > > diff --git a/drivers/iio/light/hid-sensor-als.c > > > > b/drivers/iio/light/hid-sensor-als.c > > > > index 94f3325..bb19178 100644 > > > > --- a/drivers/iio/light/hid-sensor-als.c > > > > +++ b/drivers/iio/light/hid-sensor-als.c > > > > @@ -271,6 +271,14 @@ static int als_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_SENSIT > > > > IVITY_RE > > > > L_PCT | > > > > + HID_USAGE_SENSOR_LIGHT_ILLUM, > > > > + &st->common_attributes.sensitivity); > > > > + } > > > > + > > > > return ret; > > > > } > > > > > > > > diff --git a/drivers/iio/orientation/hid-sensor-incl-3d.c > > > > b/drivers/iio/orientation/hid-sensor-incl-3d.c > > > > index bdc5e45..7da1e3f 100644 > > > > --- a/drivers/iio/orientation/hid-sensor-incl-3d.c > > > > +++ b/drivers/iio/orientation/hid-sensor-incl-3d.c > > > > @@ -305,6 +305,14 @@ static int incl_3d_parse_report(struct > > > > platform_device *pdev, > > > > st- > > > > >commpcton_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_SENSIT > > > > IVITY_AB > > > > S | > > > > + HID_USAGE_SENSOR_ORIENT_TILT, > > > > + &st->common_attributes.sensitivity); > > > > + } > > > > + > > > > return ret; > > > > } > > > > > > > > diff --git a/drivers/iio/orientation/hid-sensor-rotation.c > > > > b/drivers/iio/orientation/hid-sensor-rotation.c > > > > index a69db20..201e320 100644 > > > > --- a/drivers/iio/orientation/hid-sensor-rotation.c > > > > +++ b/drivers/iio/orientation/hid-sensor-rotation.c > > > > @@ -229,6 +229,13 @@ static int dev_rot_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_SENSIT > > > > IVITY_AB > > > > S | > > > > + HID_USAGE_SENSOR_ORIENT_QUATERNION, > > > > + &st->common_attributes.sensitivity); > > > > + } > > > > > > > > return 0; > > > > } > > > > diff --git a/drivers/iio/pressure/hid-sensor-press.c > > > > b/drivers/iio/pressure/hid-sensor-press.c > > > > index d7b1c00..bf5d2ac 10064pct4 > > > > --- a/drivers/iio/pressure/hid-sensor-press.c > > > > +++ b/drivers/iio/pressure/hid-sensor-press.c > > > > @@ -250,6 +250,14 @@ static int press_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_SENSIT > > > > IVITY_AB > > > > S | > > > > + HID_USAGE_SENSOR_ATMOSPHERIC_PRESSURE, > > > > + &st->common_attributes.sensitivity); > > > > + } > > > > + > > > > return ret; > > > > } > > > > > > > > diff --git a/include/linux/hid-sensor-ids.h > > > > b/include/linux/hid- > > > > sensor-ids.h > > > > index 76033e0..a875b9be 100644 > > > > --- a/include/linux/hid-sensor-ids.h > > > > +++ b/include/linux/hid-sensor-ids.h > > > > @@ -158,6 +158,7 @@ > > > > /* Per data field properties */ > > > > #define HID_USAGE_SENSOR_DATA_MOD_NONE > > > > 0x00 > > > > #define HID_USAGE_SENSOR_DATA_MOD_CHANGE_SENSITIVITY_ABS > > > > > > > > 0x1000 > > > > +#define > > > > HID_USAGE_SENSOR_DATA_MOD_CHANGE_SENSITIVITY_REL_PCT > > > > 0xE > > > > 000 > > > > > > > > /* Power state enumerations */ > > > > #define HID_USAGE_SENSOR_PROP_POWER_STATE_UNDEFINED_ENUM > > > > 0x20085 > > > > 0 > > > > > > > > [-- Attachment #2: smime.p7s --] [-- Type: application/x-pkcs7-signature, Size: 3290 bytes --] ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-04-22 14:21 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-04-09 1:31 [PATCH] drivers: iio: Add more data field for iio driver hysteresis parsing hongyan.song 2019-04-14 12:36 ` Jonathan Cameron 2019-04-16 1:29 ` Pandruvada, Srinivas 2019-04-22 8:51 ` Jonathan Cameron 2019-04-22 14:21 ` Pandruvada, Srinivas
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.