linux-kernel - Re: [PATCH v3 1/5] i3c: dw: Add support for AMDI0015 ACPI ID

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <507a8af4-f128-4d73-9d0f-b8a6a15603d3@amd.com>
Date: Tue, 12 Nov 2024 14:18:47 +0530
From: Shyam Sundar S K <Shyam-sundar.S-k@....com>
To: Jarkko Nikula <jarkko.nikula@...ux.intel.com>,
 Alexandre Belloni <alexandre.belloni@...tlin.com>
Cc: Sanket.Goswami@....com, linux-i3c@...ts.infradead.org,
 linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3 1/5] i3c: dw: Add support for AMDI0015 ACPI ID

Hi Jarkko,

On 11/12/2024 13:46, Jarkko Nikula wrote:
> Hi
> 
> On 11/8/24 9:33 AM, Shyam Sundar S K wrote:
>> Add AMDI0015 _HID for Designware I3C driver so that the dw-i3c-master
>> driver can be probed on AMD platforms.
>>
>> Signed-off-by: Shyam Sundar S K <Shyam-sundar.S-k@....com>
>> ---
>>   drivers/i3c/master/dw-i3c-master.c | 7 +++++++
>>   1 file changed, 7 insertions(+)
>>
>> diff --git a/drivers/i3c/master/dw-i3c-master.c
>> b/drivers/i3c/master/dw-i3c-master.c
>> index 8d694672c110..1a7c300b6d45 100644
>> --- a/drivers/i3c/master/dw-i3c-master.c
>> +++ b/drivers/i3c/master/dw-i3c-master.c
>> @@ -1748,12 +1748,19 @@ static const struct of_device_id
>> dw_i3c_master_of_match[] = {
>>   };
>>   MODULE_DEVICE_TABLE(of, dw_i3c_master_of_match);
>>   +static const struct acpi_device_id amd_i3c_device_match[] = {
>> +    { "AMDI0015" },
>> +    { }
>> +};
>> +MODULE_DEVICE_TABLE(acpi, amd_i3c_device_match);
>> +
>>   static struct platform_driver dw_i3c_driver = {
>>       .probe = dw_i3c_probe,
>>       .remove_new = dw_i3c_remove,
>>       .driver = {
>>           .name = "dw-i3c-master",
>>           .of_match_table = dw_i3c_master_of_match,
>> +        .acpi_match_table = amd_i3c_device_match,
>>           .pm = &dw_i3c_pm_ops,
>>       },
>>   };
> 
> Am I right this and patch 5/5 can be independent from rest of the series?

Right. 1/5 and 5/5 can be grouped. But rest of the other patches are
equally important because they drive the usecase.

> 
> To me it looks these two patches enable bus communication and thus be
> useful without rest of the series while latter need more discussion
> (I'll have some notes coming) and Cc'ing linux-acpi.

I have Cc'ed linux-acpi in this revision. Do you have any feedback for
patches 2-4 ?

Thanks,
Shyam