Hi Alex, On Wed, 14 Nov 2018 at 06:28, Alexander Graf <ag...@suse.de> wrote: > > On 11/06/2018 11:21 PM, Simon Glass wrote: > > Generally this functionality is not useful in SPL or TPL. Update the > > Makefile so that it is not built. > > > > Signed-off-by: Simon Glass <s...@chromium.org> > > --- > > > > lib/Makefile | 8 ++++---- > > 1 file changed, 4 insertions(+), 4 deletions(-) > > > > diff --git a/lib/Makefile b/lib/Makefile > > index 4d2e22027d0..4dba3628e78 100644 > > --- a/lib/Makefile > > +++ b/lib/Makefile > > @@ -5,10 +5,10 @@ > > > > ifndef CONFIG_SPL_BUILD > > > > -obj-$(CONFIG_EFI) += efi/ > > -obj-$(CONFIG_EFI_LOADER) += efi_driver/ > > -obj-$(CONFIG_EFI_LOADER) += efi_loader/ > > -obj-$(CONFIG_EFI_LOADER) += efi_selftest/ > > +obj-$(CONFIG_$(SPL_TPL_)EFI) += efi/ > > +obj-$(CONFIG_$(SPL_TPL_)EFI_LOADER) += efi_driver/ > > +obj-$(CONFIG_$(SPL_TPL_)EFI_LOADER) += efi_loader/ > > +obj-$(CONFIG_$(SPL_TPL_)EFI_LOADER) += efi_selftest/ > > This is already in the "ifndef CONFIG_SPL_BUILD" segment. > > I'd rather like to see you remove that whole ifndef section including > lzma, bzip2, tizen etc and convert (at least most of it) to the SPL_TPL_ > magic you applied here.
Actually we can drop this patch. I wrote it thinking I might want to control whether EFI_LOADER is in SPL, but in fact in the end I didn't need to. Sorry for the noise. Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
