Hi Stephen, On Mon, 15 Jun 2020 at 16:03, Stephen Warren <[email protected]> wrote: > > On 6/13/20 8:57 PM, Simon Glass wrote: > > This has been in the Makefile long enough to ensure migration is complete. > > Drop it. > > > diff --git a/Makefile b/Makefile > > > -OBJCOPYFLAGS_u-boot-nodtb-tegra.bin = -O binary > > --pad-to=$(CONFIG_SYS_TEXT_BASE) > > -u-boot-nodtb-tegra.bin: spl/u-boot-spl u-boot-nodtb.bin FORCE > > - $(call if_changed,pad_cat) > > - > > -OBJCOPYFLAGS_u-boot-tegra.bin = -O binary --pad-to=$(CONFIG_SYS_TEXT_BASE) > > -u-boot-tegra.bin: spl/u-boot-spl u-boot.bin FORCE > > - $(call if_changed,pad_cat) > > I don't see any replacement rules for those two binaries. Is the intent > to force all users not to use them any more; is that what "migration is > complete" means? Note that there are users of these files, so if this > change/series truly is removing them it will break those users; see: > > > https://github.com/NVIDIA/tegra-uboot-flasher-scripts/blob/master/build#L166
Binman produces all three images when run, so we don't need makefile rules for them anymore. It still produces the files. But let me know if there are any problems. Migration complete just means that there was a reason we had to keep these rules before, and I hope it has gone away. Regards, Simon
