On 11/01/2017 10:48 PM, Ashish Kumar wrote:
> Disto boot support give flexibility to run distro RFS like ubuntu's

s/Disto/Distro
s/give/gives
S/ubuntu/Ubuntu

> being deployed from SD card or USB stick. If it fails
> to detect external storage, fall back to qspi/sd boot.
> 
> Signed-off-by: Ashish Kumar <ashish.ku...@nxp.com>
> Signed-off-by: Zhang Ying <zhangy...@nxp.com>
> ---
>  include/configs/ls1088ardb.h | 113 
> +++++++++++++++++++++++++++++++++----------
>  1 file changed, 88 insertions(+), 25 deletions(-)
> 
> diff --git a/include/configs/ls1088ardb.h b/include/configs/ls1088ardb.h
> index eef9a07..c7d1fd3 100644
> --- a/include/configs/ls1088ardb.h
> +++ b/include/configs/ls1088ardb.h
> @@ -258,40 +258,103 @@
>  
>  /* Initial environment variables */
>  #if defined(CONFIG_QSPI_BOOT)
> -#undef CONFIG_EXTRA_ENV_SETTINGS
> -#define CONFIG_EXTRA_ENV_SETTINGS            \
> -     "hwconfig=fsl_ddr:bank_intlv=auto\0"    \
> -     "loadaddr=0x90100000\0"                 \
> -     "kernel_addr=0x100000\0"                \
> -     "ramdisk_addr=0x800000\0"               \
> -     "ramdisk_size=0x2000000\0"              \
> -     "fdt_high=0xa0000000\0"                 \
> -     "initrd_high=0xffffffffffffffff\0"      \
> -     "kernel_start=0x1000000\0"              \
> -     "kernel_load=0xa0000000\0"              \
> -     "kernel_size=0x2800000\0"               \
> +#define MC_INIT_CMD                          \
>       "mcinitcmd=sf probe 0:0;sf read 0x80000000 0xA00000 0x100000;"  \
> -     "sf read 0x80100000 0xE00000 0x100000;" \
> -     "fsl_mc start mc 0x80000000 0x80100000\0"       \
> -     "mcmemsize=0x70000000 \0"
> +     "sf read 0x80100000 0xE00000 0x100000;"                         \
> +     "fsl_mc start mc 0x80000000 0x80100000\0"                       \
> +     "mcmemsize=0x70000000\0"
>  #elif defined(CONFIG_SD_BOOT)
> +#define MC_INIT_CMD                          \
> +     "mcinitcmd=mmcinfo;mmc read 0x80000000 0x5000 0x800;"           \
> +     "mmc read 0x80100000 0x7000 0x800;"                             \
> +     "fsl_mc start mc 0x80000000 0x80100000\0"                       \
> +     "mcmemsize=0x70000000\0"
> +#endif
> +
>  #undef CONFIG_EXTRA_ENV_SETTINGS
>  #define CONFIG_EXTRA_ENV_SETTINGS            \
> +     "BOARD=ls1088ardb\0"    \
>       "hwconfig=fsl_ddr:bank_intlv=auto\0"    \
> -     "loadaddr=0x90100000\0"                 \
> -     "kernel_addr=0x800\0"                   \
>       "ramdisk_addr=0x800000\0"               \
>       "ramdisk_size=0x2000000\0"              \
>       "fdt_high=0xa0000000\0"                 \
>       "initrd_high=0xffffffffffffffff\0"      \
> -     "kernel_start=0x8000\0"                 \
> -     "kernel_load=0xa0000000\0"              \
> -     "kernel_size=0x14000\0"                 \
> -     "mcinitcmd=mmcinfo;mmc read 0x80000000 0x5000 0x800;"   \
> -     "mmc read 0x80100000 0x7000 0x800;"     \
> -     "fsl_mc start mc 0x80000000 0x80100000\0"       \
> -     "mcmemsize=0x70000000 \0"
> -
> +     "fdt_addr=0x64f00000\0"                 \
> +     "kernel_addr=0x1000000\0"               \
> +     "kernel_addr_sd=0x8000\0"               \
> +     "kernel_start=0x580100000\0"            \
> +     "kernelheader_start=0x580800000\0"              \
> +     "scriptaddr=0x80000000\0"               \
> +     "scripthdraddr=0x80080000\0"            \
> +     "fdtheader_addr_r=0x80100000\0"         \
> +     "kernelheader_addr=0x800000\0"  \
> +     "kernelheader_addr_r=0x80200000\0"      \
> +     "kernel_addr_r=0x81000000\0"            \
> +     "kernelheader_size=0x40000\0"           \
> +     "fdt_addr_r=0x90000000\0"               \
> +     "load_addr=0xa0000000\0"                \
> +     "kernel_size=0x2800000\0"               \
> +     "kernel_size_sd=0x14000\0"              \
> +     MC_INIT_CMD                             \
> +     BOOTENV                                 \
> +     "boot_scripts=ls1088ardb_boot.scr\0"    \
> +     "boot_script_hdr=hdr_ls1088ardb_bs.out\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"                                                    \
> +     "boot_a_script="                                        \
> +             "load ${devtype} ${devnum}:${distro_bootpart} " \
> +                     "${scriptaddr} ${prefix}${script}; "    \
> +     "env exists secureboot && load ${devtype} "             \

Please pay attention to your indentation.

> +             "${devnum}:${distro_bootpart} "                 \
> +             "${scripthdraddr} ${prefix}${boot_script_hdr} " \
> +             "&& esbc_validate ${scripthdraddr};"            \
> +     "source ${scriptaddr}\0"                                \

Inconsistent indentation makes it harder to read.

York
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to