Re: [PATCH net] MAINTAINERS: merge entries for felix and ocelot drivers

Vladimir Oltean
2020-06-15T16:43:42Z

On Mon, 15 Jun 2020 at 19:37, Jakub Kicinski <kuba@kernel.org> wrote:
>
> On Sun, 14 Jun 2020 01:07:53 +0300 Vladimir Oltean wrote:
> > From: Vladimir Oltean <vladimir.oltean@nxp.com>
> >
> > The ocelot switchdev driver also provides a set of library functions for
> > the felix DSA driver, which in practice means that most of the patches
> > will be of interest to both groups of driver maintainers.
> >
> > So, as also suggested in the discussion here, let's merge the 2 entries
> > into a single larger one:
> > https://www.spinics.net/lists/netdev/msg657412.html
> >
> > Note that the entry has been renamed into "OCELOT SWITCH" since neither
> > Vitesse nor Microsemi exist any longer as company names, instead they
> > are now named Microchip (which again might be subject to change in the
> > future), so use the device family name instead.
> >
> > Suggested-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
> > Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
>
> Looks like checkpatch is unhappy about the order of files?
>
> WARNING: Misordered MAINTAINERS entry - list file patterns in alphabetic order
> #58: FILE: MAINTAINERS:12308:
> +F: include/soc/mscc/ocelot*
> +F: drivers/net/ethernet/mscc/
>
> WARNING: Misordered MAINTAINERS entry - list file patterns in alphabetic order
> #59: FILE: MAINTAINERS:12309:
> +F: drivers/net/ethernet/mscc/
> +F: drivers/net/dsa/ocelot/*
>
> total: 0 errors, 2 warnings, 0 checks, 46 lines checked

Thanks for checking, Jakub. I sent a v2 now.

-Vladimir