On Fri, Mar 06, 2020 at 04:03:38PM +0300, Sergey.Semin@xxxxxxxxxxxxxxxxxxxx wrote: > From: Serge Semin <Sergey.Semin@xxxxxxxxxxxxxxxxxxxx> > > In accordance with the syscon-driver (drivers/mfd/syscon.c) the syscon > dts-nodes may accept endian properties of the boolean type: little-endian, > big-endian, native-endian. Lets make sure that syscon bindings json-schema > also supports them. > > Signed-off-by: Serge Semin <Sergey.Semin@xxxxxxxxxxxxxxxxxxxx> > Signed-off-by: Alexey Malahov <Alexey.Malahov@xxxxxxxxxxxxxxxxxxxx> > Cc: Thomas Bogendoerfer <tsbogend@xxxxxxxxxxxxxxxx> > Cc: Paul Burton <paulburton@xxxxxxxxxx> > Cc: Ralf Baechle <ralf@xxxxxxxxxxxxxx> > --- > Documentation/devicetree/bindings/mfd/syscon.yaml | 14 ++++++++++++++ > 1 file changed, 14 insertions(+) > > diff --git a/Documentation/devicetree/bindings/mfd/syscon.yaml b/Documentation/devicetree/bindings/mfd/syscon.yaml > index 39375e4313d2..9ee404991533 100644 > --- a/Documentation/devicetree/bindings/mfd/syscon.yaml > +++ b/Documentation/devicetree/bindings/mfd/syscon.yaml > @@ -61,6 +61,11 @@ properties: > description: > Reference to a phandle of a hardware spinlock provider node. > > +patternProperties: > + "^(big|little|native)-endian$": > + $ref: /schemas/types.yaml#/definitions/flag > + description: Bytes order of the system controller memory space. Common properties should have a type definition in a common schema. For this one, I'd like it in the core schema in dtschema. I'd expect for any specific 'syscon', either none or only a subset of these are valid, so I don't think this should be added here. Rob
