diff mbox

ARM: shmobile: henninger: rename board to Porter

Message ID 2481726.lh33apx2TG@wasted.cogentembedded.com (mailing list archive)
State New, archived
Headers show

Commit Message

Sergei Shtylyov Jan. 23, 2015, 8:37 p.m. UTC
Renesas has made some changes to the Henninger low cost board hardware and at
the same time decided to  rename the board  to Porter.

Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>

---
This patch is against 'renesas-devel-20150123-v3.19-rc5' tag of Simon Horman's
'renesas.git' repo.

 arch/arm/boot/dts/Makefile                                     |  2 +-
 .../arm/boot/dts/{r8a7791-henninger.dts => r8a7791-porter.dts} | 10 +++++-----
 2 files changed, 6 insertions(+), 6 deletions(-)
 rename arch/arm/boot/dts/{r8a7791-henninger.dts => r8a7791-porter.dts} (96%)

Comments

Geert Uytterhoeven Jan. 25, 2015, 7:30 p.m. UTC | #1
Hi Sergei,

On Fri, Jan 23, 2015 at 9:37 PM, Sergei Shtylyov
<sergei.shtylyov@cogentembedded.com> wrote:
> Renesas has made some changes to the Henninger low cost board hardware and at
> the same time decided to  rename the board  to Porter.

Which changes? They're not reflected in the new DTS?

> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
>
> ---
> This patch is against 'renesas-devel-20150123-v3.19-rc5' tag of Simon Horman's
> 'renesas.git' repo.
>
>  arch/arm/boot/dts/Makefile                                     |  2 +-
>  .../arm/boot/dts/{r8a7791-henninger.dts => r8a7791-porter.dts} | 10 +++++-----
>  2 files changed, 6 insertions(+), 6 deletions(-)
>  rename arch/arm/boot/dts/{r8a7791-henninger.dts => r8a7791-porter.dts} (96%)

You forgot to update Documentation/devicetree/bindings/arm/shmobile.txt

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
Sergei Shtylyov Jan. 28, 2015, 1:41 p.m. UTC | #2
Hello.

On 1/25/2015 10:30 PM, Geert Uytterhoeven wrote:

>> Renesas has made some changes to the Henninger low cost board hardware and at
>> the same time decided to  rename the board  to Porter.

> Which changes?

   I don't have a list ready, sorry. One thing I know is that HDMI chip is 
different (the same as on Koelsch now); we don't support it yet...

> They're not reflected in the new DTS?

    Not yet.

>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>

>> ---
>> This patch is against 'renesas-devel-20150123-v3.19-rc5' tag of Simon Horman's
>> 'renesas.git' repo.
>>
>>   arch/arm/boot/dts/Makefile                                     |  2 +-
>>   .../arm/boot/dts/{r8a7791-henninger.dts => r8a7791-porter.dts} | 10 +++++-----
>>   2 files changed, 6 insertions(+), 6 deletions(-)
>>   rename arch/arm/boot/dts/{r8a7791-henninger.dts => r8a7791-porter.dts} (96%)

> You forgot to update Documentation/devicetree/bindings/arm/shmobile.txt

    No wonder -- I had no idea it is listed there...

> Gr{oetje,eeting}s,

>                          Geert

WBR, Sergei
Simon Horman Jan. 29, 2015, 12:30 a.m. UTC | #3
On Wed, Jan 28, 2015 at 04:41:44PM +0300, Sergei Shtylyov wrote:
> Hello.
> 
> On 1/25/2015 10:30 PM, Geert Uytterhoeven wrote:
> 
> >>Renesas has made some changes to the Henninger low cost board hardware and at
> >>the same time decided to  rename the board  to Porter.
> 
> >Which changes?
> 
>   I don't have a list ready, sorry. One thing I know is that HDMI chip is
> different (the same as on Koelsch now); we don't support it yet...

This sounds more like a new board than an update. I don't have any strong
opinions about this at this time but perhaps it would make more sense to
handle this as two steps: adding a new board and removing an old one.

> >They're not reflected in the new DTS?
> 
>    Not yet.
> 
> >>Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
> 
> >>---
> >>This patch is against 'renesas-devel-20150123-v3.19-rc5' tag of Simon Horman's
> >>'renesas.git' repo.
> >>
> >>  arch/arm/boot/dts/Makefile                                     |  2 +-
> >>  .../arm/boot/dts/{r8a7791-henninger.dts => r8a7791-porter.dts} | 10 +++++-----
> >>  2 files changed, 6 insertions(+), 6 deletions(-)
> >>  rename arch/arm/boot/dts/{r8a7791-henninger.dts => r8a7791-porter.dts} (96%)
> 
> >You forgot to update Documentation/devicetree/bindings/arm/shmobile.txt
> 
>    No wonder -- I had no idea it is listed there...
> 
> >Gr{oetje,eeting}s,
> 
> >                         Geert
> 
> WBR, Sergei
>
diff mbox

Patch

diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
index 59e79e9..3cc8ba5 100644
--- a/arch/arm/boot/dts/Makefile
+++ b/arch/arm/boot/dts/Makefile
@@ -419,8 +419,8 @@  dtb-$(CONFIG_ARCH_SHMOBILE_MULTI) += eme
 	r8a7740-armadillo800eva.dtb \
 	r8a7779-marzen.dtb \
 	r8a7790-lager.dtb \
-	r8a7791-henninger.dtb \
 	r8a7791-koelsch.dtb \
+	r8a7791-porter.dtb \
 	r8a7794-alt.dtb \
 	sh73a0-kzm9g.dtb
 dtb-$(CONFIG_ARCH_SOCFPGA) += socfpga_arria5_socdk.dtb \
diff --git a/arch/arm/boot/dts/r8a7791-henninger.dts b/arch/arm/boot/dts/r8a7791-porter.dts
similarity index 96%
rename from arch/arm/boot/dts/r8a7791-henninger.dts
rename to arch/arm/boot/dts/r8a7791-porter.dts
index d2ebf11..4a616dc 100644
--- a/arch/arm/boot/dts/r8a7791-henninger.dts
+++ b/arch/arm/boot/dts/r8a7791-porter.dts
@@ -1,8 +1,8 @@ 
 /*
- * Device Tree Source for the Henninger board
+ * Device Tree Source for the Porter board
  *
- * Copyright (C) 2014 Renesas Solutions Corp.
- * Copyright (C) 2014 Cogent Embedded, Inc.
+ * Copyright (C) 2014-2015 Renesas Solutions Corp.
+ * Copyright (C) 2014-2015 Cogent Embedded, Inc.
  *
  * This file is licensed under the terms of the GNU General Public License
  * version 2.  This program is licensed "as is" without any warranty of any
@@ -14,8 +14,8 @@ 
 #include <dt-bindings/gpio/gpio.h>
 
 / {
-	model = "Henninger";
-	compatible = "renesas,henninger", "renesas,r8a7791";
+	model = "Porter";
+	compatible = "renesas,porter", "renesas,r8a7791";
 
 	aliases {
 		serial0 = &scif0;