| 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 |
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 >
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 > > >
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?
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
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 --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);
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(-)