Re: [PATCH 0/5] spi: zynqmp-gqspi: Improve error recovery by resetting

[Miquel Raynal <miquel.raynal@xxxxxxxxxxx>]

On 17/01/2025 at 13:21:58 GMT, Mark Brown <broonie@xxxxxxxxxx> wrote:

> On Thu, Jan 16, 2025 at 05:55:16PM -0500, Sean Anderson wrote:
>> This series adds support for resetting the QSPI controller if we have a
>> timeout. I find this greatly improves the stability of the device, which
>> would tend to break after any timeout.
>
> If you're hitting a timeout that tends to indicate there's already a
> serious stability problem...

Yes, unless the timeout is reached for "good reasons", ie. you request
substantial amounts of data (typically from a memory device) and the
timeout is too short compared to the theoretical time spent in the
transfer. A loaded machine can also increase the number of false
positives I guess.

Cheers,
Miquèl