Hi Marek,

> On Jul 27, 2015, at 23:39 , Marek Vasut <ma...@denx.de> wrote:
> 
> The driver didn't stop the bounce buffer in case a data transfer
> failed. This would lead to memory leakage if the communication
> between the CPU and the card is unreliable. Add the missing call
> to stop the bounce buffer.
> 
> Signed-off-by: Marek Vasut <ma...@denx.de>
> Cc: Dinh Nguyen <dingu...@opensource.altera.com>
> Cc: Pantelis Antoniou <pa...@antoniou-consulting.com>
> Cc: Tom Rini <tr...@konsulko.com>
> ---
> drivers/mmc/dw_mmc.c | 1 +
> 1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/mmc/dw_mmc.c b/drivers/mmc/dw_mmc.c
> index 53a8aca..3fffa71 100644
> --- a/drivers/mmc/dw_mmc.c
> +++ b/drivers/mmc/dw_mmc.c
> @@ -215,6 +215,7 @@ static int dwmci_send_cmd(struct mmc *mmc, struct mmc_cmd 
> *cmd,
>                       mask = dwmci_readl(host, DWMCI_RINTSTS);
>                       if (mask & (DWMCI_DATA_ERR | DWMCI_DATA_TOUT)) {
>                               printf("%s: DATA ERROR!\n", __func__);
> +                             bounce_buffer_stop(&bbstate);
>                               return -1;
>                       }
>               } while (!(mask & DWMCI_INTMSK_DTO));
> -- 
> 2.1.4
> 

Applied, thanks.

— Pantelis

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to