From mboxrd@z Thu Jan 1 00:00:00 1970 From: lee.jones@linaro.org (Lee Jones) Date: Fri, 30 Oct 2015 16:51:57 +0000 Subject: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag In-Reply-To: References: <1446030399.2757.83.camel@perches.com> <20151028113932.GS5828@x1> <20151028121454.GA4058@x1> <20151028133438.GC4058@x1> <20151028143838.GG4058@x1> <563160B2.7070305@samsung.com> Message-ID: <20151030165157.GX4058@x1> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thu, 29 Oct 2015, Javier Martinez Canillas wrote: > Hello Lee, > > On Thu, Oct 29, 2015 at 12:56 AM, Krzysztof Kozlowski > wrote: > > On 28.10.2015 23:38, Lee Jones wrote: > >> On Wed, 28 Oct 2015, Javier Martinez Canillas wrote: > >>> They are not maintainers according to your definition of maintainer > >>> that doesn't seem what most people agree with. > >> > >> "most people" so far are 3 people that I assume still want to be > >> Maintainers despite not actually conducting Maintainer duties, but > >> are rather Reviewers. I also have 2 Acks for this patch, so thus far > >> that's 3 that agree and 3 that do not. Unsurprisingly the ones that > >> agree are Maintainers and the ones who are not are (by my definition) > >> Reviewers -- go figure. > >> > > > > I am not sure on which side you put me finally. :) > > If there is a consensus among some more experienced developers that > > maintainer means branch and patches maintaining, then I won't see any > > problem with the patch nor with switching Samsung PMIC entries to review. > > > > In that case, that would be: > > Acked-by: Krzysztof Kozlowski > > > > Same for me, what I don't want is to have different meanings per > subsystems of what maintainers and reviewers mean since that could > confuse developers posting patches. > > But if there is a kernel wide consensus and all subsystems entries are > going to be updated to use the same semantics and list as Reviewer to > people that don't keep git trees, then I'm OK with this patch and you > can add my: > > Reviewed-by: Javier Martinez Canillas Thanks Javier, but I'm going to drop this patch. I can see it causing more issues than it'll solve. -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org ? Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog