* [PATCH v2] dt-bindings: iio: maxbotix,mb1232.yaml: transform to yaml @ 2019-10-06 14:27 Andreas Klinger 2019-10-07 12:42 ` Rob Herring 0 siblings, 1 reply; 4+ messages in thread From: Andreas Klinger @ 2019-10-06 14:27 UTC (permalink / raw) To: robh+dt, jic23 Cc: knaack.h, lars, pmeerw, mark.rutland, linux-iio, devicetree, linux-kernel transform existing documentation of maxbotix,mb1232 ultrasonic ranger from text documentation format into yaml. Changes in v2: - removed description of reg property - added a line: additionalProperties: false Signed-off-by: Andreas Klinger <ak@it-klinger.de> --- .../bindings/iio/proximity/maxbotix,mb1232.txt | 29 ----------- .../bindings/iio/proximity/maxbotix,mb1232.yaml | 56 ++++++++++++++++++++++ 2 files changed, 56 insertions(+), 29 deletions(-) delete mode 100644 Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.txt create mode 100644 Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.yaml diff --git a/Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.txt b/Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.txt deleted file mode 100644 index dd1058fbe9c3..000000000000 --- a/Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.txt +++ /dev/null @@ -1,29 +0,0 @@ -* MaxBotix I2CXL-MaxSonar ultrasonic distance sensor of type mb1202, - mb1212, mb1222, mb1232, mb1242, mb7040 or mb7137 using the i2c interface - for ranging - -Required properties: - - compatible: "maxbotix,mb1202", - "maxbotix,mb1212", - "maxbotix,mb1222", - "maxbotix,mb1232", - "maxbotix,mb1242", - "maxbotix,mb7040" or - "maxbotix,mb7137" - - - reg: i2c address of the device, see also i2c/i2c.txt - -Optional properties: - - interrupts: Interrupt used to announce the preceding reading - request has finished and that data is available. - If no interrupt is specified the device driver - falls back to wait a fixed amount of time until - data can be retrieved. - -Example: -proximity@70 { - compatible = "maxbotix,mb1232"; - reg = <0x70>; - interrupt-parent = <&gpio2>; - interrupts = <2 IRQ_TYPE_EDGE_FALLING>; -}; diff --git a/Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.yaml b/Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.yaml new file mode 100644 index 000000000000..e2fb1f6d4dbe --- /dev/null +++ b/Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.yaml @@ -0,0 +1,56 @@ +# SPDX-License-Identifier: GPL-2.0 +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/iio/proximity/maxbotix,mb1232.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: MaxBotix I2CXL-MaxSonar ultrasonic distance sensor + +maintainers: + - Andreas Klinger <ak@it-klinger.de> + +description: | + MaxBotix I2CXL-MaxSonar ultrasonic distance sensor of type mb1202, + mb1212, mb1222, mb1232, mb1242, mb7040 or mb7137 using the i2c interface + for ranging + + Specifications about the devices can be found at: + https://www.maxbotix.com/documents/I2CXL-MaxSonar-EZ_Datasheet.pdf + +properties: + compatible: + enum: + - maxbotix,mb1202 + - maxbotix,mb1212 + - maxbotix,mb1222 + - maxbotix,mb1232 + - maxbotix,mb1242 + - maxbotix,mb7040 + - maxbotix,mb7137 + + reg: + maxItems: 1 + + interrupts: + description: + Interrupt used to announce the preceding reading request has finished + and that data is available. If no interrupt is specified the device + driver falls back to wait a fixed amount of time until data can be + retrieved. + maxItems: 1 + +required: + - compatible + - reg + +additionalProperties: false + +examples: + - | + #include <dt-bindings/interrupt-controller/irq.h> + proximity@70 { + compatible = "maxbotix,mb1232"; + reg = <0x70>; + interrupt-parent = <&gpio2>; + interrupts = <2 IRQ_TYPE_EDGE_FALLING>; + }; -- 2.11.0 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2] dt-bindings: iio: maxbotix,mb1232.yaml: transform to yaml 2019-10-06 14:27 [PATCH v2] dt-bindings: iio: maxbotix,mb1232.yaml: transform to yaml Andreas Klinger @ 2019-10-07 12:42 ` Rob Herring 2019-10-07 17:00 ` Andreas Klinger 0 siblings, 1 reply; 4+ messages in thread From: Rob Herring @ 2019-10-07 12:42 UTC (permalink / raw) To: Andreas Klinger Cc: Jonathan Cameron, Hartmut Knaack, Lars-Peter Clausen, Peter Meerwald, Mark Rutland, open list:IIO SUBSYSTEM AND DRIVERS, devicetree, linux-kernel On Sun, Oct 6, 2019 at 9:27 AM Andreas Klinger <ak@it-klinger.de> wrote: > > transform existing documentation of maxbotix,mb1232 ultrasonic ranger > from text documentation format into yaml. > > Changes in v2: > - removed description of reg property > - added a line: > additionalProperties: false > > Signed-off-by: Andreas Klinger <ak@it-klinger.de> > --- > .../bindings/iio/proximity/maxbotix,mb1232.txt | 29 ----------- > .../bindings/iio/proximity/maxbotix,mb1232.yaml | 56 ++++++++++++++++++++++ > 2 files changed, 56 insertions(+), 29 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.txt > create mode 100644 Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.yaml > > diff --git a/Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.txt b/Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.txt > deleted file mode 100644 > index dd1058fbe9c3..000000000000 > --- a/Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.txt > +++ /dev/null > @@ -1,29 +0,0 @@ > -* MaxBotix I2CXL-MaxSonar ultrasonic distance sensor of type mb1202, > - mb1212, mb1222, mb1232, mb1242, mb7040 or mb7137 using the i2c interface > - for ranging > - > -Required properties: > - - compatible: "maxbotix,mb1202", > - "maxbotix,mb1212", > - "maxbotix,mb1222", > - "maxbotix,mb1232", > - "maxbotix,mb1242", > - "maxbotix,mb7040" or > - "maxbotix,mb7137" > - > - - reg: i2c address of the device, see also i2c/i2c.txt > - > -Optional properties: > - - interrupts: Interrupt used to announce the preceding reading > - request has finished and that data is available. > - If no interrupt is specified the device driver > - falls back to wait a fixed amount of time until > - data can be retrieved. > - > -Example: > -proximity@70 { > - compatible = "maxbotix,mb1232"; > - reg = <0x70>; > - interrupt-parent = <&gpio2>; > - interrupts = <2 IRQ_TYPE_EDGE_FALLING>; > -}; > diff --git a/Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.yaml b/Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.yaml > new file mode 100644 > index 000000000000..e2fb1f6d4dbe > --- /dev/null > +++ b/Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.yaml > @@ -0,0 +1,56 @@ > +# SPDX-License-Identifier: GPL-2.0 > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/iio/proximity/maxbotix,mb1232.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: MaxBotix I2CXL-MaxSonar ultrasonic distance sensor > + > +maintainers: > + - Andreas Klinger <ak@it-klinger.de> > + > +description: | > + MaxBotix I2CXL-MaxSonar ultrasonic distance sensor of type mb1202, > + mb1212, mb1222, mb1232, mb1242, mb7040 or mb7137 using the i2c interface > + for ranging > + > + Specifications about the devices can be found at: > + https://www.maxbotix.com/documents/I2CXL-MaxSonar-EZ_Datasheet.pdf > + > +properties: > + compatible: > + enum: > + - maxbotix,mb1202 > + - maxbotix,mb1212 > + - maxbotix,mb1222 > + - maxbotix,mb1232 > + - maxbotix,mb1242 > + - maxbotix,mb7040 > + - maxbotix,mb7137 > + > + reg: > + maxItems: 1 > + > + interrupts: > + description: > + Interrupt used to announce the preceding reading request has finished > + and that data is available. If no interrupt is specified the device > + driver falls back to wait a fixed amount of time until data can be > + retrieved. > + maxItems: 1 > + > +required: > + - compatible > + - reg > + > +additionalProperties: false > + > +examples: > + - | > + #include <dt-bindings/interrupt-controller/irq.h> > + proximity@70 { Fails to build with 'make dt_binding_check': Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.example.dts:20.11-24: Warning (reg_format): /example-0/proximity@70:reg: property has invalid length (4 bytes) (#address-cells == 1, #size-cells == 1) Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.example.dt.yaml: Warning (pci_device_bus_num): Failed prerequisite 'reg_format' Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.example.dt.yaml: Warning (i2c_bus_reg): Failed prerequisite 'reg_format' Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.example.dt.yaml: Warning (spi_bus_reg): Failed prerequisite 'reg_format' You have to put this under an i2c bus node. i2c { #address-cells = <1>; #size-cells = <0>; ... }; > + compatible = "maxbotix,mb1232"; > + reg = <0x70>; > + interrupt-parent = <&gpio2>; > + interrupts = <2 IRQ_TYPE_EDGE_FALLING>; > + }; > -- > 2.11.0 ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] dt-bindings: iio: maxbotix,mb1232.yaml: transform to yaml 2019-10-07 12:42 ` Rob Herring @ 2019-10-07 17:00 ` Andreas Klinger 2019-10-07 18:04 ` Rob Herring 0 siblings, 1 reply; 4+ messages in thread From: Andreas Klinger @ 2019-10-07 17:00 UTC (permalink / raw) To: Rob Herring Cc: Jonathan Cameron, Hartmut Knaack, Lars-Peter Clausen, Peter Meerwald, Mark Rutland, open list:IIO SUBSYSTEM AND DRIVERS, devicetree, linux-kernel Hi Rob, i don't get this error. Is there anything i'm doing wrong here? ak@arbad:/project/opt-sw/linux-robh$ make O=../build-wega-robh/ dt_binding_check make[1]: Verzeichnis „/project/opt-sw/build-wega-robh“ wird betreten SCHEMA Documentation/devicetree/bindings/processed-schema.yaml /project/opt-sw/linux-robh/Documentation/devicetree/bindings/net/adi,adin.yaml: ignoring, error in schema 'adi,fifo-depth-bits' warning: no schema found in file: /project/opt-sw/linux-robh/Documentation/devicetree/bindings/net/adi,adin.yaml /project/opt-sw/linux-robh/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml: ignoring, error in schema '0' warning: no schema found in file: /project/opt-sw/linux-robh/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml CHKDT Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.yaml CHKDT [...] But anyway i've fixed and send it out again. Andreas Rob Herring <robh+dt@kernel.org> schrieb am Mo, 07. Okt 07:42: > On Sun, Oct 6, 2019 at 9:27 AM Andreas Klinger <ak@it-klinger.de> wrote: > > > > transform existing documentation of maxbotix,mb1232 ultrasonic ranger > > from text documentation format into yaml. > > > > Changes in v2: > > - removed description of reg property > > - added a line: > > additionalProperties: false > > > > Signed-off-by: Andreas Klinger <ak@it-klinger.de> > > --- > > .../bindings/iio/proximity/maxbotix,mb1232.txt | 29 ----------- > > .../bindings/iio/proximity/maxbotix,mb1232.yaml | 56 ++++++++++++++++++++++ > > 2 files changed, 56 insertions(+), 29 deletions(-) > > delete mode 100644 Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.txt > > create mode 100644 Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.yaml > > > > diff --git a/Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.txt b/Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.txt > > deleted file mode 100644 > > index dd1058fbe9c3..000000000000 > > --- a/Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.txt > > +++ /dev/null > > @@ -1,29 +0,0 @@ > > -* MaxBotix I2CXL-MaxSonar ultrasonic distance sensor of type mb1202, > > - mb1212, mb1222, mb1232, mb1242, mb7040 or mb7137 using the i2c interface > > - for ranging > > - > > -Required properties: > > - - compatible: "maxbotix,mb1202", > > - "maxbotix,mb1212", > > - "maxbotix,mb1222", > > - "maxbotix,mb1232", > > - "maxbotix,mb1242", > > - "maxbotix,mb7040" or > > - "maxbotix,mb7137" > > - > > - - reg: i2c address of the device, see also i2c/i2c.txt > > - > > -Optional properties: > > - - interrupts: Interrupt used to announce the preceding reading > > - request has finished and that data is available. > > - If no interrupt is specified the device driver > > - falls back to wait a fixed amount of time until > > - data can be retrieved. > > - > > -Example: > > -proximity@70 { > > - compatible = "maxbotix,mb1232"; > > - reg = <0x70>; > > - interrupt-parent = <&gpio2>; > > - interrupts = <2 IRQ_TYPE_EDGE_FALLING>; > > -}; > > diff --git a/Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.yaml b/Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.yaml > > new file mode 100644 > > index 000000000000..e2fb1f6d4dbe > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.yaml > > @@ -0,0 +1,56 @@ > > +# SPDX-License-Identifier: GPL-2.0 > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/iio/proximity/maxbotix,mb1232.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: MaxBotix I2CXL-MaxSonar ultrasonic distance sensor > > + > > +maintainers: > > + - Andreas Klinger <ak@it-klinger.de> > > + > > +description: | > > + MaxBotix I2CXL-MaxSonar ultrasonic distance sensor of type mb1202, > > + mb1212, mb1222, mb1232, mb1242, mb7040 or mb7137 using the i2c interface > > + for ranging > > + > > + Specifications about the devices can be found at: > > + https://www.maxbotix.com/documents/I2CXL-MaxSonar-EZ_Datasheet.pdf > > + > > +properties: > > + compatible: > > + enum: > > + - maxbotix,mb1202 > > + - maxbotix,mb1212 > > + - maxbotix,mb1222 > > + - maxbotix,mb1232 > > + - maxbotix,mb1242 > > + - maxbotix,mb7040 > > + - maxbotix,mb7137 > > + > > + reg: > > + maxItems: 1 > > + > > + interrupts: > > + description: > > + Interrupt used to announce the preceding reading request has finished > > + and that data is available. If no interrupt is specified the device > > + driver falls back to wait a fixed amount of time until data can be > > + retrieved. > > + maxItems: 1 > > + > > +required: > > + - compatible > > + - reg > > + > > +additionalProperties: false > > + > > +examples: > > + - | > > + #include <dt-bindings/interrupt-controller/irq.h> > > + proximity@70 { > > Fails to build with 'make dt_binding_check': > > Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.example.dts:20.11-24: > Warning (reg_format): /example-0/proximity@70:reg: property has > invalid length (4 bytes) (#address-cells == 1, #size-cells == 1) > Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.example.dt.yaml: > Warning (pci_device_bus_num): Failed prerequisite 'reg_format' > Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.example.dt.yaml: > Warning (i2c_bus_reg): Failed prerequisite 'reg_format' > Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.example.dt.yaml: > Warning (spi_bus_reg): Failed prerequisite 'reg_format' > > You have to put this under an i2c bus node. > > i2c { > #address-cells = <1>; > #size-cells = <0>; > ... > }; > > > + compatible = "maxbotix,mb1232"; > > + reg = <0x70>; > > + interrupt-parent = <&gpio2>; > > + interrupts = <2 IRQ_TYPE_EDGE_FALLING>; > > + }; > > -- > > 2.11.0 -- Andreas Klinger Grabenreith 27 84508 Burgkirchen +49 8623 919966 ak@it-klinger.de www.it-klinger.de www.grabenreith.de ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] dt-bindings: iio: maxbotix,mb1232.yaml: transform to yaml 2019-10-07 17:00 ` Andreas Klinger @ 2019-10-07 18:04 ` Rob Herring 0 siblings, 0 replies; 4+ messages in thread From: Rob Herring @ 2019-10-07 18:04 UTC (permalink / raw) To: Andreas Klinger Cc: Jonathan Cameron, Hartmut Knaack, Lars-Peter Clausen, Peter Meerwald, Mark Rutland, open list:IIO SUBSYSTEM AND DRIVERS, devicetree, linux-kernel On Mon, Oct 7, 2019 at 12:00 PM Andreas Klinger <ak@it-klinger.de> wrote: > > Hi Rob, > > i don't get this error. Is there anything i'm doing wrong here? > > ak@arbad:/project/opt-sw/linux-robh$ make O=../build-wega-robh/ dt_binding_check > make[1]: Verzeichnis „/project/opt-sw/build-wega-robh“ wird betreten > SCHEMA Documentation/devicetree/bindings/processed-schema.yaml > /project/opt-sw/linux-robh/Documentation/devicetree/bindings/net/adi,adin.yaml: > ignoring, error in schema 'adi,fifo-depth-bits' > warning: no schema found in file: > /project/opt-sw/linux-robh/Documentation/devicetree/bindings/net/adi,adin.yaml > /project/opt-sw/linux-robh/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml: > ignoring, error in schema '0' > warning: no schema found in file: > /project/opt-sw/linux-robh/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml > CHKDT Documentation/devicetree/bindings/iio/proximity/maxbotix,mb1232.yaml > CHKDT [...] It would be in the later stage when dtc builds the exmaple. The unrelated errors here may be making things error out before that. BTW, update dtschema and it will fix the first error. The 2nd one is pending in linux-next still. Rob ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-10-07 18:04 UTC | newest] Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-10-06 14:27 [PATCH v2] dt-bindings: iio: maxbotix,mb1232.yaml: transform to yaml Andreas Klinger 2019-10-07 12:42 ` Rob Herring 2019-10-07 17:00 ` Andreas Klinger 2019-10-07 18:04 ` 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).