diff mbox series

[v2,6/8] arm64: renesas: document Eagle board bindings

Message ID 20170915194442.464361390@cogentembedded.com
State Not Applicable, archived
Headers show
Series Add Renesas R8A77970/Eagle board support | expand

Commit Message

Sergei Shtylyov Sept. 15, 2017, 7:43 p.m. UTC
Document the Eagle device tree bindings, listing it as a supported board.

This allows to use checkpatch.pl to validate .dts files referring to the
Eagle board.

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

---
Changes in version 2:
- renamed the patch.

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


--
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

Comments

Geert Uytterhoeven Sept. 19, 2017, 7:49 a.m. UTC | #1
On Fri, Sep 15, 2017 at 9:43 PM, Sergei Shtylyov
<sergei.shtylyov@cogentembedded.com> wrote:
> Document the Eagle device tree bindings, listing it as a supported board.
>
> This allows to use checkpatch.pl to validate .dts files referring to the
> Eagle board.
>
> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>

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
--
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) Sept. 20, 2017, 8:52 p.m. UTC | #2
On Fri, Sep 15, 2017 at 10:43:24PM +0300, Sergei Shtylyov wrote:
> Document the Eagle device tree bindings, listing it as a supported board.
> 
> This allows to use checkpatch.pl to validate .dts files referring to the
> Eagle board.
> 
> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
> 
> ---
> Changes in version 2:
> - renamed the patch.
> 
>  Documentation/devicetree/bindings/arm/shmobile.txt |    2 ++
>  1 file changed, 2 insertions(+)
> 
> Index: renesas/Documentation/devicetree/bindings/arm/shmobile.txt
> ===================================================================
> --- renesas.orig/Documentation/devicetree/bindings/arm/shmobile.txt
> +++ renesas/Documentation/devicetree/bindings/arm/shmobile.txt
> @@ -59,6 +59,8 @@ Boards:
>      compatible = "renesas,bockw", "renesas,r8a7778"
>    - Draak (RTP0RC77995SEB0010S)
>      compatible = "renesas,draak", "renesas,r8a77995"
> +  - Eagle (RTP0RC77970SEB0010S)
> +    compatible = "renesas,eagle", "renesas,r8a77970";

You're not consistent here with the ';'.

Otherwise,

Acked-by: Rob Herring <robh@kernel.org>

>    - Genmai (RTK772100BC00000BR)
>      compatible = "renesas,genmai", "renesas,r7s72100"
>    - GR-Peach (X28A-M01-E/F)
> 
> --
> 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
--
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 Sept. 21, 2017, 9:07 a.m. UTC | #3
On Wed, Sep 20, 2017 at 03:52:33PM -0500, Rob Herring wrote:
> On Fri, Sep 15, 2017 at 10:43:24PM +0300, Sergei Shtylyov wrote:
> > Document the Eagle device tree bindings, listing it as a supported board.
> > 
> > This allows to use checkpatch.pl to validate .dts files referring to the
> > Eagle board.
> > 
> > Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
> > 
> > ---
> > Changes in version 2:
> > - renamed the patch.
> > 
> >  Documentation/devicetree/bindings/arm/shmobile.txt |    2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > Index: renesas/Documentation/devicetree/bindings/arm/shmobile.txt
> > ===================================================================
> > --- renesas.orig/Documentation/devicetree/bindings/arm/shmobile.txt
> > +++ renesas/Documentation/devicetree/bindings/arm/shmobile.txt
> > @@ -59,6 +59,8 @@ Boards:
> >      compatible = "renesas,bockw", "renesas,r8a7778"
> >    - Draak (RTP0RC77995SEB0010S)
> >      compatible = "renesas,draak", "renesas,r8a77995"
> > +  - Eagle (RTP0RC77970SEB0010S)
> > +    compatible = "renesas,eagle", "renesas,r8a77970";
> 
> You're not consistent here with the ';'.
> 
> Otherwise,
> 
> Acked-by: Rob Herring <robh@kernel.org>

Thanks, applied with Rob and Geert's tags and without the ';'.
--
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

Index: renesas/Documentation/devicetree/bindings/arm/shmobile.txt
===================================================================
--- renesas.orig/Documentation/devicetree/bindings/arm/shmobile.txt
+++ renesas/Documentation/devicetree/bindings/arm/shmobile.txt
@@ -59,6 +59,8 @@  Boards:
     compatible = "renesas,bockw", "renesas,r8a7778"
   - Draak (RTP0RC77995SEB0010S)
     compatible = "renesas,draak", "renesas,r8a77995"
+  - Eagle (RTP0RC77970SEB0010S)
+    compatible = "renesas,eagle", "renesas,r8a77970";
   - Genmai (RTK772100BC00000BR)
     compatible = "renesas,genmai", "renesas,r7s72100"
   - GR-Peach (X28A-M01-E/F)