[PATCH v3 2/3] i3c: dw: Add sysfs support for Device NACK Retry count
Adrian Hunter
adrian.hunter at intel.com
Tue Nov 25 06:05:53 PST 2025
On 25/11/2025 05:19, Ng, Adrian Ho Yin wrote:
> On 11/25/2025 1:59 AM, Frank Li wrote:
>
>>> +
>>> + master->dev_nack_retry_cnt = val;
>>> +
>>> + spin_lock_irqsave(&master->devs_lock, flags);
>>
>> I think you'd better to hold i3c_bus_maintenance_lock() lock to make
>> sure not transfer on going.
>>
>> Frank
> Hi Frank
>
> i3c_bus_maintenance_lock is a static function in master.c. So should i update it and expose a public helper to take the maintenance lock or continue using the existing spin lock to protect the DAT updates?
>
> Thank You
> Adrian
Other controllers (e.g. MIPI I3C) support dev_nack_retry_cnt.
Seems to me this should be in master.c anyway, so it can be shared.
More information about the linux-i3c
mailing list