> On 01 Jun 2017, at 03:11, Andy Yan <andys...@gmail.com> wrote: > > 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.
Excellent point! I completely missed that use-case. However, then we’d need to implement support (an additional commandline flag?) for communicating this into mkimage: with the current state of the code, the new code path (i.e. assuming its a pre-padded image) will be always be taken and ddr.bin can’t be processed. The alternative would be to clearly document that a ddr.bin needs to be prepadded and then pad the ddr.bin manually before passing it to mkimage. I have no preference either way, but would recommend/request that if we leave the code-path in, then there should be a way to invoke it. Best regards, Philipp. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
