diff mbox series

dt-bindings: arm: consistently name r8a77965 as M3-N

Message ID 20180424055442.19710-1-horms+renesas@verge.net.au
State Not Applicable, archived
Headers show
Series dt-bindings: arm: consistently name r8a77965 as M3-N | expand

Commit Message

Simon Horman April 24, 2018, 5:54 a.m. UTC
There is an inconsistency between the use of M3N and M3-N.
This patch resolves this by consistently using the latter.

Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
Based on renesas-devel-20180423-v4.17-rc2
---
 Documentation/devicetree/bindings/arm/shmobile.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Geert Uytterhoeven April 24, 2018, 7:16 a.m. UTC | #1
Hi Simon,

On Tue, Apr 24, 2018 at 7:54 AM, Simon Horman
<horms+renesas@verge.net.au> wrote:
> There is an inconsistency between the use of M3N and M3-N.
> This patch resolves this by consistently using the latter.
>
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>

Note that this is for a custom Salvator-X board where the M3-W SiP was
replaced by an M3-N SiP.  So I have no idea what's on the sticker, if it was
replaced at all, and thus kept the text from the BSP.

> --- a/Documentation/devicetree/bindings/arm/shmobile.txt
> +++ b/Documentation/devicetree/bindings/arm/shmobile.txt
> @@ -116,7 +116,7 @@ Boards:
>      compatible = "renesas,salvator-x", "renesas,r8a7795"
>    - Salvator-X (RTP0RC7796SIPB0011S)
>      compatible = "renesas,salvator-x", "renesas,r8a7796"
> -  - Salvator-X (RTP0RC7796SIPB0011S (M3N))
> +  - Salvator-X (RTP0RC7796SIPB0011S (M3-N))
>      compatible = "renesas,salvator-x", "renesas,r8a77965"
>    - Salvator-XS (Salvator-X 2nd version, RTP0RC7795SIPB0012S)
>      compatible = "renesas,salvator-xs", "renesas,r8a7795"
\
Gr{oetje,eeting}s,

                        Geert
Simon Horman April 24, 2018, 8:26 a.m. UTC | #2
On Tue, Apr 24, 2018 at 09:16:17AM +0200, Geert Uytterhoeven wrote:
> Hi Simon,
> 
> On Tue, Apr 24, 2018 at 7:54 AM, Simon Horman
> <horms+renesas@verge.net.au> wrote:
> > There is an inconsistency between the use of M3N and M3-N.
> > This patch resolves this by consistently using the latter.
> >
> > Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> 
> Note that this is for a custom Salvator-X board where the M3-W SiP was
> replaced by an M3-N SiP.  So I have no idea what's on the sticker, if it was
> replaced at all, and thus kept the text from the BSP.

Understood. But I think we should make the naming consistent regardless.

> 
> > --- a/Documentation/devicetree/bindings/arm/shmobile.txt
> > +++ b/Documentation/devicetree/bindings/arm/shmobile.txt
> > @@ -116,7 +116,7 @@ Boards:
> >      compatible = "renesas,salvator-x", "renesas,r8a7795"
> >    - Salvator-X (RTP0RC7796SIPB0011S)
> >      compatible = "renesas,salvator-x", "renesas,r8a7796"
> > -  - Salvator-X (RTP0RC7796SIPB0011S (M3N))
> > +  - Salvator-X (RTP0RC7796SIPB0011S (M3-N))
> >      compatible = "renesas,salvator-x", "renesas,r8a77965"
> >    - Salvator-XS (Salvator-X 2nd version, RTP0RC7795SIPB0012S)
> >      compatible = "renesas,salvator-xs", "renesas,r8a7795"
> \
> 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
> 
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Rob Herring (Arm) May 1, 2018, 1:41 p.m. UTC | #3
On Tue, Apr 24, 2018 at 07:54:42AM +0200, Simon Horman wrote:
> There is an inconsistency between the use of M3N and M3-N.
> This patch resolves this by consistently using the latter.
> 
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> ---
> Based on renesas-devel-20180423-v4.17-rc2
> ---
>  Documentation/devicetree/bindings/arm/shmobile.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Rob Herring <robh@kernel.org>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Simon Horman May 2, 2018, 6:44 a.m. UTC | #4
On Tue, May 01, 2018 at 08:41:03AM -0500, Rob Herring wrote:
> On Tue, Apr 24, 2018 at 07:54:42AM +0200, Simon Horman wrote:
> > There is an inconsistency between the use of M3N and M3-N.
> > This patch resolves this by consistently using the latter.
> > 
> > Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> > ---
> > Based on renesas-devel-20180423-v4.17-rc2
> > ---
> >  Documentation/devicetree/bindings/arm/shmobile.txt | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> Reviewed-by: Rob Herring <robh@kernel.org>

Thanks, applied.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/arm/shmobile.txt b/Documentation/devicetree/bindings/arm/shmobile.txt
index 61b486fb5793..02c5d261683f 100644
--- a/Documentation/devicetree/bindings/arm/shmobile.txt
+++ b/Documentation/devicetree/bindings/arm/shmobile.txt
@@ -116,7 +116,7 @@  Boards:
     compatible = "renesas,salvator-x", "renesas,r8a7795"
   - Salvator-X (RTP0RC7796SIPB0011S)
     compatible = "renesas,salvator-x", "renesas,r8a7796"
-  - Salvator-X (RTP0RC7796SIPB0011S (M3N))
+  - Salvator-X (RTP0RC7796SIPB0011S (M3-N))
     compatible = "renesas,salvator-x", "renesas,r8a77965"
   - Salvator-XS (Salvator-X 2nd version, RTP0RC7795SIPB0012S)
     compatible = "renesas,salvator-xs", "renesas,r8a7795"