netdev - Re: [PATCH 1/3] dt-bindings: net: renesas-ravb: Make stream buffer optional

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAMuHMdWCvNq2zhK+Qwsv_3oDVVGJk3mV3d4PR=-g7ajgD6uakg@mail.gmail.com>
Date:   Tue, 6 Feb 2018 16:12:23 +0100
From:   Geert Uytterhoeven <geert@...ux-m68k.org>
To:     Sergei Shtylyov <sergei.shtylyov@...entembedded.com>
Cc:     Geert Uytterhoeven <geert+renesas@...der.be>,
        Rob Herring <robh+dt@...nel.org>,
        Mark Rutland <mark.rutland@....com>,
        Simon Horman <horms@...ge.net.au>,
        Magnus Damm <magnus.damm@...il.com>,
        "David S . Miller" <davem@...emloft.net>,
        Yoshihiro Shimoda <yoshihiro.shimoda.uh@...esas.com>,
        netdev <netdev@...r.kernel.org>,
        Linux-Renesas <linux-renesas-soc@...r.kernel.org>,
        "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" 
        <devicetree@...r.kernel.org>
Subject: Re: [PATCH 1/3] dt-bindings: net: renesas-ravb: Make stream buffer optional

Hi Sergei,

On Tue, Feb 6, 2018 at 3:56 PM, Sergei Shtylyov
<sergei.shtylyov@...entembedded.com> wrote:
> On 02/06/2018 04:05 PM, Geert Uytterhoeven wrote:
>> The Stream Buffer for EtherAVB-IF (STBE) is an optional component, and
>> is not present on all SoCs.
>>
>> Document this in the DT bindings, including a list of SoCs that do have
>> it.
>>
>> Fixes: 785ec87483d1e24a ("ravb: document R8A77970 bindings")
>> Fixes: f231c4178a655b09 ("dt-bindings: net: renesas-ravb: Add support for R8A77995 RAVB")
>> Signed-off-by: Geert Uytterhoeven <geert+renesas@...der.be>
>
> Acked-by: Sergei Shtylyov <sergei.shtylyov@...entembedded.com>

Thanks!

>> --- a/Documentation/devicetree/bindings/net/renesas,ravb.txt
>> +++ b/Documentation/devicetree/bindings/net/renesas,ravb.txt
>> @@ -26,7 +26,10 @@ Required properties:
>>       SoC-specific version corresponding to the platform first followed by
>>       the generic version.
>>
>> -- reg: offset and length of (1) the register block and (2) the stream buffer.
>> +- reg: Offset and length of (1) the register block and (2) the stream buffer.
>> +       The region for the register block is mandatory.
>> +       The region for the stream buffer is optional, as it is only present on
>> +       R-Car Gen2, RZ/G1, and R-Car H3 and M3-W.
>
>    Did you deliberately omit M3-N for now?

Of course. It's not yet upstream.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@...ux-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