[PATCH net-next] dt-bindings: net: ravb: Add support for r8a7744 SoC

Document RZ/G1N (R8A7744) SoC bindings.

Signed-off-by: Biju Das <redacted>
Reviewed-by: Fabrizio Castro <redacted>
---
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

Hello!

On 09/21/2018 05:25 PM, Biju Das wrote:

Document RZ/G1N (R8A7744) SoC bindings.

Signed-off-by: Biju Das <redacted>
Reviewed-by: Fabrizio Castro <redacted>

Reviewed-by: Sergei Shtylyov <redacted>

   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

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:

quoted

Document RZ/G1N (R8A7744) SoC bindings.

Signed-off-by: Biju Das <redacted>
Reviewed-by: Fabrizio Castro <redacted>

Reviewed-by: Sergei Shtylyov <redacted>

quoted

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

quoted

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.

quoted

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

On Fri, Sep 21, 2018 at 03:57:33PM +0000, Biju Das wrote:

Hi Sergei,

Thanks  for the feedback.

quoted

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:

quoted

Document RZ/G1N (R8A7744) SoC bindings.

Signed-off-by: Biju Das <redacted>
Reviewed-by: Fabrizio Castro <redacted>

Reviewed-by: Sergei Shtylyov <redacted>

quoted

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

quoted

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

quoted

quoted

       - "renesas,etheravb-r8a7745" for the R8A7745 SoC.
       - "renesas,etheravb-r8a77470" for the R8A77470 SoC.
       - "renesas,etheravb-r8a7790" for the R8A7790 SoC.

On Fri, Sep 21, 2018 at 4:32 PM Biju Das [off-list ref] wrote:

Document RZ/G1N (R8A7744) SoC bindings.

Signed-off-by: Biju Das <redacted>
Reviewed-by: Fabrizio Castro <redacted>

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