[PATCH V2 0/2] dt-bindings: soc: bcm: Convert brcm,bcm2835-vchiq to json-schemas

[PATCH V2 0/2] dt-bindings: soc: bcm: Convert brcm,bcm2835-vchiq to json-schemas

[Stefan Wahren]

This series converts the VCHIQ bindings to YAML format.

Changes in V2:
- drop some unnecessary points in the binding as suggested by Rob Herring

Stefan Wahren (2):
  ARM: dts: bcm283x: Drop fallback compatible for vchiq
  dt-bindings: soc: bcm: Convert brcm,bcm2835-vchiq to json-schema

 .../bindings/soc/bcm/brcm,bcm2835-vchiq.txt        | 17 --------
 .../bindings/soc/bcm/brcm,bcm2835-vchiq.yaml       | 45 ++++++++++++++++++++++
 arch/arm/boot/dts/bcm2836-rpi.dtsi                 |  2 +-
 3 files changed, 46 insertions(+), 18 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/soc/bcm/brcm,bcm2835-vchiq.txt
 create mode 100644 Documentation/devicetree/bindings/soc/bcm/brcm,bcm2835-vchiq.yaml

-- 
2.7.4

[PATCH V2 1/2] ARM: dts: bcm283x: Drop fallback compatible for vchiq

[Stefan Wahren]

With introduction of SoC specific vchiq compatible in 2018, we kept the
original "brcm,bcm2835-vchiq" as a fallback for older Kernel versions.
Now this fallback isn't necessary anymore. So before we create a DT schema
which also handles this fallback better fix this on the DTS side.

Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
---
 arch/arm/boot/dts/bcm2836-rpi.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/bcm2836-rpi.dtsi b/arch/arm/boot/dts/bcm2836-rpi.dtsi
index c4c858b..f98e0e4 100644
--- a/arch/arm/boot/dts/bcm2836-rpi.dtsi
+++ b/arch/arm/boot/dts/bcm2836-rpi.dtsi
@@ -2,5 +2,5 @@
 #include "bcm2835-rpi.dtsi"
 
 &vchiq {
-	compatible = "brcm,bcm2836-vchiq", "brcm,bcm2835-vchiq";
+	compatible = "brcm,bcm2836-vchiq";
 };
-- 
2.7.4

Re: [PATCH V2 1/2] ARM: dts: bcm283x: Drop fallback compatible for vchiq

[Stefan Wahren]

Am 31.10.21 um 13:08 schrieb Stefan Wahren:
> With introduction of SoC specific vchiq compatible in 2018, we kept the
> original "brcm,bcm2835-vchiq" as a fallback for older Kernel versions.
> Now this fallback isn't necessary anymore. So before we create a DT schema
> which also handles this fallback better fix this on the DTS side.
>
> Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
> ---
>  arch/arm/boot/dts/bcm2836-rpi.dtsi | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/bcm2836-rpi.dtsi b/arch/arm/boot/dts/bcm2836-rpi.dtsi
> index c4c858b..f98e0e4 100644
> --- a/arch/arm/boot/dts/bcm2836-rpi.dtsi
> +++ b/arch/arm/boot/dts/bcm2836-rpi.dtsi
> @@ -2,5 +2,5 @@
>  #include "bcm2835-rpi.dtsi"
>  
>  &vchiq {
> -	compatible = "brcm,bcm2836-vchiq", "brcm,bcm2835-vchiq";
> +	compatible = "brcm,bcm2836-vchiq";
>  };
gentle ping ...

Re: [PATCH V2 1/2] ARM: dts: bcm283x: Drop fallback compatible for vchiq

[nicolas saenz julienne]

On Sat, 2021-11-27 at 11:57 +0100, Stefan Wahren wrote:
> Am 31.10.21 um 13:08 schrieb Stefan Wahren:
> > With introduction of SoC specific vchiq compatible in 2018, we kept the
> > original "brcm,bcm2835-vchiq" as a fallback for older Kernel versions.
> > Now this fallback isn't necessary anymore. So before we create a DT schema
> > which also handles this fallback better fix this on the DTS side.
> > 
> > Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
> > ---
> >  arch/arm/boot/dts/bcm2836-rpi.dtsi | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/arch/arm/boot/dts/bcm2836-rpi.dtsi b/arch/arm/boot/dts/bcm2836-rpi.dtsi
> > index c4c858b..f98e0e4 100644
> > --- a/arch/arm/boot/dts/bcm2836-rpi.dtsi
> > +++ b/arch/arm/boot/dts/bcm2836-rpi.dtsi
> > @@ -2,5 +2,5 @@
> >  #include "bcm2835-rpi.dtsi"
> >  
> >  &vchiq {
> > -	compatible = "brcm,bcm2836-vchiq", "brcm,bcm2835-vchiq";
> > +	compatible = "brcm,bcm2836-vchiq";
> >  };
> gentle ping ...

Sorry for the wait. :)

[PATCH V2 2/2] dt-bindings: soc: bcm: Convert brcm,bcm2835-vchiq to json-schema

[Stefan Wahren]

This converts the VCHIQ bindings to YAML format.

Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
---
 .../bindings/soc/bcm/brcm,bcm2835-vchiq.txt        | 17 --------
 .../bindings/soc/bcm/brcm,bcm2835-vchiq.yaml       | 45 ++++++++++++++++++++++
 2 files changed, 45 insertions(+), 17 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/soc/bcm/brcm,bcm2835-vchiq.txt
 create mode 100644 Documentation/devicetree/bindings/soc/bcm/brcm,bcm2835-vchiq.yaml

diff --git a/Documentation/devicetree/bindings/soc/bcm/brcm,bcm2835-vchiq.txt b/Documentation/devicetree/bindings/soc/bcm/brcm,bcm2835-vchiq.txt
deleted file mode 100644
index f331316..0000000
--- a/Documentation/devicetree/bindings/soc/bcm/brcm,bcm2835-vchiq.txt
+++ /dev/null
@@ -1,17 +0,0 @@
-Broadcom VCHIQ firmware services
-
-Required properties:
-
-- compatible:	Should be "brcm,bcm2835-vchiq" on BCM2835, otherwise
-		"brcm,bcm2836-vchiq".
-- reg:		Physical base address and length of the doorbell register pair
-- interrupts:	The interrupt number
-		  See bindings/interrupt-controller/brcm,bcm2835-armctrl-ic.txt
-
-Example:
-
-mailbox@7e00b840 {
-	compatible = "brcm,bcm2835-vchiq";
-	reg = <0x7e00b840 0xf>;
-	interrupts = <0 2>;
-};
diff --git a/Documentation/devicetree/bindings/soc/bcm/brcm,bcm2835-vchiq.yaml b/Documentation/devicetree/bindings/soc/bcm/brcm,bcm2835-vchiq.yaml
new file mode 100644
index 0000000..a1d5cd4
--- /dev/null
+++ b/Documentation/devicetree/bindings/soc/bcm/brcm,bcm2835-vchiq.yaml
@@ -0,0 +1,45 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/soc/bcm/brcm,bcm2835-vchiq.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Broadcom VCHIQ firmware services
+
+maintainers:
+  - Nicolas Saenz Julienne <nsaenz@kernel.org>
+
+description:
+  The VCHIQ communication channel can be provided by BCM283x and Capri SoCs,
+  to communicate with the VPU-side OS services.
+
+properties:
+  compatible:
+    enum:
+      - brcm,bcm2835-vchiq
+      - brcm,bcm2836-vchiq
+
+  reg:
+    description: Physical base address and length of the doorbell register pair
+    minItems: 1
+
+  interrupts:
+    description: Interrupt number of the doorbell interrupt
+    minItems: 1
+
+required:
+  - compatible
+  - reg
+  - interrupts
+
+additionalProperties: false
+
+examples:
+  - |
+    mailbox@7e00b840 {
+      compatible = "brcm,bcm2835-vchiq";
+      reg = <0x7e00b840 0xf>;
+      interrupts = <0 2>;
+    };
+
+...
-- 
2.7.4

Re: [PATCH V2 2/2] dt-bindings: soc: bcm: Convert brcm,bcm2835-vchiq to json-schema

[Rob Herring]

On Sun, 31 Oct 2021 13:08:06 +0100, Stefan Wahren wrote:
> This converts the VCHIQ bindings to YAML format.
> 
> Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
> ---
>  .../bindings/soc/bcm/brcm,bcm2835-vchiq.txt        | 17 --------
>  .../bindings/soc/bcm/brcm,bcm2835-vchiq.yaml       | 45 ++++++++++++++++++++++
>  2 files changed, 45 insertions(+), 17 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/soc/bcm/brcm,bcm2835-vchiq.txt
>  create mode 100644 Documentation/devicetree/bindings/soc/bcm/brcm,bcm2835-vchiq.yaml
> 

Reviewed-by: Rob Herring <robh@kernel.org>

Re: [PATCH V2 0/2] dt-bindings: soc: bcm: Convert brcm,bcm2835-vchiq to json-schemas

[nicolas saenz julienne]

On Sun, 2021-10-31 at 13:08 +0100, Stefan Wahren wrote:
> This series converts the VCHIQ bindings to YAML format.


Applied for-next,

Thanks!