Re: [PATCH] UFS: Make TM command timeout configurable from host side

[Peter Wang (王信友)]

On Tue, 2025-11-11 at 08:37 -0800, Bart Van Assche wrote:
> 
> Why a quirk? A quirk will select a single specific timeout. The
> approach
> of this patch lets the host driver set the timeout. This seems more
> flexible to me than introducing a new quirk. Additionally, I think
> this
> is a better solution than a new kernel module parameter.
> 
> Thanks,
> 
> Bart.

Hi Bart,

It is not reasonable because the timeout value does not depend
on the host, it depends on the device. It could set a large 
timoeut value for those devices.

By the way, this patch also doesn't change any host timeout 
value if you insist that the timeout value depends on the host.

Using a module parameter is a flexible method if the customer 
is using a device that may require an extended timeout value.
Moreover, this approach would help maintain consistency. 
Otherwise, with so many different timeouts (uic/dev/tm),
it would be quite chaotic if each is handled in a different way.

Thanks
Peter