On Thu, 26 Apr 2018 19:17:09 -0700 Matt Ranostay <matt.ranostay@xxxxxxxxxxxx> wrote: > On Thu, Apr 26, 2018 at 7:15 PM, Matt Ranostay > <matt.ranostay@xxxxxxxxxxxx> wrote: > > LMP91002 is register compatible so add devicetree and i2c client ids > > > > Cc: device-tree@xxxxxxxxxxxxxxx > > *sigh* should be devicetree@xxxxxxxxxxxxxxx ... adding correct CC > > > Signed-off-by: Matt Ranostay <matt.ranostay@xxxxxxxxxxxx> > > --- > > Documentation/devicetree/bindings/iio/potentiostat/lmp91000.txt | 7 ++++--- > > drivers/iio/potentiostat/lmp91000.c | 2 ++ > > 2 files changed, 6 insertions(+), 3 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/iio/potentiostat/lmp91000.txt b/Documentation/devicetree/bindings/iio/potentiostat/lmp91000.txt > > index b9b621e94cd7..d51639f0ebbf 100644 > > --- a/Documentation/devicetree/bindings/iio/potentiostat/lmp91000.txt > > +++ b/Documentation/devicetree/bindings/iio/potentiostat/lmp91000.txt > > @@ -1,10 +1,11 @@ > > -* 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 "ti,lmp91000" or "ti,lmp91002" Please put each entry on a new line. Reduces churn going forward. Thanks, Jonathan > > - 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); > > -- > > 2.14.1 > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-iio" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html