> -----Original Message-----
> From: Mingkai Hu
> Sent: Wednesday, November 01, 2017 7:14 PM
> To: York Sun <york....@nxp.com>
> Cc: Shengzhou Liu <shengzhou....@nxp.com>; u-boot@lists.denx.de
> Subject: RE: [U-Boot] [PATCH 1/4] arm64: ls1043ardb: Add sd_bootcmd for
> distro fallback in case of sdboot
> 
> 
> 
> > -----Original Message-----
> > From: York Sun
> > Sent: Tuesday, October 31, 2017 2:45 AM
> > To: Mingkai Hu <mingkai...@nxp.com>
> > Cc: Shengzhou Liu <shengzhou....@nxp.com>; u-boot@lists.denx.de
> > Subject: Re: [U-Boot] [PATCH 1/4] arm64: ls1043ardb: Add sd_bootcmd
> > for distro fallback in case of sdboot
> >
> > On 10/23/2017 01:53 PM, York Sun wrote:
> > > On 09/14/2017 02:26 PM, York Sun wrote:
> > >> On 08/30/2017 03:43 AM, Shengzhou Liu wrote:
> > >>> Signed-off-by: Shengzhou Liu <shengzhou....@nxp.com>
> > >>> ---
> > >>>   include/configs/ls1043a_common.h | 11 ++++++++++-
> > >>>   1 file changed, 10 insertions(+), 1 deletion(-)
> > >>>
> > >>> diff --git a/include/configs/ls1043a_common.h
> > >>> b/include/configs/ls1043a_common.h
> > >>> index f064d5c24a..976c031574 100644
> > >>> --- a/include/configs/ls1043a_common.h
> > >>> +++ b/include/configs/ls1043a_common.h
> > >>> @@ -281,6 +281,8 @@
> > >>>         "fdt_addr_r=0x90000000\0"               \
> > >>>         "load_addr=0xa0000000\0"                \
> > >>>         "kernel_size=0x2800000\0"               \
> > >>> +       "kernel_addr_sd=0x8000\0"               \
> > >>> +       "kernel_size_sd=0x14000\0"              \
> > >>>         "console=ttyS0,115200\0"                \
> > >>>         "mtdparts=" MTDPARTS_DEFAULT "\0"       \
> > >>>         BOOTENV                                 \
> > >>> @@ -318,12 +320,19 @@
> > >>>                 "$kernel_addr $kernel_size && bootm
> > $load_addr#$board\0" \
> > >>>         "nor_bootcmd=echo Trying load from nor..;"      \
> > >>>                 "cp.b $kernel_addr $load_addr " \
> > >>> -               "$kernel_size && bootm $load_addr#$board\0"
> > >>> +               "$kernel_size && bootm $load_addr#$board\0"     \
> > >>> +       "sd_bootcmd=echo Trying load from SD ..;"       \
> > >>> +               "mmcinfo; mmc read $load_addr "         \
> > >>> +               "$kernel_addr_sd $kernel_size_sd ;"     \
> > >>> +               " bootm $load_addr#$board\0"
> > >>>
> > >>
> > >> Shouldn't you be using "&&" instead of ";" between commands?
> > >>
> > >
> > > Shengzhou,
> > >
> > > Are you going to update the patches?
> >
> > Mingkai,
> >
> > Do you need this change for LS1043ARDB? Please respond to my
> comment
> > if so. Otherwise, I will drop this set.
> >
> > York
> 
> Hi York,
> 
> This will be needed in order to keep consistent with our LSDK.
> 
> Thanks,
> Mingkai

York, 
please drop this version, I will update it and send next new version to add 
distro secureboot together. 

Thanks,
Shengzhou



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

Reply via email to