* [PATCH net-next] dt-bindings: net: ravb: Add support for r8a7744 SoC
@ 2018-09-21 14:25 Biju Das
2018-09-21 15:48 ` Sergei Shtylyov
2018-09-24 9:26 ` Geert Uytterhoeven
0 siblings, 2 replies; 5+ messages in thread
From: Biju Das @ 2018-09-21 14:25 UTC (permalink / raw)
To: Rob Herring, Mark Rutland
Cc: Biju Das, Sergei Shtylyov, David S. Miller, Geert Uytterhoeven,
Simon Horman, Fabrizio Castro, Jacopo Mondi, Yoshihiro Shimoda,
netdev, linux-renesas-soc, devicetree, Simon Horman,
Chris Paterson
Document RZ/G1N (R8A7744) SoC bindings.
Signed-off-by: Biju Das <biju.das@bp.renesas.com>
Reviewed-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
---
This patch is tested against net-next
---
Documentation/devicetree/bindings/net/renesas,ravb.txt | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/net/renesas,ravb.txt b/Documentation/devicetree/bindings/net/renesas,ravb.txt
index da249b7..3530256 100644
--- a/Documentation/devicetree/bindings/net/renesas,ravb.txt
+++ b/Documentation/devicetree/bindings/net/renesas,ravb.txt
@@ -6,6 +6,7 @@ interface contains.
Required properties:
- compatible: Must contain one or more of the following:
- "renesas,etheravb-r8a7743" for the R8A7743 SoC.
+ - "renesas,etheravb-r8a7744" for the R8A7744 SoC.
- "renesas,etheravb-r8a7745" for the R8A7745 SoC.
- "renesas,etheravb-r8a77470" for the R8A77470 SoC.
- "renesas,etheravb-r8a7790" for the R8A7790 SoC.
--
2.7.4
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH net-next] dt-bindings: net: ravb: Add support for r8a7744 SoC
2018-09-21 14:25 [PATCH net-next] dt-bindings: net: ravb: Add support for r8a7744 SoC Biju Das
@ 2018-09-21 15:48 ` Sergei Shtylyov
2018-09-21 15:57 ` Biju Das
2018-09-24 9:26 ` Geert Uytterhoeven
1 sibling, 1 reply; 5+ messages in thread
From: Sergei Shtylyov @ 2018-09-21 15:48 UTC (permalink / raw)
To: Biju Das, Rob Herring, Mark Rutland
Cc: David S. Miller, Geert Uytterhoeven, Simon Horman,
Fabrizio Castro, Jacopo Mondi, Yoshihiro Shimoda, netdev,
linux-renesas-soc, devicetree, Simon Horman, Chris Paterson
Hello!
On 09/21/2018 05:25 PM, Biju Das wrote:
> Document RZ/G1N (R8A7744) SoC bindings.
>
> Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> Reviewed-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
Reviewed-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
> ---
> Documentation/devicetree/bindings/net/renesas,ravb.txt | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/devicetree/bindings/net/renesas,ravb.txt b/Documentation/devicetree/bindings/net/renesas,ravb.txt
> index da249b7..3530256 100644
> --- a/Documentation/devicetree/bindings/net/renesas,ravb.txt
> +++ b/Documentation/devicetree/bindings/net/renesas,ravb.txt
> @@ -6,6 +6,7 @@ interface contains.
> Required properties:
> - compatible: Must contain one or more of the following:
> - "renesas,etheravb-r8a7743" for the R8A7743 SoC.
> + - "renesas,etheravb-r8a7744" for the R8A7744 SoC.
Mmm, I thought it was decided to go with the full model #s, like with R8A77470 below?
> - "renesas,etheravb-r8a7745" for the R8A7745 SoC.
> - "renesas,etheravb-r8a77470" for the R8A77470 SoC.
> - "renesas,etheravb-r8a7790" for the R8A7790 SoC.
MBR, Sergei
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH net-next] dt-bindings: net: ravb: Add support for r8a7744 SoC
2018-09-21 15:48 ` Sergei Shtylyov
@ 2018-09-21 15:57 ` Biju Das
2018-09-24 8:45 ` Simon Horman
0 siblings, 1 reply; 5+ messages in thread
From: Biju Das @ 2018-09-21 15:57 UTC (permalink / raw)
To: Sergei Shtylyov, Rob Herring, Mark Rutland
Cc: David S. Miller, Geert Uytterhoeven, Simon Horman,
Fabrizio Castro, Jacopo Mondi, Yoshihiro Shimoda,
netdev@vger.kernel.org, linux-renesas-soc@vger.kernel.org,
devicetree@vger.kernel.org, Simon Horman, Chris Paterson
Hi Sergei,
Thanks for the feedback.
> Subject: Re: [PATCH net-next] dt-bindings: net: ravb: Add support for
> r8a7744 SoC
>
> Hello!
>
> On 09/21/2018 05:25 PM, Biju Das wrote:
>
> > Document RZ/G1N (R8A7744) SoC bindings.
> >
> > Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> > Reviewed-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
>
> Reviewed-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
>
> > ---
> > Documentation/devicetree/bindings/net/renesas,ravb.txt | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/Documentation/devicetree/bindings/net/renesas,ravb.txt
> b/Documentation/devicetree/bindings/net/renesas,ravb.txt
> > index da249b7..3530256 100644
> > --- a/Documentation/devicetree/bindings/net/renesas,ravb.txt
> > +++ b/Documentation/devicetree/bindings/net/renesas,ravb.txt
> > @@ -6,6 +6,7 @@ interface contains.
> > Required properties:
> > - compatible: Must contain one or more of the following:
> > - "renesas,etheravb-r8a7743" for the R8A7743 SoC.
> > + - "renesas,etheravb-r8a7744" for the R8A7744 SoC.
>
> Mmm, I thought it was decided to go with the full model #s, like with
> R8A77470 below?
For RZ/G1C case, there are 2 variants 1 with HDMI(R8A77470) and the other without HDMI(R8A77471).
Current support is for R8A77470.
> > - "renesas,etheravb-r8a7745" for the R8A7745 SoC.
> > - "renesas,etheravb-r8a77470" for the R8A77470 SoC.
> > - "renesas,etheravb-r8a7790" for the R8A7790 SoC.
Regards,
Biju
Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH net-next] dt-bindings: net: ravb: Add support for r8a7744 SoC
2018-09-21 15:57 ` Biju Das
@ 2018-09-24 8:45 ` Simon Horman
0 siblings, 0 replies; 5+ messages in thread
From: Simon Horman @ 2018-09-24 8:45 UTC (permalink / raw)
To: Biju Das
Cc: Sergei Shtylyov, Rob Herring, Mark Rutland, David S. Miller,
Geert Uytterhoeven, Fabrizio Castro, Jacopo Mondi,
Yoshihiro Shimoda, netdev@vger.kernel.org,
linux-renesas-soc@vger.kernel.org, devicetree@vger.kernel.org,
Chris Paterson
On Fri, Sep 21, 2018 at 03:57:33PM +0000, Biju Das wrote:
> Hi Sergei,
>
> Thanks for the feedback.
>
> > Subject: Re: [PATCH net-next] dt-bindings: net: ravb: Add support for
> > r8a7744 SoC
> >
> > Hello!
> >
> > On 09/21/2018 05:25 PM, Biju Das wrote:
> >
> > > Document RZ/G1N (R8A7744) SoC bindings.
> > >
> > > Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> > > Reviewed-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
> >
> > Reviewed-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
> >
> > > ---
> > > Documentation/devicetree/bindings/net/renesas,ravb.txt | 1 +
> > > 1 file changed, 1 insertion(+)
> > >
> > > diff --git a/Documentation/devicetree/bindings/net/renesas,ravb.txt
> > b/Documentation/devicetree/bindings/net/renesas,ravb.txt
> > > index da249b7..3530256 100644
> > > --- a/Documentation/devicetree/bindings/net/renesas,ravb.txt
> > > +++ b/Documentation/devicetree/bindings/net/renesas,ravb.txt
> > > @@ -6,6 +6,7 @@ interface contains.
> > > Required properties:
> > > - compatible: Must contain one or more of the following:
> > > - "renesas,etheravb-r8a7743" for the R8A7743 SoC.
> > > + - "renesas,etheravb-r8a7744" for the R8A7744 SoC.
> >
> > Mmm, I thought it was decided to go with the full model #s, like with
> > R8A77470 below?
>
> For RZ/G1C case, there are 2 variants 1 with HDMI(R8A77470) and the other without HDMI(R8A77471).
> Current support is for R8A77470.
r8a7744 is consistent with other bindings for this SoC and my understanding
is that it is a) sufficient to differentiate this SoC and b) in keeping
with most other RZ/G1 SoCs (RZ/G1C being the exception).
In short, this seems fine to me.
Reviewed-by: Simon Horman <horms+renesas@verge.net.au>
> > > - "renesas,etheravb-r8a7745" for the R8A7745 SoC.
> > > - "renesas,etheravb-r8a77470" for the R8A77470 SoC.
> > > - "renesas,etheravb-r8a7790" for the R8A7790 SoC.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH net-next] dt-bindings: net: ravb: Add support for r8a7744 SoC
2018-09-21 14:25 [PATCH net-next] dt-bindings: net: ravb: Add support for r8a7744 SoC Biju Das
2018-09-21 15:48 ` Sergei Shtylyov
@ 2018-09-24 9:26 ` Geert Uytterhoeven
1 sibling, 0 replies; 5+ messages in thread
From: Geert Uytterhoeven @ 2018-09-24 9:26 UTC (permalink / raw)
To: Biju Das
Cc: Rob Herring, Mark Rutland, Sergei Shtylyov, David S. Miller,
Geert Uytterhoeven, Simon Horman, Fabrizio Castro, Jacopo Mondi,
Yoshihiro Shimoda, netdev, Linux-Renesas,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
Simon Horman, Chris Paterson
On Fri, Sep 21, 2018 at 4:32 PM Biju Das <biju.das@bp.renesas.com> wrote:
> Document RZ/G1N (R8A7744) SoC bindings.
>
> Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> Reviewed-by: Fabrizio Castro <fabrizio.castro@bp.renesas.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
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2018-09-24 9:26 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-09-21 14:25 [PATCH net-next] dt-bindings: net: ravb: Add support for r8a7744 SoC Biju Das
2018-09-21 15:48 ` Sergei Shtylyov
2018-09-21 15:57 ` Biju Das
2018-09-24 8:45 ` Simon Horman
2018-09-24 9:26 ` Geert Uytterhoeven
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).