RE: [PATCH net] MAINTAINERS: I don't want to review Renesas Ethernet Switch driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello,

Thank you for adding CC to me.

> From: Sergey Shtylyov, Sent: Friday, January 5, 2024 5:47 AM
> 
> On 1/4/24 12:28 AM, Simon Horman wrote:
> [...]
> >> I don't know this hardware, I don't have the manuals for it, so I can't
> >> provide a good review.  Let's exclude the Ethernet Switch related files.
> >>
> >> Signed-off-by: Sergey Shtylyov <s.shtylyov@xxxxxx>
> >
> > Hi Sergey,
> 
>   Hi Simon, long time, no see... :-)
> 
> > I don't know the back story to this, if there is one.
> 
>    Not much of a history: I got a lot of the rswitch patches in my
> inbox and I mostly ignored them... but kept deferring this patch due
> to a high load with fixing Svace's reports at work...
> 
> > But could I suggest that:
> >
> > 1. The patch also updates the title MAINTAINERS section to cover the
> >    remaining two drivers.
> >
> >    e.g.: RENESAS ETHERNET DRIVERS ->
> >          RENESAS ETHERNET AVB AND SUPERH ETHERNET DRIVERS
> >
> >    Or alternatively, create separate sections for each driver.
> 
>    Yeah, this 2nd option seems cleaner. Still not sure about Kconfig/
> Makefile though...

How about adding "COMMON PARTS" section?

- RENESAS ETHERNET AVB AND SUPERH ETHERNET DRIVERS : For ravb and sh_eth
- RENESAS ETHERNET DRIVERS (COMMON PARTS) : For Kconfig/Makefile in drivers/net/ethernet/renesas
- RENESAS ETHERNET SWITCH DRIVER : For rswitch

> >    n.b.: This may involve moving sections to maintain alphabetical order
> >          by section title
> >
> > 2. Reaching out to Shimoda-san (CCed) or other relevant parties
> >    to see if an appropriate maintainer or maintainers for the
> >    Renesas Ethernet Switch driver can be found.

I'm happy if I'm a maintainer for Renesas Ethernet Switch driver.
So, if my idea above is acceptable, perhaps the maintainers can be:

- RENESAS ETHERNET AVB AND SUPERH ETHERNET DRIVERS : Sergey
- RENESAS ETHERNET DRIVERS (COMMON PARTS) : Sergey and Shimoda
- RENESAS ETHERNET SWITCH DRIVER : Shimoda

> >    n.b.: It may still be a holiday period in Japan for the rest of the week.
> 
>    It's a holiday period here in Russia as well, till the 8th of Jaunary. :-)

I'm back from today, but next Monday (8th) is a holiday in Japan :)

Best regards,
Yoshihiro Shimoda

> > 3. Rephrase the subject and patch description as splitting out maintenance of
> >    the Renesas Ethernet Switch driver .
> 
> [...]
> 
> MBR, Sergey




[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux