Hi Tom,

> -----Original Message-----
> From: Tom Rini <tr...@konsulko.com>
> Sent: Wednesday, January 16, 2019 7:19 PM
> To: Alexey Brodkin <alexey.brod...@synopsys.com>
> Cc: u-boot@lists.denx.de; uboot-snps-...@synopsys.com; Michal Simek 
> <michal.si...@xilinx.com>; Simon
> Glass <s...@chromium.org>
> Subject: Re: [PATCH] ARC: Move ENV_SIZE and ENV_OFFSET to Kconfig
> 
> On Wed, Jan 16, 2019 at 07:11:15PM +0300, Alexey Brodkin wrote:
> 
> > Join the party of some ARM boards and drop more
> > items from include/configs/xxx.h.
> >
> > Signed-off-by: Alexey Brodkin <abrod...@synopsys.com>
> > Cc: Michal Simek <michal.si...@xilinx.com>
> > Cc: Simon Glass <s...@chromium.org>
> > Cc: Tom Rini <tr...@konsulko.com>
> > ---

[snip]

> >  config ENV_OFFSET
> >     hex "Environment Offset"
> > @@ -441,6 +441,7 @@ config ENV_OFFSET
> >     default 0x88000 if ARCH_SUNXI
> >     default 0xE0000 if ARCH_ZYNQ
> >     default 0x1E00000 if ARCH_ZYNQMP
> > +   default 0
> 
> Please make this "if ARC", thanks!

Actually I don't like having here listed all the boards
because with more boards/arches moving ENV settings to Kconfig
we'll get here all of them listed at some point.
Why default ENV_OFFSET [as well as] ENV_SIZE are not set in
arch/soc/board top-level Kconfig?

-Alexey
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to