Hi Philipp: 2017-05-31 18:58 GMT+08:00 Dr. Philipp Tomsich < philipp.toms...@theobroma-systems.com>:
> Now that there are no remaining cases where (spl_boot0 == false), we can > drop the spl_boot0 field from struct spl_info and also remove the if-check > (leaving only the code-path for true in place). > > I think we still have one case: use mkimage to pack ddr.bin from rockchip as spl.bin. > There should be no need to keep the legacy code-path around, as GIT will > always remember them anyway. > > Regards, > Philipp. > > > On 31 May 2017, at 12:50, Kever Yang <kever.y...@rock-chips.com> wrote: > > > > Enable the spl_boot0 in SPL and use the pre-padding TAG memory, > > the mkimage do not need to pad it but only need to replace the value > > with correct TAG value. > > > > Signed-off-by: Kever Yang <kever.y...@rock-chips.com> > > --- > > > > tools/rkcommon.c | 8 ++++---- > > 1 file changed, 4 insertions(+), 4 deletions(-) > > > > diff --git a/tools/rkcommon.c b/tools/rkcommon.c > > index 836a5a5..b58dfae 100644 > > --- a/tools/rkcommon.c > > +++ b/tools/rkcommon.c > > @@ -68,10 +68,10 @@ struct spl_info { > > }; > > > > static struct spl_info spl_infos[] = { > > - { "rk3036", "RK30", 0x1000, false, false }, > > - { "rk3188", "RK31", 0x8000 - 0x800, true, false }, > > - { "rk3288", "RK32", 0x8000, false, false }, > > - { "rk3328", "RK32", 0x8000 - 0x1000, false, false }, > > + { "rk3036", "RK30", 0x1000, false, true }, > > + { "rk3188", "RK31", 0x8000 - 0x800, true, true }, > > + { "rk3288", "RK32", 0x8000, false, true }, > > + { "rk3328", "RK32", 0x8000 - 0x1000, false, true }, > > { "rk3399", "RK33", 0x20000, false, true }, > > }; > > > > -- > > 1.9.1 > > > > _______________________________________________ > U-Boot mailing list > U-Boot@lists.denx.de > https://lists.denx.de/listinfo/u-boot > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot