[U-Boot] [PATCH 2/5] sandbox: Drop 'const' from sandbox_write()
    sjg at google.com 
    sjg at google.com
       
    Tue Oct 29 23:21:37 UTC 2019
    
    
  
Hi Simon,
On Sat, Oct 12, 2019 at 6:22 AM Simon Glass <sjg at chromium.org> wrote:
>
> This function writes to its address so the address should not be declared
> as const. Fix it.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
>  arch/sandbox/cpu/cpu.c        |  3 +--
>  arch/sandbox/include/asm/io.h | 11 +++++------
>  2 files changed, 6 insertions(+), 8 deletions(-)
>
It looks I did something wrong when testing the previous patch :) I
remember adding const was to fix build warning somewhere else.
Anyway the changes look good to me. But I suspect we need give it a
full build to see whether it generates warning somewhere.
Reviewed-by: Bin Meng <bmeng.cn at gmail.com>
Applied to u-boot-dm, thanks!
    
    
More information about the U-Boot
mailing list