From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753049AbeCWM7h convert rfc822-to-8bit (ORCPT ); Fri, 23 Mar 2018 08:59:37 -0400 Received: from szxga06-in.huawei.com ([45.249.212.32]:41739 "EHLO huawei.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1752541AbeCWM7f (ORCPT ); Fri, 23 Mar 2018 08:59:35 -0400 Date: Fri, 23 Mar 2018 12:59:19 +0000 From: Jonathan Cameron To: =?ISO-8859-1?Q?Hern=E1n?= Gonzalez CC: , , , , , , Subject: Re: [PATCH 11/11] Move cdc ad7746 driver out of staging to mainline iio Message-ID: <20180323135919.00005084@huawei.com> In-Reply-To: <1521642539-4845-12-git-send-email-hernan@vanguardiasur.com.ar> References: <1521642539-4845-1-git-send-email-hernan@vanguardiasur.com.ar> <1521642539-4845-12-git-send-email-hernan@vanguardiasur.com.ar> X-Mailer: Claws Mail 3.16.0 (GTK+ 2.24.31; i686-w64-mingw32) MIME-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 8BIT X-Originating-IP: [10.202.226.45] X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 21 Mar 2018 11:28:59 -0300 Hernán Gonzalez wrote: > Also modify the proper Kconfigs and move documentation. > > Signed-off-by: Hernán Gonzalez Please disable git move detection for this patch in v2. This only applies when moving drivers out of staging (and possibly only me who asks for it then ;). The point is to allow full review of the driver we are actually moving. That is hard to do if we don't have the code in the email. Anyhow, good work on the series in general. I look forward to V2. Jonathan > --- > .../devicetree/bindings/{staging => }/iio/cdc/ad7746.txt | 0 > drivers/iio/Kconfig | 1 + > drivers/iio/cdc/Kconfig | 16 ++++++++++++++++ > drivers/{staging => }/iio/cdc/ad7746.c | 2 +- > drivers/staging/iio/cdc/Kconfig | 10 ---------- > {drivers/staging => include/linux}/iio/cdc/ad7746.h | 4 ---- > 6 files changed, 18 insertions(+), 15 deletions(-) > rename Documentation/devicetree/bindings/{staging => }/iio/cdc/ad7746.txt (100%) > create mode 100644 drivers/iio/cdc/Kconfig > rename drivers/{staging => }/iio/cdc/ad7746.c (99%) > rename {drivers/staging => include/linux}/iio/cdc/ad7746.h (88%) > > diff --git a/Documentation/devicetree/bindings/staging/iio/cdc/ad7746.txt b/Documentation/devicetree/bindings/iio/cdc/ad7746.txt > similarity index 100% > rename from Documentation/devicetree/bindings/staging/iio/cdc/ad7746.txt > rename to Documentation/devicetree/bindings/iio/cdc/ad7746.txt > diff --git a/drivers/iio/Kconfig b/drivers/iio/Kconfig > index b3c8c6e..d1c309b 100644 > --- a/drivers/iio/Kconfig > +++ b/drivers/iio/Kconfig > @@ -71,6 +71,7 @@ config IIO_TRIGGERED_EVENT > source "drivers/iio/accel/Kconfig" > source "drivers/iio/adc/Kconfig" > source "drivers/iio/amplifiers/Kconfig" > +source "drivers/iio/cdc/Kconfig" > source "drivers/iio/chemical/Kconfig" > source "drivers/iio/common/Kconfig" > source "drivers/iio/counter/Kconfig" > diff --git a/drivers/iio/cdc/Kconfig b/drivers/iio/cdc/Kconfig > new file mode 100644 > index 0000000..d3a8600 > --- /dev/null > +++ b/drivers/iio/cdc/Kconfig > @@ -0,0 +1,16 @@ > +# > +# CDC drivers > +# > +menu "Capacitance to digital converters" > + > +config AD7746 > + tristate "Analog Devices AD7745, AD7746 AD7747 capacitive sensor driver" > + depends on I2C > + help > + Say yes here to build support for Analog Devices capacitive sensors. > + (AD7745, AD7746, AD7747) Provides direct access via sysfs. > + > + To compile this driver as a module, choose M here: the > + module will be called ad7746. > + > +endmenu > diff --git a/drivers/staging/iio/cdc/ad7746.c b/drivers/iio/cdc/ad7746.c > similarity index 99% > rename from drivers/staging/iio/cdc/ad7746.c > rename to drivers/iio/cdc/ad7746.c > index c1f76fc..23c9f61 100644 > --- a/drivers/staging/iio/cdc/ad7746.c > +++ b/drivers/iio/cdc/ad7746.c > @@ -18,8 +18,8 @@ > > #include > #include > +#include > > -#include "ad7746.h" > > /* > * AD7746 Register Definition > diff --git a/drivers/staging/iio/cdc/Kconfig b/drivers/staging/iio/cdc/Kconfig > index 80211df..a170ab3 100644 > --- a/drivers/staging/iio/cdc/Kconfig > +++ b/drivers/staging/iio/cdc/Kconfig > @@ -23,14 +23,4 @@ config AD7152 > To compile this driver as a module, choose M here: the > module will be called ad7152. > > -config AD7746 > - tristate "Analog Devices AD7745, AD7746 AD7747 capacitive sensor driver" > - depends on I2C > - help > - Say yes here to build support for Analog Devices capacitive sensors. > - (AD7745, AD7746, AD7747) Provides direct access via sysfs. > - > - To compile this driver as a module, choose M here: the > - module will be called ad7746. > - > endmenu > diff --git a/drivers/staging/iio/cdc/ad7746.h b/include/linux/iio/cdc/ad7746.h > similarity index 88% > rename from drivers/staging/iio/cdc/ad7746.h > rename to include/linux/iio/cdc/ad7746.h > index 2fbcee8..46ff25e 100644 > --- a/drivers/staging/iio/cdc/ad7746.h > +++ b/include/linux/iio/cdc/ad7746.h > @@ -9,10 +9,6 @@ > #ifndef IIO_CDC_AD7746_H_ > #define IIO_CDC_AD7746_H_ > > -/* > - * TODO: struct ad7746_platform_data needs to go into include/linux/iio > - */ > - > struct ad7746_platform_data { > unsigned char exclvl; /*Excitation Voltage Level */ > bool exca_en; /* enables EXCA pin as the excitation output */