Received-SPF: pass (google.com: domain of linux-kernel+bounces-155646-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Date: Tue, 23 Apr 2024 10:13:03 -0700
From: Dan Williams <dan.j.williams@intel.com>
To: Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
	<workflows@vger.kernel.org>, <linux-doc@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>
CC: Jonathan Corbet <corbet@lwn.net>
Subject: Re: [PATCH v1 1/2] Documentation: process: Avoid unneeded Cc: tags
Message-ID: <6627ec1f71a4b_a96f294f1@dwillia2-mobl3.amr.corp.intel.com.notmuch>
References: <20240423132024.2368662-1-andriy.shevchenko@linux.intel.com>
 <20240423132024.2368662-2-andriy.shevchenko@linux.intel.com>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <20240423132024.2368662-2-andriy.shevchenko@linux.intel.com>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?PM/Rvq7ESGedqHgSTiMI+W06RJvWZW3irTLY810RkH9WR4aX2p3GKUXDcq3p?=
 =?us-ascii?Q?IrH94Umoocgxi/tS3A1MVjtm8TIkj4472wSHWdtJniiRrNM+PuczX2duzl+l?=
 =?us-ascii?Q?/5YgIfHML+jbO+qjlEBMxG01DwaIY/I9e3uB+xSFB3pEF3gDxJ9YMeXH5PQo?=
 =?us-ascii?Q?OUccgfESPDa4ZJ8V7G41jkSU7w3gMOLIEbkvCQ3TprFbDe6cR1VY684LJVJZ?=
 =?us-ascii?Q?GipTCD6XfGzeKQ+uHbyW9DYZIBBkBWzYpgd71XuEOelYY1we2puX/w+9T0W8?=
 =?us-ascii?Q?ij+SZ2C6jgHhmRoC+VhuDfbQlm3pOxkPqOlfSZArESKh1RirUMUB6X65bFE6?=
 =?us-ascii?Q?Yo4yKLiO1ALrKepKQ2my73DX5bWlIrk4AB45sZjqUWded0wiSldQ/7C5eOjV?=
 =?us-ascii?Q?m9oRYP/cNr/exPiCdz09EoFNGAVz602Mi++nb+dSBC+qJLJUa7o53Yq6PN80?=
 =?us-ascii?Q?f/M7kIjfzjkI5akmXF9brduL7j0vh07wOaIRC1yi3C9bfQCfVUKfBew5IeBG?=
 =?us-ascii?Q?D68oUL7WaaHusjD4B+tWZXrDIfBB66lAEFksPdWTgTywYd/unYSSP6CtaRnE?=
 =?us-ascii?Q?5oqDGnb41+h/+gAFTLM1RyxmZw3ON46EqPIldRvb6XX4MsB1vFp+Lg6dEXOF?=
 =?us-ascii?Q?FDr2Tr8TJ8lDtme/MDcmdpHQVK24zimGhbks0NYRL7Crq27YKbhxCEktgL+n?=
 =?us-ascii?Q?0rw3DqqBcF4krJZ9T0MdByQWMsVhnkVo901GnFuUV16skuPPp7Mh3WR0NMSV?=
 =?us-ascii?Q?Nl5MFlhdWtfLMo8J5xZCPlIDxTdhGj4GHo+KZANWm+nffYIazu3Pc6ETEBVt?=
 =?us-ascii?Q?EZDxvA7KRofggtgilEvMJrrmrq4CNc9yRvo+aNA65THca9kCiQmhMHN2bR77?=
 =?us-ascii?Q?NAvsn9AXnKP5rUV0SKlV71xLrfaY7BWsbV5Q3c1iM3rEfLpN1z5t0crH3Yk0?=
 =?us-ascii?Q?exVE9yKjDKrh3QTtl8WOCEa/04TEH28+OQxReq1L8oUAxF7KGqWqFF9WK6g1?=
 =?us-ascii?Q?4mvF046mHa+lhKFSpJ2dYKY+EUrNJOv9TF4/Hm3bqB/Hf8F4Uhw2hVLYwi98?=
 =?us-ascii?Q?tTTEA5XYYBybbkmFt7yasCfIQWpoD4rAmI/YZlj+YJ6ar5aQbUGISma9PgOo?=
 =?us-ascii?Q?iH4cTMrQkWuXTtT6N0ZCASBpIjTjhId6UdsIOToFqFVtiaADtLoiCZ1HpxwX?=
 =?us-ascii?Q?NTKA+vWvq+qeDw5J163bGKUxBb0iBX5S25VboGsx/+wrxQ4fsS58HfmntPC/?=
 =?us-ascii?Q?Im+MXEAF0cyZJh/EHH6VZV3ZpIQ0xgQqb9VNpX0nNcXqEiB1QJmzwyJ7mkAf?=
 =?us-ascii?Q?xgHgvEAvJKMyE2Aqdd+w1JFMj3LOyFaLb8Zc0qQiEcjnkq/z7aykSKpX93hC?=
 =?us-ascii?Q?ebA55xn2I2TUMTQBXRmxOqxWSTsgbFGf4QnZcfMQwjUfa5OQtw4akXuYiuqt?=
 =?us-ascii?Q?/ql/77ZjQCdLJbnX0QyBB43nDMANiTAb1ET3O0nrBxL7qPpc+LKyYYVXv5gR?=
 =?us-ascii?Q?9c2oxMLczLzG48Ia/DaonAwULJZzTomzKia6SY2MITWwI3ghOlVYUXncSWpQ?=
 =?us-ascii?Q?5kBOsezG6zLyBuyCqw1phR8SPqf3M4cHDvlr97KB36D6Pf6mrgP0xv3I6I9R?=
 =?us-ascii?Q?iQ=3D=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: ac914cde-805f-4f54-f73d-08dc63b8a3e0
X-MS-Exchange-CrossTenant-AuthSource: PH8PR11MB8107.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Apr 2024 17:13:05.5582
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: GmtZa3jvXMUoBAURjswCI/ApFIoxnc60wpeu9Hb1VkKDejzoMAcOnEYn3vj/vYFsTttEtGweqWt4XZrbvDuB/ebNp5Cn0eXNBetXml+1gaQ=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW5PR11MB5932
X-OriginatorOrg: intel.com

Andy Shevchenko wrote:
> Add a note that explains that  Cc: email header is implied by other
> tags, such as Reviewed-by:. In this case an explicit Cc: is _not_
> needed.
> 
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> ---
>  Documentation/process/5.Posting.rst          | 4 +++-
>  Documentation/process/submitting-patches.rst | 5 ++++-
>  2 files changed, 7 insertions(+), 2 deletions(-)
[..]
> 
> diff --git a/Documentation/process/submitting-patches.rst b/Documentation/process/submitting-patches.rst
> index 66029999b587..6775f0698136 100644
> --- a/Documentation/process/submitting-patches.rst
> +++ b/Documentation/process/submitting-patches.rst
> @@ -486,7 +486,10 @@ provided such comments, you may optionally add a ``Cc:`` tag to the patch.
>  This is the only tag which might be added without an explicit action by the
>  person it names - but it should indicate that this person was copied on the
>  patch.  This tag documents that potentially interested parties
> -have been included in the discussion.
> +have been included in the discussion. Note that other formal tags are
> +automatically converted to the Cc: email header and you do not need to
> +have an explicit ``Cc:`` tag, if the person is already mentioned by another
> +tag.

This just looks like a licsense to needle people that happen to ship a
duplicate tag. It does not feel like a net improvement to community
discourse.

Instead, one positive contribution in this area might be to patch "b4
am" to cleanup redundant tags when a Cc: is repeated by another tag.
For example:

    b4 am 20231018115038.0000433d@Huawei.com

..could have elided the Cc: for Jonathan after applying his
Reviewed-by:.