W dniu 28.10.2015 o 18:55, Lee Jones pisze: > 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 <[email protected]> (maintainer:ARM/FREESCALE VYBRID ARM > ARCHITECTURE) > Sascha Hauer <[email protected]> (maintainer:ARM/FREESCALE VYBRID ARM > ARCHITECTURE) > Stefan Agner <[email protected]> (reviewer) > Russell King <[email protected]> (maintainer:ARM PORT) > [email protected] (moderated list:ARM/FREESCALE VYBRID > ARM ARCHITECTURE) > [email protected] (open list:OPEN FIRMWARE AND FLATTENED DEVICE > TREE BINDINGS) > [email protected] (open list) > > I'm sure we can make the output even more similar by listing the > MAINTAINERS tag after "reviewer" too.
And here is the output of drivers/mfd/sec-core.c *before*: Sangbeom Kim <[email protected]> (supporter:SAMSUNG MULTIFUNCTION PMIC DEVICE DRIVERS) Krzysztof Kozlowski <[email protected]> (supporter:SAMSUNG MULTIFUNCTION PMIC DEVICE DRIVERS) Lee Jones <[email protected]> (supporter:MULTIFUNCTION DEVICES (MFD)) and after (where I am listed as reviewer): Sangbeom Kim <[email protected]> (reviewer) Krzysztof Kozlowski <[email protected]> (reviewer) Lee Jones <[email protected]> (supporter:MULTIFUNCTION DEVICES (MFD)) Maybe it should be "supporter reviewer"? Best regards, Krzysztof -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
