On 25/04/2024 23:36, Eddie James wrote: > The IBM Power chips have a basic SPI controller. Document it. Please use subject prefixes matching the subsystem. You can get them for example with `git log --oneline -- DIRECTORY_OR_FILE` on the directory your patch is touching. > > Signed-off-by: Eddie James <eajames@xxxxxxxxxxxxx> > --- > .../devicetree/bindings/spi/ibm,p10-spi.yaml | 56 +++++++++++++++++++ > 1 file changed, 56 insertions(+) > create mode 100644 Documentation/devicetree/bindings/spi/ibm,p10-spi.yaml > > diff --git a/Documentation/devicetree/bindings/spi/ibm,p10-spi.yaml b/Documentation/devicetree/bindings/spi/ibm,p10-spi.yaml > new file mode 100644 > index 000000000000..9bf57b621c1f > --- /dev/null > +++ b/Documentation/devicetree/bindings/spi/ibm,p10-spi.yaml > @@ -0,0 +1,56 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/spi/ibm,p10-spi.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: IBM SPI Controller IBM P10 SPI Controller > + > +maintainers: > + - Eddie James <eajames@xxxxxxxxxxxxx> > + > +description: > + A basic SPI controller found on IBM Power chips, accessed over FSI. This > + node will always be a child node of an ibm,fsi2spi node. > + > +properties: > + compatible: > + enum: > + - ibm,p10-spi > + > + reg: > + maxItems: 1 > + > +required: > + - compatible > + - reg > + > +allOf: > + - $ref: spi-controller.yaml# > + > +unevaluatedProperties: false > + > +examples: > + - | > + fsi2spi@1c00 { > + compatible = "ibm,fsi2spi"; > + reg = <0x1c00 0x400>; > + #address-cells = <1>; > + #size-cells = <0>; Use rather some simple wrapper instead of node causing warnings, e.g. fsi {} with only address/size cells. Best regards, Krzysztof