Multiboot video in GRUB (Re: [RFC] Multiboot ammendment: non-VBE video)

[Robert Millan <rmh@aybabtu.com>]

On Sat, Jan 02, 2010 at 07:26:08PM +0100, Vladimir 'φ-coder/phcoder' Serbinenko wrote:
> === added file 'ChangeLog.efiboot'
> --- ChangeLog.efiboot	1970-01-01 00:00:00 +0000
> +++ ChangeLog.efiboot	2010-01-02 01:48:33 +0000
> @@ -0,0 +1,19 @@
> +2009-11-28  Vladimir Serbinenko  <phcoder@gmail.com>
> +
> +	Multiboot support on i386-efi.
> +
> +	* conf/i386-efi.rmk (pkglib_MODULES): Add multiboot.mod.
> +	(multiboot_mod_SOURCES): New variable.
> +	(multiboot_mod_CFLAGS): Likewise.
> +	(multiboot_mod_LDFLAGS): Likewise.
> +	(multiboot_mod_ASFLAGS): Likewise.
> +	* include/grub/i386/multiboot.h (grub_multiboot_real_boot): Explicitly
> +	declare as regparm (3).
> +	(grub_multiboot2_real_boot): Likewise.
> +	(grub_multiboot_boot): Finish boot services.
> +	(grub_fill_multiboot_mmap): Explicitly parse memory map types.
> +	(grub_multiboot): Declare as noreturn to avoid finishing console.
> +	* loader/multiboot_loader.c (grub_cmd_multiboot_loader): Enable
> +	multiboot1 on i386-efi and disable multiboot2 on this platform.
> +	(grub_cmd_module_loader): Likewise.

Could the EFI parts be split into a separate patch?  I definitely want both
things, but they're separate changes, would be good to have the additional
granularity if possible.

-- 
Robert Millan

  "Be the change you want to see in the world" -- Gandhi