[v2] iio: potentiostat: lmp91000: add LMP91002 support
diff mbox

Message ID 20180508053401.32485-1-matt.ranostay@konsulko.com
State New
Headers show

Commit Message

Matt Ranostay May 8, 2018, 5:34 a.m. UTC
LMP91002 is register compatible so add devicetree and i2c client ids

Cc: devicetree@vger.kernel.org
Signed-off-by: Matt Ranostay <matt.ranostay@konsulko.com>
---

Changes from v1:
* Split out compatible options in multiple lines for the lmp91000.txt documentation

 Documentation/devicetree/bindings/iio/potentiostat/lmp91000.txt | 9 ++++++---
 drivers/iio/potentiostat/lmp91000.c                             | 2 ++
 2 files changed, 8 insertions(+), 3 deletions(-)

Comments

Rob Herring May 8, 2018, 3:42 p.m. UTC | #1
On Mon, May 07, 2018 at 10:34:01PM -0700, Matt Ranostay wrote:
> LMP91002 is register compatible so add devicetree and i2c client ids
> 
> Cc: devicetree@vger.kernel.org
> Signed-off-by: Matt Ranostay <matt.ranostay@konsulko.com>
> ---
> 
> Changes from v1:
> * Split out compatible options in multiple lines for the lmp91000.txt documentation
> 
>  Documentation/devicetree/bindings/iio/potentiostat/lmp91000.txt | 9 ++++++---
>  drivers/iio/potentiostat/lmp91000.c                             | 2 ++
>  2 files changed, 8 insertions(+), 3 deletions(-)

Reviewed-by: Rob Herring <robh@kernel.org>
--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Jonathan Cameron May 12, 2018, 10:03 a.m. UTC | #2
On Tue, 8 May 2018 10:42:56 -0500
Rob Herring <robh@kernel.org> wrote:

> On Mon, May 07, 2018 at 10:34:01PM -0700, Matt Ranostay wrote:
> > LMP91002 is register compatible so add devicetree and i2c client ids
> > 
> > Cc: devicetree@vger.kernel.org
> > Signed-off-by: Matt Ranostay <matt.ranostay@konsulko.com>
> > ---
> > 
> > Changes from v1:
> > * Split out compatible options in multiple lines for the lmp91000.txt documentation
> > 
> >  Documentation/devicetree/bindings/iio/potentiostat/lmp91000.txt | 9 ++++++---
> >  drivers/iio/potentiostat/lmp91000.c                             | 2 ++
> >  2 files changed, 8 insertions(+), 3 deletions(-)  
> 
> Reviewed-by: Rob Herring <robh@kernel.org>

Applied to the togreg branch of iio.git and pushed out as testing for the
autobuilders to play with it.

Thanks,

Jonathan

--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch
diff mbox

diff --git a/Documentation/devicetree/bindings/iio/potentiostat/lmp91000.txt b/Documentation/devicetree/bindings/iio/potentiostat/lmp91000.txt
index b9b621e94cd7..e6d0c2eb345c 100644
--- a/Documentation/devicetree/bindings/iio/potentiostat/lmp91000.txt
+++ b/Documentation/devicetree/bindings/iio/potentiostat/lmp91000.txt
@@ -1,10 +1,13 @@ 
-* Texas Instruments LMP91000 potentiostat
+* Texas Instruments LMP91000 series of potentiostats
 
-http://www.ti.com/lit/ds/symlink/lmp91000.pdf
+LMP91000: http://www.ti.com/lit/ds/symlink/lmp91000.pdf
+LMP91002: http://www.ti.com/lit/ds/symlink/lmp91002.pdf
 
 Required properties:
 
-  - compatible: should be "ti,lmp91000"
+  - compatible: should be one of the following:
+                 "ti,lmp91000"
+                 "ti,lmp91002"
   - reg: the I2C address of the device
   - io-channels: the phandle of the iio provider
 
diff --git a/drivers/iio/potentiostat/lmp91000.c b/drivers/iio/potentiostat/lmp91000.c
index 85714055cc74..90e895adf997 100644
--- a/drivers/iio/potentiostat/lmp91000.c
+++ b/drivers/iio/potentiostat/lmp91000.c
@@ -411,12 +411,14 @@  static int lmp91000_remove(struct i2c_client *client)
 
 static const struct of_device_id lmp91000_of_match[] = {
 	{ .compatible = "ti,lmp91000", },
+	{ .compatible = "ti,lmp91002", },
 	{ },
 };
 MODULE_DEVICE_TABLE(of, lmp91000_of_match);
 
 static const struct i2c_device_id lmp91000_id[] = {
 	{ "lmp91000", 0 },
+	{ "lmp91002", 0 },
 	{}
 };
 MODULE_DEVICE_TABLE(i2c, lmp91000_id);