| Message ID | b7a6095a5c900fa23cc54d1ccd8e8ef0ccf6e788.1557236840.git-series.maxime.ripard@bootlin.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [1/4] dt-bindings: spi: Add YAML schemas for the generic SPI options | expand |
On Tue, May 7, 2019 at 8:48 AM Maxime Ripard <maxime.ripard@bootlin.com> wrote: > > The SPI controllers have a bunch of generic options that are needed in a > device tree. Add a YAML schemas for those. I'd started on this one, but was planning to move it to the schema repository. The issue there is re-licensing (adding BSD 2 clause). Maybe better to just move it later. > > Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> > --- > Documentation/devicetree/bindings/spi/spi-bus.txt | 111 +----- > Documentation/devicetree/bindings/spi/spi-controller.yaml | 156 +++++++- > 2 files changed, 156 insertions(+), 111 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/spi/spi-bus.txt > create mode 100644 Documentation/devicetree/bindings/spi/spi-controller.yaml > > diff --git a/Documentation/devicetree/bindings/spi/spi-bus.txt b/Documentation/devicetree/bindings/spi/spi-bus.txt > deleted file mode 100644 > index 1f6e86f787ef..000000000000 > --- a/Documentation/devicetree/bindings/spi/spi-bus.txt > +++ /dev/null > @@ -1,111 +0,0 @@ > -SPI (Serial Peripheral Interface) busses > - > -SPI busses can be described with a node for the SPI controller device > -and a set of child nodes for each SPI slave on the bus. The system's SPI > -controller may be described for use in SPI master mode or in SPI slave mode, > -but not for both at the same time. > - > -The SPI controller node requires the following properties: > -- compatible - Name of SPI bus controller following generic names > - recommended practice. > - > -In master mode, the SPI controller node requires the following additional > -properties: > -- #address-cells - number of cells required to define a chip select > - address on the SPI bus. > -- #size-cells - should be zero. > - > -In slave mode, the SPI controller node requires one additional property: > -- spi-slave - Empty property. > - > -No other properties are required in the SPI bus node. It is assumed > -that a driver for an SPI bus device will understand that it is an SPI bus. > -However, the binding does not attempt to define the specific method for > -assigning chip select numbers. Since SPI chip select configuration is > -flexible and non-standardized, it is left out of this binding with the > -assumption that board specific platform code will be used to manage > -chip selects. Individual drivers can define additional properties to > -support describing the chip select layout. > - > -Optional properties (master mode only): > -- cs-gpios - gpios chip select. > -- num-cs - total number of chipselects. > - > -If cs-gpios is used the number of chip selects will be increased automatically > -with max(cs-gpios > hw cs). > - > -So if for example the controller has 2 CS lines, and the cs-gpios > -property looks like this: > - > -cs-gpios = <&gpio1 0 0>, <0>, <&gpio1 1 0>, <&gpio1 2 0>; > - > -Then it should be configured so that num_chipselect = 4 with the > -following mapping: > - > -cs0 : &gpio1 0 0 > -cs1 : native > -cs2 : &gpio1 1 0 > -cs3 : &gpio1 2 0 > - > - > -SPI slave nodes must be children of the SPI controller node. > - > -In master mode, one or more slave nodes (up to the number of chip selects) can > -be present. Required properties are: > -- compatible - Name of SPI device following generic names recommended > - practice. > -- reg - Chip select address of device. > -- spi-max-frequency - Maximum SPI clocking speed of device in Hz. > - > -In slave mode, the (single) slave node is optional. > -If present, it must be called "slave". Required properties are: > -- compatible - Name of SPI device following generic names recommended > - practice. > - > -All slave nodes can contain the following optional properties: > -- spi-cpol - Empty property indicating device requires inverse clock > - polarity (CPOL) mode. > -- spi-cpha - Empty property indicating device requires shifted clock > - phase (CPHA) mode. > -- spi-cs-high - Empty property indicating device requires chip select > - active high. > -- spi-3wire - Empty property indicating device requires 3-wire mode. > -- spi-lsb-first - Empty property indicating device requires LSB first mode. > -- spi-tx-bus-width - The bus width (number of data wires) that is used for MOSI. > - Defaults to 1 if not present. > -- spi-rx-bus-width - The bus width (number of data wires) that is used for MISO. > - Defaults to 1 if not present. > -- spi-rx-delay-us - Microsecond delay after a read transfer. > -- spi-tx-delay-us - Microsecond delay after a write transfer. > - > -Some SPI controllers and devices support Dual and Quad SPI transfer mode. > -It allows data in the SPI system to be transferred using 2 wires (DUAL) or 4 > -wires (QUAD). > -Now the value that spi-tx-bus-width and spi-rx-bus-width can receive is > -only 1 (SINGLE), 2 (DUAL) and 4 (QUAD). > -Dual/Quad mode is not allowed when 3-wire mode is used. > - > -If a gpio chipselect is used for the SPI slave the gpio number will be passed > -via the SPI master node cs-gpios property. > - > -SPI example for an MPC5200 SPI bus: > - spi@f00 { > - #address-cells = <1>; > - #size-cells = <0>; > - compatible = "fsl,mpc5200b-spi","fsl,mpc5200-spi"; > - reg = <0xf00 0x20>; > - interrupts = <2 13 0 2 14 0>; > - interrupt-parent = <&mpc5200_pic>; > - > - ethernet-switch@0 { > - compatible = "micrel,ks8995m"; > - spi-max-frequency = <1000000>; > - reg = <0>; > - }; > - > - codec@1 { > - compatible = "ti,tlv320aic26"; > - spi-max-frequency = <100000>; > - reg = <1>; > - }; > - }; > diff --git a/Documentation/devicetree/bindings/spi/spi-controller.yaml b/Documentation/devicetree/bindings/spi/spi-controller.yaml > new file mode 100644 > index 000000000000..dc239083886c > --- /dev/null > +++ b/Documentation/devicetree/bindings/spi/spi-controller.yaml > @@ -0,0 +1,156 @@ > +# SPDX-License-Identifier: GPL-2.0 > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/spi/spi-controller.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: SPI Controller Generic Binding > + > +maintainers: > + - Mark Brown <broonie@kernel.org> > + > +description: | > + SPI busses can be described with a node for the SPI controller device > + and a set of child nodes for each SPI slave on the bus. The system SPI > + controller may be described for use in SPI master mode or in SPI slave mode, > + but not for both at the same time. > + > +properties: > + $nodename: > + pattern: "^spi(@[a-zA-Z0-9]+)?$" I think we want just "(@.*)". At a minimum, you need to allow for ','. It would be the a bus schema for the parent which should validate unit addresses, so we should pretty much just allow anything here. > + > + "#address-cells": > + const: 1 > + > + "#size-cells": > + const: 0 > + > + cs-gpios: > + description: | > + GPIOs used as chip selects. > + If that property is used, the number of chip selects will be > + increased automatically with max(cs-gpios, hardware chip selects). > + > + So if, for example, the controller has 2 CS lines, and the > + cs-gpios looks like this > + cs-gpios = <&gpio1 0 0>, <0>, <&gpio1 1 0>, <&gpio1 2 0>; > + > + Then it should be configured so that num_chipselect = 4, with > + the following mapping > + cs0 : &gpio1 0 0 > + cs1 : native > + cs2 : &gpio1 1 0 > + cs3 : &gpio1 2 0 > + > + num-cs: > + $ref: /schemas/types.yaml#/definitions/uint32 > + description: > + Total number of chip selects. > + > + spi-slave: > + $ref: /schemas/types.yaml#/definitions/flag "type: boolean" is sufficient here. Maybe we should just remove 'flag'. OTOH, maybe consistency with other types and the abstraction is better as we could add to the flag schema. > + description: > + The SPI controller acts as a slave, instead of a master. > + > +required: > + - "#address-cells" > + - "#size-cells" Only if there are child nodes... > + > +patternProperties: > + "^slave$": type: object > + properties: > + compatible: > + description: > + Compatible of the SPI device. > + > + required: > + - compatible > + > + "^[a-z]+@[0-9]+$": "^.*@[0-9a-f]+": type: object > + properties: > + compatible: > + description: > + Compatible of the SPI device. > + > + reg: > + maxItems: 1 > + description: > + Chip select used by the device. I tend to think we should limit something like this to reasonable values. We're not going to have 2^32 chip selects. 256 should be enough for anyone(TM). > + spi-3wire: > + $ref: /schemas/types.yaml#/definitions/flag > + description: > + The device requires 3-wire mode. > + > + spi-cpha: > + $ref: /schemas/types.yaml#/definitions/flag > + description: > + The device requires shifted clock phase (CPHA) mode. > + > + spi-cpol: > + $ref: /schemas/types.yaml#/definitions/flag > + description: > + The device requires inverse clock polarity (CPOL) mode. > + > + spi-cs-high: > + $ref: /schemas/types.yaml#/definitions/flag > + description: > + The device requires the chip select active high. > + > + spi-lsb-first: > + $ref: /schemas/types.yaml#/definitions/flag > + description: > + The device requires the LSB first mode. > + > + spi-rx-bus-width: > + allOf: > + - $ref: /schemas/types.yaml#/definitions/uint32 > + - enum: [ 1, 2, 4, 8 ] Is the old doc out of date and 8 is allowed now? > + - default: 1 > + description: > + Bus width to the SPI bus used for MISO. > + > + spi-rx-delay-us: > + $ref: /schemas/types.yaml#/definitions/uint32 This can actually be dropped because any property with a unit suffix is already type checked. > + description: > + Delay, in microseconds, after a read transfer. > + > + spi-tx-bus-width: > + allOf: > + - $ref: /schemas/types.yaml#/definitions/uint32 > + - enum: [ 1, 2, 4, 8 ] > + - default: 1 > + description: > + Bus width to the SPI bus used for MOSI. > + > + spi-tx-delay-us: > + $ref: /schemas/types.yaml#/definitions/uint32 > + description: > + Delay, in microseconds, after a write transfer. You missed spi-max-frequency. > + > + required: > + - compatible > + - reg > + > +examples: > + - | > + spi@f00 { > + #address-cells = <1>; > + #size-cells = <0>; > + compatible = "fsl,mpc5200b-spi","fsl,mpc5200-spi"; > + reg = <0xf00 0x20>; > + interrupts = <2 13 0 2 14 0>; > + interrupt-parent = <&mpc5200_pic>; > + > + ethernet-switch@0 { > + compatible = "micrel,ks8995m"; > + spi-max-frequency = <1000000>; > + reg = <0>; > + }; > + > + codec@1 { > + compatible = "ti,tlv320aic26"; > + spi-max-frequency = <100000>; > + reg = <1>; > + }; > + }; > > base-commit: fcdb095ad0016d77d3729dcf8ea915ca4b80fd8b > -- > git-series 0.9.1
Hi, On Tue, May 07, 2019 at 09:35:28AM -0500, Rob Herring wrote: > On Tue, May 7, 2019 at 8:48 AM Maxime Ripard <maxime.ripard@bootlin.com> wrote: > > > > The SPI controllers have a bunch of generic options that are needed in a > > device tree. Add a YAML schemas for those. > > I'd started on this one, but was planning to move it to the schema > repository. The issue there is re-licensing (adding BSD 2 clause). > Maybe better to just move it later. I just found out that dt-doc-validate also chokes on the reference URI. Maybe I should just submit it to the repo then once that is settled? > > +properties: > > + $nodename: > > + pattern: "^spi(@[a-zA-Z0-9]+)?$" > > I think we want just "(@.*)". At a minimum, you need to allow for ','. > It would be the a bus schema for the parent which should validate unit > addresses, so we should pretty much just allow anything here. The issue with this is that it will also match any node starting with spi. In the Allwinner case, that also means the pinctrl nodes with spi pins in them, but I'm sure we can find more corner cases. > > + > > + "#address-cells": > > + const: 1 > > + > > + "#size-cells": > > + const: 0 > > + > > + cs-gpios: > > + description: | > > + GPIOs used as chip selects. > > + If that property is used, the number of chip selects will be > > + increased automatically with max(cs-gpios, hardware chip selects). > > + > > + So if, for example, the controller has 2 CS lines, and the > > + cs-gpios looks like this > > + cs-gpios = <&gpio1 0 0>, <0>, <&gpio1 1 0>, <&gpio1 2 0>; > > + > > + Then it should be configured so that num_chipselect = 4, with > > + the following mapping > > + cs0 : &gpio1 0 0 > > + cs1 : native > > + cs2 : &gpio1 1 0 > > + cs3 : &gpio1 2 0 > > + > > + num-cs: > > + $ref: /schemas/types.yaml#/definitions/uint32 > > + description: > > + Total number of chip selects. > > + > > + spi-slave: > > + $ref: /schemas/types.yaml#/definitions/flag > > "type: boolean" is sufficient here. Maybe we should just remove > 'flag'. OTOH, maybe consistency with other types and the abstraction > is better as we could add to the flag schema. I was trying to be consistent. Do you want me to remove it? > > + spi-rx-bus-width: > > + allOf: > > + - $ref: /schemas/types.yaml#/definitions/uint32 > > + - enum: [ 1, 2, 4, 8 ] > > Is the old doc out of date and 8 is allowed now? It's not, it's a copy and paste mistake. I'll respin this with your fixes, thanks! Maxime -- Maxime Ripard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com
On Tue, May 7, 2019 at 1:07 PM Maxime Ripard <maxime.ripard@bootlin.com> wrote: > > Hi, > > On Tue, May 07, 2019 at 09:35:28AM -0500, Rob Herring wrote: > > On Tue, May 7, 2019 at 8:48 AM Maxime Ripard <maxime.ripard@bootlin.com> wrote: > > > > > > The SPI controllers have a bunch of generic options that are needed in a > > > device tree. Add a YAML schemas for those. > > > > I'd started on this one, but was planning to move it to the schema > > repository. The issue there is re-licensing (adding BSD 2 clause). > > Maybe better to just move it later. > > I just found out that dt-doc-validate also chokes on the reference > URI. Maybe I should just submit it to the repo then once that is > settled? I'm not really too excited about chasing down licensing on every file we want to move and I'd like to avoid per file licenses, so I'd like local $refs to work. I think I've got something figured out that will work. It will need a small kernel side change though. > > > +properties: > > > + $nodename: > > > + pattern: "^spi(@[a-zA-Z0-9]+)?$" > > > > I think we want just "(@.*)". At a minimum, you need to allow for ','. > > It would be the a bus schema for the parent which should validate unit > > addresses, so we should pretty much just allow anything here. > > The issue with this is that it will also match any node starting with > spi. In the Allwinner case, that also means the pinctrl nodes with spi > pins in them, but I'm sure we can find more corner cases. Maybe I wasn't clear, but I meant changing just the unit-address part. So: "^spi(@.*)$" > > > > + > > > + "#address-cells": > > > + const: 1 > > > + > > > + "#size-cells": > > > + const: 0 > > > + > > > + cs-gpios: > > > + description: | > > > + GPIOs used as chip selects. > > > + If that property is used, the number of chip selects will be > > > + increased automatically with max(cs-gpios, hardware chip selects). > > > + > > > + So if, for example, the controller has 2 CS lines, and the > > > + cs-gpios looks like this > > > + cs-gpios = <&gpio1 0 0>, <0>, <&gpio1 1 0>, <&gpio1 2 0>; > > > + > > > + Then it should be configured so that num_chipselect = 4, with > > > + the following mapping > > > + cs0 : &gpio1 0 0 > > > + cs1 : native > > > + cs2 : &gpio1 1 0 > > > + cs3 : &gpio1 2 0 > > > + > > > + num-cs: > > > + $ref: /schemas/types.yaml#/definitions/uint32 > > > + description: > > > + Total number of chip selects. > > > + > > > + spi-slave: > > > + $ref: /schemas/types.yaml#/definitions/flag > > > > "type: boolean" is sufficient here. Maybe we should just remove > > 'flag'. OTOH, maybe consistency with other types and the abstraction > > is better as we could add to the flag schema. > > I was trying to be consistent. Do you want me to remove it? No, it's fine. Rob
On Tue, May 7, 2019 at 4:35 PM Rob Herring <robh+dt@kernel.org> wrote: > On Tue, May 7, 2019 at 8:48 AM Maxime Ripard <maxime.ripard@bootlin.com> wrote: > > The SPI controllers have a bunch of generic options that are needed in a > > device tree. Add a YAML schemas for those. > > > > Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> > > index 1f6e86f787ef..000000000000 > > --- a/Documentation/devicetree/bindings/spi/spi-bus.txt > > +++ /dev/null > > -In master mode, the SPI controller node requires the following additional > > -properties: > > -- #address-cells - number of cells required to define a chip select > > - address on the SPI bus. > > -- #size-cells - should be zero. > > - > > -In slave mode, the SPI controller node requires one additional property: > > -- spi-slave - Empty property. > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/spi/spi-controller.yaml > > + spi-slave: > > + $ref: /schemas/types.yaml#/definitions/flag > > "type: boolean" is sufficient here. Maybe we should just remove > 'flag'. OTOH, maybe consistency with other types and the abstraction > is better as we could add to the flag schema. > > > + description: > > + The SPI controller acts as a slave, instead of a master. > > + > > +required: > > + - "#address-cells" > > + - "#size-cells" > > Only if there are child nodes... ... and spi-slave is not present. Gr{oetje,eeting}s, Geert
diff --git a/Documentation/devicetree/bindings/spi/spi-bus.txt b/Documentation/devicetree/bindings/spi/spi-bus.txt deleted file mode 100644 index 1f6e86f787ef..000000000000 --- a/Documentation/devicetree/bindings/spi/spi-bus.txt +++ /dev/null @@ -1,111 +0,0 @@ -SPI (Serial Peripheral Interface) busses - -SPI busses can be described with a node for the SPI controller device -and a set of child nodes for each SPI slave on the bus. The system's SPI -controller may be described for use in SPI master mode or in SPI slave mode, -but not for both at the same time. - -The SPI controller node requires the following properties: -- compatible - Name of SPI bus controller following generic names - recommended practice. - -In master mode, the SPI controller node requires the following additional -properties: -- #address-cells - number of cells required to define a chip select - address on the SPI bus. -- #size-cells - should be zero. - -In slave mode, the SPI controller node requires one additional property: -- spi-slave - Empty property. - -No other properties are required in the SPI bus node. It is assumed -that a driver for an SPI bus device will understand that it is an SPI bus. -However, the binding does not attempt to define the specific method for -assigning chip select numbers. Since SPI chip select configuration is -flexible and non-standardized, it is left out of this binding with the -assumption that board specific platform code will be used to manage -chip selects. Individual drivers can define additional properties to -support describing the chip select layout. - -Optional properties (master mode only): -- cs-gpios - gpios chip select. -- num-cs - total number of chipselects. - -If cs-gpios is used the number of chip selects will be increased automatically -with max(cs-gpios > hw cs). - -So if for example the controller has 2 CS lines, and the cs-gpios -property looks like this: - -cs-gpios = <&gpio1 0 0>, <0>, <&gpio1 1 0>, <&gpio1 2 0>; - -Then it should be configured so that num_chipselect = 4 with the -following mapping: - -cs0 : &gpio1 0 0 -cs1 : native -cs2 : &gpio1 1 0 -cs3 : &gpio1 2 0 - - -SPI slave nodes must be children of the SPI controller node. - -In master mode, one or more slave nodes (up to the number of chip selects) can -be present. Required properties are: -- compatible - Name of SPI device following generic names recommended - practice. -- reg - Chip select address of device. -- spi-max-frequency - Maximum SPI clocking speed of device in Hz. - -In slave mode, the (single) slave node is optional. -If present, it must be called "slave". Required properties are: -- compatible - Name of SPI device following generic names recommended - practice. - -All slave nodes can contain the following optional properties: -- spi-cpol - Empty property indicating device requires inverse clock - polarity (CPOL) mode. -- spi-cpha - Empty property indicating device requires shifted clock - phase (CPHA) mode. -- spi-cs-high - Empty property indicating device requires chip select - active high. -- spi-3wire - Empty property indicating device requires 3-wire mode. -- spi-lsb-first - Empty property indicating device requires LSB first mode. -- spi-tx-bus-width - The bus width (number of data wires) that is used for MOSI. - Defaults to 1 if not present. -- spi-rx-bus-width - The bus width (number of data wires) that is used for MISO. - Defaults to 1 if not present. -- spi-rx-delay-us - Microsecond delay after a read transfer. -- spi-tx-delay-us - Microsecond delay after a write transfer. - -Some SPI controllers and devices support Dual and Quad SPI transfer mode. -It allows data in the SPI system to be transferred using 2 wires (DUAL) or 4 -wires (QUAD). -Now the value that spi-tx-bus-width and spi-rx-bus-width can receive is -only 1 (SINGLE), 2 (DUAL) and 4 (QUAD). -Dual/Quad mode is not allowed when 3-wire mode is used. - -If a gpio chipselect is used for the SPI slave the gpio number will be passed -via the SPI master node cs-gpios property. - -SPI example for an MPC5200 SPI bus: - spi@f00 { - #address-cells = <1>; - #size-cells = <0>; - compatible = "fsl,mpc5200b-spi","fsl,mpc5200-spi"; - reg = <0xf00 0x20>; - interrupts = <2 13 0 2 14 0>; - interrupt-parent = <&mpc5200_pic>; - - ethernet-switch@0 { - compatible = "micrel,ks8995m"; - spi-max-frequency = <1000000>; - reg = <0>; - }; - - codec@1 { - compatible = "ti,tlv320aic26"; - spi-max-frequency = <100000>; - reg = <1>; - }; - }; diff --git a/Documentation/devicetree/bindings/spi/spi-controller.yaml b/Documentation/devicetree/bindings/spi/spi-controller.yaml new file mode 100644 index 000000000000..dc239083886c --- /dev/null +++ b/Documentation/devicetree/bindings/spi/spi-controller.yaml @@ -0,0 +1,156 @@ +# SPDX-License-Identifier: GPL-2.0 +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/spi/spi-controller.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: SPI Controller Generic Binding + +maintainers: + - Mark Brown <broonie@kernel.org> + +description: | + SPI busses can be described with a node for the SPI controller device + and a set of child nodes for each SPI slave on the bus. The system SPI + controller may be described for use in SPI master mode or in SPI slave mode, + but not for both at the same time. + +properties: + $nodename: + pattern: "^spi(@[a-zA-Z0-9]+)?$" + + "#address-cells": + const: 1 + + "#size-cells": + const: 0 + + cs-gpios: + description: | + GPIOs used as chip selects. + If that property is used, the number of chip selects will be + increased automatically with max(cs-gpios, hardware chip selects). + + So if, for example, the controller has 2 CS lines, and the + cs-gpios looks like this + cs-gpios = <&gpio1 0 0>, <0>, <&gpio1 1 0>, <&gpio1 2 0>; + + Then it should be configured so that num_chipselect = 4, with + the following mapping + cs0 : &gpio1 0 0 + cs1 : native + cs2 : &gpio1 1 0 + cs3 : &gpio1 2 0 + + num-cs: + $ref: /schemas/types.yaml#/definitions/uint32 + description: + Total number of chip selects. + + spi-slave: + $ref: /schemas/types.yaml#/definitions/flag + description: + The SPI controller acts as a slave, instead of a master. + +required: + - "#address-cells" + - "#size-cells" + +patternProperties: + "^slave$": + properties: + compatible: + description: + Compatible of the SPI device. + + required: + - compatible + + "^[a-z]+@[0-9]+$": + properties: + compatible: + description: + Compatible of the SPI device. + + reg: + maxItems: 1 + description: + Chip select used by the device. + + spi-3wire: + $ref: /schemas/types.yaml#/definitions/flag + description: + The device requires 3-wire mode. + + spi-cpha: + $ref: /schemas/types.yaml#/definitions/flag + description: + The device requires shifted clock phase (CPHA) mode. + + spi-cpol: + $ref: /schemas/types.yaml#/definitions/flag + description: + The device requires inverse clock polarity (CPOL) mode. + + spi-cs-high: + $ref: /schemas/types.yaml#/definitions/flag + description: + The device requires the chip select active high. + + spi-lsb-first: + $ref: /schemas/types.yaml#/definitions/flag + description: + The device requires the LSB first mode. + + spi-rx-bus-width: + allOf: + - $ref: /schemas/types.yaml#/definitions/uint32 + - enum: [ 1, 2, 4, 8 ] + - default: 1 + description: + Bus width to the SPI bus used for MISO. + + spi-rx-delay-us: + $ref: /schemas/types.yaml#/definitions/uint32 + description: + Delay, in microseconds, after a read transfer. + + spi-tx-bus-width: + allOf: + - $ref: /schemas/types.yaml#/definitions/uint32 + - enum: [ 1, 2, 4, 8 ] + - default: 1 + description: + Bus width to the SPI bus used for MOSI. + + spi-tx-delay-us: + $ref: /schemas/types.yaml#/definitions/uint32 + description: + Delay, in microseconds, after a write transfer. + + required: + - compatible + - reg + +examples: + - | + spi@f00 { + #address-cells = <1>; + #size-cells = <0>; + compatible = "fsl,mpc5200b-spi","fsl,mpc5200-spi"; + reg = <0xf00 0x20>; + interrupts = <2 13 0 2 14 0>; + interrupt-parent = <&mpc5200_pic>; + + ethernet-switch@0 { + compatible = "micrel,ks8995m"; + spi-max-frequency = <1000000>; + reg = <0>; + }; + + codec@1 { + compatible = "ti,tlv320aic26"; + spi-max-frequency = <100000>; + reg = <1>; + }; + };
The SPI controllers have a bunch of generic options that are needed in a device tree. Add a YAML schemas for those. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> --- Documentation/devicetree/bindings/spi/spi-bus.txt | 111 +----- Documentation/devicetree/bindings/spi/spi-controller.yaml | 156 +++++++- 2 files changed, 156 insertions(+), 111 deletions(-) delete mode 100644 Documentation/devicetree/bindings/spi/spi-bus.txt create mode 100644 Documentation/devicetree/bindings/spi/spi-controller.yaml base-commit: fcdb095ad0016d77d3729dcf8ea915ca4b80fd8b