diff mbox

[v3,01/18] arm: shmobile: Document Kingfisher board DT bindings

Message ID 1505587727-15543-1-git-send-email-vladimir.barinov@cogentembedded.com (mailing list archive)
State Accepted
Commit 5418a9004126992aa2bbd07d79e8305659cb0dc9
Delegated to: Simon Horman
Headers show

Commit Message

Vladimir Barinov Sept. 16, 2017, 6:48 p.m. UTC
Add Kingfisher Device tree bindings Documentation, listing it as a
supported board.
Kingfisher is the H3ULCB/M3ULCB extension board.

Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
---
Changes in version 2:
- added own compatible value "shimafuji,kingfisher"
Changes in version 3:
- seperate Kingfisher entry from H3ULCB/M3ULCB

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

Comments

Geert Uytterhoeven Sept. 18, 2017, 8:51 a.m. UTC | #1
On Sat, Sep 16, 2017 at 8:48 PM, Vladimir Barinov
<vladimir.barinov@cogentembedded.com> wrote:
> Add Kingfisher Device tree bindings Documentation, listing it as a
> supported board.
> Kingfisher is the H3ULCB/M3ULCB extension board.
>
> Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>

Acked-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
Simon Horman Sept. 18, 2017, 9:13 a.m. UTC | #2
On Mon, Sep 18, 2017 at 10:51:39AM +0200, Geert Uytterhoeven wrote:
> On Sat, Sep 16, 2017 at 8:48 PM, Vladimir Barinov
> <vladimir.barinov@cogentembedded.com> wrote:
> > Add Kingfisher Device tree bindings Documentation, listing it as a
> > supported board.
> > Kingfisher is the H3ULCB/M3ULCB extension board.
> >
> > Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
> 
> Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>

Thanks, applied.
Rob Herring (Arm) Sept. 20, 2017, 8:52 p.m. UTC | #3
On Sat, Sep 16, 2017 at 09:48:47PM +0300, Vladimir Barinov wrote:
> Add Kingfisher Device tree bindings Documentation, listing it as a
> supported board.
> Kingfisher is the H3ULCB/M3ULCB extension board.
> 
> Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
> ---
> Changes in version 2:
> - added own compatible value "shimafuji,kingfisher"
> Changes in version 3:
> - seperate Kingfisher entry from H3ULCB/M3ULCB
> 
>  Documentation/devicetree/bindings/arm/shmobile.txt | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/arm/shmobile.txt b/Documentation/devicetree/bindings/arm/shmobile.txt
> index ae75cb3..5921ac2 100644
> --- a/Documentation/devicetree/bindings/arm/shmobile.txt
> +++ b/Documentation/devicetree/bindings/arm/shmobile.txt
> @@ -76,6 +76,8 @@ Boards:
>      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";

Not consistent with the ';'. Otherwise,

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

>    - Koelsch (RTP0RC7791SEB00010S)
>      compatible = "renesas,koelsch", "renesas,r8a7791"
>    - Kyoto Microcomputer Co. KZM-A9-Dual
> -- 
> 1.9.1
>
Simon Horman Sept. 21, 2017, 8:08 a.m. UTC | #4
On Wed, Sep 20, 2017 at 03:52:58PM -0500, Rob Herring wrote:
> On Sat, Sep 16, 2017 at 09:48:47PM +0300, Vladimir Barinov wrote:
> > Add Kingfisher Device tree bindings Documentation, listing it as a
> > supported board.
> > Kingfisher is the H3ULCB/M3ULCB extension board.
> > 
> > Signed-off-by: Vladimir Barinov <vladimir.barinov@cogentembedded.com>
> > ---
> > Changes in version 2:
> > - added own compatible value "shimafuji,kingfisher"
> > Changes in version 3:
> > - seperate Kingfisher entry from H3ULCB/M3ULCB
> > 
> >  Documentation/devicetree/bindings/arm/shmobile.txt | 2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/Documentation/devicetree/bindings/arm/shmobile.txt b/Documentation/devicetree/bindings/arm/shmobile.txt
> > index ae75cb3..5921ac2 100644
> > --- a/Documentation/devicetree/bindings/arm/shmobile.txt
> > +++ b/Documentation/devicetree/bindings/arm/shmobile.txt
> > @@ -76,6 +76,8 @@ Boards:
> >      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";
> 
> Not consistent with the ';'. Otherwise,
> 
> Acked-by: Rob Herring <robh@kernel.org>

Thanks, applied with the ';' removed.
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/arm/shmobile.txt b/Documentation/devicetree/bindings/arm/shmobile.txt
index ae75cb3..5921ac2 100644
--- a/Documentation/devicetree/bindings/arm/shmobile.txt
+++ b/Documentation/devicetree/bindings/arm/shmobile.txt
@@ -76,6 +76,8 @@  Boards:
     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