Re: [PATCH v2 26/34] dt-bindings: arm: Convert Renesas board/soc bindings to json-schema

2018-12-10 Thread Simon Horman
On Thu, Dec 06, 2018 at 01:38:42PM -0600, Rob Herring wrote:
> On Wed, Dec 5, 2018 at 1:44 PM Simon Horman  wrote:
> >
> > On Tue, Dec 04, 2018 at 09:08:57AM -0600, Rob Herring wrote:
> > > On Tue, Dec 4, 2018 at 8:57 AM Geert Uytterhoeven  
> > > wrote:
> > > >
> > > > Hi Simon,
> > > >
> > > > On Tue, Dec 4, 2018 at 3:48 PM Simon Horman  wrote:
> > > > > On Mon, Dec 03, 2018 at 03:32:15PM -0600, Rob Herring wrote:
> > > > > > Convert Renesas SoC bindings to DT schema format using json-schema.
> > > > > >
> > > > > > Cc: Simon Horman 
> > > > > > Cc: Magnus Damm 
> > > > > > Cc: Mark Rutland 
> > > > > > Cc: linux-renesas-...@vger.kernel.org
> > > > > > Cc: devicet...@vger.kernel.org
> > > > > > Signed-off-by: Rob Herring 
> > > > > > ---
> > > > > >  .../devicetree/bindings/arm/shmobile.txt  | 151 
> > > > > >  .../devicetree/bindings/arm/shmobile.yaml | 218 
> > > > > > ++
> > > > > >  2 files changed, 218 insertions(+), 151 deletions(-)
> > > > > >  delete mode 100644 
> > > > > > Documentation/devicetree/bindings/arm/shmobile.txt
> > > > > >  create mode 100644 
> > > > > > Documentation/devicetree/bindings/arm/shmobile.yaml
> > > > >
> > > > > Hi Rob,
> > > > >
> > > > > what is this based on? I get a conflict when applying the .txt change
> > > > > and if I knew the base for this patch it would be rather easy to work
> > > > > out what has changed.
> > >
> > > 4.20-rc2
> > >
> > > > >
> > > > > Also, should we do an s/shmobile.txt/shmobile.yaml/ in MAINTAINERS?
> > >
> > > Yes. Though it was pointed out that get_maintainers.pl can pull emails
> > > out of this file. We'd need to get that to work by default though.
> > >
> > > > Probably even s/shmobile.yaml/renesas.yaml/, while at it?
> > >
> > > Sure, if that's what you all want.
> >
> > How about this?
> 
> LGTM

Thanks.

As my tree is already closed for v4.21 I have applied this for v4.22.


Re: [PATCH v2 26/34] dt-bindings: arm: Convert Renesas board/soc bindings to json-schema

2018-12-06 Thread Rob Herring
On Wed, Dec 5, 2018 at 1:44 PM Simon Horman  wrote:
>
> On Tue, Dec 04, 2018 at 09:08:57AM -0600, Rob Herring wrote:
> > On Tue, Dec 4, 2018 at 8:57 AM Geert Uytterhoeven  
> > wrote:
> > >
> > > Hi Simon,
> > >
> > > On Tue, Dec 4, 2018 at 3:48 PM Simon Horman  wrote:
> > > > On Mon, Dec 03, 2018 at 03:32:15PM -0600, Rob Herring wrote:
> > > > > Convert Renesas SoC bindings to DT schema format using json-schema.
> > > > >
> > > > > Cc: Simon Horman 
> > > > > Cc: Magnus Damm 
> > > > > Cc: Mark Rutland 
> > > > > Cc: linux-renesas-...@vger.kernel.org
> > > > > Cc: devicet...@vger.kernel.org
> > > > > Signed-off-by: Rob Herring 
> > > > > ---
> > > > >  .../devicetree/bindings/arm/shmobile.txt  | 151 
> > > > >  .../devicetree/bindings/arm/shmobile.yaml | 218 
> > > > > ++
> > > > >  2 files changed, 218 insertions(+), 151 deletions(-)
> > > > >  delete mode 100644 Documentation/devicetree/bindings/arm/shmobile.txt
> > > > >  create mode 100644 
> > > > > Documentation/devicetree/bindings/arm/shmobile.yaml
> > > >
> > > > Hi Rob,
> > > >
> > > > what is this based on? I get a conflict when applying the .txt change
> > > > and if I knew the base for this patch it would be rather easy to work
> > > > out what has changed.
> >
> > 4.20-rc2
> >
> > > >
> > > > Also, should we do an s/shmobile.txt/shmobile.yaml/ in MAINTAINERS?
> >
> > Yes. Though it was pointed out that get_maintainers.pl can pull emails
> > out of this file. We'd need to get that to work by default though.
> >
> > > Probably even s/shmobile.yaml/renesas.yaml/, while at it?
> >
> > Sure, if that's what you all want.
>
> How about this?

LGTM

> From: Rob Herring 
> Subject: [PATCH v2.1] dt-bindings: arm: Convert Renesas board/soc bindings to
>  json-schema
>
> Convert Renesas SoC bindings to DT schema format using json-schema.
>
> v2.1 [Simon Horman]
> - rebased on renesas-devel-20181204-v4.20-rc5
>   + Added r8a7744 development platform and SoM
>   + Correct RZ/G2E part number
> - Update MAINTAINERS
>
> Signed-off-by: Rob Herring 
> Signed-off-by: Simon Horman 
> ---
>  Documentation/devicetree/bindings/arm/renesas.yaml | 228 
> +
>  Documentation/devicetree/bindings/arm/shmobile.txt | 155 --
>  MAINTAINERS|   4 +-
>  3 files changed, 230 insertions(+), 157 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/arm/renesas.yaml
>  delete mode 100644 Documentation/devicetree/bindings/arm/shmobile.txt


Re: [PATCH v2 26/34] dt-bindings: arm: Convert Renesas board/soc bindings to json-schema

2018-12-05 Thread Simon Horman
On Tue, Dec 04, 2018 at 09:08:57AM -0600, Rob Herring wrote:
> On Tue, Dec 4, 2018 at 8:57 AM Geert Uytterhoeven  
> wrote:
> >
> > Hi Simon,
> >
> > On Tue, Dec 4, 2018 at 3:48 PM Simon Horman  wrote:
> > > On Mon, Dec 03, 2018 at 03:32:15PM -0600, Rob Herring wrote:
> > > > Convert Renesas SoC bindings to DT schema format using json-schema.
> > > >
> > > > Cc: Simon Horman 
> > > > Cc: Magnus Damm 
> > > > Cc: Mark Rutland 
> > > > Cc: linux-renesas-...@vger.kernel.org
> > > > Cc: devicet...@vger.kernel.org
> > > > Signed-off-by: Rob Herring 
> > > > ---
> > > >  .../devicetree/bindings/arm/shmobile.txt  | 151 
> > > >  .../devicetree/bindings/arm/shmobile.yaml | 218 ++
> > > >  2 files changed, 218 insertions(+), 151 deletions(-)
> > > >  delete mode 100644 Documentation/devicetree/bindings/arm/shmobile.txt
> > > >  create mode 100644 Documentation/devicetree/bindings/arm/shmobile.yaml
> > >
> > > Hi Rob,
> > >
> > > what is this based on? I get a conflict when applying the .txt change
> > > and if I knew the base for this patch it would be rather easy to work
> > > out what has changed.
> 
> 4.20-rc2
> 
> > >
> > > Also, should we do an s/shmobile.txt/shmobile.yaml/ in MAINTAINERS?
> 
> Yes. Though it was pointed out that get_maintainers.pl can pull emails
> out of this file. We'd need to get that to work by default though.
> 
> > Probably even s/shmobile.yaml/renesas.yaml/, while at it?
> 
> Sure, if that's what you all want.

How about this?

From: Rob Herring 
Subject: [PATCH v2.1] dt-bindings: arm: Convert Renesas board/soc bindings to
 json-schema

Convert Renesas SoC bindings to DT schema format using json-schema.

v2.1 [Simon Horman]
- rebased on renesas-devel-20181204-v4.20-rc5
  + Added r8a7744 development platform and SoM
  + Correct RZ/G2E part number
- Update MAINTAINERS

Signed-off-by: Rob Herring 
Signed-off-by: Simon Horman 
---
 Documentation/devicetree/bindings/arm/renesas.yaml | 228 +
 Documentation/devicetree/bindings/arm/shmobile.txt | 155 --
 MAINTAINERS|   4 +-
 3 files changed, 230 insertions(+), 157 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/arm/renesas.yaml
 delete mode 100644 Documentation/devicetree/bindings/arm/shmobile.txt

diff --git a/Documentation/devicetree/bindings/arm/renesas.yaml 
b/Documentation/devicetree/bindings/arm/renesas.yaml
new file mode 100644
index ..5e9d4864a600
--- /dev/null
+++ b/Documentation/devicetree/bindings/arm/renesas.yaml
@@ -0,0 +1,228 @@
+# SPDX-License-Identifier: GPL-2.0
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/arm/shmobile.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Renesas SH-Mobile, R-Mobile, and R-Car Platform Device Tree Bindings
+
+maintainers:
+  - Geert Uytterhoeven 
+
+properties:
+  $nodename:
+const: '/'
+  compatible:
+oneOf:
+  - description: Emma Mobile EV2
+items:
+  - enum:
+  - renesas,kzm9d # Kyoto Microcomputer Co. KZM-A9-Dual
+  - const: renesas,emev2
+
+  - description: RZ/A1H (R7S72100)
+items:
+  - enum:
+  - renesas,genmai # Genmai (RTK772100BC0BR)
+  - renesas,gr-peach # GR-Peach (X28A-M01-E/F)
+  - renesas,rskrza1 # RSKRZA1 (YR0K77210C000BE)
+  - const: renesas,r7s72100
+
+  - description: RZ/A2 (R7S9210)
+items:
+  - const: renesas,r7s9210
+
+  - description: SH-Mobile AG5 (R8A73A00/SH73A0)
+items:
+  - enum:
+  - renesas,kzm9g # Kyoto Microcomputer Co. KZM-A9-GT
+  - const: renesas,sh73a0
+
+  - description: R-Mobile APE6 (R8A73A40)
+items:
+  - enum:
+  - renesas,ape6evm
+  - const: renesas,r8a73a4
+
+  - description: R-Mobile A1 (R8A77400)
+items:
+  - enum:
+  - renesas,armadillo800eva # Atmark Techno Armadillo-800 EVA
+  - const: renesas,r8a7740
+
+  - description: RZ/G1H (R8A77420)
+items:
+  - const: renesas,r8a7742
+
+  - description: RZ/G1M (R8A77430)
+items:
+  - enum:
+  # iWave Systems RZ/G1M Qseven Development Platform 
(iW-RainboW-G20D-Qseven)
+  - iwave,g20d
+  - const: iwave,g20m
+  - const: renesas,r8a7743
+
+  - items:
+  - enum:
+  # iWave Systems RZ/G1M Qseven System On Module 
(iW-RainboW-G20M-Qseven)
+  - iwave,g20m
+  - renesas,sk-rzg1m # SK-RZG1M (YR8A77430S000BE)
+  - const: renesas,r8a7743
+
+  - description: RZ/G1N (R8A77440)
+items:
+  - enum:
+  # iWave Systems RZ/G1N Qseven Development Platform 
(iW-RainboW-G20D-Qseven)
+  - iwave,g20d
+  - const: iwave,g20m
+  - const: renesas,r8a7744
+
+  - items:
+  - enum:
+  # iWave Systems RZ/G1N 

Re: [PATCH v2 26/34] dt-bindings: arm: Convert Renesas board/soc bindings to json-schema

2018-12-04 Thread Simon Horman
On Mon, Dec 03, 2018 at 03:32:15PM -0600, Rob Herring wrote:
> Convert Renesas SoC bindings to DT schema format using json-schema.
> 
> Cc: Simon Horman 
> Cc: Magnus Damm 
> Cc: Mark Rutland 
> Cc: linux-renesas-...@vger.kernel.org
> Cc: devicet...@vger.kernel.org
> Signed-off-by: Rob Herring 
> ---
>  .../devicetree/bindings/arm/shmobile.txt  | 151 
>  .../devicetree/bindings/arm/shmobile.yaml | 218 ++
>  2 files changed, 218 insertions(+), 151 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/arm/shmobile.txt
>  create mode 100644 Documentation/devicetree/bindings/arm/shmobile.yaml

Hi Rob,

what is this based on? I get a conflict when applying the .txt change
and if I knew the base for this patch it would be rather easy to work
out what has changed.

Also, should we do an s/shmobile.txt/shmobile.yaml/ in MAINTAINERS?


Re: [PATCH v2 26/34] dt-bindings: arm: Convert Renesas board/soc bindings to json-schema

2018-12-04 Thread Rob Herring
On Tue, Dec 4, 2018 at 8:57 AM Geert Uytterhoeven  wrote:
>
> Hi Simon,
>
> On Tue, Dec 4, 2018 at 3:48 PM Simon Horman  wrote:
> > On Mon, Dec 03, 2018 at 03:32:15PM -0600, Rob Herring wrote:
> > > Convert Renesas SoC bindings to DT schema format using json-schema.
> > >
> > > Cc: Simon Horman 
> > > Cc: Magnus Damm 
> > > Cc: Mark Rutland 
> > > Cc: linux-renesas-...@vger.kernel.org
> > > Cc: devicet...@vger.kernel.org
> > > Signed-off-by: Rob Herring 
> > > ---
> > >  .../devicetree/bindings/arm/shmobile.txt  | 151 
> > >  .../devicetree/bindings/arm/shmobile.yaml | 218 ++
> > >  2 files changed, 218 insertions(+), 151 deletions(-)
> > >  delete mode 100644 Documentation/devicetree/bindings/arm/shmobile.txt
> > >  create mode 100644 Documentation/devicetree/bindings/arm/shmobile.yaml
> >
> > Hi Rob,
> >
> > what is this based on? I get a conflict when applying the .txt change
> > and if I knew the base for this patch it would be rather easy to work
> > out what has changed.

4.20-rc2

> >
> > Also, should we do an s/shmobile.txt/shmobile.yaml/ in MAINTAINERS?

Yes. Though it was pointed out that get_maintainers.pl can pull emails
out of this file. We'd need to get that to work by default though.

> Probably even s/shmobile.yaml/renesas.yaml/, while at it?

Sure, if that's what you all want.

Rob


Re: [PATCH v2 26/34] dt-bindings: arm: Convert Renesas board/soc bindings to json-schema

2018-12-04 Thread Geert Uytterhoeven
Hi Simon,

On Tue, Dec 4, 2018 at 3:48 PM Simon Horman  wrote:
> On Mon, Dec 03, 2018 at 03:32:15PM -0600, Rob Herring wrote:
> > Convert Renesas SoC bindings to DT schema format using json-schema.
> >
> > Cc: Simon Horman 
> > Cc: Magnus Damm 
> > Cc: Mark Rutland 
> > Cc: linux-renesas-...@vger.kernel.org
> > Cc: devicet...@vger.kernel.org
> > Signed-off-by: Rob Herring 
> > ---
> >  .../devicetree/bindings/arm/shmobile.txt  | 151 
> >  .../devicetree/bindings/arm/shmobile.yaml | 218 ++
> >  2 files changed, 218 insertions(+), 151 deletions(-)
> >  delete mode 100644 Documentation/devicetree/bindings/arm/shmobile.txt
> >  create mode 100644 Documentation/devicetree/bindings/arm/shmobile.yaml
>
> Hi Rob,
>
> what is this based on? I get a conflict when applying the .txt change
> and if I knew the base for this patch it would be rather easy to work
> out what has changed.
>
> Also, should we do an s/shmobile.txt/shmobile.yaml/ in MAINTAINERS?

Probably even s/shmobile.yaml/renesas.yaml/, while at it?

Gr{oetje,eeting}s,

Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds


[PATCH v2 26/34] dt-bindings: arm: Convert Renesas board/soc bindings to json-schema

2018-12-03 Thread Rob Herring
Convert Renesas SoC bindings to DT schema format using json-schema.

Cc: Simon Horman 
Cc: Magnus Damm 
Cc: Mark Rutland 
Cc: linux-renesas-...@vger.kernel.org
Cc: devicet...@vger.kernel.org
Signed-off-by: Rob Herring 
---
 .../devicetree/bindings/arm/shmobile.txt  | 151 
 .../devicetree/bindings/arm/shmobile.yaml | 218 ++
 2 files changed, 218 insertions(+), 151 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/arm/shmobile.txt
 create mode 100644 Documentation/devicetree/bindings/arm/shmobile.yaml

diff --git a/Documentation/devicetree/bindings/arm/shmobile.txt 
b/Documentation/devicetree/bindings/arm/shmobile.txt
deleted file mode 100644
index 5f18ce9cdbb8..
--- a/Documentation/devicetree/bindings/arm/shmobile.txt
+++ /dev/null
@@ -1,151 +0,0 @@
-Renesas SH-Mobile, R-Mobile, and R-Car Platform Device Tree Bindings
-
-
-SoCs:
-
-  - Emma Mobile EV2
-compatible = "renesas,emev2"
-  - RZ/A1H (R7S72100)
-compatible = "renesas,r7s72100"
-  - RZ/A2 (R7S9210)
-compatible = "renesas,r7s9210"
-  - SH-Mobile AG5 (R8A73A00/SH73A0)
-compatible = "renesas,sh73a0"
-  - R-Mobile APE6 (R8A73A40)
-compatible = "renesas,r8a73a4"
-  - R-Mobile A1 (R8A77400)
-compatible = "renesas,r8a7740"
-  - RZ/G1H (R8A77420)
-compatible = "renesas,r8a7742"
-  - RZ/G1M (R8A77430)
-compatible = "renesas,r8a7743"
-  - RZ/G1N (R8A77440)
-compatible = "renesas,r8a7744"
-  - RZ/G1E (R8A77450)
-compatible = "renesas,r8a7745"
-  - RZ/G1C (R8A77470)
-compatible = "renesas,r8a77470"
-  - RZ/G2M (R8A774A1)
-compatible = "renesas,r8a774a1"
-  - RZ/G2E (R8A774C0)
-compatible = "renesas,r8a774c0"
-  - R-Car M1A (R8A77781)
-compatible = "renesas,r8a7778"
-  - R-Car H1 (R8A77790)
-compatible = "renesas,r8a7779"
-  - R-Car H2 (R8A77900)
-compatible = "renesas,r8a7790"
-  - R-Car M2-W (R8A77910)
-compatible = "renesas,r8a7791"
-  - R-Car V2H (R8A77920)
-compatible = "renesas,r8a7792"
-  - R-Car M2-N (R8A77930)
-compatible = "renesas,r8a7793"
-  - R-Car E2 (R8A77940)
-compatible = "renesas,r8a7794"
-  - R-Car H3 (R8A77950)
-compatible = "renesas,r8a7795"
-  - R-Car M3-W (R8A77960)
-compatible = "renesas,r8a7796"
-  - R-Car M3-N (R8A77965)
-compatible = "renesas,r8a77965"
-  - R-Car V3M (R8A77970)
-compatible = "renesas,r8a77970"
-  - R-Car V3H (R8A77980)
-compatible = "renesas,r8a77980"
-  - R-Car E3 (R8A77990)
-compatible = "renesas,r8a77990"
-  - R-Car D3 (R8A77995)
-compatible = "renesas,r8a77995"
-  - RZ/N1D (R9A06G032)
-compatible = "renesas,r9a06g032"
-
-Boards:
-
-  - Alt (RTP0RC7794SEB00010S)
-compatible = "renesas,alt", "renesas,r8a7794"
-  - APE6-EVM
-compatible = "renesas,ape6evm", "renesas,r8a73a4"
-  - Atmark Techno Armadillo-800 EVA
-compatible = "renesas,armadillo800eva", "renesas,r8a7740"
-  - Blanche (RTP0RC7792SEB00010S)
-compatible = "renesas,blanche", "renesas,r8a7792"
-  - BOCK-W
-compatible = "renesas,bockw", "renesas,r8a7778"
-  - Condor (RTP0RC77980SEB0010SS/RTP0RC77980SEB0010SA01)
-compatible = "renesas,condor", "renesas,r8a77980"
-  - Draak (RTP0RC77995SEB0010S)
-compatible = "renesas,draak", "renesas,r8a77995"
-  - Eagle (RTP0RC77970SEB0010S)
-compatible = "renesas,eagle", "renesas,r8a77970"
-  - Ebisu (RTP0RC77990SEB0010S)
-compatible = "renesas,ebisu", "renesas,r8a77990"
-  - Genmai (RTK772100BC0BR)
-compatible = "renesas,genmai", "renesas,r7s72100"
-  - GR-Peach (X28A-M01-E/F)
-compatible = "renesas,gr-peach", "renesas,r7s72100"
-  - Gose (RTP0RC7793SEB00010S)
-compatible = "renesas,gose", "renesas,r8a7793"
-  - H3ULCB (R-Car Starter Kit Premier, RTP0RC7795SKBX0010SA00 (H3 ES1.1))
-H3ULCB (R-Car Starter Kit Premier, RTP0RC77951SKBX010SA00 (H3 ES2.0))
-compatible = "renesas,h3ulcb", "renesas,r8a7795"
-  - Henninger
-compatible = "renesas,henninger", "renesas,r8a7791"
-  - iWave Systems RZ/G1C Single Board Computer (iW-RainboW-G23S)
-compatible = "iwave,g23s", "renesas,r8a77470"
-  - iWave Systems RZ/G1E SODIMM SOM Development Platform (iW-RainboW-G22D)
-compatible = "iwave,g22d", "iwave,g22m", "renesas,r8a7745"
-  - iWave Systems RZ/G1E SODIMM System On Module (iW-RainboW-G22M-SM)
-compatible = "iwave,g22m", "renesas,r8a7745"
-  - iWave Systems RZ/G1M Qseven Development Platform (iW-RainboW-G20D-Qseven)
-compatible = "iwave,g20d", "iwave,g20m", "renesas,r8a7743"
-  - iWave Systems RZ/G1M Qseven System On Module (iW-RainboW-G20M-Qseven)
-compatible = "iwave,g20m", "renesas,r8a7743"
-  - Kingfisher (SBEV-RCAR-KF-M03)
-compatible = "shimafuji,kingfisher"
-  - Koelsch (RTP0RC7791SEB00010S)
-compatible = "renesas,koelsch", "renesas,r8a7791"
-  - Kyoto Microcomputer Co. KZM-A9-Dual
-compatible = "renesas,kzm9d", "renesas,emev2"
-  - Kyoto Microcomputer Co. KZM-A9-GT
-compatible =