Re: [U-Boot] [PATCH] zynq: Fix elf header generation

2014-02-04 Thread Michal Simek
On 02/04/2014 09:35 PM, Tom Rini wrote: > On Tue, Feb 04, 2014 at 04:59:45PM +0100, Michal Simek wrote: > >> This patch is here because of: >> "arm: keep all sections in ELF file" >> (sha1: 47ed5dd031d7d2c587e6afd386e79ccec1a1b7f7) >> >> Our tools expect to have elf with only LOAD header. >> Witho

Re: [U-Boot] [PATCH] zynq: Fix elf header generation

2014-02-04 Thread Michal Simek
On 02/05/2014 04:00 AM, Masahiro Yamada wrote: > Hello Michal, > > >> @@ -224,4 +226,5 @@ >> #define CONFIG_CMD_MII >> #define CONFIG_CMD_TFTPPUT >> >> + >> #endif /* __CONFIG_ZYNQ_COMMON_H */ >> -- >> 1.8.2.3 >> > > > Please do not double empty lines. Sure. Just sent v2. Thanks, Michal

Re: [U-Boot] [PATCH] zynq: Fix elf header generation

2014-02-04 Thread Masahiro Yamada
Hello Michal, > @@ -224,4 +226,5 @@ > #define CONFIG_CMD_MII > #define CONFIG_CMD_TFTPPUT > > + > #endif /* __CONFIG_ZYNQ_COMMON_H */ > -- > 1.8.2.3 > Please do not double empty lines. Best Regards Masahiro Yamada ___ U-Boot mailing list U-Boo

Re: [U-Boot] [PATCH] zynq: Fix elf header generation

2014-02-04 Thread Tom Rini
On Tue, Feb 04, 2014 at 04:59:45PM +0100, Michal Simek wrote: > This patch is here because of: > "arm: keep all sections in ELF file" > (sha1: 47ed5dd031d7d2c587e6afd386e79ccec1a1b7f7) > > Our tools expect to have elf with only LOAD header. > Without this fix also PHDR, INTERP and DYNAMIC headers

[U-Boot] [PATCH] zynq: Fix elf header generation

2014-02-04 Thread Michal Simek
This patch is here because of: "arm: keep all sections in ELF file" (sha1: 47ed5dd031d7d2c587e6afd386e79ccec1a1b7f7) Our tools expect to have elf with only LOAD header. Without this fix also PHDR, INTERP and DYNAMIC headers are available in ELF. Signed-off-by: Michal Simek --- Albert: We have d