netdev - RE: [PATCH 00/14] Add functional support for Gigabit Ethernet driver

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]

Date:   Sun, 10 Oct 2021 10:31:01 +0000
From:   Biju Das <biju.das.jz@...renesas.com>
To:     Sergey Shtylyov <s.shtylyov@....ru>,
        "David S. Miller" <davem@...emloft.net>,
        Jakub Kicinski <kuba@...nel.org>
CC:     Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@...renesas.com>,
        Andrew Lunn <andrew@...n.ch>,
        Geert Uytterhoeven <geert+renesas@...der.be>,
        Adam Ford <aford173@...il.com>,
        Yoshihiro Shimoda <yoshihiro.shimoda.uh@...esas.com>,
        "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
        "linux-renesas-soc@...r.kernel.org" 
        <linux-renesas-soc@...r.kernel.org>,
        Chris Paterson <Chris.Paterson2@...esas.com>,
        Biju Das <biju.das@...renesas.com>
Subject: RE: [PATCH 00/14] Add functional support for Gigabit Ethernet driver

Hi Sergei,

> Subject: Re: [PATCH 00/14] Add functional support for Gigabit Ethernet
> driver
> 
> On 10.10.2021 12:25, Biju Das wrote:
> 
> [...]
> >>>>>> The DMAC and EMAC blocks of Gigabit Ethernet IP found on RZ/G2L
> >>>>>> SoC are similar to the R-Car Ethernet AVB IP.
> >>>>>>
> >>>>>> The Gigabit Ethernet IP consists of Ethernet controller (E-MAC),
> >>>>>> Internal TCP/IP Offload Engine (TOE)  and Dedicated Direct memory
> >>>>>> access controller (DMAC).
> >>>>>>
> >>>>>> With a few changes in the driver we can support both IPs.
> >>>>>>
> >>>>>> This patch series is aims to add functional support for Gigabit
> >>>>>> Ethernet driver by filling all the stubs except set_features.
> >>>>>>
> >>>>>> set_feature patch will send as separate RFC patch along with
> >>>>>> rx_checksum patch, as it needs detailed discussion related to HW
> >>>>> checksum.
> >>>>>>
> >>>>>> Ref:-
> >>>>>>
> >>>>>> https://jpn01.safelinks.protection.outlook.com/?url=https%3A%2F%2
> >>>>>> Fp
> >>>>>> atc
> >>>>>> hwork.kernel.org%2Fproject%2Flinux-renesas-soc%2Flist%2F%3Fseries
> >>>>>> %3
> >>>>>> D55
> >>>>>> 7655&amp;data=04%7C01%7Cbiju.das.jz%40bp.renesas.com%7C25bc7b9155
> >>>>>> d8
> >>>>>> 402
> >>>>>> a191808d98b5ae62f%7C53d82571da1947e49cb4625a166a4a2a%7C0%7C0%7C63
> >>>>>> 76
> >>>>>> 940
> >>>>>> 44814904836%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV
> >>>>>> 2l
> >>>>>> uMz
> >>>>>> IiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=Vktj5v0GvrNf%2B
> >>>>>> DN
> >>>>>> IFs
> >>>>>> e6xjCUm6OjtzwHvK3q8aG1E5Y%3D&amp;reserved=0
> >>>>>>
> >>>>>> RFC->V1:
> >>>>>>    * Removed patch#3 will send it as RFC
> >>>>>>    * Removed rx_csum functionality from patch#7, will send it as
> >>>>>> RFC
> >>>>>>    * Renamed "nc_queue" -> "nc_queues"
> >>>>>>    * Separated the comment patch into 2 separate patches.
> >>>>>>    * Documented PFRI register bit
> >>>>>>    * Added Sergy's Rb tag
> >>>>>
> >>>>>      It's Sergey. :-)
> >>>>
> >>>> My Bad. Sorry will taken care this in future. I need to send V2, as
> >>>> accidentally I have added 2 macros in patch #6 As part of RFC
> >>>> discussion into v1. I will send V2 to remove this.
> >>>
> >>>      I'm not seeing patches #2, #4, and #9 in my inboxes... :-/
> >>
> >>      Seeing them now in the linux-renesas-soc folder in the GMail
> account.
> >> But they should have landed on the OMP account too. :-/
> >
> > Can you please confirm latest series[1] lands on your OMP account?
> > [1]
> > https://jpn01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc
> > hwork.kernel.org%2Fproject%2Fnetdevbpf%2Flist%2F%3Fseries%3D560617&amp
> > ;data=04%7C01%7Cbiju.das.jz%40bp.renesas.com%7C4ba52eb2327b42fb997c08d
> > 98bd48373%7C53d82571da1947e49cb4625a166a4a2a%7C0%7C0%7C637694567141278
> > 405%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTi
> > I6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=LyurKu2sR%2BujQkts4LRyLMfPxw
> > 7xmjON91zgS7f5Ktg%3D&amp;reserved=0
> 
>     No, as I've told you already. Was unclear again. :-)

Which patch doesn't have OMP account? I am sure, I have added your OMP account 
As first cc list in the latest series.

Regards,
Biju