diff options
author | Bin Meng <bmeng.cn@gmail.com> | 2017-09-12 19:00:23 -0700 |
---|---|---|
committer | Tom Rini <trini@konsulko.com> | 2017-09-15 08:05:10 -0400 |
commit | 723b43daec7ee2ddb600cfcb9b0253d4a71c3915 (patch) | |
tree | 44953c1e7fc2634c1e3ec329262d9a3332ba0453 /cmd/mmc.c | |
parent | 6305db960447ff2d32c335de8bbd9b44cb7d6554 (diff) |
blk: Remove various places that do flush cache after read
All these places seem to inherit the codes from the MMC driver where
a FIXME was put in the comment. However the correct operation after
read should be cache invalidate, not flush.
The underlying drivers should be responsible for the cache operation.
Remove these codes completely.
Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: Stefan Roese <sr@denx.de>
Reviewed-by: York Sun <york.sun@nxp.com>
Reviewed-by: Joe Hershberger <joe.hershberger@ni.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Tested-by: York Sun <york.sun@nxp.com>
Diffstat (limited to 'cmd/mmc.c')
-rw-r--r-- | cmd/mmc.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/cmd/mmc.c b/cmd/mmc.c index 00697fc1f2b..5def4ea1a29 100644 --- a/cmd/mmc.c +++ b/cmd/mmc.c @@ -293,8 +293,6 @@ static int do_mmc_read(cmd_tbl_t *cmdtp, int flag, curr_device, blk, cnt); n = blk_dread(mmc_get_blk_desc(mmc), blk, cnt, addr); - /* flush cache after read */ - flush_cache((ulong)addr, cnt * 512); /* FIXME */ printf("%d blocks read: %s\n", n, (n == cnt) ? "OK" : "ERROR"); return (n == cnt) ? CMD_RET_SUCCESS : CMD_RET_FAILURE; |