[U-Boot] [PATCH 3/3] efi_loader: Do not enable it by default for sunxi

[Maxime Ripard]

On Thu, Oct 19, 2017 at 10:06:15AM +0100, Peter Robinson wrote:
> On Thu, Oct 19, 2017 at 10:01 AM, Maxime Ripard
> <maxime.ripard at free-electrons.com> wrote:
> > On Thu, Oct 19, 2017 at 09:43:20AM +0100, Peter Robinson wrote:
> >> On Thu, Oct 19, 2017 at 9:26 AM, Maxime Ripard
> >> <maxime.ripard at free-electrons.com> wrote:
> >> > The EFI loader support takes around 31kB on an ARMv7 board, which makes us
> >> > trip across the size limit we've had on the U-Boot binary.
> >> >
> >> > Since it's not an essential feature, disable it by default for ARCH_SUNXI
> >> > so that we get back some extra room for user customisations.
> >>
> >> Does this disable it on aarch64 boards by default such as the Pine64?
> >> If so NAK as Fedora, SUSE and I'm pretty sure Debian all use EFI to
> >> boot aarch64 devices and this would regress this for all those
> >> distros.
> >
> > This is something that Fedora, Suse and I'm pretty sure Debian can add
> > to their defconfig. These are just default configuration, not
> > one-size-fits-all configuration.
> 
> So you're making at least three groups of users do more work? It could
> also be argued that those that need the smaller space could disable it
> if they don't need it in their configuration.

I don't think asking people to modify their default configuration in
order to have U-Boot booting is reasonable, is it?

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20171019/95512142/attachment.sig>