On Wed, Apr 06, 2022 at 04:36:42PM -0700, Brad Larson wrote: > The Pensando Elba SoC has integrated the DW APB SPI Controller > and requires the property pensando,syscon-spics for access > to the spics control register. > > Signed-off-by: Brad Larson <brad@xxxxxxxxxxx> > --- > Change from V3: > - Add required property pensando,syscon-spics to go with > pensando,elba-spi > > .../bindings/spi/snps,dw-apb-ssi.yaml | 17 +++++++++++++++++ > 1 file changed, 17 insertions(+) > > diff --git a/Documentation/devicetree/bindings/spi/snps,dw-apb-ssi.yaml b/Documentation/devicetree/bindings/spi/snps,dw-apb-ssi.yaml > index d7e08b03e204..41c3bbf5a55c 100644 > --- a/Documentation/devicetree/bindings/spi/snps,dw-apb-ssi.yaml > +++ b/Documentation/devicetree/bindings/spi/snps,dw-apb-ssi.yaml > @@ -37,6 +37,21 @@ allOf: > else: > required: > - interrupts > + - if: > + properties: > + compatible: > + contains: > + enum: > + - pensando,elba-spi > + then: > + properties: > + pensando,syscon-spics: Please see my comment to the "[PATCH 11/11] arm64: dts: Add Pensando Elba SoC support" patch regarding this property implementation. -Sergey > + $ref: /schemas/types.yaml#/definitions/phandle > + description: > + Phandle to the system control device node which provides access to > + the spics control register > + required: > + - pensando,syscon-spics > > properties: > compatible: > @@ -73,6 +88,8 @@ properties: > - renesas,r9a06g032-spi # RZ/N1D > - renesas,r9a06g033-spi # RZ/N1S > - const: renesas,rzn1-spi # RZ/N1 > + - description: Pensando SoC SPI Controller > + const: pensando,elba-spi > > reg: > minItems: 1 > -- > 2.17.1 >