From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <paul@pwsan.com>
Received: from smtp1.linuxfoundation.org (smtp1.linux-foundation.org
	[172.17.192.35])
	by mail.linuxfoundation.org (Postfix) with ESMTPS id C08F38B1
	for <ksummit-discuss@lists.linuxfoundation.org>;
	Sat, 17 Aug 2019 21:42:12 +0000 (UTC)
Received: from utopia.booyaka.com (utopia.booyaka.com [74.50.51.50])
	by smtp1.linuxfoundation.org (Postfix) with ESMTPS id 7FD2676D
	for <ksummit-discuss@lists.linuxfoundation.org>;
	Sat, 17 Aug 2019 21:42:11 +0000 (UTC)
Date: Sat, 17 Aug 2019 21:35:29 +0000 (UTC)
From: Paul Walmsley <paul@pwsan.com>
To: "Theodore Y. Ts'o" <tytso@mit.edu>
In-Reply-To: <20190715170045.GB3068@mit.edu>
Message-ID: <alpine.DEB.2.21.999.1908172131550.15337@utopia.booyaka.com>
References: <20190706142738.GA6893@kunai>
	<CAMuHMdUVtz6_3-9_+QLRWt6x7fauvA0K4p77eOcyVWo_oO9g5g@mail.gmail.com>
	<20190708115949.GC1050@kunai> <20190715125800.22a9a979@coco.lan>
	<20190715170045.GB3068@mit.edu>
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Cc: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>,
	ksummit <ksummit-discuss@lists.linuxfoundation.org>
Subject: Re: [Ksummit-discuss] [MAINTAINERS SUMMIT] Keeping reviews
 meaningful
List-Id: <ksummit-discuss.lists.linuxfoundation.org>
List-Unsubscribe: <https://lists.linuxfoundation.org/mailman/options/ksummit-discuss>,
	<mailto:ksummit-discuss-request@lists.linuxfoundation.org?subject=unsubscribe>
List-Archive: <http://lists.linuxfoundation.org/pipermail/ksummit-discuss/>
List-Post: <mailto:ksummit-discuss@lists.linuxfoundation.org>
List-Help: <mailto:ksummit-discuss-request@lists.linuxfoundation.org?subject=help>
List-Subscribe: <https://lists.linuxfoundation.org/mailman/listinfo/ksummit-discuss>,
	<mailto:ksummit-discuss-request@lists.linuxfoundation.org?subject=subscribe>

On Mon, 15 Jul 2019, Theodore Y. Ts'o wrote:

> I'd suggest changing the text to read:
> 
>  	 - Acked-by: indicates an agreement by the maintainer or
> 	   reviewer of the the relevant code that the patch is
> 	   appropriate for inclusion into the kernel.

This would be a positive step forward.  I would be in favor of this. 

It would also be good to state here, if it isn't stated already, that 
"reviewer" means "someone who is listed in an R: line in MAINTAINERS".


- Paul