Message ID | 20190927100407.1863293-3-paul.kocialkowski@bootlin.com |
---|---|
State | New |
Headers | show |
Series | LogiCVC mfd and GPIO support | expand |
On Fri, Sep 27, 2019 at 12:04:04PM +0200, Paul Kocialkowski wrote: > The LogiCVC is a display engine which also exposes GPIO functionality. > For this reason, it is described as a multi-function device that is expected > to provide register access to its children nodes for gpio and display. > > Signed-off-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com> > --- > .../bindings/mfd/xylon,logicvc.yaml | 50 +++++++++++++++++++ > 1 file changed, 50 insertions(+) > create mode 100644 Documentation/devicetree/bindings/mfd/xylon,logicvc.yaml > > diff --git a/Documentation/devicetree/bindings/mfd/xylon,logicvc.yaml b/Documentation/devicetree/bindings/mfd/xylon,logicvc.yaml > new file mode 100644 > index 000000000000..abc9937506e0 > --- /dev/null > +++ b/Documentation/devicetree/bindings/mfd/xylon,logicvc.yaml > @@ -0,0 +1,50 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +# Copyright 2019 Bootlin > +%YAML 1.2 > +--- > +$id: "http://devicetree.org/schemas/mfd/xylon,logicvc.yaml#" > +$schema: "http://devicetree.org/meta-schemas/core.yaml#" > + > +title: Xylon LogiCVC multi-function device > + > +maintainers: > + - Paul Kocialkowski <paul.kocialkowski@bootlin.com> > + > +description: | > + The LogiCVC is a display controller that also contains a GPIO controller. > + As a result, a multi-function device is exposed as parent of the display > + and GPIO blocks. > + > +properties: > + compatible: > + items: > + - enum: > + - xylon,logicvc-3.02.a > + - const: syscon > + - const: simple-mfd > + > + reg: > + maxItems: 1 > + > +select: > + properties: > + compatible: > + contains: > + enum: > + - xylon,logicvc-3.02.a I've seen a couple of these with 'syscon' today, so I fixed the schema tool to just exclude 'syscon' and 'simple-mfd' from the generated 'select'. So you can drop select now. Reviewed-by: Rob Herring <robh@kernel.org> > + > + required: > + - compatible > + > +required: > + - compatible > + - reg > + > +examples: > + - | > + logicvc: logicvc@43c00000 { > + compatible = "xylon,logicvc-3.02.a", "syscon", "simple-mfd"; > + reg = <0x43c00000 0x6000>; > + #address-cells = <1>; > + #size-cells = <1>; > + }; > -- > 2.23.0 >
On Fri, 27 Sep 2019, Rob Herring wrote: > On Fri, Sep 27, 2019 at 12:04:04PM +0200, Paul Kocialkowski wrote: > > The LogiCVC is a display engine which also exposes GPIO functionality. > > For this reason, it is described as a multi-function device that is expected > > to provide register access to its children nodes for gpio and display. > > > > Signed-off-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com> > > --- > > .../bindings/mfd/xylon,logicvc.yaml | 50 +++++++++++++++++++ > > 1 file changed, 50 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/mfd/xylon,logicvc.yaml > > > > diff --git a/Documentation/devicetree/bindings/mfd/xylon,logicvc.yaml b/Documentation/devicetree/bindings/mfd/xylon,logicvc.yaml > > new file mode 100644 > > index 000000000000..abc9937506e0 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/mfd/xylon,logicvc.yaml > > @@ -0,0 +1,50 @@ > > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > > +# Copyright 2019 Bootlin > > +%YAML 1.2 > > +--- > > +$id: "http://devicetree.org/schemas/mfd/xylon,logicvc.yaml#" > > +$schema: "http://devicetree.org/meta-schemas/core.yaml#" > > + > > +title: Xylon LogiCVC multi-function device > > + > > +maintainers: > > + - Paul Kocialkowski <paul.kocialkowski@bootlin.com> > > + > > +description: | > > + The LogiCVC is a display controller that also contains a GPIO controller. > > + As a result, a multi-function device is exposed as parent of the display > > + and GPIO blocks. > > + > > +properties: > > + compatible: > > + items: > > + - enum: > > + - xylon,logicvc-3.02.a > > + - const: syscon > > + - const: simple-mfd > > + > > + reg: > > + maxItems: 1 > > + > > +select: > > + properties: > > + compatible: > > + contains: > > + enum: > > + - xylon,logicvc-3.02.a > > I've seen a couple of these with 'syscon' today, so I fixed the schema > tool to just exclude 'syscon' and 'simple-mfd' from the generated > 'select'. So you can drop select now. Does this need to happen before this patch can be applied? > Reviewed-by: Rob Herring <robh@kernel.org>
On Fri, Oct 4, 2019 at 9:45 AM Lee Jones <lee.jones@linaro.org> wrote: > > On Fri, 27 Sep 2019, Rob Herring wrote: > > > On Fri, Sep 27, 2019 at 12:04:04PM +0200, Paul Kocialkowski wrote: > > > The LogiCVC is a display engine which also exposes GPIO functionality. > > > For this reason, it is described as a multi-function device that is expected > > > to provide register access to its children nodes for gpio and display. > > > > > > Signed-off-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com> > > > --- > > > .../bindings/mfd/xylon,logicvc.yaml | 50 +++++++++++++++++++ > > > 1 file changed, 50 insertions(+) > > > create mode 100644 Documentation/devicetree/bindings/mfd/xylon,logicvc.yaml > > > > > > diff --git a/Documentation/devicetree/bindings/mfd/xylon,logicvc.yaml b/Documentation/devicetree/bindings/mfd/xylon,logicvc.yaml > > > new file mode 100644 > > > index 000000000000..abc9937506e0 > > > --- /dev/null > > > +++ b/Documentation/devicetree/bindings/mfd/xylon,logicvc.yaml > > > @@ -0,0 +1,50 @@ > > > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > > > +# Copyright 2019 Bootlin > > > +%YAML 1.2 > > > +--- > > > +$id: "http://devicetree.org/schemas/mfd/xylon,logicvc.yaml#" > > > +$schema: "http://devicetree.org/meta-schemas/core.yaml#" > > > + > > > +title: Xylon LogiCVC multi-function device > > > + > > > +maintainers: > > > + - Paul Kocialkowski <paul.kocialkowski@bootlin.com> > > > + > > > +description: | > > > + The LogiCVC is a display controller that also contains a GPIO controller. > > > + As a result, a multi-function device is exposed as parent of the display > > > + and GPIO blocks. > > > + > > > +properties: > > > + compatible: > > > + items: > > > + - enum: > > > + - xylon,logicvc-3.02.a > > > + - const: syscon > > > + - const: simple-mfd > > > + > > > + reg: > > > + maxItems: 1 > > > + > > > +select: > > > + properties: > > > + compatible: > > > + contains: > > > + enum: > > > + - xylon,logicvc-3.02.a > > > > I've seen a couple of these with 'syscon' today, so I fixed the schema > > tool to just exclude 'syscon' and 'simple-mfd' from the generated > > 'select'. So you can drop select now. > > Does this need to happen before this patch can be applied? Drop the 'select'? Yes that should happen first. Rob
Hi, On Fri 04 Oct 19, 11:09, Rob Herring wrote: > On Fri, Oct 4, 2019 at 9:45 AM Lee Jones <lee.jones@linaro.org> wrote: > > > > On Fri, 27 Sep 2019, Rob Herring wrote: > > > > > On Fri, Sep 27, 2019 at 12:04:04PM +0200, Paul Kocialkowski wrote: > > > > The LogiCVC is a display engine which also exposes GPIO functionality. > > > > For this reason, it is described as a multi-function device that is expected > > > > to provide register access to its children nodes for gpio and display. > > > > > > > > Signed-off-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com> > > > > --- > > > > .../bindings/mfd/xylon,logicvc.yaml | 50 +++++++++++++++++++ > > > > 1 file changed, 50 insertions(+) > > > > create mode 100644 Documentation/devicetree/bindings/mfd/xylon,logicvc.yaml > > > > > > > > diff --git a/Documentation/devicetree/bindings/mfd/xylon,logicvc.yaml b/Documentation/devicetree/bindings/mfd/xylon,logicvc.yaml > > > > new file mode 100644 > > > > index 000000000000..abc9937506e0 > > > > --- /dev/null > > > > +++ b/Documentation/devicetree/bindings/mfd/xylon,logicvc.yaml > > > > @@ -0,0 +1,50 @@ > > > > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > > > > +# Copyright 2019 Bootlin > > > > +%YAML 1.2 > > > > +--- > > > > +$id: "http://devicetree.org/schemas/mfd/xylon,logicvc.yaml#" > > > > +$schema: "http://devicetree.org/meta-schemas/core.yaml#" > > > > + > > > > +title: Xylon LogiCVC multi-function device > > > > + > > > > +maintainers: > > > > + - Paul Kocialkowski <paul.kocialkowski@bootlin.com> > > > > + > > > > +description: | > > > > + The LogiCVC is a display controller that also contains a GPIO controller. > > > > + As a result, a multi-function device is exposed as parent of the display > > > > + and GPIO blocks. > > > > + > > > > +properties: > > > > + compatible: > > > > + items: > > > > + - enum: > > > > + - xylon,logicvc-3.02.a > > > > + - const: syscon > > > > + - const: simple-mfd > > > > + > > > > + reg: > > > > + maxItems: 1 > > > > + > > > > +select: > > > > + properties: > > > > + compatible: > > > > + contains: > > > > + enum: > > > > + - xylon,logicvc-3.02.a > > > > > > I've seen a couple of these with 'syscon' today, so I fixed the schema > > > tool to just exclude 'syscon' and 'simple-mfd' from the generated > > > 'select'. So you can drop select now. > > > > Does this need to happen before this patch can be applied? > > Drop the 'select'? Yes that should happen first. I'll definitely respin the series for a v3 soon so no worries, I'll get rid of it. Cheers, Paul
diff --git a/Documentation/devicetree/bindings/mfd/xylon,logicvc.yaml b/Documentation/devicetree/bindings/mfd/xylon,logicvc.yaml new file mode 100644 index 000000000000..abc9937506e0 --- /dev/null +++ b/Documentation/devicetree/bindings/mfd/xylon,logicvc.yaml @@ -0,0 +1,50 @@ +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) +# Copyright 2019 Bootlin +%YAML 1.2 +--- +$id: "http://devicetree.org/schemas/mfd/xylon,logicvc.yaml#" +$schema: "http://devicetree.org/meta-schemas/core.yaml#" + +title: Xylon LogiCVC multi-function device + +maintainers: + - Paul Kocialkowski <paul.kocialkowski@bootlin.com> + +description: | + The LogiCVC is a display controller that also contains a GPIO controller. + As a result, a multi-function device is exposed as parent of the display + and GPIO blocks. + +properties: + compatible: + items: + - enum: + - xylon,logicvc-3.02.a + - const: syscon + - const: simple-mfd + + reg: + maxItems: 1 + +select: + properties: + compatible: + contains: + enum: + - xylon,logicvc-3.02.a + + required: + - compatible + +required: + - compatible + - reg + +examples: + - | + logicvc: logicvc@43c00000 { + compatible = "xylon,logicvc-3.02.a", "syscon", "simple-mfd"; + reg = <0x43c00000 0x6000>; + #address-cells = <1>; + #size-cells = <1>; + };
The LogiCVC is a display engine which also exposes GPIO functionality. For this reason, it is described as a multi-function device that is expected to provide register access to its children nodes for gpio and display. Signed-off-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com> --- .../bindings/mfd/xylon,logicvc.yaml | 50 +++++++++++++++++++ 1 file changed, 50 insertions(+) create mode 100644 Documentation/devicetree/bindings/mfd/xylon,logicvc.yaml