Re: [PATCH] loader cleanup for i386-ieee1275

2008-01-24 Thread Yoshinori K. Okuji
On Thursday 24 January 2008 13:40, Robert Millan wrote: > On Thu, Jan 24, 2008 at 09:46:09AM +0100, Yoshinori K. Okuji wrote: > > > 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,

Re: [PATCH] loader cleanup for i386-ieee1275

2008-01-24 Thread Robert Millan
On Thu, Jan 24, 2008 at 09:46:09AM +0100, Yoshinori K. Okuji wrote: > > > > 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 > > unnecessa

Re: [PATCH] loader cleanup for i386-ieee1275

2008-01-24 Thread Yoshinori K. Okuji
On Wednesday 23 January 2008 15:53, Robert Millan wrote: > On Wed, Jan 23, 2008 at 03:05:31PM +0100, Marco Gerards wrote: > > Robert Millan <[EMAIL PROTECTED]> writes: > > > Some cleanup for i386-ieee1275. > > > > Can you please give a bit more information? > > The multiboot module on i386-ieee1275

Re: [PATCH] loader cleanup for i386-ieee1275

2008-01-23 Thread Robert Millan
On Wed, Jan 23, 2008 at 07:48:04PM +0100, Marco Gerards wrote: > >> You can make this conditional in the ChangeLog entry. > > > > What do you mean? > > * include/grub/i386/pc/memory.h: Include `'. > [!GRUB_IEEE1275] (grub_lower_mem, grub_upper_mem): Disabled. > > I didn't check if t

Re: [PATCH] loader cleanup for i386-ieee1275

2008-01-23 Thread Marco Gerards
Robert Millan <[EMAIL PROTECTED]> writes: > On Wed, Jan 23, 2008 at 03:05:31PM +0100, Marco Gerards wrote: >> Robert Millan <[EMAIL PROTECTED]> writes: >> >> > Some cleanup for i386-ieee1275. >> >> Can you please give a bit more information? > > The multiboot module on i386-ieee1275 was using va

Re: [PATCH] loader cleanup for i386-ieee1275

2008-01-23 Thread Robert Millan
On Wed, Jan 23, 2008 at 03:05:31PM +0100, Marco Gerards wrote: > Robert Millan <[EMAIL PROTECTED]> 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

Re: [PATCH] loader cleanup for i386-ieee1275

2008-01-23 Thread Marco Gerards
Robert Millan <[EMAIL PROTECTED]> writes: > Some cleanup for i386-ieee1275. Can you please give a bit more information? > * include/grub/i386/ieee1275/loader.h: Include `', > `' and `'. > (grub_multiboot2_real_boot): New function prototype. > > * include/grub/i386/pc/memo

[PATCH] loader cleanup for i386-ieee1275

2008-01-23 Thread Robert Millan
Some cleanup for i386-ieee1275. -- Robert Millan I know my rights; I want my phone call! What use is a phone call… if you are unable to speak? (as seen on /.) * include/grub/i386/ieee1275/loader.h: Include `', `' and `'. (grub_multiboot2_real_boot): New function prototype. * include/gru