Re: [U-Boot-Users] [PATCH v2][for 1.3.4] mpc85xx: workaround old binutils bug

2008-08-10 Thread Wolfgang Denk
Dear Kumar Gala,

In message <[EMAIL PROTECTED]> you wrote:
> The recent change to move the .bss outside of the image gives older
> binutils (ld from eldk4.1/binutils-2.16) some headache:
> 
> ppc_85xx-ld: u-boot: Not enough room for program headers (allocated 3, need 4)
> ppc_85xx-ld: final link failed: Bad value
> 
> We workaround it by being explicit about the program headers and not
> assigning the .bss to a program header.
> 
> Signed-off-by: Kumar Gala <[EMAIL PROTECTED]>
> ---
> 
> Fixes warning with older binutils
> 
>  board/freescale/mpc8540ads/u-boot.lds |   16 +++-
>  board/freescale/mpc8541cds/u-boot.lds |   16 +++-
>  board/freescale/mpc8544ds/u-boot.lds  |   16 +++-
>  board/freescale/mpc8548cds/u-boot.lds |   16 +++-
>  board/freescale/mpc8555cds/u-boot.lds |   16 +++-
>  board/freescale/mpc8560ads/u-boot.lds |   16 +++-
>  board/freescale/mpc8568mds/u-boot.lds |   16 +++-
>  7 files changed, 77 insertions(+), 35 deletions(-)

Applied, thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED]
I can't understand it. I can't even understand  the  people  who  can
understand it.- Queen Juliana of the Netherlands.

-
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [PATCH v2][for 1.3.4] mpc85xx: workaround old binutils bug

2008-08-07 Thread Rafal Jaworowski
Kumar Gala wrote:
> The recent change to move the .bss outside of the image gives older
> binutils (ld from eldk4.1/binutils-2.16) some headache:
> 
> ppc_85xx-ld: u-boot: Not enough room for program headers (allocated 3, need 4)
> ppc_85xx-ld: final link failed: Bad value
> 
> We workaround it by being explicit about the program headers and not
> assigning the .bss to a program header.
> 
> Signed-off-by: Kumar Gala <[EMAIL PROTECTED]>
> ---
> 
> Fixes warning with older binutils

Looks fine now, thanks a lot!

Rafal

-
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] [PATCH v2][for 1.3.4] mpc85xx: workaround old binutils bug

2008-08-07 Thread Kumar Gala
The recent change to move the .bss outside of the image gives older
binutils (ld from eldk4.1/binutils-2.16) some headache:

ppc_85xx-ld: u-boot: Not enough room for program headers (allocated 3, need 4)
ppc_85xx-ld: final link failed: Bad value

We workaround it by being explicit about the program headers and not
assigning the .bss to a program header.

Signed-off-by: Kumar Gala <[EMAIL PROTECTED]>
---

Fixes warning with older binutils

 board/freescale/mpc8540ads/u-boot.lds |   16 +++-
 board/freescale/mpc8541cds/u-boot.lds |   16 +++-
 board/freescale/mpc8544ds/u-boot.lds  |   16 +++-
 board/freescale/mpc8548cds/u-boot.lds |   16 +++-
 board/freescale/mpc8555cds/u-boot.lds |   16 +++-
 board/freescale/mpc8560ads/u-boot.lds |   16 +++-
 board/freescale/mpc8568mds/u-boot.lds |   16 +++-
 7 files changed, 77 insertions(+), 35 deletions(-)

diff --git a/board/freescale/mpc8540ads/u-boot.lds 
b/board/freescale/mpc8540ads/u-boot.lds
index 0e4f5a2..515d320 100644
--- a/board/freescale/mpc8540ads/u-boot.lds
+++ b/board/freescale/mpc8540ads/u-boot.lds
@@ -26,6 +26,12 @@
 OUTPUT_ARCH(powerpc)
 /* Do we need any of these for elf?
__DYNAMIC = 0;*/
+PHDRS
+{
+  text PT_LOAD;
+  bss PT_LOAD;
+}
+
 SECTIONS
 {
   /* Read-only sections, merged into text segment: */
@@ -57,7 +63,7 @@ SECTIONS
 *(.text)
 *(.fixup)
 *(.got1)
-   }
+   } :text
 _etext = .;
 PROVIDE (etext = .);
 .rodata:
@@ -66,7 +72,7 @@ SECTIONS
 *(.rodata1)
 *(.rodata.str1.4)
 *(.eh_frame)
-  }
+  } :text
   .fini  : { *(.fini)} =0
   .ctors : { *(.ctors)   }
   .dtors : { *(.dtors)   }
@@ -118,12 +124,12 @@ SECTIONS
   .bootpg ADDR(.text) + 0x7f000 :
   {
 cpu/mpc85xx/start.o(.bootpg)
-  } = 0x
+  } :text = 0x

   .resetvec ADDR(.text) + 0x7fffc :
   {
 *(.resetvec)
-  } = 0x
+  } :text = 0x

   . = ADDR(.text) + 0x8;

@@ -134,7 +140,7 @@ SECTIONS
*(.dynbss)
*(.bss)
*(COMMON)
-  }
+  } :bss

   . = ALIGN(4);
   _end = . ;
diff --git a/board/freescale/mpc8541cds/u-boot.lds 
b/board/freescale/mpc8541cds/u-boot.lds
index 1c583de..d728d8b 100644
--- a/board/freescale/mpc8541cds/u-boot.lds
+++ b/board/freescale/mpc8541cds/u-boot.lds
@@ -23,6 +23,12 @@
 OUTPUT_ARCH(powerpc)
 /* Do we need any of these for elf?
__DYNAMIC = 0;*/
+PHDRS
+{
+  text PT_LOAD;
+  bss PT_LOAD;
+}
+
 SECTIONS
 {
   /* Read-only sections, merged into text segment: */
@@ -54,7 +60,7 @@ SECTIONS
 *(.text)
 *(.fixup)
 *(.got1)
-   }
+   } :text
 _etext = .;
 PROVIDE (etext = .);
 .rodata:
@@ -63,7 +69,7 @@ SECTIONS
 *(.rodata1)
 *(.rodata.str1.4)
 *(.eh_frame)
-  }
+  } :text
   .fini  : { *(.fini)} =0
   .ctors : { *(.ctors)   }
   .dtors : { *(.dtors)   }
@@ -115,12 +121,12 @@ SECTIONS
   .bootpg ADDR(.text) + 0x7f000 :
   {
 cpu/mpc85xx/start.o(.bootpg)
-  } = 0x
+  } :text = 0x

   .resetvec ADDR(.text) + 0x7fffc :
   {
 *(.resetvec)
-  } = 0x
+  } :text = 0x

   . = ADDR(.text) + 0x8;

@@ -131,7 +137,7 @@ SECTIONS
*(.dynbss)
*(.bss)
*(COMMON)
-  }
+  } :bss

   . = ALIGN(4);
   _end = . ;
diff --git a/board/freescale/mpc8544ds/u-boot.lds 
b/board/freescale/mpc8544ds/u-boot.lds
index 500e647..a05ece5 100644
--- a/board/freescale/mpc8544ds/u-boot.lds
+++ b/board/freescale/mpc8544ds/u-boot.lds
@@ -23,6 +23,12 @@
 OUTPUT_ARCH(powerpc)
 /* Do we need any of these for elf?
__DYNAMIC = 0;*/
+PHDRS
+{
+  text PT_LOAD;
+  bss PT_LOAD;
+}
+
 SECTIONS
 {
   /* Read-only sections, merged into text segment: */
@@ -54,7 +60,7 @@ SECTIONS
 *(.text)
 *(.fixup)
 *(.got1)
-   }
+   } :text
 _etext = .;
 PROVIDE (etext = .);
 .rodata:
@@ -63,7 +69,7 @@ SECTIONS
 *(.rodata1)
 *(.rodata.str1.4)
 *(.eh_frame)
-  }
+  } :text
   .fini  : { *(.fini)} =0
   .ctors : { *(.ctors)   }
   .dtors : { *(.dtors)   }
@@ -115,12 +121,12 @@ SECTIONS
   .bootpg ADDR(.text) + 0x7f000 :
   {
 cpu/mpc85xx/start.o(.bootpg)
-  } = 0x
+  } :text = 0x

   .resetvec ADDR(.text) + 0x7fffc :
   {
 *(.resetvec)
-  } = 0x
+  } :text = 0x

   . = ADDR(.text) + 0x8;

@@ -131,7 +137,7 @@ SECTIONS
*(.dynbss)
*(.bss)
*(COMMON)
-  }
+  } :bss

   . = ALIGN(4);
   _end = . ;
diff --git a/board/freescale/mpc8548cds/u-boot.lds 
b/board/freescale/mpc8548cds/u-boot.lds
index 6b93395..d4a2f72 100644
--- a/board/freescale/mpc8548cds/u-boot.lds
+++ b/board/freescale/mpc8548cds/u-boot.lds
@@ -23,6 +23,12 @@
 OUTPUT_ARCH(powerpc)
 /* Do we need any of these for elf?
__DYNAMIC = 0;*/
+PHDRS
+{
+  text PT_LOAD;
+  bss PT_LOAD;
+}
+
 SECTIONS
 {
   /* Read-only sections, merged into text segment: */
@@ -54,7 +60,7 @@ SECTIONS
 *(.text)
 *(.fixup)
 *(.got1)
-   }
+   } :text
 _etext = .;
 PROVIDE (etext = .);