[U-Boot] [PATCH] mmc: omap: timeout counter fix
Nishanth Menon
nm at ti.com
Tue Oct 26 02:43:45 CEST 2010
Menon, Nishanth had written, on 10/25/2010 06:33 PM, the following:
> Having a loop with a counter is no timing guarentee for timing
> accuracy or compiler optimizations. For e.g. the same loop counter
> which runs when the MPU is running at 600MHz will timeout in around
> half the time when running at 1GHz. or the example where GCC 4.5
> compiles with different optimization compared to GCC 4.4.
> use a udelay(10) to ensure we have a predictable delay.
> use an emperical number - 100000 uSec ~= 1sec for a worst case timeout.
> This should never happen, and is adequate imaginary condition for us
> to fail with timeout.
>
> Signed-off-by: Nishanth Menon <nm at ti.com>
> ---
> drivers/mmc/omap_hsmmc.c | 5 ++++-
> 1 files changed, 4 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/mmc/omap_hsmmc.c b/drivers/mmc/omap_hsmmc.c
> index 9271470..9b03ce1 100644
> --- a/drivers/mmc/omap_hsmmc.c
> +++ b/drivers/mmc/omap_hsmmc.c
> @@ -137,7 +137,7 @@ static int mmc_send_cmd(struct mmc *mmc, struct mmc_cmd *cmd,
> {
> hsmmc_t *mmc_base = (hsmmc_t *)mmc->priv;
> unsigned int flags, mmc_stat;
> - unsigned int retry = 0x100000;
> + unsigned int retry = 100000;
>
>
> while ((readl(&mmc_base->pstate) & DATI_MASK) == DATI_CMDDIS)
> @@ -202,6 +202,9 @@ static int mmc_send_cmd(struct mmc *mmc, struct mmc_cmd *cmd,
>
> do {
> mmc_stat = readl(&mmc_base->stat);
> + if (!mmc_stat) {
> + udelay(10);
> + }
> retry--;
> } while ((mmc_stat == 0) && (retry > 0));
>
I am dropping this patch instead will post a major timeout cleanup for
omap_hsmmc.c -> caught a few other kickers as well :(.. darn the polling
logic is so broken in the code :(
--
Regards,
Nishanth Menon
More information about the U-Boot
mailing list