| Message ID | 154225759959.2499188.5163439674278041458.stgit@dwillia2-desk3.amr.corp.intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | Maintainer Handbook: Subsystem Profile | expand |
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
On Wed, 2018-11-14 at 21:39 -0800, Mauro Carvalho Chehab wrote: > 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. I'd prefer just removing all the "P:" entries. [] > > @@ -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 This is an ancient StrongARM based board that is shown as discontinued on Simtec's website. This is very obsolete and it's probably better to remove it. > > 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 [] > So, I guess this information is duplicated/obsoleted and could just > be removed. This one is still listed as a production board on Simtec's website. http://www.simtec.co.uk/products/EB2410ITX/ But I expect these 2410 boards aren't sold in any volume.
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) 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 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 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. 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
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(-)