[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