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@xxxxxxxxxxxxxx> > > Reviewed-by: Fabrizio Castro <fabrizio.castro@xxxxxxxxxxxxxx> > > Reviewed-by: Sergei Shtylyov <sergei.shtylyov@xxxxxxxxxxxxxxxxxx> > > > --- > > 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.