On Tue, Jul 11, 2017 at 1:26 PM, Jonathan Cameron <jic23@xxxxxxxxxx> wrote: > On Mon, 10 Jul 2017 21:51:58 -0500 > Rob Herring <robh@xxxxxxxxxx> wrote: > >> On Sun, Jul 09, 2017 at 06:57:03PM +0200, Lorenzo Bianconi wrote: >> > Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi@xxxxxx> >> > --- >> > Documentation/devicetree/bindings/iio/humidity/hts221.txt | 8 ++++++++ >> > 1 file changed, 8 insertions(+) >> > >> > diff --git a/Documentation/devicetree/bindings/iio/humidity/hts221.txt b/Documentation/devicetree/bindings/iio/humidity/hts221.txt >> > index fca263e13400..732b83c06c08 100644 >> > --- a/Documentation/devicetree/bindings/iio/humidity/hts221.txt >> > +++ b/Documentation/devicetree/bindings/iio/humidity/hts221.txt >> > @@ -5,6 +5,14 @@ Required properties: >> > - reg: i2c address of the sensor / spi cs line >> > >> > Optional properties: >> > +- drive-open-drain: the interrupt/data ready line will be configured >> >> Needs a vendor prefix and "drdy-open-drain" would be more specific. > This is straight out of the pinctrl bindings and has been used in a few > other places as well. I can see it should really be more obviously > associated with the drdy pin though. A few IIO drivers already > use this binding (st sensors and the lsm6dsx). If we are going > to decide it's a bad idea we should deprecate those bindings and > add whatever we do here as well. Ah, okay. Then the description should have a "see pinctrl.txt" for the description. Rob -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html