[v3,02/15] dt-bindings: reset: Convert Broadcom STB reset to YAML

Commit Message

Florian Fainelli Dec. 8, 2021, 12:37 a.m. UTC

Convert the Broadcom STB SW_INIT style reset controller binding to YAML.

Signed-off-by: Florian Fainelli <[e-mail]>
---
 .../bindings/reset/brcm,brcmstb-reset.txt     | 27 -----------
 .../bindings/reset/brcm,brcmstb-reset.yaml    | 48 +++++++++++++++++++
 2 files changed, 48 insertions(+), 27 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/reset/brcm,brcmstb-reset.txt
 create mode 100644 Documentation/devicetree/bindings/reset/brcm,brcmstb-reset.yaml

Comments

Philipp Zabel Dec. 9, 2021, 9:41 a.m. UTC | #1

On Tue, 2021-12-07 at 16:37 -0800, Florian Fainelli wrote:
> Convert the Broadcom STB SW_INIT style reset controller binding to YAML.
> 
> Signed-off-by: Florian Fainelli <[e-mail]>

Acked-by: Philipp Zabel <[e-mail]>

regards
Philipp

Florian Fainelli Dec. 9, 2021, 11:25 p.m. UTC | #2

On 12/9/21 1:41 AM, Philipp Zabel wrote:
> On Tue, 2021-12-07 at 16:37 -0800, Florian Fainelli wrote:
>> Convert the Broadcom STB SW_INIT style reset controller binding to YAML.
>>
>> Signed-off-by: Florian Fainelli <[e-mail]>
> 
> Acked-by: Philipp Zabel <[e-mail]>

Thanks, sorry for not carrying your Ack that you provided in v2 already.

Rob Herring (Arm) Dec. 14, 2021, 5:47 p.m. UTC | #3

On Tue, 07 Dec 2021 16:37:13 -0800, Florian Fainelli wrote:
> Convert the Broadcom STB SW_INIT style reset controller binding to YAML.
> 
> Signed-off-by: Florian Fainelli <[e-mail]>
> ---
>  .../bindings/reset/brcm,brcmstb-reset.txt     | 27 -----------
>  .../bindings/reset/brcm,brcmstb-reset.yaml    | 48 +++++++++++++++++++
>  2 files changed, 48 insertions(+), 27 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/reset/brcm,brcmstb-reset.txt
>  create mode 100644 Documentation/devicetree/bindings/reset/brcm,brcmstb-reset.yaml
> 

Applied, thanks!

Patch

diff --git a/Documentation/devicetree/bindings/reset/brcm,brcmstb-reset.txt b/Documentation/devicetree/bindings/reset/brcm,brcmstb-reset.txt
deleted file mode 100644
index ee59409640f2..000000000000
--- a/Documentation/devicetree/bindings/reset/brcm,brcmstb-reset.txt
+++ /dev/null
@@ -1,27 +0,0 @@ 
-Broadcom STB SW_INIT-style reset controller
-===========================================
-
-Broadcom STB SoCs have a SW_INIT-style reset controller with separate
-SET/CLEAR/STATUS registers and possibly multiple banks, each of 32 bit
-reset lines.
-
-Please also refer to reset.txt in this directory for common reset
-controller binding usage.
-
-Required properties:
-- compatible: should be brcm,brcmstb-reset
-- reg: register base and length
-- #reset-cells: must be set to 1
-
-Example:
-
-	reset: reset-controller@8404318 {
-		compatible = "brcm,brcmstb-reset";
-		reg = <0x8404318 0x30>;
-		#reset-cells = <1>;
-	};
-
-	&ethernet_switch {
-		resets = <&reset 26>;
-		reset-names = "switch";
-	};
diff --git a/Documentation/devicetree/bindings/reset/brcm,brcmstb-reset.yaml b/Documentation/devicetree/bindings/reset/brcm,brcmstb-reset.yaml
new file mode 100644
index 000000000000..e00efa88a198
--- /dev/null
+++ b/Documentation/devicetree/bindings/reset/brcm,brcmstb-reset.yaml
@@ -0,0 +1,48 @@ 
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: "http://devicetree.org/schemas/reset/brcm,brcmstb-reset.yaml#"
+$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+
+title: Broadcom STB SW_INIT-style reset controller
+
+description:
+  Broadcom STB SoCs have a SW_INIT-style reset controller with separate
+  SET/CLEAR/STATUS registers and possibly multiple banks, each of 32 bit
+  reset lines.
+
+  Please also refer to reset.txt in this directory for common reset
+  controller binding usage.
+
+maintainers:
+  - Florian Fainelli <[e-mail]>
+
+properties:
+  compatible:
+    const: brcm,brcmstb-reset
+
+  reg:
+    maxItems: 1
+
+  "#reset-cells":
+    const: 1
+
+required:
+  - compatible
+  - reg
+  - "#reset-cells"
+
+additionalProperties: false
+
+examples:
+  - |
+    reset: reset-controller@8404318 {
+      compatible = "brcm,brcmstb-reset";
+      reg = <0x8404318 0x30>;
+      #reset-cells = <1>;
+    };
+
+    ethernet_switch {
+      resets = <&reset 26>;
+      reset-names = "switch";
+    };