On Fri, Aug 25, 2023 at 2:06 PM Simon Glass <s...@chromium.org> wrote: > > On Fri, 25 Aug 2023 at 08:49, Tom Rini <tr...@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+rene...@gmail.com> > > > Suggested-by: Tom Rini <tr...@konsulko.com> > > > Signed-off-by: Jesse Taube <mr.bossman...@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 <tr...@konsulko.com> > > Reviewed-by: Simon Glass <s...@chromium.org> > > Yes please