* [PATCH 1/2] dt-bindings: iio: chemical: Document senseair,sunrise CO2 sensor [not found] <20210817154951.50208-1-jacopo@jmondi.org> @ 2021-08-17 15:49 ` Jacopo Mondi 2021-08-17 16:50 ` Andy Shevchenko 2021-08-18 16:49 ` Rob Herring 0 siblings, 2 replies; 5+ messages in thread From: Jacopo Mondi @ 2021-08-17 15:49 UTC (permalink / raw) To: Jonathan Cameron, Lars-Peter Clausen, Andy Shevchenko, Matt Ranostay, Rob Herring Cc: Jacopo Mondi, linux-iio, devicetree Add documentation for the Senseair Sunrise 006-0-0007 CO2 NDIR sensor. Signed-off-by: Jacopo Mondi <jacopo@jmondi.org> --- .../iio/chemical/senseair,sunrise.yaml | 51 +++++++++++++++++++ .../devicetree/bindings/vendor-prefixes.yaml | 2 + 2 files changed, 53 insertions(+) create mode 100644 Documentation/devicetree/bindings/iio/chemical/senseair,sunrise.yaml diff --git a/Documentation/devicetree/bindings/iio/chemical/senseair,sunrise.yaml b/Documentation/devicetree/bindings/iio/chemical/senseair,sunrise.yaml new file mode 100644 index 000000000000..b77196666187 --- /dev/null +++ b/Documentation/devicetree/bindings/iio/chemical/senseair,sunrise.yaml @@ -0,0 +1,51 @@ +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/iio/chemical/senseair,sunrise.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Senseair Sunrise 006-0-0007 CO2 Sensor + +maintainers: + - Jacopo Mondi <jacopo@jmondi.org> + +description: | + Senseair Sunrise 006-0-0007 is a NDIR CO2 sensor. It supports I2C or UART buses + for communications and control. + + Datasheets: + https://rmtplusstoragesenseair.blob.core.windows.net/docs/Dev/publicerat/PSP11704.pdf + https://rmtplusstoragesenseair.blob.core.windows.net/docs/Dev/publicerat/PSH11649.pdf + https://rmtplusstoragesenseair.blob.core.windows.net/docs/Dev/publicerat/TDE5531.pdf + https://rmtplusstoragesenseair.blob.core.windows.net/docs/Market/publicerat/TDE7318.pdf + +properties: + compatible: + const: senseair,sunrise-006-0-0007 + + reg: + maxItems: 1 + + ndry-gpios: + description: Phandle to the GPIO line connected to the nDRY pin. Active low. + + en-gpios: + description: Phandle to the GPIO line connected to the EN pin. Active high. + +required: + - compatible + - reg + +additionalProperties: false + +examples: + - | + i2c { + #address-cells = <1>; + #size-cells = <0>; + + sunrise@68 { + compatible = "senseair,sunrise-006-0-0007"; + reg = <0x68>; + }; + }; diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml index 944a14926e02..c60502eb3d36 100644 --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml @@ -1000,6 +1000,8 @@ patternProperties: description: Shenzhen SEI Robotics Co., Ltd "^semtech,.*": description: Semtech Corporation + "^senseair,.*": + description: Senseair AB "^sensirion,.*": description: Sensirion AG "^sensortek,.*": -- 2.32.0 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 1/2] dt-bindings: iio: chemical: Document senseair,sunrise CO2 sensor 2021-08-17 15:49 ` [PATCH 1/2] dt-bindings: iio: chemical: Document senseair,sunrise CO2 sensor Jacopo Mondi @ 2021-08-17 16:50 ` Andy Shevchenko 2021-08-18 7:29 ` Jacopo Mondi 2021-08-18 16:49 ` Rob Herring 1 sibling, 1 reply; 5+ messages in thread From: Andy Shevchenko @ 2021-08-17 16:50 UTC (permalink / raw) To: Jacopo Mondi Cc: Jonathan Cameron, Lars-Peter Clausen, Matt Ranostay, Rob Herring, linux-iio, devicetree On Tue, Aug 17, 2021 at 05:49:50PM +0200, Jacopo Mondi wrote: > Add documentation for the Senseair Sunrise 006-0-0007 CO2 NDIR sensor. > + ndry-gpios: > + description: Phandle to the GPIO line connected to the nDRY pin. Active low. > + > + en-gpios: > + description: Phandle to the GPIO line connected to the EN pin. Active high. Not sure you have to mention polarity. It can be changed on PCB level easily and this bindings won't satisfy those (valid) changes. -- With Best Regards, Andy Shevchenko ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/2] dt-bindings: iio: chemical: Document senseair,sunrise CO2 sensor 2021-08-17 16:50 ` Andy Shevchenko @ 2021-08-18 7:29 ` Jacopo Mondi 2021-08-18 16:47 ` Rob Herring 0 siblings, 1 reply; 5+ messages in thread From: Jacopo Mondi @ 2021-08-18 7:29 UTC (permalink / raw) To: Andy Shevchenko Cc: Jonathan Cameron, Lars-Peter Clausen, Matt Ranostay, Rob Herring, linux-iio, devicetree Hi Andy, On Tue, Aug 17, 2021 at 07:50:46PM +0300, Andy Shevchenko wrote: > On Tue, Aug 17, 2021 at 05:49:50PM +0200, Jacopo Mondi wrote: > > Add documentation for the Senseair Sunrise 006-0-0007 CO2 NDIR sensor. > > > + ndry-gpios: > > + description: Phandle to the GPIO line connected to the nDRY pin. Active low. > > + > > + en-gpios: > > + description: Phandle to the GPIO line connected to the EN pin. Active high. > > Not sure you have to mention polarity. It can be changed on PCB level easily > and this bindings won't satisfy those (valid) changes. Well, one can indeed invert the signal on the PCB, it's weird I see most of the bindings reporting the pin's active polarity though... I'll drop it. Thanks j > > -- > With Best Regards, > Andy Shevchenko > > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/2] dt-bindings: iio: chemical: Document senseair,sunrise CO2 sensor 2021-08-18 7:29 ` Jacopo Mondi @ 2021-08-18 16:47 ` Rob Herring 0 siblings, 0 replies; 5+ messages in thread From: Rob Herring @ 2021-08-18 16:47 UTC (permalink / raw) To: Jacopo Mondi Cc: Andy Shevchenko, Jonathan Cameron, Lars-Peter Clausen, Matt Ranostay, linux-iio, devicetree On Wed, Aug 18, 2021 at 09:29:43AM +0200, Jacopo Mondi wrote: > Hi Andy, > > On Tue, Aug 17, 2021 at 07:50:46PM +0300, Andy Shevchenko wrote: > > On Tue, Aug 17, 2021 at 05:49:50PM +0200, Jacopo Mondi wrote: > > > Add documentation for the Senseair Sunrise 006-0-0007 CO2 NDIR sensor. > > > > > + ndry-gpios: > > > + description: Phandle to the GPIO line connected to the nDRY pin. Active low. > > > + > > > + en-gpios: > > > + description: Phandle to the GPIO line connected to the EN pin. Active high. > > > > Not sure you have to mention polarity. It can be changed on PCB level easily > > and this bindings won't satisfy those (valid) changes. > > Well, one can indeed invert the signal on the PCB, it's weird I see most of > the bindings reporting the pin's active polarity though... > > I'll drop it. I'd keep it. It's documenting the pin on the h/w. You can prefix with 'Typically' if you want. Rob ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/2] dt-bindings: iio: chemical: Document senseair,sunrise CO2 sensor 2021-08-17 15:49 ` [PATCH 1/2] dt-bindings: iio: chemical: Document senseair,sunrise CO2 sensor Jacopo Mondi 2021-08-17 16:50 ` Andy Shevchenko @ 2021-08-18 16:49 ` Rob Herring 1 sibling, 0 replies; 5+ messages in thread From: Rob Herring @ 2021-08-18 16:49 UTC (permalink / raw) To: Jacopo Mondi Cc: Jonathan Cameron, Lars-Peter Clausen, Andy Shevchenko, Matt Ranostay, linux-iio, devicetree On Tue, Aug 17, 2021 at 05:49:50PM +0200, Jacopo Mondi wrote: > Add documentation for the Senseair Sunrise 006-0-0007 CO2 NDIR sensor. > > Signed-off-by: Jacopo Mondi <jacopo@jmondi.org> > --- > .../iio/chemical/senseair,sunrise.yaml | 51 +++++++++++++++++++ > .../devicetree/bindings/vendor-prefixes.yaml | 2 + > 2 files changed, 53 insertions(+) > create mode 100644 Documentation/devicetree/bindings/iio/chemical/senseair,sunrise.yaml > > diff --git a/Documentation/devicetree/bindings/iio/chemical/senseair,sunrise.yaml b/Documentation/devicetree/bindings/iio/chemical/senseair,sunrise.yaml > new file mode 100644 > index 000000000000..b77196666187 > --- /dev/null > +++ b/Documentation/devicetree/bindings/iio/chemical/senseair,sunrise.yaml > @@ -0,0 +1,51 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/iio/chemical/senseair,sunrise.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Senseair Sunrise 006-0-0007 CO2 Sensor > + > +maintainers: > + - Jacopo Mondi <jacopo@jmondi.org> > + > +description: | > + Senseair Sunrise 006-0-0007 is a NDIR CO2 sensor. It supports I2C or UART buses > + for communications and control. > + > + Datasheets: > + https://rmtplusstoragesenseair.blob.core.windows.net/docs/Dev/publicerat/PSP11704.pdf > + https://rmtplusstoragesenseair.blob.core.windows.net/docs/Dev/publicerat/PSH11649.pdf > + https://rmtplusstoragesenseair.blob.core.windows.net/docs/Dev/publicerat/TDE5531.pdf > + https://rmtplusstoragesenseair.blob.core.windows.net/docs/Market/publicerat/TDE7318.pdf > + > +properties: > + compatible: > + const: senseair,sunrise-006-0-0007 > + > + reg: > + maxItems: 1 > + > + ndry-gpios: > + description: Phandle to the GPIO line connected to the nDRY pin. Active low. > + > + en-gpios: > + description: Phandle to the GPIO line connected to the EN pin. Active high. You need to define how many (maxItems: 1). > + > +required: > + - compatible > + - reg > + > +additionalProperties: false > + > +examples: > + - | > + i2c { > + #address-cells = <1>; > + #size-cells = <0>; > + > + sunrise@68 { 'co2-sensor' perhaps. > + compatible = "senseair,sunrise-006-0-0007"; > + reg = <0x68>; > + }; > + }; > diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml > index 944a14926e02..c60502eb3d36 100644 > --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml > +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml > @@ -1000,6 +1000,8 @@ patternProperties: > description: Shenzhen SEI Robotics Co., Ltd > "^semtech,.*": > description: Semtech Corporation > + "^senseair,.*": > + description: Senseair AB > "^sensirion,.*": > description: Sensirion AG > "^sensortek,.*": > -- > 2.32.0 > > ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-08-18 16:49 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <20210817154951.50208-1-jacopo@jmondi.org> 2021-08-17 15:49 ` [PATCH 1/2] dt-bindings: iio: chemical: Document senseair,sunrise CO2 sensor Jacopo Mondi 2021-08-17 16:50 ` Andy Shevchenko 2021-08-18 7:29 ` Jacopo Mondi 2021-08-18 16:47 ` Rob Herring 2021-08-18 16:49 ` Rob Herring
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).