diff mbox series

[v2,1/1] iio: hid-sensors: lighten exported symbols by moving to IIO_HID namespace

Message ID 20210614162447.5392-1-andriy.shevchenko@linux.intel.com (mailing list archive)
State Mainlined
Commit 12f13d1faead80884f41781e8792ab397812c0c7
Headers show
Series [v2,1/1] iio: hid-sensors: lighten exported symbols by moving to IIO_HID namespace | expand

Commit Message

Andy Shevchenko June 14, 2021, 4:24 p.m. UTC
A namespace for exported symbols makes clear who is a provider
and who is a consumer of the certain resources. Besides that,
it doesn't pollute the common namespace.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
v2: updated RTC HID sensor driver as well (Jonathan)
 drivers/iio/accel/hid-sensor-accel-3d.c       |  1 +
 .../hid-sensors/hid-sensor-attributes.c       | 26 +++++++++----------
 .../common/hid-sensors/hid-sensor-trigger.c   |  9 ++++---
 drivers/iio/gyro/hid-sensor-gyro-3d.c         |  1 +
 drivers/iio/humidity/hid-sensor-humidity.c    |  1 +
 drivers/iio/light/hid-sensor-als.c            |  1 +
 drivers/iio/light/hid-sensor-prox.c           |  1 +
 drivers/iio/magnetometer/hid-sensor-magn-3d.c |  1 +
 drivers/iio/orientation/hid-sensor-incl-3d.c  |  1 +
 drivers/iio/orientation/hid-sensor-rotation.c |  1 +
 .../position/hid-sensor-custom-intel-hinge.c  |  1 +
 drivers/iio/pressure/hid-sensor-press.c       |  1 +
 .../iio/temperature/hid-sensor-temperature.c  |  1 +
 drivers/rtc/rtc-hid-sensor-time.c             |  1 +
 14 files changed, 30 insertions(+), 17 deletions(-)

Comments

Alexandre Belloni June 14, 2021, 8:58 p.m. UTC | #1
On 14/06/2021 19:24:47+0300, Andy Shevchenko wrote:
> A namespace for exported symbols makes clear who is a provider
> and who is a consumer of the certain resources. Besides that,
> it doesn't pollute the common namespace.
> 
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Acked-by: Alexandre Belloni <alexandre.belloni@bootlin.com>

> ---
> v2: updated RTC HID sensor driver as well (Jonathan)
>  drivers/iio/accel/hid-sensor-accel-3d.c       |  1 +
>  .../hid-sensors/hid-sensor-attributes.c       | 26 +++++++++----------
>  .../common/hid-sensors/hid-sensor-trigger.c   |  9 ++++---
>  drivers/iio/gyro/hid-sensor-gyro-3d.c         |  1 +
>  drivers/iio/humidity/hid-sensor-humidity.c    |  1 +
>  drivers/iio/light/hid-sensor-als.c            |  1 +
>  drivers/iio/light/hid-sensor-prox.c           |  1 +
>  drivers/iio/magnetometer/hid-sensor-magn-3d.c |  1 +
>  drivers/iio/orientation/hid-sensor-incl-3d.c  |  1 +
>  drivers/iio/orientation/hid-sensor-rotation.c |  1 +
>  .../position/hid-sensor-custom-intel-hinge.c  |  1 +
>  drivers/iio/pressure/hid-sensor-press.c       |  1 +
>  .../iio/temperature/hid-sensor-temperature.c  |  1 +
>  drivers/rtc/rtc-hid-sensor-time.c             |  1 +
>  14 files changed, 30 insertions(+), 17 deletions(-)
> 
> diff --git a/drivers/iio/accel/hid-sensor-accel-3d.c b/drivers/iio/accel/hid-sensor-accel-3d.c
> index 27f47e1c251e..bcafca7b2eac 100644
> --- a/drivers/iio/accel/hid-sensor-accel-3d.c
> +++ b/drivers/iio/accel/hid-sensor-accel-3d.c
> @@ -465,3 +465,4 @@ module_platform_driver(hid_accel_3d_platform_driver);
>  MODULE_DESCRIPTION("HID Sensor Accel 3D");
>  MODULE_AUTHOR("Srinivas Pandruvada <srinivas.pandruvada@intel.com>");
>  MODULE_LICENSE("GPL");
> +MODULE_IMPORT_NS(IIO_HID);
> diff --git a/drivers/iio/common/hid-sensors/hid-sensor-attributes.c b/drivers/iio/common/hid-sensors/hid-sensor-attributes.c
> index cb52b4fd6bf7..a81a0b206af6 100644
> --- a/drivers/iio/common/hid-sensors/hid-sensor-attributes.c
> +++ b/drivers/iio/common/hid-sensors/hid-sensor-attributes.c
> @@ -176,7 +176,7 @@ s32 hid_sensor_read_poll_value(struct hid_sensor_common *st)
>  
>  	return value;
>  }
> -EXPORT_SYMBOL(hid_sensor_read_poll_value);
> +EXPORT_SYMBOL_NS(hid_sensor_read_poll_value, IIO_HID_ATTRIBUTES);
>  
>  int hid_sensor_read_samp_freq_value(struct hid_sensor_common *st,
>  				int *val1, int *val2)
> @@ -203,7 +203,7 @@ int hid_sensor_read_samp_freq_value(struct hid_sensor_common *st,
>  
>  	return IIO_VAL_INT_PLUS_MICRO;
>  }
> -EXPORT_SYMBOL(hid_sensor_read_samp_freq_value);
> +EXPORT_SYMBOL_NS(hid_sensor_read_samp_freq_value, IIO_HID);
>  
>  int hid_sensor_write_samp_freq_value(struct hid_sensor_common *st,
>  				int val1, int val2)
> @@ -238,7 +238,7 @@ int hid_sensor_write_samp_freq_value(struct hid_sensor_common *st,
>  
>  	return 0;
>  }
> -EXPORT_SYMBOL(hid_sensor_write_samp_freq_value);
> +EXPORT_SYMBOL_NS(hid_sensor_write_samp_freq_value, IIO_HID);
>  
>  int hid_sensor_read_raw_hyst_value(struct hid_sensor_common *st,
>  				int *val1, int *val2)
> @@ -261,7 +261,7 @@ int hid_sensor_read_raw_hyst_value(struct hid_sensor_common *st,
>  
>  	return IIO_VAL_INT_PLUS_MICRO;
>  }
> -EXPORT_SYMBOL(hid_sensor_read_raw_hyst_value);
> +EXPORT_SYMBOL_NS(hid_sensor_read_raw_hyst_value, IIO_HID);
>  
>  int hid_sensor_read_raw_hyst_rel_value(struct hid_sensor_common *st, int *val1,
>  				       int *val2)
> @@ -283,7 +283,7 @@ int hid_sensor_read_raw_hyst_rel_value(struct hid_sensor_common *st, int *val1,
>  
>  	return IIO_VAL_INT_PLUS_MICRO;
>  }
> -EXPORT_SYMBOL(hid_sensor_read_raw_hyst_rel_value);
> +EXPORT_SYMBOL_NS(hid_sensor_read_raw_hyst_rel_value, IIO_HID);
>  
>  
>  int hid_sensor_write_raw_hyst_value(struct hid_sensor_common *st,
> @@ -315,7 +315,7 @@ int hid_sensor_write_raw_hyst_value(struct hid_sensor_common *st,
>  
>  	return 0;
>  }
> -EXPORT_SYMBOL(hid_sensor_write_raw_hyst_value);
> +EXPORT_SYMBOL_NS(hid_sensor_write_raw_hyst_value, IIO_HID);
>  
>  int hid_sensor_write_raw_hyst_rel_value(struct hid_sensor_common *st,
>  					int val1, int val2)
> @@ -346,7 +346,7 @@ int hid_sensor_write_raw_hyst_rel_value(struct hid_sensor_common *st,
>  
>  	return 0;
>  }
> -EXPORT_SYMBOL(hid_sensor_write_raw_hyst_rel_value);
> +EXPORT_SYMBOL_NS(hid_sensor_write_raw_hyst_rel_value, IIO_HID);
>  
>  /*
>   * This fuction applies the unit exponent to the scale.
> @@ -430,14 +430,14 @@ int hid_sensor_format_scale(u32 usage_id,
>  
>  	return IIO_VAL_INT_PLUS_NANO;
>  }
> -EXPORT_SYMBOL(hid_sensor_format_scale);
> +EXPORT_SYMBOL_NS(hid_sensor_format_scale, IIO_HID);
>  
>  int64_t hid_sensor_convert_timestamp(struct hid_sensor_common *st,
>  				     int64_t raw_value)
>  {
>  	return st->timestamp_ns_scale * raw_value;
>  }
> -EXPORT_SYMBOL(hid_sensor_convert_timestamp);
> +EXPORT_SYMBOL_NS(hid_sensor_convert_timestamp, IIO_HID);
>  
>  static
>  int hid_sensor_get_reporting_interval(struct hid_sensor_hub_device *hsdev,
> @@ -484,7 +484,7 @@ int hid_sensor_get_report_latency(struct hid_sensor_common *st)
>  
>  	return value;
>  }
> -EXPORT_SYMBOL(hid_sensor_get_report_latency);
> +EXPORT_SYMBOL_NS(hid_sensor_get_report_latency, IIO_HID_ATTRIBUTES);
>  
>  int hid_sensor_set_report_latency(struct hid_sensor_common *st, int latency_ms)
>  {
> @@ -492,13 +492,13 @@ int hid_sensor_set_report_latency(struct hid_sensor_common *st, int latency_ms)
>  				      st->report_latency.index,
>  				      sizeof(latency_ms), &latency_ms);
>  }
> -EXPORT_SYMBOL(hid_sensor_set_report_latency);
> +EXPORT_SYMBOL_NS(hid_sensor_set_report_latency, IIO_HID_ATTRIBUTES);
>  
>  bool hid_sensor_batch_mode_supported(struct hid_sensor_common *st)
>  {
>  	return st->report_latency.index > 0 && st->report_latency.report_id > 0;
>  }
> -EXPORT_SYMBOL(hid_sensor_batch_mode_supported);
> +EXPORT_SYMBOL_NS(hid_sensor_batch_mode_supported, IIO_HID_ATTRIBUTES);
>  
>  int hid_sensor_parse_common_attributes(struct hid_sensor_hub_device *hsdev,
>  					u32 usage_id,
> @@ -590,7 +590,7 @@ int hid_sensor_parse_common_attributes(struct hid_sensor_hub_device *hsdev,
>  
>  	return 0;
>  }
> -EXPORT_SYMBOL(hid_sensor_parse_common_attributes);
> +EXPORT_SYMBOL_NS(hid_sensor_parse_common_attributes, IIO_HID);
>  
>  MODULE_AUTHOR("Srinivas Pandruvada <srinivas.pandruvada@intel.com>");
>  MODULE_DESCRIPTION("HID Sensor common attribute processing");
> diff --git a/drivers/iio/common/hid-sensors/hid-sensor-trigger.c b/drivers/iio/common/hid-sensors/hid-sensor-trigger.c
> index c06537e106e9..60e85d675387 100644
> --- a/drivers/iio/common/hid-sensors/hid-sensor-trigger.c
> +++ b/drivers/iio/common/hid-sensors/hid-sensor-trigger.c
> @@ -150,7 +150,7 @@ static int _hid_sensor_power_state(struct hid_sensor_common *st, bool state)
>  
>  	return 0;
>  }
> -EXPORT_SYMBOL(hid_sensor_power_state);
> +EXPORT_SYMBOL_NS(hid_sensor_power_state, IIO_HID);
>  
>  int hid_sensor_power_state(struct hid_sensor_common *st, bool state)
>  {
> @@ -225,7 +225,7 @@ void hid_sensor_remove_trigger(struct iio_dev *indio_dev,
>  	iio_trigger_free(attrb->trigger);
>  	iio_triggered_buffer_cleanup(indio_dev);
>  }
> -EXPORT_SYMBOL(hid_sensor_remove_trigger);
> +EXPORT_SYMBOL_NS(hid_sensor_remove_trigger, IIO_HID);
>  
>  static const struct iio_trigger_ops hid_sensor_trigger_ops = {
>  	.set_trigger_state = &hid_sensor_data_rdy_trigger_set_state,
> @@ -291,7 +291,7 @@ int hid_sensor_setup_trigger(struct iio_dev *indio_dev, const char *name,
>  	iio_triggered_buffer_cleanup(indio_dev);
>  	return ret;
>  }
> -EXPORT_SYMBOL(hid_sensor_setup_trigger);
> +EXPORT_SYMBOL_NS(hid_sensor_setup_trigger, IIO_HID);
>  
>  static int __maybe_unused hid_sensor_suspend(struct device *dev)
>  {
> @@ -321,8 +321,9 @@ const struct dev_pm_ops hid_sensor_pm_ops = {
>  	SET_RUNTIME_PM_OPS(hid_sensor_suspend,
>  			   hid_sensor_runtime_resume, NULL)
>  };
> -EXPORT_SYMBOL(hid_sensor_pm_ops);
> +EXPORT_SYMBOL_NS(hid_sensor_pm_ops, IIO_HID);
>  
>  MODULE_AUTHOR("Srinivas Pandruvada <srinivas.pandruvada@intel.com>");
>  MODULE_DESCRIPTION("HID Sensor trigger processing");
>  MODULE_LICENSE("GPL");
> +MODULE_IMPORT_NS(IIO_HID_ATTRIBUTES);
> diff --git a/drivers/iio/gyro/hid-sensor-gyro-3d.c b/drivers/iio/gyro/hid-sensor-gyro-3d.c
> index dad26ee4fd1f..2ed2a8effb86 100644
> --- a/drivers/iio/gyro/hid-sensor-gyro-3d.c
> +++ b/drivers/iio/gyro/hid-sensor-gyro-3d.c
> @@ -400,3 +400,4 @@ module_platform_driver(hid_gyro_3d_platform_driver);
>  MODULE_DESCRIPTION("HID Sensor Gyroscope 3D");
>  MODULE_AUTHOR("Srinivas Pandruvada <srinivas.pandruvada@intel.com>");
>  MODULE_LICENSE("GPL");
> +MODULE_IMPORT_NS(IIO_HID);
> diff --git a/drivers/iio/humidity/hid-sensor-humidity.c b/drivers/iio/humidity/hid-sensor-humidity.c
> index 74383abc0d44..8a9091d71d2a 100644
> --- a/drivers/iio/humidity/hid-sensor-humidity.c
> +++ b/drivers/iio/humidity/hid-sensor-humidity.c
> @@ -295,3 +295,4 @@ module_platform_driver(hid_humidity_platform_driver);
>  MODULE_DESCRIPTION("HID Environmental humidity sensor");
>  MODULE_AUTHOR("Song Hongyan <hongyan.song@intel.com>");
>  MODULE_LICENSE("GPL v2");
> +MODULE_IMPORT_NS(IIO_HID);
> diff --git a/drivers/iio/light/hid-sensor-als.c b/drivers/iio/light/hid-sensor-als.c
> index 85c8a05b73cb..a63d577493e2 100644
> --- a/drivers/iio/light/hid-sensor-als.c
> +++ b/drivers/iio/light/hid-sensor-als.c
> @@ -392,3 +392,4 @@ module_platform_driver(hid_als_platform_driver);
>  MODULE_DESCRIPTION("HID Sensor ALS");
>  MODULE_AUTHOR("Srinivas Pandruvada <srinivas.pandruvada@intel.com>");
>  MODULE_LICENSE("GPL");
> +MODULE_IMPORT_NS(IIO_HID);
> diff --git a/drivers/iio/light/hid-sensor-prox.c b/drivers/iio/light/hid-sensor-prox.c
> index 17d167c3d595..99de268563d6 100644
> --- a/drivers/iio/light/hid-sensor-prox.c
> +++ b/drivers/iio/light/hid-sensor-prox.c
> @@ -350,3 +350,4 @@ module_platform_driver(hid_prox_platform_driver);
>  MODULE_DESCRIPTION("HID Sensor Proximity");
>  MODULE_AUTHOR("Archana Patni <archana.patni@intel.com>");
>  MODULE_LICENSE("GPL");
> +MODULE_IMPORT_NS(IIO_HID);
> diff --git a/drivers/iio/magnetometer/hid-sensor-magn-3d.c b/drivers/iio/magnetometer/hid-sensor-magn-3d.c
> index b78691523dd4..a66f9e933628 100644
> --- a/drivers/iio/magnetometer/hid-sensor-magn-3d.c
> +++ b/drivers/iio/magnetometer/hid-sensor-magn-3d.c
> @@ -587,3 +587,4 @@ module_platform_driver(hid_magn_3d_platform_driver);
>  MODULE_DESCRIPTION("HID Sensor Magnetometer 3D");
>  MODULE_AUTHOR("Srinivas Pandruvada <srinivas.pandruvada@intel.com>");
>  MODULE_LICENSE("GPL");
> +MODULE_IMPORT_NS(IIO_HID);
> diff --git a/drivers/iio/orientation/hid-sensor-incl-3d.c b/drivers/iio/orientation/hid-sensor-incl-3d.c
> index 7af48d336285..cc905b0fa835 100644
> --- a/drivers/iio/orientation/hid-sensor-incl-3d.c
> +++ b/drivers/iio/orientation/hid-sensor-incl-3d.c
> @@ -425,3 +425,4 @@ module_platform_driver(hid_incl_3d_platform_driver);
>  MODULE_DESCRIPTION("HID Sensor Inclinometer 3D");
>  MODULE_AUTHOR("Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>");
>  MODULE_LICENSE("GPL");
> +MODULE_IMPORT_NS(IIO_HID);
> diff --git a/drivers/iio/orientation/hid-sensor-rotation.c b/drivers/iio/orientation/hid-sensor-rotation.c
> index cf7f57a47681..6570bfd22035 100644
> --- a/drivers/iio/orientation/hid-sensor-rotation.c
> +++ b/drivers/iio/orientation/hid-sensor-rotation.c
> @@ -373,3 +373,4 @@ module_platform_driver(hid_dev_rot_platform_driver);
>  MODULE_DESCRIPTION("HID Sensor Device Rotation");
>  MODULE_AUTHOR("Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>");
>  MODULE_LICENSE("GPL");
> +MODULE_IMPORT_NS(IIO_HID);
> diff --git a/drivers/iio/position/hid-sensor-custom-intel-hinge.c b/drivers/iio/position/hid-sensor-custom-intel-hinge.c
> index 738b5f4626ce..4478ad9387c5 100644
> --- a/drivers/iio/position/hid-sensor-custom-intel-hinge.c
> +++ b/drivers/iio/position/hid-sensor-custom-intel-hinge.c
> @@ -376,3 +376,4 @@ module_platform_driver(hid_hinge_platform_driver);
>  MODULE_DESCRIPTION("HID Sensor INTEL Hinge");
>  MODULE_AUTHOR("Ye Xiang <xiang.ye@intel.com>");
>  MODULE_LICENSE("GPL");
> +MODULE_IMPORT_NS(IIO_HID);
> diff --git a/drivers/iio/pressure/hid-sensor-press.c b/drivers/iio/pressure/hid-sensor-press.c
> index c416d261e3e3..79b3399e4095 100644
> --- a/drivers/iio/pressure/hid-sensor-press.c
> +++ b/drivers/iio/pressure/hid-sensor-press.c
> @@ -357,3 +357,4 @@ module_platform_driver(hid_press_platform_driver);
>  MODULE_DESCRIPTION("HID Sensor Pressure");
>  MODULE_AUTHOR("Archana Patni <archana.patni@intel.com>");
>  MODULE_LICENSE("GPL");
> +MODULE_IMPORT_NS(IIO_HID);
> diff --git a/drivers/iio/temperature/hid-sensor-temperature.c b/drivers/iio/temperature/hid-sensor-temperature.c
> index dc534ed784c3..21aa952b9f6d 100644
> --- a/drivers/iio/temperature/hid-sensor-temperature.c
> +++ b/drivers/iio/temperature/hid-sensor-temperature.c
> @@ -291,3 +291,4 @@ module_platform_driver(hid_temperature_platform_driver);
>  MODULE_DESCRIPTION("HID Environmental temperature sensor");
>  MODULE_AUTHOR("Song Hongyan <hongyan.song@intel.com>");
>  MODULE_LICENSE("GPL v2");
> +MODULE_IMPORT_NS(IIO_HID);
> diff --git a/drivers/rtc/rtc-hid-sensor-time.c b/drivers/rtc/rtc-hid-sensor-time.c
> index 47cd12db2356..16fdefafec5d 100644
> --- a/drivers/rtc/rtc-hid-sensor-time.c
> +++ b/drivers/rtc/rtc-hid-sensor-time.c
> @@ -328,3 +328,4 @@ module_platform_driver(hid_time_platform_driver);
>  MODULE_DESCRIPTION("HID Sensor Time");
>  MODULE_AUTHOR("Alexander Holler <holler@ahsoftware.de>");
>  MODULE_LICENSE("GPL");
> +MODULE_IMPORT_NS(IIO_HID);
> -- 
> 2.30.2
>
Jonathan Cameron June 16, 2021, 12:41 p.m. UTC | #2
On Mon, 14 Jun 2021 22:58:46 +0200
Alexandre Belloni <alexandre.belloni@bootlin.com> wrote:

> On 14/06/2021 19:24:47+0300, Andy Shevchenko wrote:
> > A namespace for exported symbols makes clear who is a provider
> > and who is a consumer of the certain resources. Besides that,
> > it doesn't pollute the common namespace.
> > 
> > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>  
> Acked-by: Alexandre Belloni <alexandre.belloni@bootlin.com>

Thanks,  applied to the togreg branch of iio.git and pushed out as
testing for 0-day to poke at it and see if we missed anything.

Jonathan

> 
> > ---
> > v2: updated RTC HID sensor driver as well (Jonathan)
> >  drivers/iio/accel/hid-sensor-accel-3d.c       |  1 +
> >  .../hid-sensors/hid-sensor-attributes.c       | 26 +++++++++----------
> >  .../common/hid-sensors/hid-sensor-trigger.c   |  9 ++++---
> >  drivers/iio/gyro/hid-sensor-gyro-3d.c         |  1 +
> >  drivers/iio/humidity/hid-sensor-humidity.c    |  1 +
> >  drivers/iio/light/hid-sensor-als.c            |  1 +
> >  drivers/iio/light/hid-sensor-prox.c           |  1 +
> >  drivers/iio/magnetometer/hid-sensor-magn-3d.c |  1 +
> >  drivers/iio/orientation/hid-sensor-incl-3d.c  |  1 +
> >  drivers/iio/orientation/hid-sensor-rotation.c |  1 +
> >  .../position/hid-sensor-custom-intel-hinge.c  |  1 +
> >  drivers/iio/pressure/hid-sensor-press.c       |  1 +
> >  .../iio/temperature/hid-sensor-temperature.c  |  1 +
> >  drivers/rtc/rtc-hid-sensor-time.c             |  1 +
> >  14 files changed, 30 insertions(+), 17 deletions(-)
> > 
> > diff --git a/drivers/iio/accel/hid-sensor-accel-3d.c b/drivers/iio/accel/hid-sensor-accel-3d.c
> > index 27f47e1c251e..bcafca7b2eac 100644
> > --- a/drivers/iio/accel/hid-sensor-accel-3d.c
> > +++ b/drivers/iio/accel/hid-sensor-accel-3d.c
> > @@ -465,3 +465,4 @@ module_platform_driver(hid_accel_3d_platform_driver);
> >  MODULE_DESCRIPTION("HID Sensor Accel 3D");
> >  MODULE_AUTHOR("Srinivas Pandruvada <srinivas.pandruvada@intel.com>");
> >  MODULE_LICENSE("GPL");
> > +MODULE_IMPORT_NS(IIO_HID);
> > diff --git a/drivers/iio/common/hid-sensors/hid-sensor-attributes.c b/drivers/iio/common/hid-sensors/hid-sensor-attributes.c
> > index cb52b4fd6bf7..a81a0b206af6 100644
> > --- a/drivers/iio/common/hid-sensors/hid-sensor-attributes.c
> > +++ b/drivers/iio/common/hid-sensors/hid-sensor-attributes.c
> > @@ -176,7 +176,7 @@ s32 hid_sensor_read_poll_value(struct hid_sensor_common *st)
> >  
> >  	return value;
> >  }
> > -EXPORT_SYMBOL(hid_sensor_read_poll_value);
> > +EXPORT_SYMBOL_NS(hid_sensor_read_poll_value, IIO_HID_ATTRIBUTES);
> >  
> >  int hid_sensor_read_samp_freq_value(struct hid_sensor_common *st,
> >  				int *val1, int *val2)
> > @@ -203,7 +203,7 @@ int hid_sensor_read_samp_freq_value(struct hid_sensor_common *st,
> >  
> >  	return IIO_VAL_INT_PLUS_MICRO;
> >  }
> > -EXPORT_SYMBOL(hid_sensor_read_samp_freq_value);
> > +EXPORT_SYMBOL_NS(hid_sensor_read_samp_freq_value, IIO_HID);
> >  
> >  int hid_sensor_write_samp_freq_value(struct hid_sensor_common *st,
> >  				int val1, int val2)
> > @@ -238,7 +238,7 @@ int hid_sensor_write_samp_freq_value(struct hid_sensor_common *st,
> >  
> >  	return 0;
> >  }
> > -EXPORT_SYMBOL(hid_sensor_write_samp_freq_value);
> > +EXPORT_SYMBOL_NS(hid_sensor_write_samp_freq_value, IIO_HID);
> >  
> >  int hid_sensor_read_raw_hyst_value(struct hid_sensor_common *st,
> >  				int *val1, int *val2)
> > @@ -261,7 +261,7 @@ int hid_sensor_read_raw_hyst_value(struct hid_sensor_common *st,
> >  
> >  	return IIO_VAL_INT_PLUS_MICRO;
> >  }
> > -EXPORT_SYMBOL(hid_sensor_read_raw_hyst_value);
> > +EXPORT_SYMBOL_NS(hid_sensor_read_raw_hyst_value, IIO_HID);
> >  
> >  int hid_sensor_read_raw_hyst_rel_value(struct hid_sensor_common *st, int *val1,
> >  				       int *val2)
> > @@ -283,7 +283,7 @@ int hid_sensor_read_raw_hyst_rel_value(struct hid_sensor_common *st, int *val1,
> >  
> >  	return IIO_VAL_INT_PLUS_MICRO;
> >  }
> > -EXPORT_SYMBOL(hid_sensor_read_raw_hyst_rel_value);
> > +EXPORT_SYMBOL_NS(hid_sensor_read_raw_hyst_rel_value, IIO_HID);
> >  
> >  
> >  int hid_sensor_write_raw_hyst_value(struct hid_sensor_common *st,
> > @@ -315,7 +315,7 @@ int hid_sensor_write_raw_hyst_value(struct hid_sensor_common *st,
> >  
> >  	return 0;
> >  }
> > -EXPORT_SYMBOL(hid_sensor_write_raw_hyst_value);
> > +EXPORT_SYMBOL_NS(hid_sensor_write_raw_hyst_value, IIO_HID);
> >  
> >  int hid_sensor_write_raw_hyst_rel_value(struct hid_sensor_common *st,
> >  					int val1, int val2)
> > @@ -346,7 +346,7 @@ int hid_sensor_write_raw_hyst_rel_value(struct hid_sensor_common *st,
> >  
> >  	return 0;
> >  }
> > -EXPORT_SYMBOL(hid_sensor_write_raw_hyst_rel_value);
> > +EXPORT_SYMBOL_NS(hid_sensor_write_raw_hyst_rel_value, IIO_HID);
> >  
> >  /*
> >   * This fuction applies the unit exponent to the scale.
> > @@ -430,14 +430,14 @@ int hid_sensor_format_scale(u32 usage_id,
> >  
> >  	return IIO_VAL_INT_PLUS_NANO;
> >  }
> > -EXPORT_SYMBOL(hid_sensor_format_scale);
> > +EXPORT_SYMBOL_NS(hid_sensor_format_scale, IIO_HID);
> >  
> >  int64_t hid_sensor_convert_timestamp(struct hid_sensor_common *st,
> >  				     int64_t raw_value)
> >  {
> >  	return st->timestamp_ns_scale * raw_value;
> >  }
> > -EXPORT_SYMBOL(hid_sensor_convert_timestamp);
> > +EXPORT_SYMBOL_NS(hid_sensor_convert_timestamp, IIO_HID);
> >  
> >  static
> >  int hid_sensor_get_reporting_interval(struct hid_sensor_hub_device *hsdev,
> > @@ -484,7 +484,7 @@ int hid_sensor_get_report_latency(struct hid_sensor_common *st)
> >  
> >  	return value;
> >  }
> > -EXPORT_SYMBOL(hid_sensor_get_report_latency);
> > +EXPORT_SYMBOL_NS(hid_sensor_get_report_latency, IIO_HID_ATTRIBUTES);
> >  
> >  int hid_sensor_set_report_latency(struct hid_sensor_common *st, int latency_ms)
> >  {
> > @@ -492,13 +492,13 @@ int hid_sensor_set_report_latency(struct hid_sensor_common *st, int latency_ms)
> >  				      st->report_latency.index,
> >  				      sizeof(latency_ms), &latency_ms);
> >  }
> > -EXPORT_SYMBOL(hid_sensor_set_report_latency);
> > +EXPORT_SYMBOL_NS(hid_sensor_set_report_latency, IIO_HID_ATTRIBUTES);
> >  
> >  bool hid_sensor_batch_mode_supported(struct hid_sensor_common *st)
> >  {
> >  	return st->report_latency.index > 0 && st->report_latency.report_id > 0;
> >  }
> > -EXPORT_SYMBOL(hid_sensor_batch_mode_supported);
> > +EXPORT_SYMBOL_NS(hid_sensor_batch_mode_supported, IIO_HID_ATTRIBUTES);
> >  
> >  int hid_sensor_parse_common_attributes(struct hid_sensor_hub_device *hsdev,
> >  					u32 usage_id,
> > @@ -590,7 +590,7 @@ int hid_sensor_parse_common_attributes(struct hid_sensor_hub_device *hsdev,
> >  
> >  	return 0;
> >  }
> > -EXPORT_SYMBOL(hid_sensor_parse_common_attributes);
> > +EXPORT_SYMBOL_NS(hid_sensor_parse_common_attributes, IIO_HID);
> >  
> >  MODULE_AUTHOR("Srinivas Pandruvada <srinivas.pandruvada@intel.com>");
> >  MODULE_DESCRIPTION("HID Sensor common attribute processing");
> > diff --git a/drivers/iio/common/hid-sensors/hid-sensor-trigger.c b/drivers/iio/common/hid-sensors/hid-sensor-trigger.c
> > index c06537e106e9..60e85d675387 100644
> > --- a/drivers/iio/common/hid-sensors/hid-sensor-trigger.c
> > +++ b/drivers/iio/common/hid-sensors/hid-sensor-trigger.c
> > @@ -150,7 +150,7 @@ static int _hid_sensor_power_state(struct hid_sensor_common *st, bool state)
> >  
> >  	return 0;
> >  }
> > -EXPORT_SYMBOL(hid_sensor_power_state);
> > +EXPORT_SYMBOL_NS(hid_sensor_power_state, IIO_HID);
> >  
> >  int hid_sensor_power_state(struct hid_sensor_common *st, bool state)
> >  {
> > @@ -225,7 +225,7 @@ void hid_sensor_remove_trigger(struct iio_dev *indio_dev,
> >  	iio_trigger_free(attrb->trigger);
> >  	iio_triggered_buffer_cleanup(indio_dev);
> >  }
> > -EXPORT_SYMBOL(hid_sensor_remove_trigger);
> > +EXPORT_SYMBOL_NS(hid_sensor_remove_trigger, IIO_HID);
> >  
> >  static const struct iio_trigger_ops hid_sensor_trigger_ops = {
> >  	.set_trigger_state = &hid_sensor_data_rdy_trigger_set_state,
> > @@ -291,7 +291,7 @@ int hid_sensor_setup_trigger(struct iio_dev *indio_dev, const char *name,
> >  	iio_triggered_buffer_cleanup(indio_dev);
> >  	return ret;
> >  }
> > -EXPORT_SYMBOL(hid_sensor_setup_trigger);
> > +EXPORT_SYMBOL_NS(hid_sensor_setup_trigger, IIO_HID);
> >  
> >  static int __maybe_unused hid_sensor_suspend(struct device *dev)
> >  {
> > @@ -321,8 +321,9 @@ const struct dev_pm_ops hid_sensor_pm_ops = {
> >  	SET_RUNTIME_PM_OPS(hid_sensor_suspend,
> >  			   hid_sensor_runtime_resume, NULL)
> >  };
> > -EXPORT_SYMBOL(hid_sensor_pm_ops);
> > +EXPORT_SYMBOL_NS(hid_sensor_pm_ops, IIO_HID);
> >  
> >  MODULE_AUTHOR("Srinivas Pandruvada <srinivas.pandruvada@intel.com>");
> >  MODULE_DESCRIPTION("HID Sensor trigger processing");
> >  MODULE_LICENSE("GPL");
> > +MODULE_IMPORT_NS(IIO_HID_ATTRIBUTES);
> > diff --git a/drivers/iio/gyro/hid-sensor-gyro-3d.c b/drivers/iio/gyro/hid-sensor-gyro-3d.c
> > index dad26ee4fd1f..2ed2a8effb86 100644
> > --- a/drivers/iio/gyro/hid-sensor-gyro-3d.c
> > +++ b/drivers/iio/gyro/hid-sensor-gyro-3d.c
> > @@ -400,3 +400,4 @@ module_platform_driver(hid_gyro_3d_platform_driver);
> >  MODULE_DESCRIPTION("HID Sensor Gyroscope 3D");
> >  MODULE_AUTHOR("Srinivas Pandruvada <srinivas.pandruvada@intel.com>");
> >  MODULE_LICENSE("GPL");
> > +MODULE_IMPORT_NS(IIO_HID);
> > diff --git a/drivers/iio/humidity/hid-sensor-humidity.c b/drivers/iio/humidity/hid-sensor-humidity.c
> > index 74383abc0d44..8a9091d71d2a 100644
> > --- a/drivers/iio/humidity/hid-sensor-humidity.c
> > +++ b/drivers/iio/humidity/hid-sensor-humidity.c
> > @@ -295,3 +295,4 @@ module_platform_driver(hid_humidity_platform_driver);
> >  MODULE_DESCRIPTION("HID Environmental humidity sensor");
> >  MODULE_AUTHOR("Song Hongyan <hongyan.song@intel.com>");
> >  MODULE_LICENSE("GPL v2");
> > +MODULE_IMPORT_NS(IIO_HID);
> > diff --git a/drivers/iio/light/hid-sensor-als.c b/drivers/iio/light/hid-sensor-als.c
> > index 85c8a05b73cb..a63d577493e2 100644
> > --- a/drivers/iio/light/hid-sensor-als.c
> > +++ b/drivers/iio/light/hid-sensor-als.c
> > @@ -392,3 +392,4 @@ module_platform_driver(hid_als_platform_driver);
> >  MODULE_DESCRIPTION("HID Sensor ALS");
> >  MODULE_AUTHOR("Srinivas Pandruvada <srinivas.pandruvada@intel.com>");
> >  MODULE_LICENSE("GPL");
> > +MODULE_IMPORT_NS(IIO_HID);
> > diff --git a/drivers/iio/light/hid-sensor-prox.c b/drivers/iio/light/hid-sensor-prox.c
> > index 17d167c3d595..99de268563d6 100644
> > --- a/drivers/iio/light/hid-sensor-prox.c
> > +++ b/drivers/iio/light/hid-sensor-prox.c
> > @@ -350,3 +350,4 @@ module_platform_driver(hid_prox_platform_driver);
> >  MODULE_DESCRIPTION("HID Sensor Proximity");
> >  MODULE_AUTHOR("Archana Patni <archana.patni@intel.com>");
> >  MODULE_LICENSE("GPL");
> > +MODULE_IMPORT_NS(IIO_HID);
> > diff --git a/drivers/iio/magnetometer/hid-sensor-magn-3d.c b/drivers/iio/magnetometer/hid-sensor-magn-3d.c
> > index b78691523dd4..a66f9e933628 100644
> > --- a/drivers/iio/magnetometer/hid-sensor-magn-3d.c
> > +++ b/drivers/iio/magnetometer/hid-sensor-magn-3d.c
> > @@ -587,3 +587,4 @@ module_platform_driver(hid_magn_3d_platform_driver);
> >  MODULE_DESCRIPTION("HID Sensor Magnetometer 3D");
> >  MODULE_AUTHOR("Srinivas Pandruvada <srinivas.pandruvada@intel.com>");
> >  MODULE_LICENSE("GPL");
> > +MODULE_IMPORT_NS(IIO_HID);
> > diff --git a/drivers/iio/orientation/hid-sensor-incl-3d.c b/drivers/iio/orientation/hid-sensor-incl-3d.c
> > index 7af48d336285..cc905b0fa835 100644
> > --- a/drivers/iio/orientation/hid-sensor-incl-3d.c
> > +++ b/drivers/iio/orientation/hid-sensor-incl-3d.c
> > @@ -425,3 +425,4 @@ module_platform_driver(hid_incl_3d_platform_driver);
> >  MODULE_DESCRIPTION("HID Sensor Inclinometer 3D");
> >  MODULE_AUTHOR("Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>");
> >  MODULE_LICENSE("GPL");
> > +MODULE_IMPORT_NS(IIO_HID);
> > diff --git a/drivers/iio/orientation/hid-sensor-rotation.c b/drivers/iio/orientation/hid-sensor-rotation.c
> > index cf7f57a47681..6570bfd22035 100644
> > --- a/drivers/iio/orientation/hid-sensor-rotation.c
> > +++ b/drivers/iio/orientation/hid-sensor-rotation.c
> > @@ -373,3 +373,4 @@ module_platform_driver(hid_dev_rot_platform_driver);
> >  MODULE_DESCRIPTION("HID Sensor Device Rotation");
> >  MODULE_AUTHOR("Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>");
> >  MODULE_LICENSE("GPL");
> > +MODULE_IMPORT_NS(IIO_HID);
> > diff --git a/drivers/iio/position/hid-sensor-custom-intel-hinge.c b/drivers/iio/position/hid-sensor-custom-intel-hinge.c
> > index 738b5f4626ce..4478ad9387c5 100644
> > --- a/drivers/iio/position/hid-sensor-custom-intel-hinge.c
> > +++ b/drivers/iio/position/hid-sensor-custom-intel-hinge.c
> > @@ -376,3 +376,4 @@ module_platform_driver(hid_hinge_platform_driver);
> >  MODULE_DESCRIPTION("HID Sensor INTEL Hinge");
> >  MODULE_AUTHOR("Ye Xiang <xiang.ye@intel.com>");
> >  MODULE_LICENSE("GPL");
> > +MODULE_IMPORT_NS(IIO_HID);
> > diff --git a/drivers/iio/pressure/hid-sensor-press.c b/drivers/iio/pressure/hid-sensor-press.c
> > index c416d261e3e3..79b3399e4095 100644
> > --- a/drivers/iio/pressure/hid-sensor-press.c
> > +++ b/drivers/iio/pressure/hid-sensor-press.c
> > @@ -357,3 +357,4 @@ module_platform_driver(hid_press_platform_driver);
> >  MODULE_DESCRIPTION("HID Sensor Pressure");
> >  MODULE_AUTHOR("Archana Patni <archana.patni@intel.com>");
> >  MODULE_LICENSE("GPL");
> > +MODULE_IMPORT_NS(IIO_HID);
> > diff --git a/drivers/iio/temperature/hid-sensor-temperature.c b/drivers/iio/temperature/hid-sensor-temperature.c
> > index dc534ed784c3..21aa952b9f6d 100644
> > --- a/drivers/iio/temperature/hid-sensor-temperature.c
> > +++ b/drivers/iio/temperature/hid-sensor-temperature.c
> > @@ -291,3 +291,4 @@ module_platform_driver(hid_temperature_platform_driver);
> >  MODULE_DESCRIPTION("HID Environmental temperature sensor");
> >  MODULE_AUTHOR("Song Hongyan <hongyan.song@intel.com>");
> >  MODULE_LICENSE("GPL v2");
> > +MODULE_IMPORT_NS(IIO_HID);
> > diff --git a/drivers/rtc/rtc-hid-sensor-time.c b/drivers/rtc/rtc-hid-sensor-time.c
> > index 47cd12db2356..16fdefafec5d 100644
> > --- a/drivers/rtc/rtc-hid-sensor-time.c
> > +++ b/drivers/rtc/rtc-hid-sensor-time.c
> > @@ -328,3 +328,4 @@ module_platform_driver(hid_time_platform_driver);
> >  MODULE_DESCRIPTION("HID Sensor Time");
> >  MODULE_AUTHOR("Alexander Holler <holler@ahsoftware.de>");
> >  MODULE_LICENSE("GPL");
> > +MODULE_IMPORT_NS(IIO_HID);
> > -- 
> > 2.30.2
> >   
>
Andy Shevchenko June 16, 2021, 12:50 p.m. UTC | #3
On Wed, Jun 16, 2021 at 01:41:53PM +0100, Jonathan Cameron wrote:
> On Mon, 14 Jun 2021 22:58:46 +0200
> Alexandre Belloni <alexandre.belloni@bootlin.com> wrote:
> 
> > On 14/06/2021 19:24:47+0300, Andy Shevchenko wrote:
> > > A namespace for exported symbols makes clear who is a provider
> > > and who is a consumer of the certain resources. Besides that,
> > > it doesn't pollute the common namespace.
> > > 
> > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>  
> > Acked-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
> 
> Thanks,  applied to the togreg branch of iio.git and pushed out as
> testing for 0-day to poke at it

Thanks!

> and see if we missed anything.

Hopefully nothing in the code.

What I have missed is the Ack from Srinivas. Can you add it later on?
Jonathan Cameron June 16, 2021, 1:41 p.m. UTC | #4
On Wed, 16 Jun 2021 15:50:28 +0300
Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote:

> On Wed, Jun 16, 2021 at 01:41:53PM +0100, Jonathan Cameron wrote:
> > On Mon, 14 Jun 2021 22:58:46 +0200
> > Alexandre Belloni <alexandre.belloni@bootlin.com> wrote:
> >   
> > > On 14/06/2021 19:24:47+0300, Andy Shevchenko wrote:  
> > > > A namespace for exported symbols makes clear who is a provider
> > > > and who is a consumer of the certain resources. Besides that,
> > > > it doesn't pollute the common namespace.
> > > > 
> > > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>    
> > > Acked-by: Alexandre Belloni <alexandre.belloni@bootlin.com>  
> > 
> > Thanks,  applied to the togreg branch of iio.git and pushed out as
> > testing for 0-day to poke at it  
> 
> Thanks!
> 
> > and see if we missed anything.  
> 
> Hopefully nothing in the code.
> 
> What I have missed is the Ack from Srinivas. Can you add it later on?
> 

Done which was easy because I hadn't pushed it out anywhere yet!

Jonathan
Andy Shevchenko June 16, 2021, 1:46 p.m. UTC | #5
On Wed, Jun 16, 2021 at 02:41:08PM +0100, Jonathan Cameron wrote:
> On Wed, 16 Jun 2021 15:50:28 +0300
> Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote:
> > On Wed, Jun 16, 2021 at 01:41:53PM +0100, Jonathan Cameron wrote:
> > > On Mon, 14 Jun 2021 22:58:46 +0200
> > > Alexandre Belloni <alexandre.belloni@bootlin.com> wrote:
> > > > On 14/06/2021 19:24:47+0300, Andy Shevchenko wrote:

...

> > > > Acked-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
> > > 
> > > Thanks,  applied to the togreg branch of iio.git and pushed out as
> > > testing for 0-day to poke at it
> > 
> > Thanks!
> > 
> > > and see if we missed anything.
> > 
> > Hopefully nothing in the code.
> > 
> > What I have missed is the Ack from Srinivas. Can you add it later on?

> Done which was easy because I hadn't pushed it out anywhere yet!

Thanks!
diff mbox series

Patch

diff --git a/drivers/iio/accel/hid-sensor-accel-3d.c b/drivers/iio/accel/hid-sensor-accel-3d.c
index 27f47e1c251e..bcafca7b2eac 100644
--- a/drivers/iio/accel/hid-sensor-accel-3d.c
+++ b/drivers/iio/accel/hid-sensor-accel-3d.c
@@ -465,3 +465,4 @@  module_platform_driver(hid_accel_3d_platform_driver);
 MODULE_DESCRIPTION("HID Sensor Accel 3D");
 MODULE_AUTHOR("Srinivas Pandruvada <srinivas.pandruvada@intel.com>");
 MODULE_LICENSE("GPL");
+MODULE_IMPORT_NS(IIO_HID);
diff --git a/drivers/iio/common/hid-sensors/hid-sensor-attributes.c b/drivers/iio/common/hid-sensors/hid-sensor-attributes.c
index cb52b4fd6bf7..a81a0b206af6 100644
--- a/drivers/iio/common/hid-sensors/hid-sensor-attributes.c
+++ b/drivers/iio/common/hid-sensors/hid-sensor-attributes.c
@@ -176,7 +176,7 @@  s32 hid_sensor_read_poll_value(struct hid_sensor_common *st)
 
 	return value;
 }
-EXPORT_SYMBOL(hid_sensor_read_poll_value);
+EXPORT_SYMBOL_NS(hid_sensor_read_poll_value, IIO_HID_ATTRIBUTES);
 
 int hid_sensor_read_samp_freq_value(struct hid_sensor_common *st,
 				int *val1, int *val2)
@@ -203,7 +203,7 @@  int hid_sensor_read_samp_freq_value(struct hid_sensor_common *st,
 
 	return IIO_VAL_INT_PLUS_MICRO;
 }
-EXPORT_SYMBOL(hid_sensor_read_samp_freq_value);
+EXPORT_SYMBOL_NS(hid_sensor_read_samp_freq_value, IIO_HID);
 
 int hid_sensor_write_samp_freq_value(struct hid_sensor_common *st,
 				int val1, int val2)
@@ -238,7 +238,7 @@  int hid_sensor_write_samp_freq_value(struct hid_sensor_common *st,
 
 	return 0;
 }
-EXPORT_SYMBOL(hid_sensor_write_samp_freq_value);
+EXPORT_SYMBOL_NS(hid_sensor_write_samp_freq_value, IIO_HID);
 
 int hid_sensor_read_raw_hyst_value(struct hid_sensor_common *st,
 				int *val1, int *val2)
@@ -261,7 +261,7 @@  int hid_sensor_read_raw_hyst_value(struct hid_sensor_common *st,
 
 	return IIO_VAL_INT_PLUS_MICRO;
 }
-EXPORT_SYMBOL(hid_sensor_read_raw_hyst_value);
+EXPORT_SYMBOL_NS(hid_sensor_read_raw_hyst_value, IIO_HID);
 
 int hid_sensor_read_raw_hyst_rel_value(struct hid_sensor_common *st, int *val1,
 				       int *val2)
@@ -283,7 +283,7 @@  int hid_sensor_read_raw_hyst_rel_value(struct hid_sensor_common *st, int *val1,
 
 	return IIO_VAL_INT_PLUS_MICRO;
 }
-EXPORT_SYMBOL(hid_sensor_read_raw_hyst_rel_value);
+EXPORT_SYMBOL_NS(hid_sensor_read_raw_hyst_rel_value, IIO_HID);
 
 
 int hid_sensor_write_raw_hyst_value(struct hid_sensor_common *st,
@@ -315,7 +315,7 @@  int hid_sensor_write_raw_hyst_value(struct hid_sensor_common *st,
 
 	return 0;
 }
-EXPORT_SYMBOL(hid_sensor_write_raw_hyst_value);
+EXPORT_SYMBOL_NS(hid_sensor_write_raw_hyst_value, IIO_HID);
 
 int hid_sensor_write_raw_hyst_rel_value(struct hid_sensor_common *st,
 					int val1, int val2)
@@ -346,7 +346,7 @@  int hid_sensor_write_raw_hyst_rel_value(struct hid_sensor_common *st,
 
 	return 0;
 }
-EXPORT_SYMBOL(hid_sensor_write_raw_hyst_rel_value);
+EXPORT_SYMBOL_NS(hid_sensor_write_raw_hyst_rel_value, IIO_HID);
 
 /*
  * This fuction applies the unit exponent to the scale.
@@ -430,14 +430,14 @@  int hid_sensor_format_scale(u32 usage_id,
 
 	return IIO_VAL_INT_PLUS_NANO;
 }
-EXPORT_SYMBOL(hid_sensor_format_scale);
+EXPORT_SYMBOL_NS(hid_sensor_format_scale, IIO_HID);
 
 int64_t hid_sensor_convert_timestamp(struct hid_sensor_common *st,
 				     int64_t raw_value)
 {
 	return st->timestamp_ns_scale * raw_value;
 }
-EXPORT_SYMBOL(hid_sensor_convert_timestamp);
+EXPORT_SYMBOL_NS(hid_sensor_convert_timestamp, IIO_HID);
 
 static
 int hid_sensor_get_reporting_interval(struct hid_sensor_hub_device *hsdev,
@@ -484,7 +484,7 @@  int hid_sensor_get_report_latency(struct hid_sensor_common *st)
 
 	return value;
 }
-EXPORT_SYMBOL(hid_sensor_get_report_latency);
+EXPORT_SYMBOL_NS(hid_sensor_get_report_latency, IIO_HID_ATTRIBUTES);
 
 int hid_sensor_set_report_latency(struct hid_sensor_common *st, int latency_ms)
 {
@@ -492,13 +492,13 @@  int hid_sensor_set_report_latency(struct hid_sensor_common *st, int latency_ms)
 				      st->report_latency.index,
 				      sizeof(latency_ms), &latency_ms);
 }
-EXPORT_SYMBOL(hid_sensor_set_report_latency);
+EXPORT_SYMBOL_NS(hid_sensor_set_report_latency, IIO_HID_ATTRIBUTES);
 
 bool hid_sensor_batch_mode_supported(struct hid_sensor_common *st)
 {
 	return st->report_latency.index > 0 && st->report_latency.report_id > 0;
 }
-EXPORT_SYMBOL(hid_sensor_batch_mode_supported);
+EXPORT_SYMBOL_NS(hid_sensor_batch_mode_supported, IIO_HID_ATTRIBUTES);
 
 int hid_sensor_parse_common_attributes(struct hid_sensor_hub_device *hsdev,
 					u32 usage_id,
@@ -590,7 +590,7 @@  int hid_sensor_parse_common_attributes(struct hid_sensor_hub_device *hsdev,
 
 	return 0;
 }
-EXPORT_SYMBOL(hid_sensor_parse_common_attributes);
+EXPORT_SYMBOL_NS(hid_sensor_parse_common_attributes, IIO_HID);
 
 MODULE_AUTHOR("Srinivas Pandruvada <srinivas.pandruvada@intel.com>");
 MODULE_DESCRIPTION("HID Sensor common attribute processing");
diff --git a/drivers/iio/common/hid-sensors/hid-sensor-trigger.c b/drivers/iio/common/hid-sensors/hid-sensor-trigger.c
index c06537e106e9..60e85d675387 100644
--- a/drivers/iio/common/hid-sensors/hid-sensor-trigger.c
+++ b/drivers/iio/common/hid-sensors/hid-sensor-trigger.c
@@ -150,7 +150,7 @@  static int _hid_sensor_power_state(struct hid_sensor_common *st, bool state)
 
 	return 0;
 }
-EXPORT_SYMBOL(hid_sensor_power_state);
+EXPORT_SYMBOL_NS(hid_sensor_power_state, IIO_HID);
 
 int hid_sensor_power_state(struct hid_sensor_common *st, bool state)
 {
@@ -225,7 +225,7 @@  void hid_sensor_remove_trigger(struct iio_dev *indio_dev,
 	iio_trigger_free(attrb->trigger);
 	iio_triggered_buffer_cleanup(indio_dev);
 }
-EXPORT_SYMBOL(hid_sensor_remove_trigger);
+EXPORT_SYMBOL_NS(hid_sensor_remove_trigger, IIO_HID);
 
 static const struct iio_trigger_ops hid_sensor_trigger_ops = {
 	.set_trigger_state = &hid_sensor_data_rdy_trigger_set_state,
@@ -291,7 +291,7 @@  int hid_sensor_setup_trigger(struct iio_dev *indio_dev, const char *name,
 	iio_triggered_buffer_cleanup(indio_dev);
 	return ret;
 }
-EXPORT_SYMBOL(hid_sensor_setup_trigger);
+EXPORT_SYMBOL_NS(hid_sensor_setup_trigger, IIO_HID);
 
 static int __maybe_unused hid_sensor_suspend(struct device *dev)
 {
@@ -321,8 +321,9 @@  const struct dev_pm_ops hid_sensor_pm_ops = {
 	SET_RUNTIME_PM_OPS(hid_sensor_suspend,
 			   hid_sensor_runtime_resume, NULL)
 };
-EXPORT_SYMBOL(hid_sensor_pm_ops);
+EXPORT_SYMBOL_NS(hid_sensor_pm_ops, IIO_HID);
 
 MODULE_AUTHOR("Srinivas Pandruvada <srinivas.pandruvada@intel.com>");
 MODULE_DESCRIPTION("HID Sensor trigger processing");
 MODULE_LICENSE("GPL");
+MODULE_IMPORT_NS(IIO_HID_ATTRIBUTES);
diff --git a/drivers/iio/gyro/hid-sensor-gyro-3d.c b/drivers/iio/gyro/hid-sensor-gyro-3d.c
index dad26ee4fd1f..2ed2a8effb86 100644
--- a/drivers/iio/gyro/hid-sensor-gyro-3d.c
+++ b/drivers/iio/gyro/hid-sensor-gyro-3d.c
@@ -400,3 +400,4 @@  module_platform_driver(hid_gyro_3d_platform_driver);
 MODULE_DESCRIPTION("HID Sensor Gyroscope 3D");
 MODULE_AUTHOR("Srinivas Pandruvada <srinivas.pandruvada@intel.com>");
 MODULE_LICENSE("GPL");
+MODULE_IMPORT_NS(IIO_HID);
diff --git a/drivers/iio/humidity/hid-sensor-humidity.c b/drivers/iio/humidity/hid-sensor-humidity.c
index 74383abc0d44..8a9091d71d2a 100644
--- a/drivers/iio/humidity/hid-sensor-humidity.c
+++ b/drivers/iio/humidity/hid-sensor-humidity.c
@@ -295,3 +295,4 @@  module_platform_driver(hid_humidity_platform_driver);
 MODULE_DESCRIPTION("HID Environmental humidity sensor");
 MODULE_AUTHOR("Song Hongyan <hongyan.song@intel.com>");
 MODULE_LICENSE("GPL v2");
+MODULE_IMPORT_NS(IIO_HID);
diff --git a/drivers/iio/light/hid-sensor-als.c b/drivers/iio/light/hid-sensor-als.c
index 85c8a05b73cb..a63d577493e2 100644
--- a/drivers/iio/light/hid-sensor-als.c
+++ b/drivers/iio/light/hid-sensor-als.c
@@ -392,3 +392,4 @@  module_platform_driver(hid_als_platform_driver);
 MODULE_DESCRIPTION("HID Sensor ALS");
 MODULE_AUTHOR("Srinivas Pandruvada <srinivas.pandruvada@intel.com>");
 MODULE_LICENSE("GPL");
+MODULE_IMPORT_NS(IIO_HID);
diff --git a/drivers/iio/light/hid-sensor-prox.c b/drivers/iio/light/hid-sensor-prox.c
index 17d167c3d595..99de268563d6 100644
--- a/drivers/iio/light/hid-sensor-prox.c
+++ b/drivers/iio/light/hid-sensor-prox.c
@@ -350,3 +350,4 @@  module_platform_driver(hid_prox_platform_driver);
 MODULE_DESCRIPTION("HID Sensor Proximity");
 MODULE_AUTHOR("Archana Patni <archana.patni@intel.com>");
 MODULE_LICENSE("GPL");
+MODULE_IMPORT_NS(IIO_HID);
diff --git a/drivers/iio/magnetometer/hid-sensor-magn-3d.c b/drivers/iio/magnetometer/hid-sensor-magn-3d.c
index b78691523dd4..a66f9e933628 100644
--- a/drivers/iio/magnetometer/hid-sensor-magn-3d.c
+++ b/drivers/iio/magnetometer/hid-sensor-magn-3d.c
@@ -587,3 +587,4 @@  module_platform_driver(hid_magn_3d_platform_driver);
 MODULE_DESCRIPTION("HID Sensor Magnetometer 3D");
 MODULE_AUTHOR("Srinivas Pandruvada <srinivas.pandruvada@intel.com>");
 MODULE_LICENSE("GPL");
+MODULE_IMPORT_NS(IIO_HID);
diff --git a/drivers/iio/orientation/hid-sensor-incl-3d.c b/drivers/iio/orientation/hid-sensor-incl-3d.c
index 7af48d336285..cc905b0fa835 100644
--- a/drivers/iio/orientation/hid-sensor-incl-3d.c
+++ b/drivers/iio/orientation/hid-sensor-incl-3d.c
@@ -425,3 +425,4 @@  module_platform_driver(hid_incl_3d_platform_driver);
 MODULE_DESCRIPTION("HID Sensor Inclinometer 3D");
 MODULE_AUTHOR("Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>");
 MODULE_LICENSE("GPL");
+MODULE_IMPORT_NS(IIO_HID);
diff --git a/drivers/iio/orientation/hid-sensor-rotation.c b/drivers/iio/orientation/hid-sensor-rotation.c
index cf7f57a47681..6570bfd22035 100644
--- a/drivers/iio/orientation/hid-sensor-rotation.c
+++ b/drivers/iio/orientation/hid-sensor-rotation.c
@@ -373,3 +373,4 @@  module_platform_driver(hid_dev_rot_platform_driver);
 MODULE_DESCRIPTION("HID Sensor Device Rotation");
 MODULE_AUTHOR("Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>");
 MODULE_LICENSE("GPL");
+MODULE_IMPORT_NS(IIO_HID);
diff --git a/drivers/iio/position/hid-sensor-custom-intel-hinge.c b/drivers/iio/position/hid-sensor-custom-intel-hinge.c
index 738b5f4626ce..4478ad9387c5 100644
--- a/drivers/iio/position/hid-sensor-custom-intel-hinge.c
+++ b/drivers/iio/position/hid-sensor-custom-intel-hinge.c
@@ -376,3 +376,4 @@  module_platform_driver(hid_hinge_platform_driver);
 MODULE_DESCRIPTION("HID Sensor INTEL Hinge");
 MODULE_AUTHOR("Ye Xiang <xiang.ye@intel.com>");
 MODULE_LICENSE("GPL");
+MODULE_IMPORT_NS(IIO_HID);
diff --git a/drivers/iio/pressure/hid-sensor-press.c b/drivers/iio/pressure/hid-sensor-press.c
index c416d261e3e3..79b3399e4095 100644
--- a/drivers/iio/pressure/hid-sensor-press.c
+++ b/drivers/iio/pressure/hid-sensor-press.c
@@ -357,3 +357,4 @@  module_platform_driver(hid_press_platform_driver);
 MODULE_DESCRIPTION("HID Sensor Pressure");
 MODULE_AUTHOR("Archana Patni <archana.patni@intel.com>");
 MODULE_LICENSE("GPL");
+MODULE_IMPORT_NS(IIO_HID);
diff --git a/drivers/iio/temperature/hid-sensor-temperature.c b/drivers/iio/temperature/hid-sensor-temperature.c
index dc534ed784c3..21aa952b9f6d 100644
--- a/drivers/iio/temperature/hid-sensor-temperature.c
+++ b/drivers/iio/temperature/hid-sensor-temperature.c
@@ -291,3 +291,4 @@  module_platform_driver(hid_temperature_platform_driver);
 MODULE_DESCRIPTION("HID Environmental temperature sensor");
 MODULE_AUTHOR("Song Hongyan <hongyan.song@intel.com>");
 MODULE_LICENSE("GPL v2");
+MODULE_IMPORT_NS(IIO_HID);
diff --git a/drivers/rtc/rtc-hid-sensor-time.c b/drivers/rtc/rtc-hid-sensor-time.c
index 47cd12db2356..16fdefafec5d 100644
--- a/drivers/rtc/rtc-hid-sensor-time.c
+++ b/drivers/rtc/rtc-hid-sensor-time.c
@@ -328,3 +328,4 @@  module_platform_driver(hid_time_platform_driver);
 MODULE_DESCRIPTION("HID Sensor Time");
 MODULE_AUTHOR("Alexander Holler <holler@ahsoftware.de>");
 MODULE_LICENSE("GPL");
+MODULE_IMPORT_NS(IIO_HID);