drivers/spi: fsl_qspi: fix read timeout
During QSPI reads, current is_controller_busy function sporadically fails with -ETIMEDOUT due to fixed number of 5 test loops. Using timer functions to wait 1000 us instead will fix this. Signed-off-by: Thomas Schaefer <thomas.schaefer@kontron.com> Reviewed-by: Fabio Estevam <festevam@gmail.com> Reviewed-by: Jagan Teki <jagan@amarulasolutions.com>
This commit is contained in:
parent
535b1fdb8e
commit
c6d0c5eb30
@ -152,7 +152,7 @@ static inline int is_controller_busy(const struct fsl_qspi_priv *priv)
|
||||
u32 val;
|
||||
const u32 mask = QSPI_SR_BUSY_MASK | QSPI_SR_AHB_ACC_MASK |
|
||||
QSPI_SR_IP_ACC_MASK;
|
||||
unsigned int retry = 5;
|
||||
unsigned long timeout = timer_get_us() + 1000;
|
||||
|
||||
do {
|
||||
val = qspi_read32(priv->flags, &priv->regs->sr);
|
||||
@ -160,10 +160,9 @@ static inline int is_controller_busy(const struct fsl_qspi_priv *priv)
|
||||
if ((~val & mask) == mask)
|
||||
return 0;
|
||||
|
||||
udelay(1);
|
||||
} while (--retry);
|
||||
|
||||
return -ETIMEDOUT;
|
||||
if (timer_get_us() > timeout )
|
||||
return -ETIMEDOUT;
|
||||
} while (1);
|
||||
}
|
||||
|
||||
/* QSPI support swapping the flash read/write data
|
||||
|
Loading…
Reference in New Issue
Block a user