[U-Boot] [Patch] fsl_esdhc: Fix DMA transfer completion waiting loop

Dirk Behme dirk.behme at gmail.com
Sun Apr 14 09:19:58 CEST 2013


Am 08.04.2013 11:06, schrieb Andrew Gabbasov:
> Rework the waiting for transfer completion loop condition
> to continue waiting until both Transfer Complete and DMA End
> interrupts occur. Checking of DLA bit in Present State register
> looks not needed in addition to interrupts status checking,
> so it can be removed from the condition. Also, DMA Error
> condition is added to the list of data errors, checked in the loop.
>
> Signed-off-by: Andrew Gabbasov <andrew_gabbasov at mentor.com>
> ---
>   drivers/mmc/fsl_esdhc.c |    3 +--
>   include/fsl_esdhc.h     |    4 +++-
>   2 files changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/mmc/fsl_esdhc.c b/drivers/mmc/fsl_esdhc.c
> index 54b5363..814bba4 100644
> --- a/drivers/mmc/fsl_esdhc.c
> +++ b/drivers/mmc/fsl_esdhc.c
> @@ -400,8 +400,7 @@ esdhc_send_cmd(struct mmc *mmc, struct mmc_cmd *cmd, struct mmc_data *data)
>
>   			if (irqstat & DATA_ERR)
>   				return COMM_ERR;
> -		} while (!(irqstat & IRQSTAT_TC) &&
> -				(esdhc_read32(&regs->prsstat) & PRSSTAT_DLA));
> +		} while ((irqstat & DATA_COMPLETE) != DATA_COMPLETE);
>   #endif
>   	}
>
> diff --git a/include/fsl_esdhc.h b/include/fsl_esdhc.h
> index 47d2fe4..ea0880b 100644
> --- a/include/fsl_esdhc.h
> +++ b/include/fsl_esdhc.h
> @@ -63,7 +63,9 @@
>   #define IRQSTAT_CC		(0x00000001)
>
>   #define CMD_ERR		(IRQSTAT_CIE | IRQSTAT_CEBE | IRQSTAT_CCE)
> -#define DATA_ERR	(IRQSTAT_DEBE | IRQSTAT_DCE | IRQSTAT_DTOE)
> +#define DATA_ERR	(IRQSTAT_DEBE | IRQSTAT_DCE | IRQSTAT_DTOE | \
> +				IRQSTAT_DMAE)
> +#define DATA_COMPLETE	(IRQSTAT_TC | IRQSTAT_DINT)
>
>   #define IRQSTATEN		0x0002e034
>   #define IRQSTATEN_DMAE		(0x10000000)

I can't say anything to the content of the patches ;)

But are this patch and the patch from Eric

http://patchwork.ozlabs.org/patch/233595/

fine and should be applied together, now?

Thanks

Dirk



More information about the U-Boot mailing list