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@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 > > 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
