[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240105110738.GV31813@kernel.org>
Date: Fri, 5 Jan 2024 11:07:38 +0000
From: Simon Horman <horms@...nel.org>
To: Sergey Shtylyov <s.shtylyov@....ru>
Cc: 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>
Subject: Re: [PATCH net] MAINTAINERS: I don't want to review Renesas Ethernet
Switch driver
On Fri, Jan 05, 2024 at 01:46:32PM +0300, Sergey Shtylyov wrote:
> On 1/5/24 5:42 AM, Yoshihiro Shimoda 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@....ru>
> >>>
> >>> 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
>
> No, this seems too much for these poor little couple files. :-)
> Listing them in each new MAINTAINERS entry should achieve the same goal
> and seems a biut cleaner...
>
> >>> 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.
>
> Do you want to be a maintainer or just a reviewer (like me)?
FWIIW, either option is fine by me.
>
> > So, if my idea above is acceptable, perhaps the maintainers can be:
> >
> > - RENESAS ETHERNET AVB AND SUPERH ETHERNET DRIVERS : Sergey
>
> There will be separate entries for those...
>
> > - RENESAS ETHERNET DRIVERS (COMMON PARTS) : Sergey and Shimoda
>
> I'd like to avoid that... :-)
>
> > - 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 :)
>
> You're back for 1 day? :-)
>
> > Best regards,
> > Yoshihiro Shimoda
> >
> >>> 3. Rephrase the subject and patch description as splitting out maintenance of
> >>> the Renesas Ethernet Switch driver .
>
> The question that remains still unaddressed: should I do the MAINTAINERS
> "directory split" with a single patch?
That seems easiest to me.
>
> [...]
>
> MBR, Sergey
Powered by blists - more mailing lists