On Fri, Jul 12, 2019 at 08:04:59AM +0200, Heinrich Schuchardt wrote:
> On 7/12/19 2:59 AM, AKASHI Takahiro wrote:
> >Heinrich,
> >
> >I wonder why we need "asm-generic/pe.h".
>
> The assembler cannot understand includes that contain C specific syntax.
> So we cannot include "pe.h". This is why "asm-generic/pe.h" has been
> split off.
If so, why not use "#if(n)def __ASSEMBLY__"?
It is a common usage in lots of headers.
-Takahiro Akashi
> >If IMAGE_SUBSYSTEM_EFI_APPLICATION can be arch-specific,
> >why not move other definitions, like * relocation types, too?
>
> We need those symbols in include/asm-generic/pe.h that are used by
> assembler code. IMAGE_SUBSYSTEM_EFI_APPLICATION is not arch-specific but
> it is used in arch/arm/lib/crt0_arm_efi.S and others.
>
> Best regards
>
> Heinrich
>
> >
> >-Takahiro Akashi
> >
> >On Thu, Jul 11, 2019 at 06:28:24PM +0200, Heinrich Schuchardt wrote:
> >>IMAGE_SUBSYSTEM_EFI_APPLICATION is already defined in
> >>include/asm-generic/pe.h which is included in include/pe.h. We should
> >>not define it twice.
> >>
> >>Signed-off-by: Heinrich Schuchardt <xypron.g...@gmx.de>
> >>---
> >> include/pe.h | 1 -
> >> 1 file changed, 1 deletion(-)
> >>
> >>diff --git a/include/pe.h b/include/pe.h
> >>index 36e1908b7e..c7aeb2dcb6 100644
> >>--- a/include/pe.h
> >>+++ b/include/pe.h
> >>@@ -48,7 +48,6 @@ typedef struct _IMAGE_DOS_HEADER {
> >>
> >> #define IMAGE_NT_OPTIONAL_HDR32_MAGIC 0x10b
> >> #define IMAGE_NT_OPTIONAL_HDR64_MAGIC 0x20b
> >>-#define IMAGE_SUBSYSTEM_EFI_APPLICATION 10
> >>
> >> typedef struct _IMAGE_FILE_HEADER {
> >> uint16_t Machine;
> >>--
> >>2.20.1
> >>
> >
>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
