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

From: Bao D. Nguyen
Date: Wed May 22 2024 - 16:52:05 EST


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.