linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: lee.jones@linaro.org (Lee Jones)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] get_maintainer: Add subsystem to reviewer output
Date: Thu, 29 Oct 2015 09:20:07 +0000	[thread overview]
Message-ID: <20151029092007.GN4058@x1> (raw)
In-Reply-To: <1446054996.2757.139.camel@perches.com>

On Wed, 28 Oct 2015, Joe Perches wrote:
> On Wed, 2015-10-28 at 17:49 +0000, Lee Jones wrote:
> > On Wed, 28 Oct 2015, Joe Perches wrote:
> > > On Wed, 2015-10-28 at 17:22 +0000, Lee Jones wrote:
> > > > On Wed, 28 Oct 2015, Joe Perches wrote:
> > > > > >   Acked-by: Lee Jones <lee.jones@linaro.org>
> > > > > It'd be better if you could add a "tested-by:" instead.
> > > > 
> > > > Patch works as expected:
> > > >   Tested-by: Lee Jones <lee.jones@linaro.org>
> > > []
> > > >       "I'm sure we can make the output even more similar by listing
> > > >       the MAINTAINERS tag after "reviewer" too."
> > > 
> > > I guess we'll disagree on that one too.
> > 
> > You just happened to think of it a few hours after I mentioned it.
> 
> Nope, it seems a maintainers tag is what Krzysztof was suggesting
> with his "supported reviewer" bit.

Which was in reply to the bit above.

> I added the subsystem name.

(subsystem name == MAINTAINERS tag) in my comment above.

> An issue for that will be how multiple subsystem section matching
> affects the output for reviewers who are also maintainers.

I think that's okay, becuase the 'MAINTAINERS tag' will be different.

As an example:

  Lee Jones <lee.jones@linaro.org> (supporter:DRIVER SUBSYSTEM)
  Lee Jones <lee.jones@linaro.org> (reviewer:VENDOR DRIVER NAME)

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org ? Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

  reply	other threads:[~2015-10-29  9:20 UTC|newest]

Thread overview: 49+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-27 15:42 [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Lee Jones
2015-10-27 17:24 ` Sebastian Reichel
2015-10-27 18:15   ` Lee Jones
2015-10-27 18:44     ` Joe Perches
2015-10-28  1:46       ` Krzysztof Kozlowski
2015-10-28  8:24         ` Lee Jones
2015-10-28  9:21           ` Javier Martinez Canillas
2015-10-28  9:25             ` Javier Martinez Canillas
2015-10-28  9:31             ` Krzysztof Kozlowski
2015-10-28 10:28             ` Lee Jones
2015-10-28 10:53               ` Javier Martinez Canillas
2015-10-28 11:06                 ` Joe Perches
2015-10-28 11:25                   ` Javier Martinez Canillas
2015-10-28 11:39                     ` Lee Jones
2015-10-28 12:14                       ` Lee Jones
2015-10-28 12:20                         ` Joe Perches
2015-10-28 12:24                           ` Lee Jones
2015-10-28 12:46                             ` Joe Perches
2015-10-28 13:06                         ` Javier Martinez Canillas
2015-10-28 13:34                           ` Lee Jones
2015-10-28 14:09                             ` Javier Martinez Canillas
2015-10-28 14:38                               ` Lee Jones
2015-10-28 14:54                                 ` Javier Martinez Canillas
2015-10-28 23:56                                 ` Krzysztof Kozlowski
2015-10-29  0:12                                   ` Javier Martinez Canillas
2015-10-30 16:51                                     ` Lee Jones
2015-10-30 16:51                                   ` Lee Jones
2015-10-28  9:23           ` Krzysztof Kozlowski
2015-10-28  9:39             ` Uwe Kleine-König
2015-10-28  9:55               ` Lee Jones
2015-10-28 13:13                 ` Krzysztof Kozlowski
2015-10-28 16:41                   ` [PATCH] get_maintainer: Add subsystem to reviewer output Joe Perches
2015-10-28 17:01                     ` Lee Jones
2015-10-28 17:08                       ` Joe Perches
2015-10-28 17:22                         ` Lee Jones
2015-10-28 17:30                           ` Joe Perches
2015-10-28 17:49                             ` Lee Jones
2015-10-28 17:56                               ` Joe Perches
2015-10-29  9:20                                 ` Lee Jones [this message]
2015-10-29 14:14                                   ` Joe Perches
2015-10-29 16:17                                     ` Lee Jones
2015-10-28 17:18                       ` Joe Perches
2015-10-28 23:49                     ` Krzysztof Kozlowski
2015-10-28 10:14             ` [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag Lee Jones
2015-10-28 13:27               ` Krzysztof Kozlowski
2015-10-28 13:49                 ` Lee Jones
2015-10-28 16:26           ` Bartlomiej Zolnierkiewicz
2015-10-28 16:33             ` Lee Jones
2015-10-28  8:57 ` Chanwoo Choi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20151029092007.GN4058@x1 \
    --to=lee.jones@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).