[PATCH 00/03] Further product name updates

[PATCH 00/03] Further product name updates

[Magnus Damm]

Further product name updates

[PATCH 01/03] dt-bindings: arm: consistently name SH-Mobile AG5 as R8A73A00
[PATCH 02/03] arm: dts: Include R-Car Gen2 product name in DTSI files
[PATCH 03/03] arm: dts: R-Car Gen1 board comment update

Clean up and update the DTS/DTSI files and SoC DT binding documentation.

Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
---

 Developed on top of renesas-devel-20180914-v4.19-rc3

 Documentation/devicetree/bindings/arm/shmobile.txt |    2 +-
 arch/arm/boot/dts/r8a7778-bockw.dts                |    2 +-
 arch/arm/boot/dts/r8a7779-marzen.dts               |    2 +-
 arch/arm/boot/dts/r8a7790.dtsi                     |    2 +-
 arch/arm/boot/dts/r8a7791.dtsi                     |    2 +-
 arch/arm/boot/dts/r8a7792.dtsi                     |    2 +-
 arch/arm/boot/dts/r8a7793.dtsi                     |    2 +-
 arch/arm/boot/dts/r8a7794.dtsi                     |    2 +-
 8 files changed, 8 insertions(+), 8 deletions(-)

[PATCH 01/03] dt-bindings: arm: consistently name SH-Mobile AG5 as R8A73A00

[Magnus Damm]

From: Magnus Damm <damm@opensource.se>

Update the description of the AG5 entry to follow same style as other SoCs.

The actual DT binding remains unchanged.

Signed-off-by: Magnus Damm <damm@opensource.se>
---

 Developed on top of renesas-devel-20180914-v4.19-rc3

 Documentation/devicetree/bindings/arm/shmobile.txt |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--- 0001/Documentation/devicetree/bindings/arm/shmobile.txt
+++ work/Documentation/devicetree/bindings/arm/shmobile.txt	2018-09-17 13:00:41.773502304 +0900
@@ -9,7 +9,7 @@ SoCs:
     compatible = "renesas,r7s72100"
   - RZ/A2 (R7S9210)
     compatible = "renesas,r7s9210"
-  - SH-Mobile AG5 (R8A73A00/SH73A0)
+  - SH-Mobile AG5 (R8A73A00)
     compatible = "renesas,sh73a0"
   - R-Mobile APE6 (R8A73A40)
     compatible = "renesas,r8a73a4"

[PATCH 02/03] arm: dts: Include R-Car Gen2 product name in DTSI files

[Magnus Damm]

From: Magnus Damm <damm@opensource.se>

Improve the user friendliness of the DTS code base by including the
R-Car product name in each R-Car Gen2 DTSI file.

The product names are taken from:
Documentation/devicetree/bindings/arm/shmobile.txt

Signed-off-by: Magnus Damm <damm@opensource.se>
---

 Developed on top of renesas-devel-20180914-v4.19-rc3

 arch/arm/boot/dts/r8a7790.dtsi |    2 +-
 arch/arm/boot/dts/r8a7791.dtsi |    2 +-
 arch/arm/boot/dts/r8a7792.dtsi |    2 +-
 arch/arm/boot/dts/r8a7793.dtsi |    2 +-
 arch/arm/boot/dts/r8a7794.dtsi |    2 +-
 5 files changed, 5 insertions(+), 5 deletions(-)

--- 0001/arch/arm/boot/dts/r8a7790.dtsi
+++ work/arch/arm/boot/dts/r8a7790.dtsi	2018-09-17 16:22:22.002897845 +0900
@@ -1,6 +1,6 @@
 // SPDX-License-Identifier: GPL-2.0
 /*
- * Device Tree Source for the r8a7790 SoC
+ * Device Tree Source for the R-Car H2 (R8A77900) SoC
  *
  * Copyright (C) 2015 Renesas Electronics Corporation
  * Copyright (C) 2013-2014 Renesas Solutions Corp.
--- 0001/arch/arm/boot/dts/r8a7791.dtsi
+++ work/arch/arm/boot/dts/r8a7791.dtsi	2018-09-17 16:22:47.249221466 +0900
@@ -1,6 +1,6 @@
 // SPDX-License-Identifier: GPL-2.0
 /*
- * Device Tree Source for the r8a7791 SoC
+ * Device Tree Source for the R-Car M2-W (R8A77910) SoC
  *
  * Copyright (C) 2013-2015 Renesas Electronics Corporation
  * Copyright (C) 2013-2014 Renesas Solutions Corp.
--- 0001/arch/arm/boot/dts/r8a7792.dtsi
+++ work/arch/arm/boot/dts/r8a7792.dtsi	2018-09-17 16:23:16.352747315 +0900
@@ -1,6 +1,6 @@
 // SPDX-License-Identifier: GPL-2.0
 /*
- * Device Tree Source for the r8a7792 SoC
+ * Device Tree Source for the R-Car V2H (R8A77920) SoC
  *
  * Copyright (C) 2016 Cogent Embedded Inc.
  */
--- 0001/arch/arm/boot/dts/r8a7793.dtsi
+++ work/arch/arm/boot/dts/r8a7793.dtsi	2018-09-17 16:23:38.337899965 +0900
@@ -1,6 +1,6 @@
 // SPDX-License-Identifier: GPL-2.0
 /*
- * Device Tree Source for the r8a7793 SoC
+ * Device Tree Source for the R-Car M2-N (R8A77930) SoC
  *
  * Copyright (C) 2014-2015 Renesas Electronics Corporation
  */
--- 0001/arch/arm/boot/dts/r8a7794.dtsi
+++ work/arch/arm/boot/dts/r8a7794.dtsi	2018-09-17 16:23:53.752708146 +0900
@@ -1,6 +1,6 @@
 // SPDX-License-Identifier: GPL-2.0
 /*
- * Device Tree Source for the r8a7794 SoC
+ * Device Tree Source for the R-Car E2 (R8A77940) SoC
  *
  * Copyright (C) 2014 Renesas Electronics Corporation
  * Copyright (C) 2014 Ulrich Hecht

[PATCH 03/03] arm: dts: R-Car Gen1 board comment update

[Magnus Damm]

From: Magnus Damm <damm@opensource.se>

Include R-Car Gen1 product names for Bock-W and Marzen.

The product names are taken from:
Documentation/devicetree/bindings/arm/shmobile.txt

Signed-off-by: Magnus Damm <damm@opensource.se>
---

 Developed on top of renesas-devel-20180914-v4.19-rc3

 arch/arm/boot/dts/r8a7778-bockw.dts  |    2 +-
 arch/arm/boot/dts/r8a7779-marzen.dts |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

--- 0001/arch/arm/boot/dts/r8a7778-bockw.dts
+++ work/arch/arm/boot/dts/r8a7778-bockw.dts	2018-09-17 12:47:10.561971790 +0900
@@ -1,6 +1,6 @@
 // SPDX-License-Identifier: GPL-2.0
 /*
- * Reference Device Tree Source for the Bock-W board
+ * Reference Device Tree Source for the R-Car M1A (R8A77781) Bock-W board
  *
  * Copyright (C) 2013  Renesas Solutions Corp.
  * Copyright (C) 2013  Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
--- 0001/arch/arm/boot/dts/r8a7779-marzen.dts
+++ work/arch/arm/boot/dts/r8a7779-marzen.dts	2018-09-17 12:47:35.744292059 +0900
@@ -1,6 +1,6 @@
 // SPDX-License-Identifier: GPL-2.0
 /*
- * Device Tree Source for the Marzen board
+ * Device Tree Source for the R-Car H1 (R8A77790) Marzen board
  *
  * Copyright (C) 2013 Renesas Solutions Corp.
  * Copyright (C) 2013 Simon Horman

Re: [PATCH 00/03] Further product name updates

[Simon Horman]

On Mon, Sep 17, 2018 at 05:39:22PM +0900, Magnus Damm wrote:
> Further product name updates
> 
> [PATCH 01/03] dt-bindings: arm: consistently name SH-Mobile AG5 as R8A73A00
> [PATCH 02/03] arm: dts: Include R-Car Gen2 product name in DTSI files
> [PATCH 03/03] arm: dts: R-Car Gen1 board comment update
> 
> Clean up and update the DTS/DTSI files and SoC DT binding documentation.
> 
> Signed-off-by: Magnus Damm <damm+renesas@opensource.se>

Thanks Magnus,

This looks fine to me but I will wait to see if there are other reviews
before applying.

Reviewed-by: Simon Horman <horms+renesas@verge.net.au>

Re: [PATCH 01/03] dt-bindings: arm: consistently name SH-Mobile AG5 as R8A73A00

[Geert Uytterhoeven]

Hi Magnus,

On Mon, Sep 17, 2018 at 10:39 AM Magnus Damm <magnus.damm@gmail.com> wrote:
> From: Magnus Damm <damm@opensource.se>
>
> Update the description of the AG5 entry to follow same style as other SoCs.
>
> The actual DT binding remains unchanged.
>
> Signed-off-by: Magnus Damm <damm@opensource.se>

Thanks for your patch!

Do you plan to update the other occurrences under
Documentation/devicetree/bindings/, too?

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@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

Re: [PATCH 03/03] arm: dts: R-Car Gen1 board comment update

[Geert Uytterhoeven]

On Mon, Sep 17, 2018 at 10:39 AM Magnus Damm <magnus.damm@gmail.com> wrote:
> From: Magnus Damm <damm@opensource.se>
>
> Include R-Car Gen1 product names for Bock-W and Marzen.
>
> The product names are taken from:
> Documentation/devicetree/bindings/arm/shmobile.txt
>
> Signed-off-by: Magnus Damm <damm@opensource.se>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@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

Re: [PATCH 02/03] arm: dts: Include R-Car Gen2 product name in DTSI files

[Geert Uytterhoeven]

On Mon, Sep 17, 2018 at 10:39 AM Magnus Damm <magnus.damm@gmail.com> wrote:
> From: Magnus Damm <damm@opensource.se>
>
> Improve the user friendliness of the DTS code base by including the
> R-Car product name in each R-Car Gen2 DTSI file.
>
> The product names are taken from:
> Documentation/devicetree/bindings/arm/shmobile.txt
>
> Signed-off-by: Magnus Damm <damm@opensource.se>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@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

Re: [PATCH 01/03] dt-bindings: arm: consistently name SH-Mobile AG5 as R8A73A00

[Simon Horman]

On Mon, Sep 17, 2018 at 01:20:57PM +0200, Geert Uytterhoeven wrote:
> Hi Magnus,
> 
> On Mon, Sep 17, 2018 at 10:39 AM Magnus Damm <magnus.damm@gmail.com> wrote:
> > From: Magnus Damm <damm@opensource.se>
> >
> > Update the description of the AG5 entry to follow same style as other SoCs.
> >
> > The actual DT binding remains unchanged.
> >
> > Signed-off-by: Magnus Damm <damm@opensource.se>
> 
> Thanks for your patch!
> 
> Do you plan to update the other occurrences under
> Documentation/devicetree/bindings/, too?

Which occurrences are you concerned about?

Re: [PATCH 02/03] arm: dts: Include R-Car Gen2 product name in DTSI files

[Simon Horman]

On Mon, Sep 17, 2018 at 01:22:22PM +0200, Geert Uytterhoeven wrote:
> On Mon, Sep 17, 2018 at 10:39 AM Magnus Damm <magnus.damm@gmail.com> wrote:
> > From: Magnus Damm <damm@opensource.se>
> >
> > Improve the user friendliness of the DTS code base by including the
> > R-Car product name in each R-Car Gen2 DTSI file.
> >
> > The product names are taken from:
> > Documentation/devicetree/bindings/arm/shmobile.txt
> >
> > Signed-off-by: Magnus Damm <damm@opensource.se>
> 
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Thanks, applied for v4.20.

Re: [PATCH 03/03] arm: dts: R-Car Gen1 board comment update

[Simon Horman]

On Mon, Sep 17, 2018 at 01:21:42PM +0200, Geert Uytterhoeven wrote:
> On Mon, Sep 17, 2018 at 10:39 AM Magnus Damm <magnus.damm@gmail.com> wrote:
> > From: Magnus Damm <damm@opensource.se>
> >
> > Include R-Car Gen1 product names for Bock-W and Marzen.
> >
> > The product names are taken from:
> > Documentation/devicetree/bindings/arm/shmobile.txt
> >
> > Signed-off-by: Magnus Damm <damm@opensource.se>
> 
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Thanks, applied for v4.20.

Re: [PATCH 01/03] dt-bindings: arm: consistently name SH-Mobile AG5 as R8A73A00

[Geert Uytterhoeven]

On Wed, Sep 19, 2018 at 10:31 AM Simon Horman <horms@verge.net.au> wrote:
> On Mon, Sep 17, 2018 at 01:20:57PM +0200, Geert Uytterhoeven wrote:
> > On Mon, Sep 17, 2018 at 10:39 AM Magnus Damm <magnus.damm@gmail.com> wrote:
> > > From: Magnus Damm <damm@opensource.se>
> > >
> > > Update the description of the AG5 entry to follow same style as other SoCs.
> > >
> > > The actual DT binding remains unchanged.
> > >
> > > Signed-off-by: Magnus Damm <damm@opensource.se>
> >
> > Thanks for your patch!
> >
> > Do you plan to update the other occurrences under
> > Documentation/devicetree/bindings/, too?
>
> Which occurrences are you concerned about?

git grep SH73A0 -- Documentation/devicetree/bindings/

I have mixed feelings about this patch. AFAIU, it was called SH73A0 originally,
but renamed R8A73A0 after SuperHitachi and successors morphed into Renesas?

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@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

Re: [PATCH 01/03] dt-bindings: arm: consistently name SH-Mobile AG5 as R8A73A00

[Simon Horman]

On Wed, Sep 19, 2018 at 10:43:53AM +0200, Geert Uytterhoeven wrote:
> On Wed, Sep 19, 2018 at 10:31 AM Simon Horman <horms@verge.net.au> wrote:
> > On Mon, Sep 17, 2018 at 01:20:57PM +0200, Geert Uytterhoeven wrote:
> > > On Mon, Sep 17, 2018 at 10:39 AM Magnus Damm <magnus.damm@gmail.com> wrote:
> > > > From: Magnus Damm <damm@opensource.se>
> > > >
> > > > Update the description of the AG5 entry to follow same style as other SoCs.
> > > >
> > > > The actual DT binding remains unchanged.
> > > >
> > > > Signed-off-by: Magnus Damm <damm@opensource.se>
> > >
> > > Thanks for your patch!
> > >
> > > Do you plan to update the other occurrences under
> > > Documentation/devicetree/bindings/, too?
> >
> > Which occurrences are you concerned about?
> 
> git grep SH73A0 -- Documentation/devicetree/bindings/
> 
> I have mixed feelings about this patch. AFAIU, it was called SH73A0 originally,
> but renamed R8A73A0 after SuperHitachi and successors morphed into Renesas?

Thanks.

I agree that we should strive to be consistent.
Lets see if anyone has some more information regarding the
history of the name for this SoC.