diff mbox series

[V3,3/5] dt-bindings: gpio: Add bindings for gpio-virtio

Message ID 605b87ce93b9a528810a5857984bfe6b913d5e73.1627273794.git.viresh.kumar@linaro.org
State Superseded
Headers show
Series virtio: Add virtio-device bindings | expand

Commit Message

Viresh Kumar July 26, 2021, 4:51 a.m. UTC
This patch adds binding for virtio GPIO controller, it is based on
virtio-device bindings.

Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>

---
 .../devicetree/bindings/gpio/gpio-virtio.yaml | 60 +++++++++++++++++++
 1 file changed, 60 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/gpio/gpio-virtio.yaml

-- 
2.31.1.272.g89b43f80a514

Comments

Arnd Bergmann July 26, 2021, 8:13 a.m. UTC | #1
On Mon, Jul 26, 2021 at 6:53 AM Viresh Kumar <viresh.kumar@linaro.org> wrote:
>
> This patch adds binding for virtio GPIO controller, it is based on
> virtio-device bindings.
>
> Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>

Reviewed-by: Arnd Bergmann <arnd@arndb.de>, except

> +
> +properties:
> +  $nodename:
> +    pattern: '^gpio-virtio(-[a-z0-9]+)?$'
> +
> +        gpio: gpio-virtio {
> +            compatible = "virtio,29";

The node name here does not appear to be mandated by the schema, but
most others name it "gpio", so I would do the same here instead of
"gpio-virtio".

       Arnd
Rob Herring July 26, 2021, 3:11 p.m. UTC | #2
On Sun, Jul 25, 2021 at 10:52 PM Viresh Kumar <viresh.kumar@linaro.org> wrote:
>

> This patch adds binding for virtio GPIO controller, it is based on

> virtio-device bindings.

>

> Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>

> ---

>  .../devicetree/bindings/gpio/gpio-virtio.yaml | 60 +++++++++++++++++++

>  1 file changed, 60 insertions(+)

>  create mode 100644 Documentation/devicetree/bindings/gpio/gpio-virtio.yaml

>

> diff --git a/Documentation/devicetree/bindings/gpio/gpio-virtio.yaml b/Documentation/devicetree/bindings/gpio/gpio-virtio.yaml

> new file mode 100644

> index 000000000000..96108cfb7a08

> --- /dev/null

> +++ b/Documentation/devicetree/bindings/gpio/gpio-virtio.yaml

> @@ -0,0 +1,60 @@

> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)

> +%YAML 1.2

> +---

> +$id: http://devicetree.org/schemas/gpio/gpio-virtio.yaml#

> +$schema: http://devicetree.org/meta-schemas/core.yaml#

> +

> +title: Virtio GPIO controller

> +

> +maintainers:

> +  - Viresh Kumar <viresh.kumar@linaro.org>

> +

> +allOf:

> +  - $ref: /schemas/gpio/gpio.yaml#


You don't need to include this.

> +  - $ref: /schemas/virtio/virtio-device.yaml#

> +

> +description:

> +  Virtio GPIO controller, see /schemas/virtio/virtio-device.yaml for more

> +  details.

> +

> +properties:

> +  $nodename:

> +    pattern: '^gpio-virtio(-[a-z0-9]+)?$'

> +

> +  compatible:

> +    const: virtio,29

> +

> +  gpio-controller: true

> +

> +  "#gpio-cells":

> +    const: 2

> +

> +  interrupt-controller: true

> +

> +  "#interrupt-cells":

> +    const: 2

> +

> +required:

> +  - compatible

> +  - gpio-controller

> +  - "#gpio-cells"

> +

> +unevaluatedProperties: false

> +

> +examples:

> +  - |

> +    virtio@3000 {

> +        compatible = "virtio,mmio";

> +        reg = <0x3000 0x100>;

> +        interrupts = <41>;

> +

> +        gpio: gpio-virtio {

> +            compatible = "virtio,29";

> +            gpio-controller;

> +            #gpio-cells = <2>;

> +            interrupt-controller;

> +            #interrupt-cells = <2>;

> +        };

> +    };

> +

> +...

> --

> 2.31.1.272.g89b43f80a514

>
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/gpio/gpio-virtio.yaml b/Documentation/devicetree/bindings/gpio/gpio-virtio.yaml
new file mode 100644
index 000000000000..96108cfb7a08
--- /dev/null
+++ b/Documentation/devicetree/bindings/gpio/gpio-virtio.yaml
@@ -0,0 +1,60 @@ 
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/gpio/gpio-virtio.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Virtio GPIO controller
+
+maintainers:
+  - Viresh Kumar <viresh.kumar@linaro.org>
+
+allOf:
+  - $ref: /schemas/gpio/gpio.yaml#
+  - $ref: /schemas/virtio/virtio-device.yaml#
+
+description:
+  Virtio GPIO controller, see /schemas/virtio/virtio-device.yaml for more
+  details.
+
+properties:
+  $nodename:
+    pattern: '^gpio-virtio(-[a-z0-9]+)?$'
+
+  compatible:
+    const: virtio,29
+
+  gpio-controller: true
+
+  "#gpio-cells":
+    const: 2
+
+  interrupt-controller: true
+
+  "#interrupt-cells":
+    const: 2
+
+required:
+  - compatible
+  - gpio-controller
+  - "#gpio-cells"
+
+unevaluatedProperties: false
+
+examples:
+  - |
+    virtio@3000 {
+        compatible = "virtio,mmio";
+        reg = <0x3000 0x100>;
+        interrupts = <41>;
+
+        gpio: gpio-virtio {
+            compatible = "virtio,29";
+            gpio-controller;
+            #gpio-cells = <2>;
+            interrupt-controller;
+            #interrupt-cells = <2>;
+        };
+    };
+
+...