[U-Boot] [PATCH v2] blk: Remove various places that do flush cache after read
Bin Meng
bmeng.cn at gmail.com
Fri Sep 15 01:25:32 UTC 2017
Hi Tom,
On Wed, Sep 13, 2017 at 10:00 AM, Bin Meng <bmeng.cn at gmail.com> wrote:
> 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 at gmail.com>
> Reviewed-by: Stefan Roese <sr at denx.de>
> Reviewed-by: York Sun <york.sun at nxp.com>
> Reviewed-by: Joe Hershberger <joe.hershberger at ni.com>
> Reviewed-by: Simon Glass <sjg at chromium.org>
> Tested-by: York Sun <york.sun at nxp.com>
>
> ---
>
> Changes in v2:
> - rebase on top of u-boot/master
>
> arch/arm/cpu/armv8/fsl-layerscape/ppa.c | 15 ---------------
> board/toradex/common/tdx-cfg-block.c | 2 --
> cmd/mmc.c | 2 --
> drivers/block/blk-uclass.c | 3 ---
> drivers/block/blk_legacy.c | 3 ---
> drivers/net/fm/fm.c | 2 --
> drivers/net/phy/cortina.c | 2 --
> drivers/qe/qe.c | 2 --
> 8 files changed, 31 deletions(-)
>
Will you apply this one?
Regards,
Bin
More information about the U-Boot
mailing list