From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759394AbbJ3QwE (ORCPT ); Fri, 30 Oct 2015 12:52:04 -0400 Received: from mail-wm0-f42.google.com ([74.125.82.42]:37389 "EHLO mail-wm0-f42.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751337AbbJ3QwC (ORCPT ); Fri, 30 Oct 2015 12:52:02 -0400 Date: Fri, 30 Oct 2015 16:51:57 +0000 From: Lee Jones To: Javier Martinez Canillas Cc: Joe Perches , Krzysztof Kozlowski , Wolfram Sang , Linus Walleij , Sebastian Reichel , Linux Kernel , Chanwoo Choi , Mark Brown , Ben Dooks , "linux-arm-kernel@lists.infradead.org" Subject: Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Message-ID: <20151030165157.GX4058@x1> References: <1446030399.2757.83.camel@perches.com> <20151028113932.GS5828@x1> <20151028121454.GA4058@x1> <20151028133438.GC4058@x1> <20151028143838.GG4058@x1> <563160B2.7070305@samsung.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.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