All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paul Menzel <paulepanter@users.sourceforge.net>
To: grub-devel@gnu.org
Subject: Re: `all_video.mod` missing loading `grub.cfg` generated by grub-pc in GRUB payload (coreboot)
Date: Sun, 16 Apr 2017 21:31:30 +0200	[thread overview]
Message-ID: <1492371090.2561.114.camel@users.sourceforge.net> (raw)
In-Reply-To: <fc8e835b-b518-4d9c-711e-bc9a4a862722@gmail.com>

[-- Attachment #1: Type: text/plain, Size: 893 bytes --]

Dear Andrei,


Am Samstag, den 15.04.2017, 14:22 +0300 schrieb Andrei Borzenkov:
> 15.04.2017 13:25, Paul Menzel пишет:

> > When using GRUB as a coreboot payload, that means passing `--with-
> > platform=coreboot` to configure, and configuring it to load the GRUB
> > configuration file from disk, `/boot/grub/grub.cfg` in Debian 8.7
> > (Jessie/stable) and 9 (Stretch/testing), generated by GRUB shipped by
> > the GNU/Linux distribution, I get a warning about the missing module
> > `all_video.mod`, and the enter key has to be pressed to continue
> > booting.
> 
> ...
> 
> > Do you know of a way to make that compatible with grub-pc and GRUB
> > payload? 
> 
> Include all_video in coreboot payload?

Thank you. That indeed works. I thought, it wouldn’t be available for
that platform, and it is. I’ll send a patch to add it by default.


Thanks,

Paul

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

  reply	other threads:[~2017-04-16 19:31 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-04-15 10:25 `all_video.mod` missing loading `grub.cfg` generated by grub-pc in GRUB payload (coreboot) Paul Menzel
2017-04-15 11:22 ` Andrei Borzenkov
2017-04-16 19:31   ` Paul Menzel [this message]
2017-04-16 21:45     ` [PATCH] Makefile.am: Add `all_video` to `default_payload.elf` Paul Menzel
2018-03-21  8:28       ` [PATCH v2] " Paul Menzel
2018-04-23 19:45         ` Paul Menzel
2018-04-24 10:20           ` Daniel Kiper
2018-08-09  6:20             ` [PATCH v3] " Paul Menzel
2018-09-06 12:58               ` Daniel Kiper
2018-09-06 13:07                 ` Vladimir 'phcoder' Serbinenko
2018-09-06 13:21                   ` Daniel Kiper
2018-09-19 16:12                     ` Daniel Kiper
2018-09-27 16:52                       ` Daniel Kiper
2018-10-01  9:00                         ` Paul Menzel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1492371090.2561.114.camel@users.sourceforge.net \
    --to=paulepanter@users.sourceforge.net \
    --cc=grub-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.