[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] loader cleanup for i386-ieee1275

From: Marco Gerards
Subject: Re: [PATCH] loader cleanup for i386-ieee1275
Date: Wed, 23 Jan 2008 19:48:04 +0100
User-agent: Gnus/5.110006 (No Gnus v0.6) Emacs/21.4 (gnu/linux)

Robert Millan <address@hidden> writes:

> On Wed, Jan 23, 2008 at 03:05:31PM +0100, Marco Gerards wrote:
>> Robert Millan <address@hidden> writes:
>> > Some cleanup for i386-ieee1275.
>> Can you please give a bit more information?
> The multiboot module on i386-ieee1275 was using variables that only have a
> meaning on i386-pc (and i386-linuxbios):  grub_os_area_addr,
> grub_os_area_size, grub_lower_mem and grub_upper_mem.  A bunch of
> unnecessary cruft from i386-pc port was being included, because at the
> beginning I thought I would base the multiboot loader on i386-pc (like I
> did for i386-linuxbios), but this turned out to be a wrong approach.

I see.

>> >    * include/grub/i386/ieee1275/loader.h: Include `<grub/types.h>',
>> >    `<grub/symbol.h>' and `<grub/multiboot.h>'.
>> >    (grub_multiboot2_real_boot): New function prototype.
>> >
>> >    * include/grub/i386/pc/memory.h: Include `<grub/machine/machine.h>'.
>> >    (grub_lower_mem, grub_upper_mem): Disable for IEEE1275.
>> You can make this conditional in the ChangeLog entry.
> What do you mean?

        * include/grub/i386/pc/memory.h: Include `<grub/machine/machine.h>'.
        [!GRUB_IEEE1275] (grub_lower_mem, grub_upper_mem): Disabled.

I didn't check if this is the name of the macro, so please do not
blindly copy this.  But this is the right approach.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]