[PATCH v2] Convert CFG_SYS_UBOOT_START to Kconfig
Jesse T
mr.bossman075 at gmail.com
Fri Aug 25 21:01:26 CEST 2023
On Fri, Aug 25, 2023 at 2:06 PM Simon Glass <sjg at chromium.org> wrote:
>
> On Fri, 25 Aug 2023 at 08:49, Tom Rini <trini at konsulko.com> wrote:
> >
> > On Thu, Aug 24, 2023 at 09:59:48PM -0400, Jesse Taube wrote:
> >
> > > Commit 65cc0e2a65d2 ("global: Move remaining CONFIG_SYS_* to CFG_SYS_*")
> > > renamed CONFIG_SYS_UBOOT_START to CFG_SYS_UBOOT_START. Unfortunately,
> > > this meant that the value was no longer available to the Makefile. This
> > > caused imxrt to fail to boot. All the other boards that used this
> > > variable were unaffected because they were using the default value
> > > which is CONFIG_TEXT_BASE.
> > >
> > > This commit converts CFG_SYS_UBOOT_START to Kconfig and sets the default
> > > value to CONFIG_TEXT_BASE.
> > >
> > > Suggested-by: Marek Vasut <marek.vasut+renesas at gmail.com>
> > > Suggested-by: Tom Rini <trini at konsulko.com>
> > > Signed-off-by: Jesse Taube <Mr.Bossman075 at gmail.com>
> > [snip]
> > > diff --git a/boot/Kconfig b/boot/Kconfig
> > > index 5e2d4286ae..3883cae2ba 100644
> > > --- a/boot/Kconfig
> > > +++ b/boot/Kconfig
> > > @@ -738,7 +738,24 @@ config TEXT_BASE
> > > default 0x42e00000 if SUNXI_MINIMUM_DRAM_MB >= 64
> > > hex "Text Base"
> > > help
> > > - The address in memory that U-Boot will be running from, initially.
> > > + The address in memory that U-Boot will be copied and executed from initially.
> > > +
> > > +config HAVE_SYS_UBOOT_START
> > > + bool "Use custom U-Boot Start"
>
> start ?
This was the original name I did not change it.
Uncapitalizing start, or changing it to "U-Boot entry" is fine if
that's what you were asking
>
> > > + depends on HAVE_TEXT_BASE
> > > + help
> > > + By default, the address in memory that U-Boot will be copied from (TEXT_BASE) and the
> > > + entry point are the same. Select this to start the execution of U-Boot from a different address.
> > > + This may be required if a header or vector table needs to be copied but not executed.
>
> Nice help!
>
> > > +
> > > +config SYS_UBOOT_START
> > > + hex
> > > + depends on HAVE_TEXT_BASE
> > > + default TEXT_BASE
> > > + prompt "U-Boot entry" if HAVE_SYS_UBOOT_START
> > > + help
> > > + If TEXT_BASE differs from the start of execution, this sets the address in memory that
> > > + U-Boot will start execution from initially.
> >
> > A small point, these help texts should wrap at 80 columns. If there's
> > no other feedback I'll just fix that up when applying unless you want
> > to.
Oops, it's faster if you wrap to 80 columns.
Thanks,
Jesse Taube
> >
> > Reviewed-by: Tom Rini <trini at konsulko.com>
>
> Reviewed-by: Simon Glass <sjg at chromium.org>
>
> Yes please
More information about the U-Boot
mailing list