diff mbox series

[1/3] iio: Add relative sensitivity support

Message ID 20210201074358.18435-2-xiang.ye@intel.com
State Superseded
Headers show
Series Add relative hysteresis support for hid sensors | expand

Commit Message

Ye Xiang Feb. 1, 2021, 7:43 a.m. UTC 
Some hid sensors may use relative sensitivity such as als sensor.
This patch adds relative sensitivity checking for all hid sensors.

Signed-off-by: Ye Xiang <xiang.ye@intel.com>
---
 .../hid-sensors/hid-sensor-attributes.c       | 75 ++++++++++++++++++-
 drivers/iio/industrialio-core.c               |  1 +
 include/linux/hid-sensor-hub.h                |  5 ++
 include/linux/hid-sensor-ids.h                |  1 +
 include/linux/iio/types.h                     |  1 +
 5 files changed, 79 insertions(+), 4 deletions(-)

Comments

Jonathan Cameron Feb. 6, 2021, 4:51 p.m. UTC | #1 
On Mon,  1 Feb 2021 15:43:56 +0800
Ye Xiang <xiang.ye@intel.com> wrote:

> Some hid sensors may use relative sensitivity such as als sensor.

> This patch adds relative sensitivity checking for all hid sensors.

> 

> Signed-off-by: Ye Xiang <xiang.ye@intel.com>


One trivial formatting thing inline + ideally I'd like an ack from Jiri for the
HID related header changes.

Otherwise look good to me!

Jonathan

> ---

>  .../hid-sensors/hid-sensor-attributes.c       | 75 ++++++++++++++++++-

>  drivers/iio/industrialio-core.c               |  1 +

>  include/linux/hid-sensor-hub.h                |  5 ++

>  include/linux/hid-sensor-ids.h                |  1 +

>  include/linux/iio/types.h                     |  1 +

>  5 files changed, 79 insertions(+), 4 deletions(-)

> 

> diff --git a/drivers/iio/common/hid-sensors/hid-sensor-attributes.c b/drivers/iio/common/hid-sensors/hid-sensor-attributes.c

> index d349ace2e33f..7a555262ff9a 100644

> --- a/drivers/iio/common/hid-sensors/hid-sensor-attributes.c

> +++ b/drivers/iio/common/hid-sensors/hid-sensor-attributes.c

> @@ -263,6 +263,30 @@ int hid_sensor_read_raw_hyst_value(struct hid_sensor_common *st,

>  }

>  EXPORT_SYMBOL(hid_sensor_read_raw_hyst_value);

>  

> +int hid_sensor_read_raw_hyst_rel_value(struct hid_sensor_common *st, int *val1,

> +				       int *val2)

> +{

> +	s32 value;

> +	int ret;

> +

> +	ret = sensor_hub_get_feature(st->hsdev,

> +				     st->sensitivity_rel.report_id,

> +				     st->sensitivity_rel.index, sizeof(value),

> +				     &value);

> +	if (ret < 0 || value < 0) {

> +		*val1 = *val2 = 0;

> +		return -EINVAL;

> +	} else {


This isn't really an 'else' path given we've returned for the error path
above.

hence I'd just drop the indentation of this.  That is

	if (ret < 0 || value < 0) {
...
	}

	convert_from_vtf...


> +		convert_from_vtf_format(value, st->sensitivity_rel.size,

> +					st->sensitivity_rel.unit_expo,

> +					val1, val2);

> +	}

> +

> +	return IIO_VAL_INT_PLUS_MICRO;

> +}

> +EXPORT_SYMBOL(hid_sensor_read_raw_hyst_rel_value);

> +

> +

>  int hid_sensor_write_raw_hyst_value(struct hid_sensor_common *st,

>  					int val1, int val2)

>  {

> @@ -294,6 +318,37 @@ int hid_sensor_write_raw_hyst_value(struct hid_sensor_common *st,

>  }

>  EXPORT_SYMBOL(hid_sensor_write_raw_hyst_value);

>  

> +int hid_sensor_write_raw_hyst_rel_value(struct hid_sensor_common *st,

> +					int val1, int val2)

> +{

> +	s32 value;

> +	int ret;

> +

> +	if (val1 < 0 || val2 < 0)

> +		return -EINVAL;

> +

> +	value = convert_to_vtf_format(st->sensitivity_rel.size,

> +				st->sensitivity_rel.unit_expo,

> +				val1, val2);

> +	ret = sensor_hub_set_feature(st->hsdev, st->sensitivity_rel.report_id,

> +				     st->sensitivity_rel.index, sizeof(value),

> +				     &value);

> +	if (ret < 0 || value < 0)

> +		return -EINVAL;

> +

> +	ret = sensor_hub_get_feature(st->hsdev,

> +				     st->sensitivity_rel.report_id,

> +				     st->sensitivity_rel.index, sizeof(value),

> +				     &value);

> +	if (ret < 0 || value < 0)

> +		return -EINVAL;

> +

> +	st->raw_hystersis = value;

> +

> +	return 0;

> +}

> +EXPORT_SYMBOL(hid_sensor_write_raw_hyst_rel_value);

> +

>  /*

>   * This fuction applies the unit exponent to the scale.

>   * For example:

> @@ -478,16 +533,28 @@ int hid_sensor_parse_common_attributes(struct hid_sensor_hub_device *hsdev,

>  			HID_USAGE_SENSOR_PROP_SENSITIVITY_ABS,

>  			 &st->sensitivity);

>  

> +	sensor_hub_input_get_attribute_info(hsdev,

> +			HID_FEATURE_REPORT, usage_id,

> +			HID_USAGE_SENSOR_PROP_SENSITIVITY_REL_PCT,

> +			&st->sensitivity_rel);

>  	/*

>  	 * Set Sensitivity field ids, when there is no individual modifier, will

> -	 * check absolute sensitivity of data field

> +	 * check absolute sensitivity and relative sensitivity of data field

>  	 */

> -	for (i = 0; i < sensitivity_addresses_len && st->sensitivity.index < 0; i++) {

> -		sensor_hub_input_get_attribute_info(hsdev,

> -				HID_FEATURE_REPORT, usage_id,

> +	for (i = 0; i < sensitivity_addresses_len; i++) {

> +		if (st->sensitivity.index < 0)

> +			sensor_hub_input_get_attribute_info(

> +				hsdev, HID_FEATURE_REPORT, usage_id,

>  				HID_USAGE_SENSOR_DATA_MOD_CHANGE_SENSITIVITY_ABS |

>  					sensitivity_addresses[i],

>  				&st->sensitivity);

> +

> +		if (st->sensitivity_rel.index < 0)

> +			sensor_hub_input_get_attribute_info(

> +				hsdev, HID_FEATURE_REPORT, usage_id,

> +				HID_USAGE_SENSOR_DATA_MOD_CHANGE_SENSITIVITY_REL_PCT |

> +					sensitivity_addresses[i],

> +				&st->sensitivity_rel);

>  	}

>  

>  	st->raw_hystersis = -1;

> diff --git a/drivers/iio/industrialio-core.c b/drivers/iio/industrialio-core.c

> index 7db761afa578..3da8fcec3c16 100644

> --- a/drivers/iio/industrialio-core.c

> +++ b/drivers/iio/industrialio-core.c

> @@ -157,6 +157,7 @@ static const char * const iio_chan_info_postfix[] = {

>  	[IIO_CHAN_INFO_PHASE] = "phase",

>  	[IIO_CHAN_INFO_HARDWAREGAIN] = "hardwaregain",

>  	[IIO_CHAN_INFO_HYSTERESIS] = "hysteresis",

> +	[IIO_CHAN_INFO_HYSTERESIS_RELATIVE] = "hysteresis_relative",

>  	[IIO_CHAN_INFO_INT_TIME] = "integration_time",

>  	[IIO_CHAN_INFO_ENABLE] = "en",

>  	[IIO_CHAN_INFO_CALIBHEIGHT] = "calibheight",

> diff --git a/include/linux/hid-sensor-hub.h b/include/linux/hid-sensor-hub.h

> index 8b2599348554..5e7bc309172e 100644

> --- a/include/linux/hid-sensor-hub.h

> +++ b/include/linux/hid-sensor-hub.h

> @@ -230,6 +230,7 @@ struct hid_sensor_common {

>  	struct hid_sensor_hub_attribute_info report_state;

>  	struct hid_sensor_hub_attribute_info power_state;

>  	struct hid_sensor_hub_attribute_info sensitivity;

> +	struct hid_sensor_hub_attribute_info sensitivity_rel;

>  	struct hid_sensor_hub_attribute_info report_latency;

>  	struct work_struct work;

>  };

> @@ -252,8 +253,12 @@ int hid_sensor_parse_common_attributes(struct hid_sensor_hub_device *hsdev,

>  					u32 sensitivity_addresses_len);

>  int hid_sensor_write_raw_hyst_value(struct hid_sensor_common *st,

>  					int val1, int val2);

> +int hid_sensor_write_raw_hyst_rel_value(struct hid_sensor_common *st, int val1,

> +					int val2);

>  int hid_sensor_read_raw_hyst_value(struct hid_sensor_common *st,

>  					int *val1, int *val2);

> +int hid_sensor_read_raw_hyst_rel_value(struct hid_sensor_common *st,

> +				       int *val1, int *val2);

>  int hid_sensor_write_samp_freq_value(struct hid_sensor_common *st,

>  					int val1, int val2);

>  int hid_sensor_read_samp_freq_value(struct hid_sensor_common *st,

> diff --git a/include/linux/hid-sensor-ids.h b/include/linux/hid-sensor-ids.h

> index 3bbdbccc5805..ac631159403a 100644

> --- a/include/linux/hid-sensor-ids.h

> +++ b/include/linux/hid-sensor-ids.h

> @@ -149,6 +149,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

> diff --git a/include/linux/iio/types.h b/include/linux/iio/types.h

> index 1e3ed6f55bca..5aa7f66d4345 100644

> --- a/include/linux/iio/types.h

> +++ b/include/linux/iio/types.h

> @@ -50,6 +50,7 @@ enum iio_chan_info_enum {

>  	IIO_CHAN_INFO_PHASE,

>  	IIO_CHAN_INFO_HARDWAREGAIN,

>  	IIO_CHAN_INFO_HYSTERESIS,

> +	IIO_CHAN_INFO_HYSTERESIS_RELATIVE,

>  	IIO_CHAN_INFO_INT_TIME,

>  	IIO_CHAN_INFO_ENABLE,

>  	IIO_CHAN_INFO_CALIBHEIGHT,
diff mbox series

Patch

diff --git a/drivers/iio/common/hid-sensors/hid-sensor-attributes.c b/drivers/iio/common/hid-sensors/hid-sensor-attributes.c
index d349ace2e33f..7a555262ff9a 100644
--- a/drivers/iio/common/hid-sensors/hid-sensor-attributes.c
+++ b/drivers/iio/common/hid-sensors/hid-sensor-attributes.c
@@ -263,6 +263,30 @@  int hid_sensor_read_raw_hyst_value(struct hid_sensor_common *st,
 }
 EXPORT_SYMBOL(hid_sensor_read_raw_hyst_value);
 
+int hid_sensor_read_raw_hyst_rel_value(struct hid_sensor_common *st, int *val1,
+				       int *val2)
+{
+	s32 value;
+	int ret;
+
+	ret = sensor_hub_get_feature(st->hsdev,
+				     st->sensitivity_rel.report_id,
+				     st->sensitivity_rel.index, sizeof(value),
+				     &value);
+	if (ret < 0 || value < 0) {
+		*val1 = *val2 = 0;
+		return -EINVAL;
+	} else {
+		convert_from_vtf_format(value, st->sensitivity_rel.size,
+					st->sensitivity_rel.unit_expo,
+					val1, val2);
+	}
+
+	return IIO_VAL_INT_PLUS_MICRO;
+}
+EXPORT_SYMBOL(hid_sensor_read_raw_hyst_rel_value);
+
+
 int hid_sensor_write_raw_hyst_value(struct hid_sensor_common *st,
 					int val1, int val2)
 {
@@ -294,6 +318,37 @@  int hid_sensor_write_raw_hyst_value(struct hid_sensor_common *st,
 }
 EXPORT_SYMBOL(hid_sensor_write_raw_hyst_value);
 
+int hid_sensor_write_raw_hyst_rel_value(struct hid_sensor_common *st,
+					int val1, int val2)
+{
+	s32 value;
+	int ret;
+
+	if (val1 < 0 || val2 < 0)
+		return -EINVAL;
+
+	value = convert_to_vtf_format(st->sensitivity_rel.size,
+				st->sensitivity_rel.unit_expo,
+				val1, val2);
+	ret = sensor_hub_set_feature(st->hsdev, st->sensitivity_rel.report_id,
+				     st->sensitivity_rel.index, sizeof(value),
+				     &value);
+	if (ret < 0 || value < 0)
+		return -EINVAL;
+
+	ret = sensor_hub_get_feature(st->hsdev,
+				     st->sensitivity_rel.report_id,
+				     st->sensitivity_rel.index, sizeof(value),
+				     &value);
+	if (ret < 0 || value < 0)
+		return -EINVAL;
+
+	st->raw_hystersis = value;
+
+	return 0;
+}
+EXPORT_SYMBOL(hid_sensor_write_raw_hyst_rel_value);
+
 /*
  * This fuction applies the unit exponent to the scale.
  * For example:
@@ -478,16 +533,28 @@  int hid_sensor_parse_common_attributes(struct hid_sensor_hub_device *hsdev,
 			HID_USAGE_SENSOR_PROP_SENSITIVITY_ABS,
 			 &st->sensitivity);
 
+	sensor_hub_input_get_attribute_info(hsdev,
+			HID_FEATURE_REPORT, usage_id,
+			HID_USAGE_SENSOR_PROP_SENSITIVITY_REL_PCT,
+			&st->sensitivity_rel);
 	/*
 	 * Set Sensitivity field ids, when there is no individual modifier, will
-	 * check absolute sensitivity of data field
+	 * check absolute sensitivity and relative sensitivity of data field
 	 */
-	for (i = 0; i < sensitivity_addresses_len && st->sensitivity.index < 0; i++) {
-		sensor_hub_input_get_attribute_info(hsdev,
-				HID_FEATURE_REPORT, usage_id,
+	for (i = 0; i < sensitivity_addresses_len; i++) {
+		if (st->sensitivity.index < 0)
+			sensor_hub_input_get_attribute_info(
+				hsdev, HID_FEATURE_REPORT, usage_id,
 				HID_USAGE_SENSOR_DATA_MOD_CHANGE_SENSITIVITY_ABS |
 					sensitivity_addresses[i],
 				&st->sensitivity);
+
+		if (st->sensitivity_rel.index < 0)
+			sensor_hub_input_get_attribute_info(
+				hsdev, HID_FEATURE_REPORT, usage_id,
+				HID_USAGE_SENSOR_DATA_MOD_CHANGE_SENSITIVITY_REL_PCT |
+					sensitivity_addresses[i],
+				&st->sensitivity_rel);
 	}
 
 	st->raw_hystersis = -1;
diff --git a/drivers/iio/industrialio-core.c b/drivers/iio/industrialio-core.c
index 7db761afa578..3da8fcec3c16 100644
--- a/drivers/iio/industrialio-core.c
+++ b/drivers/iio/industrialio-core.c
@@ -157,6 +157,7 @@  static const char * const iio_chan_info_postfix[] = {
 	[IIO_CHAN_INFO_PHASE] = "phase",
 	[IIO_CHAN_INFO_HARDWAREGAIN] = "hardwaregain",
 	[IIO_CHAN_INFO_HYSTERESIS] = "hysteresis",
+	[IIO_CHAN_INFO_HYSTERESIS_RELATIVE] = "hysteresis_relative",
 	[IIO_CHAN_INFO_INT_TIME] = "integration_time",
 	[IIO_CHAN_INFO_ENABLE] = "en",
 	[IIO_CHAN_INFO_CALIBHEIGHT] = "calibheight",
diff --git a/include/linux/hid-sensor-hub.h b/include/linux/hid-sensor-hub.h
index 8b2599348554..5e7bc309172e 100644
--- a/include/linux/hid-sensor-hub.h
+++ b/include/linux/hid-sensor-hub.h
@@ -230,6 +230,7 @@  struct hid_sensor_common {
 	struct hid_sensor_hub_attribute_info report_state;
 	struct hid_sensor_hub_attribute_info power_state;
 	struct hid_sensor_hub_attribute_info sensitivity;
+	struct hid_sensor_hub_attribute_info sensitivity_rel;
 	struct hid_sensor_hub_attribute_info report_latency;
 	struct work_struct work;
 };
@@ -252,8 +253,12 @@  int hid_sensor_parse_common_attributes(struct hid_sensor_hub_device *hsdev,
 					u32 sensitivity_addresses_len);
 int hid_sensor_write_raw_hyst_value(struct hid_sensor_common *st,
 					int val1, int val2);
+int hid_sensor_write_raw_hyst_rel_value(struct hid_sensor_common *st, int val1,
+					int val2);
 int hid_sensor_read_raw_hyst_value(struct hid_sensor_common *st,
 					int *val1, int *val2);
+int hid_sensor_read_raw_hyst_rel_value(struct hid_sensor_common *st,
+				       int *val1, int *val2);
 int hid_sensor_write_samp_freq_value(struct hid_sensor_common *st,
 					int val1, int val2);
 int hid_sensor_read_samp_freq_value(struct hid_sensor_common *st,
diff --git a/include/linux/hid-sensor-ids.h b/include/linux/hid-sensor-ids.h
index 3bbdbccc5805..ac631159403a 100644
--- a/include/linux/hid-sensor-ids.h
+++ b/include/linux/hid-sensor-ids.h
@@ -149,6 +149,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
diff --git a/include/linux/iio/types.h b/include/linux/iio/types.h
index 1e3ed6f55bca..5aa7f66d4345 100644
--- a/include/linux/iio/types.h
+++ b/include/linux/iio/types.h
@@ -50,6 +50,7 @@  enum iio_chan_info_enum {
 	IIO_CHAN_INFO_PHASE,
 	IIO_CHAN_INFO_HARDWAREGAIN,
 	IIO_CHAN_INFO_HYSTERESIS,
+	IIO_CHAN_INFO_HYSTERESIS_RELATIVE,
 	IIO_CHAN_INFO_INT_TIME,
 	IIO_CHAN_INFO_ENABLE,
 	IIO_CHAN_INFO_CALIBHEIGHT,