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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id A6414C433EF for ; Sun, 30 Jan 2022 20:50:51 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1347586AbiA3Uuv (ORCPT ); Sun, 30 Jan 2022 15:50:51 -0500 Received: from dfw.source.kernel.org ([139.178.84.217]:56710 "EHLO dfw.source.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S243587AbiA3Uuu (ORCPT ); Sun, 30 Jan 2022 15:50:50 -0500 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 2F39C60C65 for ; Sun, 30 Jan 2022 20:50:50 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 8A7FDC340E4; Sun, 30 Jan 2022 20:50:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1643575849; bh=DjsTKlh38WggMN/Hq8JYjz3b7Er5pNid4Z8WMjfubI0=; h=From:To:Cc:Subject:Date:From; b=mXtB88fvSySmZgkZxLymGlzJMRq2z2iQMZImOsSHZJheiNDuTsp8og5tyyEpkTZXB 728JCcyVYJmxxrRm/q01yJiymBrf3ewmm9Y6TckyrjTrmfddU50JjxCKHqgbNrrUIZ n+ca0Trdxi0wSM7i0UEjksGijSpxvrs4XjZW/AOE1esVWIDRqs/Tgdco/sAs/K5r8O jIUgdTwv/PRaKV9WkBGNLXN5N4Wm3VTjwLSVLTeMNEH0XFU2fpam8ipkJr+sKtoNd9 9z/+RQsIb5mmF5KYjZ08EqDsXsFiYEp/fjV2Lh4BQY7Cxho2fN/sEBUY9ON3icEYtL Ox3CDe8jRM2og== From: Jonathan Cameron To: linux-iio@vger.kernel.org Cc: Andy Shevchenko , Gwendal Grignou , lars@metafoo.de, Jonathan Cameron , Himanshu Jha , Song Qiang , Tomasz Duszynski , Song Bao Hua , Paul Cercueil , Alexandre Belloni , William Markezana , Ludovic Tancerel , Renato Lui Geh , Michael Hennerich Subject: [PATCH 00/16] 2nd set of IIO export namespaces Date: Sun, 30 Jan 2022 20:56:45 +0000 Message-Id: <20220130205701.334592-1-jic23@kernel.org> X-Mailer: git-send-email 2.35.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-iio@vger.kernel.org From: Jonathan Cameron 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. Cc: Himanshu Jha Cc: Song Qiang Cc: Tomasz Duszynski Cc: Lars-Peter Clausen Cc: Song Bao Hua (Barry Song) Cc: Paul Cercueil Cc: Alexandre Belloni Cc: William Markezana Cc: Ludovic Tancerel Cc: Renato Lui Geh Cc: Michael Hennerich 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