On Thu, Dec 14, 2023 at 06:36:37PM +0100, Jan Kiszka wrote:
> On 31.10.23 08:14, Stefan Roese wrote:
> > On 10/30/23 17:20, Jan Kiszka wrote:
> >> From: Jan Kiszka <jan.kis...@siemens.com>
> >>
> >> We are not iterating CQSPI_REG_RETRY, we are waiting 'timeout' ms, since
> >> day 1.
> >>
> >> Signed-off-by: Jan Kiszka <jan.kis...@siemens.com>
> >
> > Reviewed-by: Stefan Roese <s...@denx.de>
> >
> > Thanks,
> > Stefan
> >
> >> ---
> >>
> >> We are unfortunately seeing that message right now, rarely but then
> >> prominently...
> >>
> >> drivers/spi/cadence_qspi_apb.c | 3 +--
> >> 1 file changed, 1 insertion(+), 2 deletions(-)
> >>
> >> diff --git a/drivers/spi/cadence_qspi_apb.c
> >> b/drivers/spi/cadence_qspi_apb.c
> >> index 9ce2c0f254f..d033184aa46 100644
> >> --- a/drivers/spi/cadence_qspi_apb.c
> >> +++ b/drivers/spi/cadence_qspi_apb.c
> >> @@ -171,8 +171,7 @@ static unsigned int cadence_qspi_wait_idle(void
> >> *reg_base)
> >> }
> >> /* Timeout, still in busy mode. */
> >> - printf("QSPI: QSPI is still busy after poll for %d times.\n",
> >> - CQSPI_REG_RETRY);
> >> + printf("QSPI: QSPI is still busy after poll for %d ms.\n", timeout);
> >> return 0;
> >> }
> >>
> >
> > Viele Grüße,
> > Stefan Roese
> >
>
> Ping.Added to my dont-forget bundle. -- Tom
signature.asc
Description: PGP signature
