From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp1.linuxfoundation.org (smtp1.linux-foundation.org [172.17.192.35]) by mail.linuxfoundation.org (Postfix) with ESMTPS id 18E0F84B for ; Thu, 15 Nov 2018 05:39:16 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) by smtp1.linuxfoundation.org (Postfix) with ESMTPS id 7FF898B for ; Thu, 15 Nov 2018 05:39:15 +0000 (UTC) Date: Wed, 14 Nov 2018 21:39:01 -0800 From: Mauro Carvalho Chehab To: Dan Williams Message-ID: <20181114213901.75768212@silica.lan> In-Reply-To: <154225759959.2499188.5163439674278041458.stgit@dwillia2-desk3.amr.corp.intel.com> References: <154225759358.2499188.15268218778137905050.stgit@dwillia2-desk3.amr.corp.intel.com> <154225759959.2499188.5163439674278041458.stgit@dwillia2-desk3.amr.corp.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: vishal.l.verma@intel.com, linux-kernel@vger.kernel.org, ksummit-discuss@lists.linuxfoundation.org, linux-nvdimm@lists.01.org Subject: Re: [Ksummit-discuss] [RFC PATCH 1/3] MAINTAINERS: Reclaim the P: tag for Subsystem Profile List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Em Wed, 14 Nov 2018 20:53:19 -0800 Dan Williams 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 > Signed-off-by: Dan Williams > --- > 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 > R: Designated reviewer: FullName > 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 > +M: Andres Salomon > 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 > -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 > 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 > M: Helmut Schaa > 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 > +E: Ben Dooks > +M: Vincent Sanders > M: Simtec Linux Team > W: http://www.simtec.co.uk/products/EB110ATX/ > S: Supported > > SIMTEC EB2410ITX (BAST) > -P: Ben Dooks > -P: Vincent Sanders > +E: Ben Dooks > +M: Vincent Sanders > M: Simtec Linux Team > 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