[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20200630100841.GA553950@ravnborg.org>
Date: Tue, 30 Jun 2020 12:08:41 +0200
From: Sam Ravnborg <sam@...nborg.org>
To: Paul Cercueil <paul@...pouillou.net>
Cc: David Airlie <airlied@...ux.ie>, Daniel Vetter <daniel@...ll.ch>,
Rob Herring <robh+dt@...nel.org>, devicetree@...r.kernel.org,
linux-mips@...r.kernel.org, dri-devel@...ts.freedesktop.org,
linux-kernel@...r.kernel.org, od@...c.me
Subject: Re: [PATCH v2 02/10] dt-bindings: display: Add ingenic,ipu.yaml
On Tue, Jun 30, 2020 at 01:52:02AM +0200, Paul Cercueil wrote:
> Add documentation of the Device Tree bindings for the Image Processing
> Unit (IPU) found in most Ingenic SoCs.
>
> Signed-off-by: Paul Cercueil <paul@...pouillou.net>
Do not know the IPU but looks correct.
Acked-by: Sam Ravnborg <sam@...nborg.org>
> ---
>
> Notes:
> v2: Add missing 'const' in items list
>
> .../bindings/display/ingenic,ipu.yaml | 65 +++++++++++++++++++
> 1 file changed, 65 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/display/ingenic,ipu.yaml
>
> diff --git a/Documentation/devicetree/bindings/display/ingenic,ipu.yaml b/Documentation/devicetree/bindings/display/ingenic,ipu.yaml
> new file mode 100644
> index 000000000000..5bfc33eb32c9
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/display/ingenic,ipu.yaml
> @@ -0,0 +1,65 @@
> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/display/ingenic,ipu.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Ingenic SoCs Image Processing Unit (IPU) devicetree bindings
> +
> +maintainers:
> + - Paul Cercueil <paul@...pouillou.net>
> +
> +properties:
> + compatible:
> + oneOf:
> + - enum:
> + - ingenic,jz4725b-ipu
> + - ingenic,jz4760-ipu
> + - items:
> + - const: ingenic,jz4770-ipu
> + - const: ingenic,jz4760-ipu
> +
> + reg:
> + maxItems: 1
> +
> + interrupts:
> + maxItems: 1
> +
> + clocks:
> + maxItems: 1
> +
> + clock-names:
> + const: ipu
> +
> +patternProperties:
> + "^ports?$":
> + description: OF graph bindings (specified in bindings/graph.txt).
> +
> +required:
> + - compatible
> + - reg
> + - interrupts
> + - clocks
> + - clock-names
> +
> +additionalProperties: false
> +
> +examples:
> + - |
> + #include <dt-bindings/clock/jz4770-cgu.h>
> + ipu@...80000 {
> + compatible = "ingenic,jz4770-ipu", "ingenic,jz4760-ipu";
> + reg = <0x13080000 0x800>;
> +
> + interrupt-parent = <&intc>;
> + interrupts = <29>;
> +
> + clocks = <&cgu JZ4770_CLK_IPU>;
> + clock-names = "ipu";
> +
> + port {
> + ipu_ep: endpoint {
> + remote-endpoint = <&lcdc_ep>;
> + };
> + };
> + };
> --
> 2.27.0
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@...ts.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
Powered by blists - more mailing lists