From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-13.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 36CB5C47097 for ; Fri, 4 Jun 2021 01:11:22 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 247206120F for ; Fri, 4 Jun 2021 01:11:22 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229957AbhFDBNG (ORCPT ); Thu, 3 Jun 2021 21:13:06 -0400 Received: from mga01.intel.com ([192.55.52.88]:3644 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229754AbhFDBNE (ORCPT ); Thu, 3 Jun 2021 21:13:04 -0400 IronPort-SDR: nGqHLVMynEF7YLDpFBsURfvTVlQTjD6SYfVHLB6OUIT5h3N8VvmUzl3jfX51WJNrhhlaBRmEin WXaQXLk2BjYw== X-IronPort-AV: E=McAfee;i="6200,9189,10004"; a="225508186" X-IronPort-AV: E=Sophos;i="5.83,246,1616482800"; d="scan'208";a="225508186" Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 03 Jun 2021 18:11:08 -0700 IronPort-SDR: iEZg2eOfgQQdO6an+RlQX8obiRikjN4S0sB4FOL6YSBHoIce+h3SMGYSf6yX4PuOCz8Ud5yr3H e9R/XIQ2PkvQ== X-IronPort-AV: E=Sophos;i="5.83,246,1616482800"; d="scan'208";a="475281916" Received: from kmillard-mobl.amr.corp.intel.com ([10.212.170.118]) by fmsmga003-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 03 Jun 2021 18:11:06 -0700 Message-ID: Subject: Re: [PATCH v1 1/1] iio: hid-sensors: lighten the exported symbols by introducing IIO_HID namespace From: srinivas pandruvada To: Andy Shevchenko Cc: Jonathan Cameron , Andy Shevchenko , Jonathan Cameron , Ye Xiang , "linux-input@vger.kernel.org" , "linux-iio@vger.kernel.org" , "linux-kernel@vger.kernel.org" , Jiri Kosina , Lars-Peter Clausen Date: Thu, 03 Jun 2021 18:10:56 -0700 In-Reply-To: References: <20210525132119.21800-1-andriy.shevchenko@linux.intel.com> <20210603180700.3206eda0@jic23-huawei> Content-Type: text/plain; charset="UTF-8" User-Agent: Evolution 3.38.4 (3.38.4-1.fc33) MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 2021-06-04 at 01:50 +0300, Andy Shevchenko wrote: > > > On Thursday, June 3, 2021, Srinivas Pandruvada > wrote: > > On Thu, 2021-06-03 at 18:07 +0100, Jonathan Cameron wrote: > > > On Tue, 25 May 2021 16:21:19 +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 > > > > > > > > > > I'm fine with this, but looking for input from Srinivas and Jiri > > > before > > > I apply it. > > > > I see these names are using "IIO_HID" and "IIO_HID_ATTRIBUTES". > > Since they are only used in hid-sensors, may be explicitly define > > > > I think we need to stick with a provider name which is IIO subsystem, > HID driver. That’s my logic behind. >   OK > > "HID_SENSOR_IIO". Also prefer only one name space. > > > > > > It may not be one namespace IIUC, because same module has to request > and provide it at the same time, moreover, two C modules are exporting > something, I dunno how it’s supposed to work. > I am fine with the changes. Acked-by: Srinivas Pandruvada Thanks, Srinivas >   > > Thanks, > > Srinivas > > > > > > > > > --- > > > > > > > > Jonothan et al. This is just a work against HID IIO sensor > > > > drivers, > > > > I think > > > > that entire IIO can gain from namespaces. But I leave it for you > > > > to > > > > decide and > > > > how to proceed / convert. > > > > > > It's been on the todo list since Jon Corbetts recent LWN article > > > pointing out > > > how underused this feature is which reminded me of it's existence. > > > > > > I agree in principle with doing it, but that is going to be a > > > rather > > > big > > > single shot patch.  One for early in a cycle I think. > > > > > > We have a lot of multi-module drivers as well where this might have > > > benefits > > > and would be a rather more contained change. > > > > > > Jonathan > > > > > > > Yes, due to nature of this (kind of) change it may not be split. > > > > > > > >   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 + > > > >   13 files changed, 29 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 > > > > > > > > "); > > > >   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 > > > > > > > > "); > > > >   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 > > > > > > > > "); > > > >   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 > > > > > > > > "); > > > >   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 "); > > > >   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 > > > > > > > > "); > > > >   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 "); > > > >   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 > > > > > > > > "); > > > >   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 "); > > > >   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 "); > > > >   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 "); > > > >   MODULE_LICENSE("GPL v2"); > > > > +MODULE_IMPORT_NS(IIO_HID); > > > > > >