Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:4 as permitted sender) client-ip=2620:137:e000::3:4;
Message-ID: <e29d0306-329b-b5b8-aa50-4e3ab0ded1e2@intel.com>
Date:   Mon, 30 Oct 2023 10:05:49 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.3.1
Subject: Re: [PATCH] checkpatch: allow tags between co-developed-by and their
 sign-off
To:     Krzysztof Kozlowski <krzk@kernel.org>,
        Sean Christopherson <seanjc@google.com>
CC:     Andy Whitcroft <apw@canonical.com>, Joe Perches <joe@perches.com>,
        Dwaipayan Ray <dwaipayanray1@gmail.com>,
        Lukas Bulwahn <lukas.bulwahn@gmail.com>,
        <workflows@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
        Jonathan Corbet <corbet@lwn.net>, <linux-doc@vger.kernel.org>,
        Jacob Keller <jacob.e.keller@intel.com>,
        Mateusz Polchlopek <mateusz.polchlopek@intel.com>
References: <20231023102846.14830-1-przemyslaw.kitszel@intel.com>
 <ZTZ9Bk3-NrK_cVDu@google.com>
 <887636af-6be7-f418-bfa4-d85501f2f40c@intel.com>
 <0693460e-2104-49af-b1dc-5ffae0f279c5@kernel.org>
Content-Language: en-US
From:   Przemek Kitszel <przemyslaw.kitszel@intel.com>
In-Reply-To: <0693460e-2104-49af-b1dc-5ffae0f279c5@kernel.org>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?S3FMb0NpdlNBaDRQNDFjWEtSZ1E2K1VYWGpZZnM0ZUdCbDV3ZnlMbVRFRXhH?=
 =?utf-8?B?YUVFZUlycC9qS3czc3d5RzFuVXA4N3lXYlJJWWVaUEdDMWRQWlFrbzd0a3Nh?=
 =?utf-8?B?aldRWnY5NnJ6YUFUMk1LQVVIbURQT1BQbVRVU1dsL015cVpVdHpOZVh4c0NS?=
 =?utf-8?B?MzJCV0wrVHhWM1lBYlcwSjljMmIyTXZOa0pSVXE5djZQYnlkMkxTRWg2ZU5I?=
 =?utf-8?B?SGdxbjM4OFRkTFVSQ0FzdFVGME5mTEFvNWRtelJKcGxTMnVRNnFQKzZuYmJR?=
 =?utf-8?B?WTJKb1oxYnR5NE96bHdwd0lPSEh0TGdyUUFLZDMxZkJWSVJuTVF3ZnBjd2Rl?=
 =?utf-8?B?Yks3WjQvT0lNdUpzaVNxb1UvTmpWZHVLbXMvbG16ZVM0cTNvZ0YrYUtsOE9o?=
 =?utf-8?B?Q2xCQzhzWmd2NlFicVFoY3greC9PeGt0Nlp5Vk1oNk9XejJjVVM2Q0JtRWpZ?=
 =?utf-8?B?c0o4eHY5Y2UzRUJvaEcrUTNEek1uME5wOXpWMjI4REQ4NU5tRHZTSTZPTmFZ?=
 =?utf-8?B?NytOcksyblliM09QcGdRZ2I5cmNadU9BYzNFQkZQeXlmRUNWTjlDWkFyempV?=
 =?utf-8?B?cGplVHlhZmxvVC9sV292Z3JTbVJtODJxd1FOSTFmdnpHMVNQTnM4SUFLUHZQ?=
 =?utf-8?B?ckVYOW5OblZldHNiMFhVUXFpMkJHa0RySjVTU3lsem1UTG4xY1FFcDBZdlBl?=
 =?utf-8?B?eUxiRytzMEYvN2s0aFF6dTR0Uk9XVGkvZjJUZmk4RW1STWJoTWJxVVhROCtV?=
 =?utf-8?B?UFB1Z1UwS1hoMnNsL2FXUktBZkk5VktPMmdlamtQNm5ScTdGNnU4bXh5eEVq?=
 =?utf-8?B?dGxUUjczejV2dEhrM2xhOC8xWGs0bjdDWjNMaGV5QlNMZHd6bmovVEo3ajAx?=
 =?utf-8?B?UC9TVmlnMWY2VVd5a3pyb0ZGU1RQelhzZkQ5czU3VVR3alRSdk1rWXBFRjZP?=
 =?utf-8?B?ZlBwT3EralFMa250a0c4dG0zb2FqMkNsSStXQk9FVTI5a1ZrMkg1TS85SXd3?=
 =?utf-8?B?SjNJS05CdjFtZHVuVXE2WXExSXBIK3RmVHcvdzF6bTIwczkrdFlOdXRJSmhB?=
 =?utf-8?B?WTBaT1RHZ2ZPMUNMTnRsVTFJbnRvL0pGa08zMmdTRU5rcHNmdG1OOTQxWVlP?=
 =?utf-8?B?em1GYU8zQ0xkUlpkK3ZUSXdXQjVHUHJSVlhhbkZkV1VyandpZGxhQzdWMGc0?=
 =?utf-8?B?QXZtY3AzV2ZqYkxkY1g2Ui9MVkZSeXlJanVOd09WSmlUTy84VXE2NDJoOXM0?=
 =?utf-8?B?OXU0R2YvVXBWZTNQOHhJMGt4bm40QURyTm44VmlwZm01V3B4QVQ2MnZ1RmtD?=
 =?utf-8?B?bFJaZFFNLzU3THh5cjhQRGU2c0pxY2psOWJ4UHd4ZCtGS3VkbHRUNU11dFZh?=
 =?utf-8?B?ZHRGMzhMMnI3SDJ1SUtkbVlOa2lJc2d1VlJuS0gxbHd4MlFiMmdaS2wwSWpV?=
 =?utf-8?B?RzB0MHhZMm1pYlpNMTB2dlUzR3pLbHFxMWtNMEF2WVpSem1SZTdZTXhRdGVw?=
 =?utf-8?B?SHZVNENtWG84a0FESDFkZ2Vscm5PVzhEQ3djaHJNNHVLNUpVMzNTL1Jnd2Rv?=
 =?utf-8?B?UFZnTVBSOEZiMW0ya2swZGRJR0ducFl1dmNLVEE3cnFpTDdpUEpreklJeVdE?=
 =?utf-8?B?RjgzWmZoMklYZXhLNjJwTC9ZaDlPYmpGdklWTTJmMER6OWljS2dydzVhdUxF?=
 =?utf-8?B?RjdWTkh3NHJvQzFTM3J6MmlzcmJLZzdMZ1hnNE1ETGNFTnZLQ01BU1FKOWlX?=
 =?utf-8?B?YU5sazFpeTQ5RklrTWg4ejlpbENQOXJGOUU0NzQ3MTFUaXp0K1pUS2JubkRY?=
 =?utf-8?B?VWQ4S2ZMU1ZpRW5LN2FYZjhST05PU3BTb1A3VWRYei9LZk5DakdZYUFSKzlO?=
 =?utf-8?B?Mit1MllmV3FUWEVIaE9POGNDSVFhREpEYmliS0xjMnBvUFovVTNyUmxvTmtM?=
 =?utf-8?B?cmlqeWsvem0zSXhRWjMxVGh4amtOeDB5TDE2MFBPankzZFFaQTBrcFRhc1ly?=
 =?utf-8?B?eitqdk5DYWViejFSY0t4eGlId2c4ajk4NzV4dnNWOTg0MG91Njd4czU3UFl3?=
 =?utf-8?B?VmM0TDRJVWprdUE4dXFIeUlvZ1NPbXNFZGtWR3JBQVp3azByNkhoWUF6MkdK?=
 =?utf-8?B?MmhnTGgyK0lSazZCNFNvL3MyaTFWc3Bvc3c3aUZIT21TZDZSMU9OeWlLZE5N?=
 =?utf-8?B?N1E9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 0fccf393-e945-4151-9548-08dbd9276dfd
X-MS-Exchange-CrossTenant-AuthSource: DM6PR11MB3674.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 Oct 2023 09:05:59.0162
 (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: ppbeqBbmz0rME15Uxx7SlhTWa0WdiCdiedzX3/P2bmufHCA0uST+aVNQM9aNttcpTOQwLLx6DT8h7Zi9Mh7fvB46PbpmH6CaRkn/Y2lRMn0=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH0PR11MB5426
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-5.8 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS
	autolearn=unavailable autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on howler.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (howler.vger.email [0.0.0.0]); Mon, 30 Oct 2023 02:06:28 -0700 (PDT)

On 10/29/23 10:35, Krzysztof Kozlowski wrote:
> On 24/10/2023 11:15, Przemek Kitszel wrote:
>> On 10/23/23 16:02, Sean Christopherson wrote:
>>> +Mateusz
>>>
>>> On Mon, Oct 23, 2023, Przemek Kitszel wrote:
>>>> Additional tags between Co-developed-by and corresponding Signed-off-by
>>>> could include Reviewed-by tags collected by Submitter, which is also
>>>> a Co-developer, but should sign-off at the very end of tags provided by
>>>> the Submitter.
>>>
>>> ...
>>>
>>>> Mateusz Polchlopek <mateusz.polchlopek@intel.com> has reported this to me.
>>>
>>> Heh, there's a tag for that...
>>>
>>>     Reported-by: Mateusz Polchlopek <mateusz.polchlopek@intel.com>
>>>
>>> And it's usually a good idea to Cc the reporter in case there are questions they
>>> can help answer.
>>
>> Heh ;) then I would get a checkpatch warning for not providing Link: to
>> the report, somehow I wanted to avoid those for checkpatch contrib :)
> 
> You wanted Suggested-by. There is no bug here, so Reported-by is not
> suitable.
> 
> Best regards,
> Krzysztof
> 

I would really like to see Inspired-by: and use it a lot, for cases that
some talk ignites an idea, but it's Suggested-by