[Buildroot] [PATCH v2 2/6] fs/iso9660: add support to Grub EFI bootloader in the image
Yann E. MORIN
yann.morin.1998 at free.fr
Tue Sep 21 19:52:03 UTC 2021
Köry, All,
On 2021-09-21 20:41 +0200, Köry Maincent spake thusly:
> On Tue, 21 Sep 2021 17:36:09 +0200
> "Yann E. MORIN" <yann.morin.1998 at free.fr> wrote:
[--SNIP--]
> > > + -b $(ROOTFS_ISO9660_BOOT_IMAGE)
> > > +else ifeq ($(BR2_TARGET_ROOTFS_ISO9660_EFI_BOOTLOADER),y)
> > Note that BR2_TARGET_ROOTFS_ISO9660_BIOS_BOOTLOADER and
> > BR2_TARGET_ROOTFS_ISO9660_EFI_BOOTLOADER are mutually exclusive, by
> > definition, but we also know at least one will be set, also by
> > definition.
> > So, the else clause can be a simple one, without the ifeq.
>
> doh! you raise a point. In fact in the current configuration they are not
> mutually exclusive. I need to fix that with a Kconfig menu.
Ah, right! And this patch is not (yet) about making a fully hybrid
image, so you *have* to have the user choose what boot payload to use,
probably something like:
choice
bool "Boot payload"
config BR2_TARGET_ROOTFS_ISO9660_BIOS_BOOTLOADER
bool "legacy bios"
depends on something I guess...
config BR2_TARGET_ROOTFS_ISO9660_EFI_BOOTLOADER
bool "UEFI"
depends on something else I guess...
endchoice
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
More information about the buildroot
mailing list