[v10,6/7] dt-bindings: media: wave5: add yaml devicetree bindings

Message ID 20221022000506.221933-7-sebastian.fricke@collabora.com
State New
Headers
Series Wave5 codec driver |

Commit Message

Sebastian Fricke Oct. 22, 2022, 12:05 a.m. UTC
  From: Robert Beckett <bob.beckett@collabora.com>

Add bindings for the wave5 chips&media codec driver

Signed-off-by: Robert Beckett <bob.beckett@collabora.com>
Signed-off-by: Dafna Hirschfeld <dafna.hirschfeld@collabora.com>
Signed-off-by: Sebastian Fricke <sebastian.fricke@collabora.com>
---
 .../devicetree/bindings/media/wave5.yml       | 73 +++++++++++++++++++
 1 file changed, 73 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/media/wave5.yml
  

Comments

Sebastian Fricke Oct. 23, 2022, 8:53 a.m. UTC | #1
Hey all,

I have noticed that I renamed the file incorrectly, in V11 it will be
called: "cnm,wave5.yml"

Greetings,
Sebastian

On 22.10.2022 02:05, Sebastian Fricke wrote:
>From: Robert Beckett <bob.beckett@collabora.com>
>
>Add bindings for the wave5 chips&media codec driver
>
>Signed-off-by: Robert Beckett <bob.beckett@collabora.com>
>Signed-off-by: Dafna Hirschfeld <dafna.hirschfeld@collabora.com>
>Signed-off-by: Sebastian Fricke <sebastian.fricke@collabora.com>
>---
> .../devicetree/bindings/media/wave5.yml       | 73 +++++++++++++++++++
> 1 file changed, 73 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/media/wave5.yml
>
>diff --git a/Documentation/devicetree/bindings/media/wave5.yml b/Documentation/devicetree/bindings/media/wave5.yml
>new file mode 100644
>index 000000000000..85dc3c744876
>--- /dev/null
>+++ b/Documentation/devicetree/bindings/media/wave5.yml
>@@ -0,0 +1,73 @@
>+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
>+%YAML 1.2
>+---
>+$id: http://devicetree.org/schemas/media/wave5.yaml#
>+$schema: http://devicetree.org/meta-schemas/core.yaml#
>+
>+title: Chips&Media Wave 5 Series multi-standard codec IP
>+
>+maintainers:
>+  - Nas Chung <nas.chung@chipsnmedia.com>
>+  - Robert Beckett <bob.beckett@collabora.com>
>+  - Sebastian Fricke <sebastian.fricke@collabora.com>
>+
>+description: |-
>+  The Chips&Media WAVE codec IP is a multi format video encoder/decoder
>+
>+properties:
>+  compatible:
>+    anyOf:
>+      - items:
>+        - enum:
>+            - cnm,cm511-vpu
>+            - cnm,cm517-vpu
>+            - cnm,cm521-vpu
>+            - cnm,cm521c-vpu
>+            - cnm,cm521c-dual-vpu
>+            - cnm,cm521e1-vpu
>+            - cnm,cm537-vpu
>+  reg:
>+    maxItems: 1
>+
>+  clocks:
>+    minItems: 1
>+    maxItems: 4
>+
>+  clock-names:
>+    minItems: 1
>+    maxItems: 4
>+
>+  interrupts:
>+    maxItems: 1
>+
>+  power-domains:
>+    maxItems: 1
>+
>+  resets:
>+    maxItems: 1
>+
>+  sram:
>+    $ref: /schemas/types.yaml#/definitions/phandle
>+    description: phandle pointing to the SRAM device node
>+    maxItems: 1
>+
>+required:
>+  - compatible
>+  - reg
>+  - interrupts
>+  - clocks
>+  - clock-names
>+
>+additionalProperties: false
>+
>+examples:
>+  - |
>+    vpu: video-codec@12345678 {
>+        compatible = "cnm,cm521-vpu";
>+        reg = <0x12345678 0x1000>;
>+        interrupts = <42>;
>+        clocks = <&clks 42>;
>+        clock-names = "vcodec";
>+        sram = <&sram>;
>+    };
>+
>-- 
>2.25.1
>
  
Krzysztof Kozlowski Oct. 28, 2022, 11:39 p.m. UTC | #2
On 23/10/2022 04:53, Sebastian Fricke wrote:
> Hey all,
> 
> I have noticed that I renamed the file incorrectly, in V11 it will be
> called: "cnm,wave5.yml"
> 

When resending be sure you cced proper people... You miss here a lot so
this clearly is not the process.

Best regards,
Krzysztof
  

Patch

diff --git a/Documentation/devicetree/bindings/media/wave5.yml b/Documentation/devicetree/bindings/media/wave5.yml
new file mode 100644
index 000000000000..85dc3c744876
--- /dev/null
+++ b/Documentation/devicetree/bindings/media/wave5.yml
@@ -0,0 +1,73 @@ 
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/media/wave5.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Chips&Media Wave 5 Series multi-standard codec IP
+
+maintainers:
+  - Nas Chung <nas.chung@chipsnmedia.com>
+  - Robert Beckett <bob.beckett@collabora.com>
+  - Sebastian Fricke <sebastian.fricke@collabora.com>
+
+description: |-
+  The Chips&Media WAVE codec IP is a multi format video encoder/decoder
+
+properties:
+  compatible:
+    anyOf:
+      - items:
+        - enum:
+            - cnm,cm511-vpu
+            - cnm,cm517-vpu
+            - cnm,cm521-vpu
+            - cnm,cm521c-vpu
+            - cnm,cm521c-dual-vpu
+            - cnm,cm521e1-vpu
+            - cnm,cm537-vpu
+  reg:
+    maxItems: 1
+
+  clocks:
+    minItems: 1
+    maxItems: 4
+
+  clock-names:
+    minItems: 1
+    maxItems: 4
+
+  interrupts:
+    maxItems: 1
+
+  power-domains:
+    maxItems: 1
+
+  resets:
+    maxItems: 1
+
+  sram:
+    $ref: /schemas/types.yaml#/definitions/phandle
+    description: phandle pointing to the SRAM device node
+    maxItems: 1
+
+required:
+  - compatible
+  - reg
+  - interrupts
+  - clocks
+  - clock-names
+
+additionalProperties: false
+
+examples:
+  - |
+    vpu: video-codec@12345678 {
+        compatible = "cnm,cm521-vpu";
+        reg = <0x12345678 0x1000>;
+        interrupts = <42>;
+        clocks = <&clks 42>;
+        clock-names = "vcodec";
+        sram = <&sram>;
+    };
+