[PATCH] configs: layerscape: Disable the EFI_LOADER feature

Z.Q. Hou zhiqiang.hou at nxp.com
Mon Jul 26 08:18:45 CEST 2021


Hi Tom,

> -----Original Message-----
> From: Tom Rini <trini at konsulko.com>
> Sent: 2021年7月22日 23:26
> To: Z.Q. Hou <zhiqiang.hou at nxp.com>; Michael Walle <michael at walle.cc>;
> Heinrich Schuchardt <xypron.glpk at gmx.de>
> Cc: u-boot at lists.denx.de; Priyanka Jain <priyanka.jain at nxp.com>
> Subject: Re: [PATCH] configs: layerscape: Disable the EFI_LOADER feature
> 
> On Thu, Jul 22, 2021 at 02:25:59PM +0800, Zhiqiang Hou wrote:
> 
> > From: Hou Zhiqiang <Zhiqiang.Hou at nxp.com>
> >
> > The feature BOOTENV_SHARED_EFI is not supported on layerscape boards,
> > it didn't result kernel boot crash previously since there isn't the
> > efi/boot/"BOOTEFI_NAME" and it skip calling of 'boot_efi_binary'.
> >
> > But since the commit f3866909e350 ("distro_bootcmd: call EFI bootmgr
> > even without having /EFI/boot"), it will cause kernel boot crash as
> > there isn't a valid fdt_addr and it finially uses the device tree blob
> > of U-Boot and further cause errors.
> >
> > As this feature is enabled by default for armv7 and armv8, so disable
> > it explicitly to avoid calling the 'scan_dev_for_efi'.
> 
> I'm not thrilled with this.  Why isn't the solution to get and keep in sync the
> device trees, so that the tree U-Boot has is valid for the kernel?  I'm also
> open to discussing f3866909e350 more.  But I'm really opposed to disabling
> EFI_LOADER on modern platforms as that will make adoption of U-Boot in
> device harder I feel.
> 
 
I think it doesn't make sense for the platforms on which the EFI boot is not planed.
As there isn't EFI boot needed components in the search path, finally the EFI boot
will be skipped. I don't want to look into the EFI boot process, so I trend to disable
the feature, is it acceptable?

Thanks,
Zhiqiang

> --
> Tom


More information about the U-Boot mailing list