* [PATCH] iio: adc128s052: add support for adc122s021 @ 2015-05-18 12:04 Urs Fässler 2015-05-18 12:04 ` Urs Fässler 0 siblings, 1 reply; 6+ messages in thread From: Urs Fässler @ 2015-05-18 12:04 UTC (permalink / raw) To: jic23, angelo.compagnucci, urs.fassler Cc: iivanov, ezequiel.garcia, jacob.jun.pan, devicetree, linux-kernel, linux-iio This patch adds support for the 2 channel ADC adc122s021. Since I only have an adc122s021 at hand, I can not test the adc128s052 part. Patch is built against v4.1-rc2. Urs Fässler (1): iio: adc128s052: add support for adc122s021 .../devicetree/bindings/iio/adc/ti-adc128s052.txt | 6 ++--- drivers/iio/adc/Kconfig | 4 ++-- drivers/iio/adc/ti-adc128s052.c | 27 ++++++++++++++++++---- 3 files changed, 27 insertions(+), 10 deletions(-) -- 2.1.4 ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] iio: adc128s052: add support for adc122s021 2015-05-18 12:04 [PATCH] iio: adc128s052: add support for adc122s021 Urs Fässler @ 2015-05-18 12:04 ` Urs Fässler 2015-05-18 12:19 ` Lars-Peter Clausen 2015-05-18 12:40 ` Peter Meerwald 0 siblings, 2 replies; 6+ messages in thread From: Urs Fässler @ 2015-05-18 12:04 UTC (permalink / raw) To: jic23, angelo.compagnucci, urs.fassler Cc: iivanov, ezequiel.garcia, jacob.jun.pan, devicetree, linux-kernel, linux-iio Signed-off-by: Urs Fässler <urs.fassler@bytesatwork.ch> --- .../devicetree/bindings/iio/adc/ti-adc128s052.txt | 6 ++--- drivers/iio/adc/Kconfig | 4 ++-- drivers/iio/adc/ti-adc128s052.c | 27 ++++++++++++++++++---- 3 files changed, 27 insertions(+), 10 deletions(-) diff --git a/Documentation/devicetree/bindings/iio/adc/ti-adc128s052.txt b/Documentation/devicetree/bindings/iio/adc/ti-adc128s052.txt index 42ca7de..7cefcf7 100644 --- a/Documentation/devicetree/bindings/iio/adc/ti-adc128s052.txt +++ b/Documentation/devicetree/bindings/iio/adc/ti-adc128s052.txt @@ -1,7 +1,7 @@ -* Texas Instruments' ADC128S052 ADC chip +* Texas Instruments' ADC128S052 and ADC122S021 ADC chip Required properties: - - compatible: Should be "ti,adc128s052" + - compatible: Should be "adc128s052" or "adc122s021" - reg: spi chip select number for the device - vref-supply: The regulator supply for ADC reference voltage @@ -11,7 +11,7 @@ Recommended properties: Example: adc@0 { - compatible = "ti,adc128s052"; + compatible = "adc128s052"; reg = <0>; vref-supply = <&vdd_supply>; spi-max-frequency = <1000000>; diff --git a/drivers/iio/adc/Kconfig b/drivers/iio/adc/Kconfig index e36a73e..92c319c 100644 --- a/drivers/iio/adc/Kconfig +++ b/drivers/iio/adc/Kconfig @@ -285,11 +285,11 @@ config TI_ADC081C called ti-adc081c. config TI_ADC128S052 - tristate "Texas Instruments ADC128S052" + tristate "Texas Instruments ADC128S052/ADC122S021" depends on SPI help If you say yes here you get support for Texas Instruments ADC128S052 - chip. + and ADC122S021 chip. This driver can also be built as a module. If so, the module will be called ti-adc128s052. diff --git a/drivers/iio/adc/ti-adc128s052.c b/drivers/iio/adc/ti-adc128s052.c index 655cb56..42c7772 100644 --- a/drivers/iio/adc/ti-adc128s052.c +++ b/drivers/iio/adc/ti-adc128s052.c @@ -1,7 +1,7 @@ /* * Copyright (C) 2014 Angelo Compagnucci <angelo.compagnucci@gmail.com> * - * Driver for Texas Instruments' ADC128S052 ADC chip. + * Driver for Texas Instruments' ADC128S052 and ADC122S021 ADC chip. * Datasheet can be found here: * http://www.ti.com/lit/ds/symlink/adc128s052.pdf * @@ -16,6 +16,11 @@ #include <linux/iio/iio.h> #include <linux/regulator/consumer.h> +struct adc128_configuration { + const struct iio_chan_spec *channels; + u8 num_channels; +}; + struct adc128 { struct spi_device *spi; @@ -92,7 +97,7 @@ static int adc128_read_raw(struct iio_dev *indio_dev, .info_mask_shared_by_type = BIT(IIO_CHAN_INFO_SCALE) \ } -static const struct iio_chan_spec adc128_channels[] = { +static const struct iio_chan_spec adc128s052_channels[] = { ADC128_VOLTAGE_CHANNEL(0), ADC128_VOLTAGE_CHANNEL(1), ADC128_VOLTAGE_CHANNEL(2), @@ -103,6 +108,16 @@ static const struct iio_chan_spec adc128_channels[] = { ADC128_VOLTAGE_CHANNEL(7), }; +static const struct iio_chan_spec adc122s021_channels[] = { + ADC128_VOLTAGE_CHANNEL(0), + ADC128_VOLTAGE_CHANNEL(1), +}; + +static const struct adc128_configuration adc128_config[] = { + { adc128s052_channels, ARRAY_SIZE(adc128s052_channels) }, + { adc122s021_channels, ARRAY_SIZE(adc122s021_channels) }, +}; + static const struct iio_info adc128_info = { .read_raw = adc128_read_raw, .driver_module = THIS_MODULE, @@ -112,6 +127,7 @@ static int adc128_probe(struct spi_device *spi) { struct iio_dev *indio_dev; struct adc128 *adc; + int config = spi_get_device_id(spi)->driver_data; int ret; indio_dev = devm_iio_device_alloc(&spi->dev, sizeof(*adc)); @@ -128,8 +144,8 @@ static int adc128_probe(struct spi_device *spi) indio_dev->modes = INDIO_DIRECT_MODE; indio_dev->info = &adc128_info; - indio_dev->channels = adc128_channels; - indio_dev->num_channels = ARRAY_SIZE(adc128_channels); + indio_dev->channels = adc128_config[config].channels; + indio_dev->num_channels = adc128_config[config].num_channels; adc->reg = devm_regulator_get(&spi->dev, "vref"); if (IS_ERR(adc->reg)) @@ -158,7 +174,8 @@ static int adc128_remove(struct spi_device *spi) } static const struct spi_device_id adc128_id[] = { - { "adc128s052", 0}, + { "adc128s052", 0}, /* index into adc128_config */ + { "adc122s021", 1}, { } }; MODULE_DEVICE_TABLE(spi, adc128_id); -- 2.1.4 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] iio: adc128s052: add support for adc122s021 2015-05-18 12:04 ` Urs Fässler @ 2015-05-18 12:19 ` Lars-Peter Clausen 2015-05-18 12:40 ` Peter Meerwald 1 sibling, 0 replies; 6+ messages in thread From: Lars-Peter Clausen @ 2015-05-18 12:19 UTC (permalink / raw) To: Urs Fässler, jic23, angelo.compagnucci Cc: iivanov, ezequiel.garcia, jacob.jun.pan, devicetree, linux-kernel, linux-iio > --- a/Documentation/devicetree/bindings/iio/adc/ti-adc128s052.txt > +++ b/Documentation/devicetree/bindings/iio/adc/ti-adc128s052.txt > @@ -1,7 +1,7 @@ > -* Texas Instruments' ADC128S052 ADC chip > +* Texas Instruments' ADC128S052 and ADC122S021 ADC chip > > Required properties: > - - compatible: Should be "ti,adc128s052" > + - compatible: Should be "adc128s052" or "adc122s021" Why are you removing the vendor prefix? The compatible string is not valid without a vendor prefix. > - reg: spi chip select number for the device > - vref-supply: The regulator supply for ADC reference voltage > > @@ -11,7 +11,7 @@ Recommended properties: > > Example: > adc@0 { > - compatible = "ti,adc128s052"; > + compatible = "adc128s052"; Indention. ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] iio: adc128s052: add support for adc122s021 2015-05-18 12:04 ` Urs Fässler 2015-05-18 12:19 ` Lars-Peter Clausen @ 2015-05-18 12:40 ` Peter Meerwald 1 sibling, 0 replies; 6+ messages in thread From: Peter Meerwald @ 2015-05-18 12:40 UTC (permalink / raw) To: Urs Fässler; +Cc: jic23, angelo.compagnucci, linux-iio [-- Attachment #1: Type: TEXT/PLAIN, Size: 4755 bytes --] looks good, maybe add the link to the datasheet: http://www.ti.com/lit/ds/symlink/adc122s021.pdf why do you drop the "ti," in the DT compatible? > Signed-off-by: Urs Fässler <urs.fassler@bytesatwork.ch> > --- > .../devicetree/bindings/iio/adc/ti-adc128s052.txt | 6 ++--- > drivers/iio/adc/Kconfig | 4 ++-- > drivers/iio/adc/ti-adc128s052.c | 27 ++++++++++++++++++---- > 3 files changed, 27 insertions(+), 10 deletions(-) > > diff --git a/Documentation/devicetree/bindings/iio/adc/ti-adc128s052.txt b/Documentation/devicetree/bindings/iio/adc/ti-adc128s052.txt > index 42ca7de..7cefcf7 100644 > --- a/Documentation/devicetree/bindings/iio/adc/ti-adc128s052.txt > +++ b/Documentation/devicetree/bindings/iio/adc/ti-adc128s052.txt > @@ -1,7 +1,7 @@ > -* Texas Instruments' ADC128S052 ADC chip > +* Texas Instruments' ADC128S052 and ADC122S021 ADC chip > > Required properties: > - - compatible: Should be "ti,adc128s052" > + - compatible: Should be "adc128s052" or "adc122s021" > - reg: spi chip select number for the device > - vref-supply: The regulator supply for ADC reference voltage > > @@ -11,7 +11,7 @@ Recommended properties: > > Example: > adc@0 { > - compatible = "ti,adc128s052"; > + compatible = "adc128s052"; > reg = <0>; > vref-supply = <&vdd_supply>; > spi-max-frequency = <1000000>; > diff --git a/drivers/iio/adc/Kconfig b/drivers/iio/adc/Kconfig > index e36a73e..92c319c 100644 > --- a/drivers/iio/adc/Kconfig > +++ b/drivers/iio/adc/Kconfig > @@ -285,11 +285,11 @@ config TI_ADC081C > called ti-adc081c. > > config TI_ADC128S052 > - tristate "Texas Instruments ADC128S052" > + tristate "Texas Instruments ADC128S052/ADC122S021" > depends on SPI > help > If you say yes here you get support for Texas Instruments ADC128S052 > - chip. > + and ADC122S021 chip. > > This driver can also be built as a module. If so, the module will be > called ti-adc128s052. > diff --git a/drivers/iio/adc/ti-adc128s052.c b/drivers/iio/adc/ti-adc128s052.c > index 655cb56..42c7772 100644 > --- a/drivers/iio/adc/ti-adc128s052.c > +++ b/drivers/iio/adc/ti-adc128s052.c > @@ -1,7 +1,7 @@ > /* > * Copyright (C) 2014 Angelo Compagnucci <angelo.compagnucci@gmail.com> > * > - * Driver for Texas Instruments' ADC128S052 ADC chip. > + * Driver for Texas Instruments' ADC128S052 and ADC122S021 ADC chip. > * Datasheet can be found here: > * http://www.ti.com/lit/ds/symlink/adc128s052.pdf > * > @@ -16,6 +16,11 @@ > #include <linux/iio/iio.h> > #include <linux/regulator/consumer.h> > > +struct adc128_configuration { > + const struct iio_chan_spec *channels; > + u8 num_channels; > +}; > + > struct adc128 { > struct spi_device *spi; > > @@ -92,7 +97,7 @@ static int adc128_read_raw(struct iio_dev *indio_dev, > .info_mask_shared_by_type = BIT(IIO_CHAN_INFO_SCALE) \ > } > > -static const struct iio_chan_spec adc128_channels[] = { > +static const struct iio_chan_spec adc128s052_channels[] = { > ADC128_VOLTAGE_CHANNEL(0), > ADC128_VOLTAGE_CHANNEL(1), > ADC128_VOLTAGE_CHANNEL(2), > @@ -103,6 +108,16 @@ static const struct iio_chan_spec adc128_channels[] = { > ADC128_VOLTAGE_CHANNEL(7), > }; > > +static const struct iio_chan_spec adc122s021_channels[] = { > + ADC128_VOLTAGE_CHANNEL(0), > + ADC128_VOLTAGE_CHANNEL(1), > +}; > + > +static const struct adc128_configuration adc128_config[] = { > + { adc128s052_channels, ARRAY_SIZE(adc128s052_channels) }, > + { adc122s021_channels, ARRAY_SIZE(adc122s021_channels) }, > +}; > + > static const struct iio_info adc128_info = { > .read_raw = adc128_read_raw, > .driver_module = THIS_MODULE, > @@ -112,6 +127,7 @@ static int adc128_probe(struct spi_device *spi) > { > struct iio_dev *indio_dev; > struct adc128 *adc; > + int config = spi_get_device_id(spi)->driver_data; > int ret; > > indio_dev = devm_iio_device_alloc(&spi->dev, sizeof(*adc)); > @@ -128,8 +144,8 @@ static int adc128_probe(struct spi_device *spi) > indio_dev->modes = INDIO_DIRECT_MODE; > indio_dev->info = &adc128_info; > > - indio_dev->channels = adc128_channels; > - indio_dev->num_channels = ARRAY_SIZE(adc128_channels); > + indio_dev->channels = adc128_config[config].channels; > + indio_dev->num_channels = adc128_config[config].num_channels; > > adc->reg = devm_regulator_get(&spi->dev, "vref"); > if (IS_ERR(adc->reg)) > @@ -158,7 +174,8 @@ static int adc128_remove(struct spi_device *spi) > } > > static const struct spi_device_id adc128_id[] = { > - { "adc128s052", 0}, > + { "adc128s052", 0}, /* index into adc128_config */ > + { "adc122s021", 1}, > { } > }; > MODULE_DEVICE_TABLE(spi, adc128_id); > -- Peter Meerwald +43-664-2444418 (mobile) ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2] iio: adc128s052: add support for adc122s021 @ 2015-05-18 13:22 Urs Fässler 2015-05-18 13:22 ` [PATCH] " Urs Fässler 0 siblings, 1 reply; 6+ messages in thread From: Urs Fässler @ 2015-05-18 13:22 UTC (permalink / raw) To: jic23, angelo.compagnucci, urs.fassler Cc: iivanov, ezequiel.garcia, jacob.jun.pan, devicetree, linux-kernel, linux-iio Changes in version 2: - leave vendor prefix in compatible string of device tree (and rendering indention problem obsolete) - add link to datasheet Urs Fässler (1): iio: adc128s052: add support for adc122s021 .../devicetree/bindings/iio/adc/ti-adc128s052.txt | 4 +-- drivers/iio/adc/Kconfig | 4 +-- drivers/iio/adc/ti-adc128s052.c | 30 +++++++++++++++++----- 3 files changed, 28 insertions(+), 10 deletions(-) -- 2.1.4 ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] iio: adc128s052: add support for adc122s021 2015-05-18 13:22 [PATCH v2] " Urs Fässler @ 2015-05-18 13:22 ` Urs Fässler 2015-05-23 11:43 ` Jonathan Cameron 0 siblings, 1 reply; 6+ messages in thread From: Urs Fässler @ 2015-05-18 13:22 UTC (permalink / raw) To: jic23, angelo.compagnucci, urs.fassler Cc: iivanov, ezequiel.garcia, jacob.jun.pan, devicetree, linux-kernel, linux-iio Signed-off-by: Urs Fässler <urs.fassler@bytesatwork.ch> --- .../devicetree/bindings/iio/adc/ti-adc128s052.txt | 4 +-- drivers/iio/adc/Kconfig | 4 +-- drivers/iio/adc/ti-adc128s052.c | 30 +++++++++++++++++----- 3 files changed, 28 insertions(+), 10 deletions(-) diff --git a/Documentation/devicetree/bindings/iio/adc/ti-adc128s052.txt b/Documentation/devicetree/bindings/iio/adc/ti-adc128s052.txt index 42ca7de..15ca6b4 100644 --- a/Documentation/devicetree/bindings/iio/adc/ti-adc128s052.txt +++ b/Documentation/devicetree/bindings/iio/adc/ti-adc128s052.txt @@ -1,7 +1,7 @@ -* Texas Instruments' ADC128S052 ADC chip +* Texas Instruments' ADC128S052 and ADC122S021 ADC chip Required properties: - - compatible: Should be "ti,adc128s052" + - compatible: Should be "ti,adc128s052" or "ti,adc122s021" - reg: spi chip select number for the device - vref-supply: The regulator supply for ADC reference voltage diff --git a/drivers/iio/adc/Kconfig b/drivers/iio/adc/Kconfig index e36a73e..92c319c 100644 --- a/drivers/iio/adc/Kconfig +++ b/drivers/iio/adc/Kconfig @@ -285,11 +285,11 @@ config TI_ADC081C called ti-adc081c. config TI_ADC128S052 - tristate "Texas Instruments ADC128S052" + tristate "Texas Instruments ADC128S052/ADC122S021" depends on SPI help If you say yes here you get support for Texas Instruments ADC128S052 - chip. + and ADC122S021 chip. This driver can also be built as a module. If so, the module will be called ti-adc128s052. diff --git a/drivers/iio/adc/ti-adc128s052.c b/drivers/iio/adc/ti-adc128s052.c index 655cb56..915be6b 100644 --- a/drivers/iio/adc/ti-adc128s052.c +++ b/drivers/iio/adc/ti-adc128s052.c @@ -1,9 +1,10 @@ /* * Copyright (C) 2014 Angelo Compagnucci <angelo.compagnucci@gmail.com> * - * Driver for Texas Instruments' ADC128S052 ADC chip. - * Datasheet can be found here: + * Driver for Texas Instruments' ADC128S052 and ADC122S021 ADC chip. + * Datasheets can be found here: * http://www.ti.com/lit/ds/symlink/adc128s052.pdf + * http://www.ti.com/lit/ds/symlink/adc122s021.pdf * * This program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License version 2 as @@ -16,6 +17,11 @@ #include <linux/iio/iio.h> #include <linux/regulator/consumer.h> +struct adc128_configuration { + const struct iio_chan_spec *channels; + u8 num_channels; +}; + struct adc128 { struct spi_device *spi; @@ -92,7 +98,7 @@ static int adc128_read_raw(struct iio_dev *indio_dev, .info_mask_shared_by_type = BIT(IIO_CHAN_INFO_SCALE) \ } -static const struct iio_chan_spec adc128_channels[] = { +static const struct iio_chan_spec adc128s052_channels[] = { ADC128_VOLTAGE_CHANNEL(0), ADC128_VOLTAGE_CHANNEL(1), ADC128_VOLTAGE_CHANNEL(2), @@ -103,6 +109,16 @@ static const struct iio_chan_spec adc128_channels[] = { ADC128_VOLTAGE_CHANNEL(7), }; +static const struct iio_chan_spec adc122s021_channels[] = { + ADC128_VOLTAGE_CHANNEL(0), + ADC128_VOLTAGE_CHANNEL(1), +}; + +static const struct adc128_configuration adc128_config[] = { + { adc128s052_channels, ARRAY_SIZE(adc128s052_channels) }, + { adc122s021_channels, ARRAY_SIZE(adc122s021_channels) }, +}; + static const struct iio_info adc128_info = { .read_raw = adc128_read_raw, .driver_module = THIS_MODULE, @@ -112,6 +128,7 @@ static int adc128_probe(struct spi_device *spi) { struct iio_dev *indio_dev; struct adc128 *adc; + int config = spi_get_device_id(spi)->driver_data; int ret; indio_dev = devm_iio_device_alloc(&spi->dev, sizeof(*adc)); @@ -128,8 +145,8 @@ static int adc128_probe(struct spi_device *spi) indio_dev->modes = INDIO_DIRECT_MODE; indio_dev->info = &adc128_info; - indio_dev->channels = adc128_channels; - indio_dev->num_channels = ARRAY_SIZE(adc128_channels); + indio_dev->channels = adc128_config[config].channels; + indio_dev->num_channels = adc128_config[config].num_channels; adc->reg = devm_regulator_get(&spi->dev, "vref"); if (IS_ERR(adc->reg)) @@ -158,7 +175,8 @@ static int adc128_remove(struct spi_device *spi) } static const struct spi_device_id adc128_id[] = { - { "adc128s052", 0}, + { "adc128s052", 0}, /* index into adc128_config */ + { "adc122s021", 1}, { } }; MODULE_DEVICE_TABLE(spi, adc128_id); -- 2.1.4 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] iio: adc128s052: add support for adc122s021 2015-05-18 13:22 ` [PATCH] " Urs Fässler @ 2015-05-23 11:43 ` Jonathan Cameron 0 siblings, 0 replies; 6+ messages in thread From: Jonathan Cameron @ 2015-05-23 11:43 UTC (permalink / raw) To: Urs Fässler, angelo.compagnucci Cc: iivanov, ezequiel.garcia, jacob.jun.pan, devicetree, linux-kernel, linux-iio On 18/05/15 14:22, Urs Fässler wrote: > Signed-off-by: Urs Fässler <urs.fassler@bytesatwork.ch> looks good. Applied to the togreg branch of iio.git with the chip->chips change. Thanks, Jonathan > --- > .../devicetree/bindings/iio/adc/ti-adc128s052.txt | 4 +-- > drivers/iio/adc/Kconfig | 4 +-- > drivers/iio/adc/ti-adc128s052.c | 30 +++++++++++++++++----- > 3 files changed, 28 insertions(+), 10 deletions(-) > > diff --git a/Documentation/devicetree/bindings/iio/adc/ti-adc128s052.txt b/Documentation/devicetree/bindings/iio/adc/ti-adc128s052.txt > index 42ca7de..15ca6b4 100644 > --- a/Documentation/devicetree/bindings/iio/adc/ti-adc128s052.txt > +++ b/Documentation/devicetree/bindings/iio/adc/ti-adc128s052.txt > @@ -1,7 +1,7 @@ > -* Texas Instruments' ADC128S052 ADC chip > +* Texas Instruments' ADC128S052 and ADC122S021 ADC chip > > Required properties: > - - compatible: Should be "ti,adc128s052" > + - compatible: Should be "ti,adc128s052" or "ti,adc122s021" > - reg: spi chip select number for the device > - vref-supply: The regulator supply for ADC reference voltage > > diff --git a/drivers/iio/adc/Kconfig b/drivers/iio/adc/Kconfig > index e36a73e..92c319c 100644 > --- a/drivers/iio/adc/Kconfig > +++ b/drivers/iio/adc/Kconfig > @@ -285,11 +285,11 @@ config TI_ADC081C > called ti-adc081c. > > config TI_ADC128S052 > - tristate "Texas Instruments ADC128S052" > + tristate "Texas Instruments ADC128S052/ADC122S021" > depends on SPI > help > If you say yes here you get support for Texas Instruments ADC128S052 > - chip. > + and ADC122S021 chip. chips ;) > > This driver can also be built as a module. If so, the module will be > called ti-adc128s052. > diff --git a/drivers/iio/adc/ti-adc128s052.c b/drivers/iio/adc/ti-adc128s052.c > index 655cb56..915be6b 100644 > --- a/drivers/iio/adc/ti-adc128s052.c > +++ b/drivers/iio/adc/ti-adc128s052.c > @@ -1,9 +1,10 @@ > /* > * Copyright (C) 2014 Angelo Compagnucci <angelo.compagnucci@gmail.com> > * > - * Driver for Texas Instruments' ADC128S052 ADC chip. > - * Datasheet can be found here: > + * Driver for Texas Instruments' ADC128S052 and ADC122S021 ADC chip. > + * Datasheets can be found here: > * http://www.ti.com/lit/ds/symlink/adc128s052.pdf > + * http://www.ti.com/lit/ds/symlink/adc122s021.pdf > * > * This program is free software; you can redistribute it and/or modify > * it under the terms of the GNU General Public License version 2 as > @@ -16,6 +17,11 @@ > #include <linux/iio/iio.h> > #include <linux/regulator/consumer.h> > > +struct adc128_configuration { > + const struct iio_chan_spec *channels; > + u8 num_channels; > +}; > + > struct adc128 { > struct spi_device *spi; > > @@ -92,7 +98,7 @@ static int adc128_read_raw(struct iio_dev *indio_dev, > .info_mask_shared_by_type = BIT(IIO_CHAN_INFO_SCALE) \ > } > > -static const struct iio_chan_spec adc128_channels[] = { > +static const struct iio_chan_spec adc128s052_channels[] = { > ADC128_VOLTAGE_CHANNEL(0), > ADC128_VOLTAGE_CHANNEL(1), > ADC128_VOLTAGE_CHANNEL(2), > @@ -103,6 +109,16 @@ static const struct iio_chan_spec adc128_channels[] = { > ADC128_VOLTAGE_CHANNEL(7), > }; > > +static const struct iio_chan_spec adc122s021_channels[] = { > + ADC128_VOLTAGE_CHANNEL(0), > + ADC128_VOLTAGE_CHANNEL(1), > +}; > + > +static const struct adc128_configuration adc128_config[] = { > + { adc128s052_channels, ARRAY_SIZE(adc128s052_channels) }, > + { adc122s021_channels, ARRAY_SIZE(adc122s021_channels) }, > +}; This is a good flexible solution as opposed to doing it via a switch in the probe function which would have been shorter, but not allowed for as easy an extension to moer parts in future. Looks like there are lots of fairly similar parts! > + > static const struct iio_info adc128_info = { > .read_raw = adc128_read_raw, > .driver_module = THIS_MODULE, > @@ -112,6 +128,7 @@ static int adc128_probe(struct spi_device *spi) > { > struct iio_dev *indio_dev; > struct adc128 *adc; > + int config = spi_get_device_id(spi)->driver_data; > int ret; > > indio_dev = devm_iio_device_alloc(&spi->dev, sizeof(*adc)); > @@ -128,8 +145,8 @@ static int adc128_probe(struct spi_device *spi) > indio_dev->modes = INDIO_DIRECT_MODE; > indio_dev->info = &adc128_info; > > - indio_dev->channels = adc128_channels; > - indio_dev->num_channels = ARRAY_SIZE(adc128_channels); > + indio_dev->channels = adc128_config[config].channels; > + indio_dev->num_channels = adc128_config[config].num_channels; > > adc->reg = devm_regulator_get(&spi->dev, "vref"); > if (IS_ERR(adc->reg)) > @@ -158,7 +175,8 @@ static int adc128_remove(struct spi_device *spi) > } > > static const struct spi_device_id adc128_id[] = { > - { "adc128s052", 0}, > + { "adc128s052", 0}, /* index into adc128_config */ > + { "adc122s021", 1}, > { } > }; > MODULE_DEVICE_TABLE(spi, adc128_id); > ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-05-23 11:43 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-05-18 12:04 [PATCH] iio: adc128s052: add support for adc122s021 Urs Fässler 2015-05-18 12:04 ` Urs Fässler 2015-05-18 12:19 ` Lars-Peter Clausen 2015-05-18 12:40 ` Peter Meerwald -- strict thread matches above, loose matches on Subject: below -- 2015-05-18 13:22 [PATCH v2] " Urs Fässler 2015-05-18 13:22 ` [PATCH] " Urs Fässler 2015-05-23 11:43 ` Jonathan Cameron
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).