[U-Boot] [PATCH v2 03/18] bootm: Align cache flush end address correctly

Andreas Bießmann andreas at biessmann.org
Thu May 5 16:19:48 CEST 2016


On 05.05.16 15:28, Simon Glass wrote:
> Flushing part of the cache should be done on cache boundaries. Trying to
> flush part of a cache line is not supported and the request may be ignored
> or print warnings.
> 
> Adjust the bootm code to align the end address to prevent this problem.
> 
> Signed-off-by: Simon Glass <sjg at chromium.org>
> Reviewed-by: Heiko Schocher <hs at denx.de>
> Tested-on: smartweb, corvus, taurus, axm
> Tested-by: Heiko Schocher <hs at denx.de>
> Reviewed-by: Joe Hershberger <joe.hershberger at ni.com>
> ---
> 
> Changes in v2: None
> 
>  common/bootm.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/common/bootm.c b/common/bootm.c
> index c965326..7469c61 100644
> --- a/common/bootm.c
> +++ b/common/bootm.c
> @@ -435,7 +435,7 @@ static int bootm_load_os(bootm_headers_t *images, unsigned long *load_end,
>  		bootstage_error(BOOTSTAGE_ID_DECOMP_IMAGE);
>  		return err;
>  	}
> -	flush_cache(load, *load_end - load);
> +	flush_cache(load, ALIGN(*load_end - load, ARCH_DMA_MINALIGN));
>  
>  	debug("   kernel loaded at 0x%08lx, end = 0x%08lx\n", load, *load_end);
>  	bootstage_mark(BOOTSTAGE_ID_KERNEL_LOADED);
> 

Reviewed-by: Andreas Bießmann <andreas at biessmann.org>


More information about the U-Boot mailing list