diff mbox series

dt-bindings: arm: Convert RDA Micro board/soc bindings to json-schema

Message ID 20190618212229.32302-4-robh@kernel.org (mailing list archive)
State Mainlined
Commit f21ce913a82faa512b583640226d4ab39b38735e
Headers show
Series dt-bindings: arm: Convert RDA Micro board/soc bindings to json-schema | expand

Commit Message

Rob Herring June 18, 2019, 9:22 p.m. UTC
Convert RDA Micro SoC bindings to DT schema format using json-schema.

Cc: "Andreas Färber" <afaerber@suse.de>
Cc: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Signed-off-by: Rob Herring <robh@kernel.org>
---
Andreas, Update the license on this one too to dual licensed if you 
want.

 Documentation/devicetree/bindings/arm/rda.txt | 17 ----------------
 .../devicetree/bindings/arm/rda.yaml          | 20 +++++++++++++++++++
 MAINTAINERS                                   |  2 +-
 3 files changed, 21 insertions(+), 18 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/arm/rda.txt
 create mode 100644 Documentation/devicetree/bindings/arm/rda.yaml

Comments

Rob Herring July 9, 2019, 5:03 p.m. UTC | #1
On Tue, Jun 18, 2019 at 03:22:28PM -0600, Rob Herring wrote:
> Convert RDA Micro SoC bindings to DT schema format using json-schema.
> 
> Cc: "Andreas Färber" <afaerber@suse.de>
> Cc: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
> Andreas, Update the license on this one too to dual licensed if you 
> want.

Ping.

> 
>  Documentation/devicetree/bindings/arm/rda.txt | 17 ----------------
>  .../devicetree/bindings/arm/rda.yaml          | 20 +++++++++++++++++++
>  MAINTAINERS                                   |  2 +-
>  3 files changed, 21 insertions(+), 18 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/arm/rda.txt
>  create mode 100644 Documentation/devicetree/bindings/arm/rda.yaml
Manivannan Sadhasivam July 9, 2019, 5:15 p.m. UTC | #2
Hi Rob,

On Tue, Jun 18, 2019 at 03:22:28PM -0600, Rob Herring wrote:
> Convert RDA Micro SoC bindings to DT schema format using json-schema.
> 
> Cc: "Andreas Färber" <afaerber@suse.de>
> Cc: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
> Andreas, Update the license on this one too to dual licensed if you 
> want.

I'm fine with GPL-2.0. Since there isn't any other RDA specific patches
in my tree, you want to take this patch? Else I have to send the Pull
Request to ARM SoC folks with this patch alone.

Thanks,
Mani

> 
>  Documentation/devicetree/bindings/arm/rda.txt | 17 ----------------
>  .../devicetree/bindings/arm/rda.yaml          | 20 +++++++++++++++++++
>  MAINTAINERS                                   |  2 +-
>  3 files changed, 21 insertions(+), 18 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/arm/rda.txt
>  create mode 100644 Documentation/devicetree/bindings/arm/rda.yaml
> 
> diff --git a/Documentation/devicetree/bindings/arm/rda.txt b/Documentation/devicetree/bindings/arm/rda.txt
> deleted file mode 100644
> index 43c80762c428..000000000000
> --- a/Documentation/devicetree/bindings/arm/rda.txt
> +++ /dev/null
> @@ -1,17 +0,0 @@
> -RDA Micro platforms device tree bindings
> -----------------------------------------
> -
> -RDA8810PL SoC
> -=============
> -
> -Required root node properties:
> -
> - - compatible :  must contain "rda,8810pl"
> -
> -
> -Boards:
> -
> -Root node property compatible must contain, depending on board:
> -
> - - Orange Pi 2G-IoT: "xunlong,orangepi-2g-iot"
> - - Orange Pi i96: "xunlong,orangepi-i96"
> diff --git a/Documentation/devicetree/bindings/arm/rda.yaml b/Documentation/devicetree/bindings/arm/rda.yaml
> new file mode 100644
> index 000000000000..51cec2b63b04
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/arm/rda.yaml
> @@ -0,0 +1,20 @@
> +# SPDX-License-Identifier: GPL-2.0
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/arm/rda.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: RDA Micro platforms device tree bindings
> +
> +maintainers:
> +  - Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> +
> +properties:
> +  compatible:
> +    items:
> +      - enum:
> +          - xunlong,orangepi-2g-iot     # Orange Pi 2G-IoT
> +          - xunlong,orangepi-i96        # Orange Pi i96
> +      - const: rda,8810pl
> +
> +...
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 2c6f4d15805e..56ee276088eb 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -2111,7 +2111,7 @@ F:	arch/arm/boot/dts/rda8810pl-*
>  F:	drivers/clocksource/timer-rda.c
>  F:	drivers/irqchip/irq-rda-intc.c
>  F:	drivers/tty/serial/rda-uart.c
> -F:	Documentation/devicetree/bindings/arm/rda.txt
> +F:	Documentation/devicetree/bindings/arm/rda.yaml
>  F:	Documentation/devicetree/bindings/interrupt-controller/rda,8810pl-intc.txt
>  F:	Documentation/devicetree/bindings/serial/rda,8810pl-uart.txt
>  F:	Documentation/devicetree/bindings/timer/rda,8810pl-timer.txt
> -- 
> 2.20.1
>
Rob Herring July 9, 2019, 6:29 p.m. UTC | #3
On Tue, Jul 9, 2019 at 11:15 AM Manivannan Sadhasivam
<manivannan.sadhasivam@linaro.org> wrote:
>
> Hi Rob,
>
> On Tue, Jun 18, 2019 at 03:22:28PM -0600, Rob Herring wrote:
> > Convert RDA Micro SoC bindings to DT schema format using json-schema.
> >
> > Cc: "Andreas Färber" <afaerber@suse.de>
> > Cc: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> > Signed-off-by: Rob Herring <robh@kernel.org>
> > ---
> > Andreas, Update the license on this one too to dual licensed if you
> > want.
>
> I'm fine with GPL-2.0. Since there isn't any other RDA specific patches
> in my tree, you want to take this patch? Else I have to send the Pull
> Request to ARM SoC folks with this patch alone.

Okay, I can take it. The preference is actually dual licensed which is
why I asked, but that can be done later.

Rob
Rob Herring July 9, 2019, 6:41 p.m. UTC | #4
On Tue, Jul 9, 2019 at 12:29 PM Rob Herring <robh@kernel.org> wrote:
>
> On Tue, Jul 9, 2019 at 11:15 AM Manivannan Sadhasivam
> <manivannan.sadhasivam@linaro.org> wrote:
> >
> > Hi Rob,
> >
> > On Tue, Jun 18, 2019 at 03:22:28PM -0600, Rob Herring wrote:
> > > Convert RDA Micro SoC bindings to DT schema format using json-schema.
> > >
> > > Cc: "Andreas Färber" <afaerber@suse.de>
> > > Cc: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> > > Signed-off-by: Rob Herring <robh@kernel.org>
> > > ---
> > > Andreas, Update the license on this one too to dual licensed if you
> > > want.
> >
> > I'm fine with GPL-2.0. Since there isn't any other RDA specific patches
> > in my tree, you want to take this patch? Else I have to send the Pull
> > Request to ARM SoC folks with this patch alone.
>
> Okay, I can take it. The preference is actually dual licensed which is
> why I asked, but that can be done later.

And is that an ack from you?

Rob
Manivannan Sadhasivam July 9, 2019, 6:44 p.m. UTC | #5
On Tue, Jul 09, 2019 at 12:41:19PM -0600, Rob Herring wrote:
> On Tue, Jul 9, 2019 at 12:29 PM Rob Herring <robh@kernel.org> wrote:
> >
> > On Tue, Jul 9, 2019 at 11:15 AM Manivannan Sadhasivam
> > <manivannan.sadhasivam@linaro.org> wrote:
> > >
> > > Hi Rob,
> > >
> > > On Tue, Jun 18, 2019 at 03:22:28PM -0600, Rob Herring wrote:
> > > > Convert RDA Micro SoC bindings to DT schema format using json-schema.
> > > >
> > > > Cc: "Andreas Färber" <afaerber@suse.de>
> > > > Cc: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> > > > Signed-off-by: Rob Herring <robh@kernel.org>
> > > > ---
> > > > Andreas, Update the license on this one too to dual licensed if you
> > > > want.
> > >
> > > I'm fine with GPL-2.0. Since there isn't any other RDA specific patches
> > > in my tree, you want to take this patch? Else I have to send the Pull
> > > Request to ARM SoC folks with this patch alone.
> >
> > Okay, I can take it. The preference is actually dual licensed which is
> > why I asked, but that can be done later.
> 
> And is that an ack from you?

Yes, here is the formal tag:

Acked-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>

Also, Andreas asked to switch to Dual license for Actions platform since he
is the maintainer of it. But for RDA he is not, so I don't think there will
be a concern here (I'm fine with GPL-2.0).

Thanks,
Mani
> 
> Rob
Rob Herring July 9, 2019, 7:02 p.m. UTC | #6
On Tue, Jul 9, 2019 at 12:45 PM Manivannan Sadhasivam
<manivannan.sadhasivam@linaro.org> wrote:
>
> On Tue, Jul 09, 2019 at 12:41:19PM -0600, Rob Herring wrote:
> > On Tue, Jul 9, 2019 at 12:29 PM Rob Herring <robh@kernel.org> wrote:
> > >
> > > On Tue, Jul 9, 2019 at 11:15 AM Manivannan Sadhasivam
> > > <manivannan.sadhasivam@linaro.org> wrote:
> > > >
> > > > Hi Rob,
> > > >
> > > > On Tue, Jun 18, 2019 at 03:22:28PM -0600, Rob Herring wrote:
> > > > > Convert RDA Micro SoC bindings to DT schema format using json-schema.
> > > > >
> > > > > Cc: "Andreas Färber" <afaerber@suse.de>
> > > > > Cc: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> > > > > Signed-off-by: Rob Herring <robh@kernel.org>
> > > > > ---
> > > > > Andreas, Update the license on this one too to dual licensed if you
> > > > > want.
> > > >
> > > > I'm fine with GPL-2.0. Since there isn't any other RDA specific patches
> > > > in my tree, you want to take this patch? Else I have to send the Pull
> > > > Request to ARM SoC folks with this patch alone.
> > >
> > > Okay, I can take it. The preference is actually dual licensed which is
> > > why I asked, but that can be done later.
> >
> > And is that an ack from you?
>
> Yes, here is the formal tag:
>
> Acked-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
>
> Also, Andreas asked to switch to Dual license for Actions platform since he
> is the maintainer of it. But for RDA he is not, so I don't think there will
> be a concern here (I'm fine with GPL-2.0).

It's authorship that matters and Andreas was the author on the
original file. The maintainer only has a say in what license they'll
accept.

Rob
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/arm/rda.txt b/Documentation/devicetree/bindings/arm/rda.txt
deleted file mode 100644
index 43c80762c428..000000000000
--- a/Documentation/devicetree/bindings/arm/rda.txt
+++ /dev/null
@@ -1,17 +0,0 @@ 
-RDA Micro platforms device tree bindings
-----------------------------------------
-
-RDA8810PL SoC
-=============
-
-Required root node properties:
-
- - compatible :  must contain "rda,8810pl"
-
-
-Boards:
-
-Root node property compatible must contain, depending on board:
-
- - Orange Pi 2G-IoT: "xunlong,orangepi-2g-iot"
- - Orange Pi i96: "xunlong,orangepi-i96"
diff --git a/Documentation/devicetree/bindings/arm/rda.yaml b/Documentation/devicetree/bindings/arm/rda.yaml
new file mode 100644
index 000000000000..51cec2b63b04
--- /dev/null
+++ b/Documentation/devicetree/bindings/arm/rda.yaml
@@ -0,0 +1,20 @@ 
+# SPDX-License-Identifier: GPL-2.0
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/arm/rda.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: RDA Micro platforms device tree bindings
+
+maintainers:
+  - Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
+
+properties:
+  compatible:
+    items:
+      - enum:
+          - xunlong,orangepi-2g-iot     # Orange Pi 2G-IoT
+          - xunlong,orangepi-i96        # Orange Pi i96
+      - const: rda,8810pl
+
+...
diff --git a/MAINTAINERS b/MAINTAINERS
index 2c6f4d15805e..56ee276088eb 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2111,7 +2111,7 @@  F:	arch/arm/boot/dts/rda8810pl-*
 F:	drivers/clocksource/timer-rda.c
 F:	drivers/irqchip/irq-rda-intc.c
 F:	drivers/tty/serial/rda-uart.c
-F:	Documentation/devicetree/bindings/arm/rda.txt
+F:	Documentation/devicetree/bindings/arm/rda.yaml
 F:	Documentation/devicetree/bindings/interrupt-controller/rda,8810pl-intc.txt
 F:	Documentation/devicetree/bindings/serial/rda,8810pl-uart.txt
 F:	Documentation/devicetree/bindings/timer/rda,8810pl-timer.txt