On Sun, Jan 30, 2022 at 10:50 PM Jonathan Cameron <jic23@xxxxxxxxxx> wrote: > > From: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx> > > Whilst I'm still looking for additional review on the first set, > I'll take the quiet as meaning no one objects and send out a second > set. Note that I don't have anyone cc'd on some of these because > I couldn't immediately figure out who was still active and familiar > with the drivers. As these should be simple to review, please take > a look at as many as you have time for! > > Here's the reasoning as per the first set cover letter. > Note that we have some clashes with patches that I'd like to clean > up the handling of dev_pm_ops and CONFIG_PM* guards so for now > I've dropped those cases from this series. > > https://lwn.net/Articles/760045/ provides a good overview of this feature > and why it might be used. In this particular case we fall under the multi > module case. For each group of modules, I've moved the exports into > an independent namespace. > > Whilst I'm not entirely sure it is worth the noise of moving the main > IIO exports into a namespace, it is definitely worthwhile for these small > clusters of related modules (typically a core + i2c and spi modules). > > I've just done those within drivers/accel (including all of st-sensors as > I did not want to do that piecemeal) as an initial set to get feedback on > what people think of the approach and in particular naming of the namespaces. > > As you can see it is a straight forward change to makes so I'll do the > rest of the IIO drivers once we are agreed on the generic details. > Note that I'll be expecting this for any new cases of this pattern going > forwards. > > Includes a small diversion in st-sensors where we had a bunch of duplicated > MODULE_* macros for some of the core modules which were made up of multiple > c files. I briefly looked at the series and found no drastical issues (hope CI will find, if any), so Reviewed-by: Andy Shevchenko <andy.shevchenko@xxxxxxxxx> > Cc: Himanshu Jha <himanshujha199640@xxxxxxxxx> > Cc: Song Qiang <songqiang1304521@xxxxxxxxx> > Cc: Tomasz Duszynski <tduszyns@xxxxxxxxx> > Cc: Lars-Peter Clausen <lars@xxxxxxxxxx> > Cc: Song Bao Hua (Barry Song) <song.bao.hua@xxxxxxxxxxxxx> > Cc: Paul Cercueil <paul@xxxxxxxxxxxxxxx> > Cc: Alexandre Belloni <alexandre.belloni@xxxxxxxxxxx> > Cc: William Markezana <william.markezana@xxxxxxxxxxxxx> > Cc: Ludovic Tancerel <ludovic.tancerel@xxxxxxxxxxxxxxxxx> > Cc: Renato Lui Geh <renatogeh@xxxxxxxxx> > Cc: Michael Hennerich <Michael.Hennerich@xxxxxxxxxx> > > > Jonathan Cameron (16): > iio:adc:ad_sigma_delta: Move exports into IIO_AD_SIGMA_DELTA namespace > iio:adc:ad7091r: Move exports into IIO_AD7091R namespace. > iio:adc:ad76060: Move exports into IIO_AD7606 namespace. > iio:common:meas-spec: Move exports into IIO_MEAS_SPEC_SENSORS > iio:common:ssp_sensors: Move exports into IIO_SSP_SENSORS namespace > iio:dac:ad5592r: Move exports into IIO_AD5592R namespace > iio:dac:ad5686: Move exports into IIO_AD5686 namespace > iio:imu:adis: Move exports into IIO_ADISLIB namespace > iio:pressure:zpa2326: Move exports into IIO_ZPA2326 namespace > iio:pressure:ms5611: Move exports into IIO_MS5611 namespace > iio:pressure:mpl115: Move exports into IIO_MPL115 namespace > iio:magnetometer:rm3100: Move exports to IIO_RM3100 namespace > iio:magnetometer:bmc150: Move exports to IIO_BMC150_MAGN namespace > iio:magnetometer:hmc5843: Move exports to IIO_HMC5843 namespace > iio:light:st_uvis25: Move exports to IIO_UVIS25 namespace > iio:chemical:bme680: Move exports to IIO_BME680 namespace > > drivers/iio/accel/adis16201.c | 1 + > drivers/iio/accel/adis16209.c | 1 + > drivers/iio/accel/ssp_accel_sensor.c | 1 + > drivers/iio/adc/ad7091r-base.c | 4 ++-- > drivers/iio/adc/ad7091r5.c | 1 + > drivers/iio/adc/ad7124.c | 1 + > drivers/iio/adc/ad7192.c | 1 + > drivers/iio/adc/ad7606.c | 4 ++-- > drivers/iio/adc/ad7606_par.c | 1 + > drivers/iio/adc/ad7606_spi.c | 1 + > drivers/iio/adc/ad7780.c | 1 + > drivers/iio/adc/ad7791.c | 1 + > drivers/iio/adc/ad7793.c | 1 + > drivers/iio/adc/ad_sigma_delta.c | 20 ++++++++-------- > drivers/iio/chemical/bme680_core.c | 4 ++-- > drivers/iio/chemical/bme680_i2c.c | 1 + > drivers/iio/chemical/bme680_spi.c | 1 + > .../iio/common/ms_sensors/ms_sensors_i2c.c | 24 +++++++++---------- > drivers/iio/common/ssp_sensors/ssp_dev.c | 10 ++++---- > drivers/iio/common/ssp_sensors/ssp_iio.c | 7 +++--- > drivers/iio/dac/ad5592r-base.c | 4 ++-- > drivers/iio/dac/ad5592r.c | 1 + > drivers/iio/dac/ad5593r.c | 1 + > drivers/iio/dac/ad5686-spi.c | 1 + > drivers/iio/dac/ad5686.c | 4 ++-- > drivers/iio/dac/ad5696-i2c.c | 1 + > drivers/iio/gyro/adis16136.c | 1 + > drivers/iio/gyro/adis16260.c | 1 + > drivers/iio/gyro/ssp_gyro_sensor.c | 1 + > drivers/iio/humidity/htu21.c | 1 + > drivers/iio/imu/adis.c | 20 ++++++++-------- > drivers/iio/imu/adis16400.c | 1 + > drivers/iio/imu/adis16460.c | 1 + > drivers/iio/imu/adis16475.c | 1 + > drivers/iio/imu/adis16480.c | 1 + > drivers/iio/imu/adis_buffer.c | 4 ++-- > drivers/iio/imu/adis_trigger.c | 2 +- > drivers/iio/light/st_uvis25_core.c | 4 ++-- > drivers/iio/light/st_uvis25_i2c.c | 1 + > drivers/iio/light/st_uvis25_spi.c | 1 + > drivers/iio/magnetometer/bmc150_magn.c | 8 +++---- > drivers/iio/magnetometer/bmc150_magn_i2c.c | 1 + > drivers/iio/magnetometer/bmc150_magn_spi.c | 1 + > drivers/iio/magnetometer/hmc5843_core.c | 8 +++---- > drivers/iio/magnetometer/hmc5843_i2c.c | 1 + > drivers/iio/magnetometer/hmc5843_spi.c | 1 + > drivers/iio/magnetometer/rm3100-core.c | 8 +++---- > drivers/iio/magnetometer/rm3100-i2c.c | 1 + > drivers/iio/magnetometer/rm3100-spi.c | 1 + > drivers/iio/pressure/mpl115.c | 2 +- > drivers/iio/pressure/mpl115_i2c.c | 1 + > drivers/iio/pressure/mpl115_spi.c | 1 + > drivers/iio/pressure/ms5611_core.c | 4 ++-- > drivers/iio/pressure/ms5611_i2c.c | 1 + > drivers/iio/pressure/ms5611_spi.c | 1 + > drivers/iio/pressure/ms5637.c | 1 + > drivers/iio/pressure/zpa2326.c | 12 +++++----- > drivers/iio/pressure/zpa2326_i2c.c | 1 + > drivers/iio/pressure/zpa2326_spi.c | 1 + > drivers/iio/temperature/tsys01.c | 1 + > drivers/iio/temperature/tsys02d.c | 1 + > drivers/staging/iio/accel/adis16203.c | 1 + > drivers/staging/iio/accel/adis16240.c | 1 + > 63 files changed, 121 insertions(+), 76 deletions(-) > > -- > 2.35.1 > -- With Best Regards, Andy Shevchenko
