[PATCH] imx: ventana: correct splashimage load address

[ZHIZHIKIN Andrey]

Hello Tim,

> -----Original Message-----
> From: Tim Harvey <tharvey at gateworks.com>
> Sent: Friday, January 28, 2022 12:20 AM
> To: ZHIZHIKIN Andrey <andrey.zhizhikin at leica-geosystems.com>
> Cc: u-boot <u-boot at lists.denx.de>
> Subject: Re: [PATCH] imx: ventana: correct splashimage load address
> 
> 
> On Tue, Jan 18, 2022 at 8:40 AM Tim Harvey <tharvey at gateworks.com> wrote:
> >
> > On Mon, Jan 17, 2022 at 3:32 AM Andrey Zhizhikin
> > <andrey.zhizhikin at leica-geosystems.com> wrote:
> > >
> > > Commit 72d81360aabd ("global: Convert CONFIG_LOADADDR to
> > > CONFIG_SYS_LOADADDR") dropped the usage of LOADADDR and replaced it with
> > > SYS_LOADADDR.
> > >
> > > Use the correct macro in environment by replacing CONFIG_LOADADDR with
> > > CONFIG_SYS_LOADADDR.
> > >
> > > Fixes: d75ebf3482c3 ("imx: ventana: fix splash logo drawing")
> > > Signed-off-by: Andrey Zhizhikin <andrey.zhizhikin at leica-geosystems.com>
> > > Cc: Tim Harvey <tharvey at gateworks.com>
> > > ---
> > >  include/configs/gw_ventana.h | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/include/configs/gw_ventana.h b/include/configs/gw_ventana.h
> > > index 513b2d4679..acd495f234 100644
> > > --- a/include/configs/gw_ventana.h
> > > +++ b/include/configs/gw_ventana.h
> > > @@ -105,7 +105,7 @@
> > >
> > >  #define CONFIG_EXTRA_ENV_SETTINGS_COMMON \
> > >         "splashpos=m,m\0" \
> > > -       "splashimage=" __stringify(CONFIG_LOADADDR) "\0" \
> > > +       "splashimage=" __stringify(CONFIG_SYS_LOAD_ADDR) "\0" \
> > >         "usb_pgood_delay=2000\0" \
> > >         "console=ttymxc1\0" \
> > >         "bootdevs=usb mmc sata flash\0" \
> > >
> > > base-commit: d71dbe657c3c22f50ecb3df133e5cdd884daae7d
> > > --
> > > 2.25.1
> > >
> >
> > Acked-By: Tim Harvey <tharvey at gateworks.com>
> >
> 
> Andrey,
> 
> While this does restore the custom logo on the gwventana boards there
> is now a default U-Boot logo in the top left corner. I assume this is
> some new addition... how do I disable the U-Boot logo?

THB, this I do not know. I stumbled upon the incorrect variable while
I was working on imx8mq-evk, and thought that this patch would be
beneficial for gwventana since this was the only place where the old
variable name was used.

I guess the question of suppression of default U-Boot logo is for a
bigger audience here.

> 
> Best Regards,
> 
> Tim

-- andrey