Messages in this thread | | | Date | Tue, 8 Sep 2020 14:44:35 -0600 | From | Rob Herring <> | Subject | Re: [PATCH 4/7] dt-bindings: fsi: fsi2spi: Document new restricted property |
| |
On Thu, Aug 20, 2020 at 12:02:25PM -0500, Eddie James wrote: > Add documentation for the "fsi2spi,restricted" property which indicates > a controller shouldn't sequence loops and therefore has a smaller > transfer size. > > Signed-off-by: Eddie James <eajames@linux.ibm.com> > Acked-by: Joel Stanley <joel@jms.id.au> > Signed-off-by: Joel Stanley <joel@jms.id.au> > --- > Documentation/devicetree/bindings/fsi/ibm,fsi2spi.yaml | 10 ++++++++++ > 1 file changed, 10 insertions(+) > > diff --git a/Documentation/devicetree/bindings/fsi/ibm,fsi2spi.yaml b/Documentation/devicetree/bindings/fsi/ibm,fsi2spi.yaml > index b26d4b4be743..0d2fb071fd00 100644 > --- a/Documentation/devicetree/bindings/fsi/ibm,fsi2spi.yaml > +++ b/Documentation/devicetree/bindings/fsi/ibm,fsi2spi.yaml > @@ -24,6 +24,16 @@ properties: > items: > - description: FSI slave address > > +patternProperties: > + "^spi(@.*|-[0-9a-f])*$":
Doesn't FSI define the unit-address format already? No need to do it again here. Plus the form 'spi-[0-9a-f]' isn't valid. Just "^spi@" should be enough.
> + type: object > + > + properties: > + fsi2spi,restricted:
[<vendor>],<property-name>
fsi2spi is not a vendor.
> + description: indicates the controller should not use looping in the > + sequencer and therefore has a smaller maximum transfer size > + type: boolean > + > required: > - compatible > - reg > -- > 2.26.2 >
| |