Re: [PATCH v3] Makefile.am: Add `all_video` to `default_payload.elf`

[Vladimir 'phcoder' Serbinenko]

Unfortunately this pulls in some problematic graphics drivers that shouldn't be in default payload

On Thu, 6 Sep 2018, 14:59 Daniel Kiper, <address@hidden> wrote:

On Thu, Aug 09, 2018 at 08:20:07AM +0200, Paul Menzel wrote:
> Date: Sun, 16 Apr 2017 21:02:58 +0200
>
> The module `all_video` is used in `util/grub.d/00_header.in`, and
> included for grub-pc by default.
>
> To make it easier to load such a GRUB configuration from disk with a
> GRUB built as a coreboot payload, add this module also to
> `default_payload.elf` by default. That avoids a missing module error by
> GRUB, forcing the user to hit enter during the start process.
>
>     error: file `/boot/grub/i386-coreboot/all_video.mod' not found.
>

Lack of of SOB... :-((( Otherwise LGTM. If I get your SOB in a reply
to this email or something like that then I will get this patch...

Daniel

_______________________________________________
Grub-devel mailing list
address@hidden
https://lists.gnu.org/mailman/listinfo/grub-devel