Re: [PATCH] MAINTAINERS: Start using the 'reviewer' (R) tag

From: Chanwoo Choi
Date: Wed Oct 28 2015 - 04:57:20 EST


Hi Lee,

On 2015ë 10ì 28ì 00:42, Lee Jones wrote:
> Since eafbaac ("MAINTAINERS: Add "R:" designated-reviewers tag") we
> have been able to tag specific people as Reviewers. These are key
> individuals who are tasked with or volunteer to review code submitted
> to a subsystem or specific file. However, according to MAINTAINERS
> we have 1046 Maintainers and only a mere 22 Reviewers. I believe
> these numbers to be incorrect, as many of these Maintainers are in
> fact Reviewers.
>
> I have taken the time to identify some of the Reviewers who pertain
> to subsystems which I look after, and have changed their status from
> Maintainer (collector of patches) to Reviewer (reviewer of code).
>
> Signed-off-by: Lee Jones <lee.jones@xxxxxxxxxx>
> ---
>
> MAINTAINERS | 22 +++++++++++-----------
> 1 file changed, 11 insertions(+), 11 deletions(-)

Although this patch don't include the entry of drivers/extcon,
I agree your opinion to add 'R' for Reviewer.

Acked-by: Chanwoo Choi <cw00.choi@xxxxxxxxxxx>

Thanks,
Chanwoo Choi

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/