[U-Boot] [PATCH v2] arm64: ls1012afrdm: Add distro boot support
Rajesh Bhagat
rajesh.bhagat at nxp.com
Mon Oct 9 05:20:13 UTC 2017
> -----Original Message-----
> From: York Sun
> Sent: Thursday, October 05, 2017 8:30 PM
> To: Rajesh Bhagat <rajesh.bhagat at nxp.com>; u-boot at lists.denx.de
> Cc: Bhaskar Upadhaya <bhaskar.upadhaya at nxp.com>
> Subject: Re: [PATCH v2] arm64: ls1012afrdm: Add distro boot support
>
> On 10/05/2017 01:47 AM, Rajesh Bhagat wrote:
> > Include common config_distro_defaults.h and config_distro_bootcmd.h
> > for u-boot enviroments to support automatical distro boot which scan
> > boot.scr from external storage devices(e.g. SD and USB) and execute
> > autoboot script.
> >
> > Signed-off-by: Bhaskar Upadhaya <Bhaskar.Upadhaya at nxp.com>
> > Signed-off-by: Rajesh Bhagat <rajesh.bhagat at nxp.com>
> > ---
> > Depends on:
> > https://emea01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fpatc
> >
> hwork.ozlabs.org%2Fpatch%2F808617%2F&data=01%7C01%7Cyork.sun%40nxp.com
> >
> %7C83457d3a96b74d7dd33d08d50bcdb2c9%7C686ea1d3bc2b4c6fa92cd99c5c301635
> >
> %7C0&sdata=sKmaJpoaeK1JXMf%2F%2Ba%2BPlUQtY3HumeB08I6MKTWchzc%3D&re
> serv
> > ed=0
> >
> > Changes in v2:
> > - Removed the extra "\" character from macro definition
> > - Dropped the #if for CONFIG_BOOTCOMMAND
> >
> > configs/ls1012afrdm_qspi_defconfig | 1 +
> > include/configs/ls1012afrdm.h | 57
> ++++++++++++++++++++++++++++++++------
> > 2 files changed, 49 insertions(+), 9 deletions(-)
> >
> > diff --git a/configs/ls1012afrdm_qspi_defconfig
> > b/configs/ls1012afrdm_qspi_defconfig
> > index fe95f04..42acff0 100644
> > --- a/configs/ls1012afrdm_qspi_defconfig
> > +++ b/configs/ls1012afrdm_qspi_defconfig
> > @@ -46,3 +46,4 @@ CONFIG_DM_USB=y
> > CONFIG_USB_XHCI_HCD=y
> > CONFIG_USB_XHCI_DWC3=y
> > CONFIG_USB_STORAGE=y
> > +CONFIG_DISTRO_DEFAULTS=y
> > diff --git a/include/configs/ls1012afrdm.h
> > b/include/configs/ls1012afrdm.h index 6b1ba57..4d6448b 100644
> > --- a/include/configs/ls1012afrdm.h
> > +++ b/include/configs/ls1012afrdm.h
> > @@ -20,16 +20,55 @@
> > #define CONFIG_SYS_MEMTEST_START 0x80000000
> > #define CONFIG_SYS_MEMTEST_END 0x9fffffff
> >
> > +#ifndef CONFIG_SPL_BUILD
> > +#undef BOOT_TARGET_DEVICES
> > +#define BOOT_TARGET_DEVICES(func) \
> > + func(USB, usb, 0)
> > +#endif
> > +
> > #undef CONFIG_EXTRA_ENV_SETTINGS
> > -#define CONFIG_EXTRA_ENV_SETTINGS \
> > - "verify=no\0" \
> > - "loadaddr=0x80100000\0" \
> > - "kernel_addr=0x100000\0" \
> > - "fdt_high=0xffffffffffffffff\0" \
> > - "initrd_high=0xffffffffffffffff\0" \
> > - "kernel_start=0xa00000\0" \
> > - "kernel_load=0x96000000\0" \
> > - "kernel_size=0x2800000\0"
> > +#define CONFIG_EXTRA_ENV_SETTINGS \
> > + "verify=no\0" \
> > + "fdt_high=0xffffffffffffffff\0" \
> > + "initrd_high=0xffffffffffffffff\0" \
> > + "fdt_addr=0x00f00000\0" \
> > + "kernel_addr=0x01000000\0" \
> > + "scriptaddr=0x80000000\0" \
> > + "fdtheader_addr_r=0x80100000\0" \
> > + "kernelheader_addr_r=0x80200000\0" \
> > + "kernel_addr_r=0x96000000\0" \
> > + "fdt_addr_r=0x90000000\0" \
> > + "load_addr=0x96000000\0" \
> > + "kernel_size=0x2800000\0" \
> > + "console=ttyS0,115200\0" \
> > + BOOTENV \
> > + "boot_scripts=ls1012afrdm_boot.scr\0" \
> > + "scan_dev_for_boot_part=" \
> > + "part list ${devtype} ${devnum} devplist; " \
> > + "env exists devplist || setenv devplist 1; " \
> > + "for distro_bootpart in ${devplist}; do " \
> > + "if fstype ${devtype} " \
> > + "${devnum}:${distro_bootpart} " \
> > + "bootfstype; then " \
> > + "run scan_dev_for_boot; " \
> > + "fi; " \
> > + "done\0" \
> > + "scan_dev_for_boot=" \
> > + "echo Scanning ${devtype} " \
> > + "${devnum}:${distro_bootpart}...; " \
> > + "for prefix in ${boot_prefixes}; do " \
> > + "run scan_dev_for_scripts; " \
> > + "done;" \
> > + "\0" \
> > + "installer=load usb 0:2 $load_addr " \
> > + "/flex_installer_arm64.itb; " \
> > + "bootm $load_addr#$board\0" \
> > + "qspi_bootcmd=echo Trying load from qspi..;" \
> > + "sf probe && sf read $load_addr " \
> > + "$kernel_addr $kernel_size && bootm $load_addr#$board\0"
> > +
>
> http://patchwork.ozlabs.org/patch/808617/ changes CONFIG_EXTRA_ENV_SETTINGS
> which you just redefined. Do you really depend on that patch? That patch needs an
> update.
>
Yes, this patch depends on below patch as that changes CONFIG_ENV_OFFSET
which is required for distro boot.
http://patchwork.ozlabs.org/patch/808617/
> York
More information about the U-Boot
mailing list