Messages in this thread | | | Date | Wed, 28 Oct 2015 09:55:24 +0000 | From | Lee Jones <> | Subject | Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag |
| |
On Wed, 28 Oct 2015, Uwe Kleine-König wrote:
> Hello, > > On Wed, Oct 28, 2015 at 06:23:37PM +0900, Krzysztof Kozlowski wrote: > > On 28.10.2015 17:24, Lee Jones wrote: > > > You guys are pushing back like this is some kind of demotion. > > > That's not the case at all. All it does is better describe the (very > > > worthy) function you *actually* provide. > > > > It is getting into dispute about entire change of yours... which is not > > what I want. I agree with your general idea but I was referring only to > > that particular case - the Samsung PMICs (and Maxim PMICs/MUICs which > > would fall into same category). > > Not being affected by this change, I wonder what the technical > difference is if someone is listed as reviewer instead of maintainer. > Does get_maintainer.pl behave differently?
Reviewers are people who should be Cc'ed on all patches, as are Maintainers, so no, fundamentally they are treated the same by a Submitter.
Here is some output for Maintainers/Reviews from get_maintainer.pl.
$ ./scripts/get_maintainer.pl -f arch/arm/boot/dts/vf500.dtsi Shawn Guo <shawnguo@kernel.org> (maintainer:ARM/FREESCALE VYBRID ARM ARCHITECTURE) Sascha Hauer <kernel@pengutronix.de> (maintainer:ARM/FREESCALE VYBRID ARM ARCHITECTURE) Stefan Agner <stefan@agner.ch> (reviewer) Russell King <linux@arm.linux.org.uk> (maintainer:ARM PORT) linux-arm-kernel@lists.infradead.org (moderated list:ARM/FREESCALE VYBRID ARM ARCHITECTURE) devicetree@vger.kernel.org (open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS) linux-kernel@vger.kernel.org (open list)
I'm sure we can make the output even more similar by listing the MAINTAINERS tag after "reviewer" too.
-- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog
| |