| Message ID | 20200426185856.38826-2-paul@crapouillou.net |
|---|---|
| State | New |
| Headers | show |
| Series | [1/8] dt-bindings: clock: Convert ingenic,cgu.txt to YAML | expand |
Hello! On 26.04.2020 21:58, Paul Cercueil wrote: > Convert the ingenic,intc.txt doc file to ingenic,intc.yaml. > > Some compatible strings now require a fallback, as the controller > generally works the same across the SoCs families. > > Signed-off-by: Paul Cercueil <paul@crapouillou.net> [...] > diff --git a/Documentation/devicetree/bindings/interrupt-controller/ingenic,intc.yaml b/Documentation/devicetree/bindings/interrupt-controller/ingenic,intc.yaml > new file mode 100644 > index 000000000000..28b27e1a6e9d > --- /dev/null > +++ b/Documentation/devicetree/bindings/interrupt-controller/ingenic,intc.yaml > @@ -0,0 +1,63 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/interrupt-controller/ingenic,intc.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Ingenic SoCs interrupt controller devicetree bindings > + > +maintainers: > + - Paul Cercueil <paul@crapouillou.net> > + > +properties: > + $nodename: > + pattern: "^interrupt-controller@[0-9a-f]+$" > + > + compatible: > + oneOf: > + - enum: > + - ingenic,jz4740-intc > + - ingenic,jz4760-intc > + - ingenic,jz4780-intc > + - items: > + - enum: > + - ingenic,jz4775-intc > + - ingenic,jz4770-intc > + - const: ingenic,jz4760-intc > + - items: > + - const: ingenic,x1000-intc > + - const: ingenic,jz4780-intc > + - items: > + - const: ingenic,jz4725b-intc > + - const: ingenic,jz4740-intc > + > + "#interrupt-cells": > + const: 1 Do double quotes work the same as the single ones? [...] MBR, Sergei
Hi Sergei, Le lun. 27 avril 2020 à 12:11, Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> a écrit : > Hello! > > On 26.04.2020 21:58, Paul Cercueil wrote: > >> Convert the ingenic,intc.txt doc file to ingenic,intc.yaml. >> >> Some compatible strings now require a fallback, as the controller >> generally works the same across the SoCs families. >> >> Signed-off-by: Paul Cercueil <paul@crapouillou.net> > [...] >> diff --git >> a/Documentation/devicetree/bindings/interrupt-controller/ingenic,intc.yaml >> b/Documentation/devicetree/bindings/interrupt-controller/ingenic,intc.yaml >> new file mode 100644 >> index 000000000000..28b27e1a6e9d >> --- /dev/null >> +++ >> b/Documentation/devicetree/bindings/interrupt-controller/ingenic,intc.yaml >> @@ -0,0 +1,63 @@ >> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) >> +%YAML 1.2 >> +--- >> +$id: >> http://devicetree.org/schemas/interrupt-controller/ingenic,intc.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: Ingenic SoCs interrupt controller devicetree bindings >> + >> +maintainers: >> + - Paul Cercueil <paul@crapouillou.net> >> + >> +properties: >> + $nodename: >> + pattern: "^interrupt-controller@[0-9a-f]+$" >> + >> + compatible: >> + oneOf: >> + - enum: >> + - ingenic,jz4740-intc >> + - ingenic,jz4760-intc >> + - ingenic,jz4780-intc >> + - items: >> + - enum: >> + - ingenic,jz4775-intc >> + - ingenic,jz4770-intc >> + - const: ingenic,jz4760-intc >> + - items: >> + - const: ingenic,x1000-intc >> + - const: ingenic,jz4780-intc >> + - items: >> + - const: ingenic,jz4725b-intc >> + - const: ingenic,jz4740-intc >> + >> + "#interrupt-cells": >> + const: 1 > > Do double quotes work the same as the single ones? Yes. The only difference is that you can escape characters in double quotes. -Paul > > [...] > > MBR, Sergei
On Sun, 26 Apr 2020 20:58:50 +0200, Paul Cercueil wrote: > Convert the ingenic,intc.txt doc file to ingenic,intc.yaml. > > Some compatible strings now require a fallback, as the controller > generally works the same across the SoCs families. > > Signed-off-by: Paul Cercueil <paul@crapouillou.net> > --- > .../interrupt-controller/ingenic,intc.txt | 28 --------- > .../interrupt-controller/ingenic,intc.yaml | 63 +++++++++++++++++++ > 2 files changed, 63 insertions(+), 28 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/interrupt-controller/ingenic,intc.txt > create mode 100644 Documentation/devicetree/bindings/interrupt-controller/ingenic,intc.yaml > Applied, thanks!
diff --git a/Documentation/devicetree/bindings/interrupt-controller/ingenic,intc.txt b/Documentation/devicetree/bindings/interrupt-controller/ingenic,intc.txt deleted file mode 100644 index d4373d0f7121..000000000000 --- a/Documentation/devicetree/bindings/interrupt-controller/ingenic,intc.txt +++ /dev/null @@ -1,28 +0,0 @@ -Ingenic SoC Interrupt Controller - -Required properties: - -- compatible : should be "ingenic,<socname>-intc". Valid strings are: - ingenic,jz4740-intc - ingenic,jz4725b-intc - ingenic,jz4770-intc - ingenic,jz4775-intc - ingenic,jz4780-intc -- reg : Specifies base physical address and size of the registers. -- interrupt-controller : Identifies the node as an interrupt controller -- #interrupt-cells : Specifies the number of cells needed to encode an - interrupt source. The value shall be 1. -- interrupts : Specifies the CPU interrupt the controller is connected to. - -Example: - -intc: interrupt-controller@10001000 { - compatible = "ingenic,jz4740-intc"; - reg = <0x10001000 0x14>; - - interrupt-controller; - #interrupt-cells = <1>; - - interrupt-parent = <&cpuintc>; - interrupts = <2>; -}; diff --git a/Documentation/devicetree/bindings/interrupt-controller/ingenic,intc.yaml b/Documentation/devicetree/bindings/interrupt-controller/ingenic,intc.yaml new file mode 100644 index 000000000000..28b27e1a6e9d --- /dev/null +++ b/Documentation/devicetree/bindings/interrupt-controller/ingenic,intc.yaml @@ -0,0 +1,63 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/interrupt-controller/ingenic,intc.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Ingenic SoCs interrupt controller devicetree bindings + +maintainers: + - Paul Cercueil <paul@crapouillou.net> + +properties: + $nodename: + pattern: "^interrupt-controller@[0-9a-f]+$" + + compatible: + oneOf: + - enum: + - ingenic,jz4740-intc + - ingenic,jz4760-intc + - ingenic,jz4780-intc + - items: + - enum: + - ingenic,jz4775-intc + - ingenic,jz4770-intc + - const: ingenic,jz4760-intc + - items: + - const: ingenic,x1000-intc + - const: ingenic,jz4780-intc + - items: + - const: ingenic,jz4725b-intc + - const: ingenic,jz4740-intc + + "#interrupt-cells": + const: 1 + + reg: + maxItems: 1 + + interrupts: + maxItems: 1 + + interrupt-controller: true + +required: + - compatible + - reg + - interrupts + - "#interrupt-cells" + - interrupt-controller + +examples: + - | + intc: interrupt-controller@10001000 { + compatible = "ingenic,jz4770-intc", "ingenic,jz4760-intc"; + reg = <0x10001000 0x40>; + + interrupt-controller; + #interrupt-cells = <1>; + + interrupt-parent = <&cpuintc>; + interrupts = <2>; + };
Convert the ingenic,intc.txt doc file to ingenic,intc.yaml. Some compatible strings now require a fallback, as the controller generally works the same across the SoCs families. Signed-off-by: Paul Cercueil <paul@crapouillou.net> --- .../interrupt-controller/ingenic,intc.txt | 28 --------- .../interrupt-controller/ingenic,intc.yaml | 63 +++++++++++++++++++ 2 files changed, 63 insertions(+), 28 deletions(-) delete mode 100644 Documentation/devicetree/bindings/interrupt-controller/ingenic,intc.txt create mode 100644 Documentation/devicetree/bindings/interrupt-controller/ingenic,intc.yaml