grub-devel
[Top][All Lists]
Advanced

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

Re: [PATCH] generic ELF version of grub-mkimage


From: Marco Gerards
Subject: Re: [PATCH] generic ELF version of grub-mkimage
Date: Fri, 12 Oct 2007 12:52:42 +0200
User-agent: Gnus/5.110006 (No Gnus v0.6) Emacs/21.4 (gnu/linux)

Robert Millan <address@hidden> writes:

> Woops.  I've been pointed out that cross-compiles were actually working and
> should be preserved.  I propose this new patch which integrates endianess
> conversion macros with include/grub/types.h.
>
> -- 
> Robert Millan
>
> <GPLv2> I know my rights; I want my phone call!
> <DRM> What use is a phone call, if you are unable to speak?
> (as seen on /.)
>
> 2007-10-12  Robert Millan  <address@hidden>
>
>       * conf/powerpc-ieee1275.rmk (grub_mkimage_SOURCES): Replace reference
>       to util/powerpc/ieee1275/grub-mkimage.c with util/elf/grub-mkimage.c.
>
>       * include/grub/types.h (#ifdef GRUB_CPU_WORDS_BIGENDIAN): Define
>       grub_host_to_target16, grub_host_to_target32, grub_host_to_target64,
>       grub_target_to_host16, grub_target_to_host32 and grub_target_to_host64.
>       (#else): Likewise.

This syntax is wrong.  It should have been something like:

>       * include/grub/types.h (#ifdef GRUB_CPU_WORDS_BIGENDIAN): Define
>       grub_host_to_target16, grub_host_to_target32, grub_host_to_target64,
>       grub_target_to_host16, grub_target_to_host32 and grub_target_to_host64.
>       (#else): Likewise.

        * include/grub/types.h (grub_host_to_target16): New macro.
        (grub_host_to_target32): Likewise.

etc...

If you are not sure, better wait a few days before committing a
patch.  Or ask me to double check it.

>       * include/grub/powerpc/ieee1275/kernel.h (GRUB_IEEE1275_MOD_ALIGN):
>       Renamed from to ...
>       (GRUB_MOD_ALIGN): ...this.  Update all users.
>
>       * util/elf/grub-mkimage.c: Replace grub_cpu_to_be16, grub_cpu_to_be32,
>       grub_be_to_cpu16 and grub_be_to_cpu32 macros with grub_host_to_target16,
>       grub_host_to_target32, grub_target_to_host16 and grub_target_to_host32,
>       respectively.

You know quite well that this is wrong.  You have to mention which
function you changed.  

If you use diff -up this is easy to see from the patch.

--
Marco





reply via email to

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