diff mbox

dt-bindings: net: renesas-ravb: Add support for R8A77995 RAVB

Message ID 1505305060-4192-1-git-send-email-yoshihiro.shimoda.uh@renesas.com (mailing list archive)
State Superseded
Delegated to: Geert Uytterhoeven
Headers show

Commit Message

Yoshihiro Shimoda Sept. 13, 2017, 12:17 p.m. UTC
Add a new compatible string for the R8A77995 (R-Car D3) RAVB.

Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
---
 Documentation/devicetree/bindings/net/renesas,ravb.txt | 1 +
 1 file changed, 1 insertion(+)

Comments

Geert Uytterhoeven Sept. 13, 2017, 2:47 p.m. UTC | #1
On Wed, Sep 13, 2017 at 2:17 PM, Yoshihiro Shimoda
<yoshihiro.shimoda.uh@renesas.com> wrote:
> Add a new compatible string for the R8A77995 (R-Car D3) RAVB.
>
> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.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
Sergei Shtylyov Sept. 13, 2017, 4:02 p.m. UTC | #2
Hello!

On 09/13/2017 03:17 PM, Yoshihiro Shimoda wrote:

> Add a new compatible string for the R8A77995 (R-Car D3) RAVB.
> 
> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.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 1672353..ae2213f 100644
> --- a/Documentation/devicetree/bindings/net/renesas,ravb.txt
> +++ b/Documentation/devicetree/bindings/net/renesas,ravb.txt
> @@ -17,6 +17,7 @@ Required properties:
>   
>         - "renesas,etheravb-r8a7795" for the R8A7795 SoC.
>         - "renesas,etheravb-r8a7796" for the R8A7796 SoC.
> +      - "renesas,etheravb-r8a77995" for the R8A77995 SoC.

    That would conflict with the R8A77970 bindings patch posted by me 
yesterday. Please respin atop of it.

>         - "renesas,etheravb-rcar-gen3" as a fallback for the above
>   		R-Car Gen3 devices.
>   

    One more fragment is needed here, about the mandatory 'interrupt-names" 
prop. My patch makes this change unnecessary, however...

MBR, Sergei
Sergei Shtylyov Sept. 13, 2017, 4:08 p.m. UTC | #3
On 09/13/2017 07:02 PM, Sergei Shtylyov wrote:

>> Add a new compatible string for the R8A77995 (R-Car D3) RAVB.
>>
>> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.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 1672353..ae2213f 100644
>> --- a/Documentation/devicetree/bindings/net/renesas,ravb.txt
>> +++ b/Documentation/devicetree/bindings/net/renesas,ravb.txt
>> @@ -17,6 +17,7 @@ Required properties:
>>         - "renesas,etheravb-r8a7795" for the R8A7795 SoC.
>>         - "renesas,etheravb-r8a7796" for the R8A7796 SoC.
>> +      - "renesas,etheravb-r8a77995" for the R8A77995 SoC.
> 
>     That would conflict with the R8A77970 bindings patch posted by me 
> yesterday. Please respin atop of it.

    Here's the link for your convenience:

https://marc.info/?l=linux-renesas-soc&m=150524655515476

[...]

MBR, Sergei
Yoshihiro Shimoda Sept. 14, 2017, 12:11 a.m. UTC | #4
Hello!

> From: Sergei Shtylyov

> Sent: Thursday, September 14, 2017 1:02 AM

> 

> Hello!

> 

> On 09/13/2017 03:17 PM, Yoshihiro Shimoda wrote:

> 

> > Add a new compatible string for the R8A77995 (R-Car D3) RAVB.

> >

> > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.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 1672353..ae2213f 100644

> > --- a/Documentation/devicetree/bindings/net/renesas,ravb.txt

> > +++ b/Documentation/devicetree/bindings/net/renesas,ravb.txt

> > @@ -17,6 +17,7 @@ Required properties:

> >

> >         - "renesas,etheravb-r8a7795" for the R8A7795 SoC.

> >         - "renesas,etheravb-r8a7796" for the R8A7796 SoC.

> > +      - "renesas,etheravb-r8a77995" for the R8A77995 SoC.

> 

>     That would conflict with the R8A77970 bindings patch posted by me

> yesterday. Please respin atop of it.


Yes, I submitted v2 patch now.

> >         - "renesas,etheravb-rcar-gen3" as a fallback for the above

> >   		R-Car Gen3 devices.

> >

> 

>     One more fragment is needed here, about the mandatory 'interrupt-names"

> prop. My patch makes this change unnecessary, however...


I understood it.

Best regards,
Yoshihiro Shimoda

> MBR, Sergei
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/net/renesas,ravb.txt b/Documentation/devicetree/bindings/net/renesas,ravb.txt
index 1672353..ae2213f 100644
--- a/Documentation/devicetree/bindings/net/renesas,ravb.txt
+++ b/Documentation/devicetree/bindings/net/renesas,ravb.txt
@@ -17,6 +17,7 @@  Required properties:
 
       - "renesas,etheravb-r8a7795" for the R8A7795 SoC.
       - "renesas,etheravb-r8a7796" for the R8A7796 SoC.
+      - "renesas,etheravb-r8a77995" for the R8A77995 SoC.
       - "renesas,etheravb-rcar-gen3" as a fallback for the above
 		R-Car Gen3 devices.