From mboxrd@z Thu Jan  1 00:00:00 1970
From: Bin Meng <bmeng.cn@gmail.com>
Date: Wed, 23 Aug 2017 13:28:35 +0800
Subject: [U-Boot] [PATCH] blk: Remove various places that do flush cache
	after read
In-Reply-To: <DB6PR0401MB25362F40E1973E968FE001FFF8850@DB6PR0401MB2536.eurprd04.prod.outlook.com>
References: <1503373600-20847-1-git-send-email-bmeng.cn@gmail.com>
 <VI1PR04MB20789932166064785652A30A9A840@VI1PR04MB2078.eurprd04.prod.outlook.com>
 <AM3PR04MB11855E0686E0CB10192B050491850@AM3PR04MB1185.eurprd04.prod.outlook.com>
 <DB6PR0401MB25362F40E1973E968FE001FFF8850@DB6PR0401MB2536.eurprd04.prod.outlook.com>
Message-ID: <CAEUhbmVFDTao=4qYghGWdTihFxieNtPO4PUnQcXT_7WrQCoHHQ@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Yangbo,

On Wed, Aug 23, 2017 at 11:29 AM, Y.b. Lu <yangbo.lu@nxp.com> wrote:
> cmd/mmc.c
>
> Maybe SD controller on some vendor platforms doesn't support SNOOP.
> flush_cache() is required to sync between DDR and cache.
> Should be careful to remove this.
>

Please do not top-posting.

What you mentioned if snoop is not supported, flush_cache() is
required, is true, however that's not the issue I am trying to fix
here. The flush_cache() here after a DMA read will cause issues
because stale data in the cache will overwrite the correct data in the
RAM that is DMAed by hardware (in this case, MMC controller).

Regards,
Bin