[U-Boot] [PATCH v3 068/108] x86: Add an option to control the position of U-Boot

[Simon Glass]

Hi Bin,

On Tue, 19 Nov 2019 at 01:34, Bin Meng <bmeng.cn at gmail.com> wrote:
>
> Hi Simon,
>
> On Mon, Oct 21, 2019 at 11:40 AM Simon Glass <sjg at chromium.org> wrote:
> >
> > The existing work-around for positioning U-Boot in the ROM when it
> > actually runs from RAM still exists and there is not obvious way to change
> > this.
> >
> > Add a proper Kconfig option to handle this case. This also adds a new bool
> > property to indicate whether CONFIG_SYS_TEXT_BASE exists.
> >
> > Signed-off-by: Simon Glass <sjg at chromium.org>
> > ---
> >
> > Changes in v3: None
> > Changes in v2: None
> >
> >  Kconfig                                |  9 ++++++---
> >  arch/x86/Kconfig                       |  5 +++++
> >  arch/x86/dts/u-boot.dtsi               | 18 +++---------------
> >  configs/chromebook_samus_tpl_defconfig |  1 +
> >  4 files changed, 15 insertions(+), 18 deletions(-)
> >

[..]

> > diff --git a/configs/chromebook_samus_tpl_defconfig b/configs/chromebook_samus_tpl_defconfig
> > index 28f23cfe125..c7f125eaa40 100644
> > --- a/configs/chromebook_samus_tpl_defconfig
> > +++ b/configs/chromebook_samus_tpl_defconfig
> > @@ -13,6 +13,7 @@ CONFIG_HAVE_REFCODE=y
> >  CONFIG_SMP=y
> >  CONFIG_HAVE_VGA_BIOS=y
> >  CONFIG_SPL_TEXT_BASE=0xffe70000
> > +CONFIG_X86_OFFSET_U_BOOT=0xfff00000
>
> Should we update other x86 board defconfig files for this option?

It defaults to the same as the text base, which works fine with XIP,
so I don't think this is necessary.

Regards,
Simon