Re: [PATCH v2] ACPI: APD: Add AMDI0015 as platform device

[Shyam Sundar S K <Shyam-sundar.S-k@amd.com>]

On 8/20/2024 20:19, Rafael J. Wysocki wrote:
> On Mon, Aug 12, 2024 at 7:24 PM Andy Shevchenko
> <andriy.shevchenko@linux.intel.com> wrote:
>>
>> On Mon, Aug 12, 2024 at 08:10:18PM +0530, Shyam Sundar S K wrote:
>>> Add AMDI0015 to the ACPI APD support list to ensure correct clock settings
>>> for the I3C device on the latest AMD platforms.
>>
>> Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
>> from the ACPI ID perspective.
> 
> I've replaced the v1 that has been applied for some time already with
> this one, but please note that it still is not entirely clean.
> 
> Namely, if there are two S-o-b targs on a patch, 2 cases are possible:
> 
> (1) The person sending it is not its author and merely sends someone
> else's work adding an S-o-b to document a "supply chain link".  In
> this case, the From: header should point to the original author (it
> can be added right before the changelog) and its value should match
> the other S-o-b tag exactly.
> 
> (2) The person sending it is one of its authors.  In this case, a
> Co-developed-by tag should be added to point to the other author and
> its value should match the other S-o-b tag exactly.
> 
> I've assumed (2) and added "Co-developed-by: Sanket Goswami
> <Sanket.Goswami@amd.com>" to the commit, but that's not something I
> should be doing.

Thanks! That was a miss.

Co-developed-by: Sanket Goswami <Sanket.Goswami@amd.com> is the right tag.

Thanks,
Shyam

> 
> Thanks!