Messages in this thread |  | | Date | Wed, 14 Nov 2018 21:39:01 -0800 | From | Mauro Carvalho Chehab <> | Subject | Re: [Ksummit-discuss] [RFC PATCH 1/3] MAINTAINERS: Reclaim the P: tag for Subsystem Profile |
| |
Em Wed, 14 Nov 2018 20:53:19 -0800 Dan Williams <dan.j.williams@intel.com> escreveu:
> Fixup some P: entries to be M: and rename the remaining ones to 'E:' for > "entity". The P: tag will be used to indicate the location of a > Subsystem Profile for a given MAINTAINERS entry. > > Cc: Joe Perches <joe@perches.com> > Signed-off-by: Dan Williams <dan.j.williams@intel.com> > --- > MAINTAINERS | 18 +++++++++--------- > 1 file changed, 9 insertions(+), 9 deletions(-) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 0abecc528dac..83b7b3943a12 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -76,7 +76,7 @@ trivial patch so apply some common sense. > > Descriptions of section entries: > > - P: Person (obsolete) > + E: Entity (obsolete)
I don't like very much the idea of renaming it, but that's just my 2 cents.
IMO, the best would be to replace them to a non-obsolete field on a patch that would remove it, then another patch would re-add with a different meaning.
On a quick look, I suspect we could get rid of all remaining entries. See below.
> M: Mail patches to: FullName <address@domain> > R: Designated reviewer: FullName <address@domain> > These reviewers should be CCed on patches. > @@ -771,7 +771,7 @@ S: Orphan > F: drivers/usb/gadget/udc/amd5536udc.* > > AMD GEODE PROCESSOR/CHIPSET SUPPORT > -P: Andres Salomon <dilinger@queued.net> > +M: Andres Salomon <dilinger@queued.net> > L: linux-geode@lists.infradead.org (moderated for non-subscribers) > W: http://www.amd.com/us-en/ConnectivitySolutions/TechnicalResources/0,,50_2334_2452_11363,00.html > S: Supported > @@ -9271,7 +9271,7 @@ F: drivers/staging/media/tegra-vde/ > > MEDIA INPUT INFRASTRUCTURE (V4L/DVB) > M: Mauro Carvalho Chehab <mchehab@kernel.org>
> -P: LinuxTV.org Project > +E: LinuxTV.org Project
LinuxTV is not really an entity. Just a name for a community-maintained site where we store data about it.
That's already a "W:" tag pointing for it, so I would just remove it.
If you decide to remove, feel free to add my ack:
Acked-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
> L: linux-media@vger.kernel.org > W: https://linuxtv.org > Q: http://patchwork.kernel.org/project/linux-media/list/ > @@ -12465,7 +12465,7 @@ S: Maintained > F: arch/mips/ralink > > RALINK RT2X00 WIRELESS LAN DRIVER > -P: rt2x00 project > +E: rt2x00 project
I suspect that the above info is not really useful, and it is probably just a left-over from the past history.
I suspect that, if removed, nobody will really miss it, as it is quite obvious :-)
> M: Stanislaw Gruszka <sgruszka@redhat.com> > M: Helmut Schaa <helmut.schaa@googlemail.com> > L: linux-wireless@vger.kernel.org > @@ -12764,7 +12764,7 @@ S: Supported > F: drivers/net/ethernet/rocker/ > > ROCKETPORT DRIVER > -P: Comtrol Corp. > +E: Comtrol Corp.
There's already a W: field pointing to the company that maintains it. So, IMHO this is simply duplicated stuff that can be removed.
> W: http://www.comtrol.com > S: Maintained > F: Documentation/serial/rocket.txt > @@ -13577,15 +13577,15 @@ F: drivers/video/fbdev/simplefb.c > F: include/linux/platform_data/simplefb.h
> SIMTEC EB110ATX (Chalice CATS) > -P: Ben Dooks > -P: Vincent Sanders <vince@simtec.co.uk> > +E: Ben Dooks > +M: Vincent Sanders <vince@simtec.co.uk> > M: Simtec Linux Team <linux@simtec.co.uk> > W: http://www.simtec.co.uk/products/EB110ATX/ > S: Supported > > SIMTEC EB2410ITX (BAST) > -P: Ben Dooks > -P: Vincent Sanders <vince@simtec.co.uk> > +E: Ben Dooks > +M: Vincent Sanders <vince@simtec.co.uk> > M: Simtec Linux Team <linux@simtec.co.uk> > W: http://www.simtec.co.uk/products/EB2410ITX/ > S: Supported
On the above two drivers, I've no idea why to keep it there. Ben Dooks seems to be a past maintainer. He is already listed at CREDITS for several things including Simtec.
So, I guess this information is duplicated/obsoleted and could just be removed.
Cheers, Mauro
|  |