Re: [PATCH v1 1/2] scsi: ufs: core: Support Updating UIC Command Timeout

Bao D. Nguyen
2024-05-22T20:51:55Z

On 5/22/2024 11:16 AM, Bart Van Assche wrote:
> On 5/22/24 00:01, Bao D. Nguyen wrote:
>> interrupt starvations happen occasionally because the uart may
>> print long debug messages from different modules in the system.
>
> I think that's a bug in the UART driver that should be fixed in the
> UART driver.

Thanks Bart.
I am not familiar with the UART drivers. I looked at some UART code and
it could be interpreted as their choice of implementation.
During product development, the UART may be used. However, when the
development completes, most likely the UART logging is disabled due to
performance reason.

This change is to give flexibility to the SoCs to use the UART
implementation of their choice and to choose the desired UIC command
timeout without affecting the system stability or the default hardcoded
UIC timeout value of 500ms that others may be using.

Bao

>
> Thanks,
>
> Bart.