* [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; 3+ 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] 3+ messages in thread
* [PATCH] iio: adc128s052: add support for adc122s021 2015-05-18 13:22 [PATCH v2] iio: adc128s052: add support for adc122s021 Urs Fässler @ 2015-05-18 13:22 ` Urs Fässler 2015-05-23 11:43 ` Jonathan Cameron 0 siblings, 1 reply; 3+ 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] 3+ 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; 3+ 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] 3+ messages in thread
end of thread, other threads:[~2015-05-23 11:43 UTC | newest] Thread overview: 3+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-05-18 13:22 [PATCH v2] iio: adc128s052: add support for adc122s021 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).