[U-Boot] [PATCH 1/2] riscv: set CONFIG_SYS_BOOTM_LEN to SZ_64M

[David Abdurachmanov]

On Thu, Apr 11, 2019 at 2:40 PM Auer, Lukas
<lukas.auer at aisec.fraunhofer.de> wrote:
>
> + Bin
> [Please use get_maintainer or patman to include all maintainers on CC]

Thanks.

Hm.. get_maintainer and patman are not mentioned on wiki page:
"Patches and Feature Requests". It only mentions to CC people from:
http://www.denx.de/wiki/U-Boot/Custodians

>
> On Tue, 2019-04-09 at 12:42 +0200, David Abdurachmanov wrote:
> > After updating Fedora/RISCV kernel to 5.1-rc3+ the size increased above
> > the current threshold. Looking into HiKey, Dragonboards, etc. seems that
> > SZ_64M is a popular option.
> >
> > This sucessfully boots Fedora/RISCV with 5.1-rc3+ kernel on QEMU 4.0 (master)
> > with OpenSBI -> U-Boot (S-mode) [extlinux] -> Kernel setup.
> >
> > Signed-off-by: David Abdurachmanov <david.abdurachmanov at gmail.com>
> > ---
> >  include/configs/qemu-riscv.h | 12 ++++++------
> >  1 file changed, 6 insertions(+), 6 deletions(-)
> >
> > diff --git a/include/configs/qemu-riscv.h b/include/configs/qemu-riscv.h
> > index 2588c5a0b2..22a5cd7365 100644
> > --- a/include/configs/qemu-riscv.h
> > +++ b/include/configs/qemu-riscv.h
> > @@ -15,7 +15,7 @@
> >
> >  #define CONFIG_SYS_MALLOC_LEN                SZ_8M
> >
> > -#define CONFIG_SYS_BOOTM_LEN         SZ_16M
> > +#define CONFIG_SYS_BOOTM_LEN         SZ_64M
> >
> >  #define CONFIG_STANDALONE_LOAD_ADDR  0x80200000
> >
> > @@ -41,11 +41,11 @@
> >  #define CONFIG_EXTRA_ENV_SETTINGS \
> >       "fdt_high=0xffffffffffffffff\0" \
> >       "initrd_high=0xffffffffffffffff\0" \
> > -     "kernel_addr_r=0x81000000\0" \
> > -     "fdt_addr_r=0x82000000\0" \
> > -     "scriptaddr=0x82100000\0" \
> > -     "pxefile_addr_r=0x82200000\0" \
> > -     "ramdisk_addr_r=0x82300000\0" \
> > +     "kernel_addr_r=0x84000000\0" \
>
> Why are you also moving kernel_addr_r?

I think there two 16MB windows here for kernel:
- 0x80000000 - 0x81000000 (kernel_addr_r),  and kernel is loaded at 0x80200000
- 0x81000000 (kernel_addr_r) - 0x82000000 (fdt_addr_r)

>From U-Boot documentation:

[..]
236 kernel_addr_r:
237
238   Mandatory. The location in RAM where the kernel will be loaded
to when
239   processing the kernel option in the extlinux.conf.
[..]

Thus I moved it to ensure that both windows are 64MB.

>
> Thanks,
> Lukas
>
> > +     "fdt_addr_r=0x88000000\0" \
> > +     "scriptaddr=0x88100000\0" \
> > +     "pxefile_addr_r=0x88200000\0" \
> > +     "ramdisk_addr_r=0x88300000\0" \
> >       BOOTENV
> >
> >  #endif /* __CONFIG_H */