On Thu, Sep 29, 2005 at 10:56:33AM -0400, Chuck Meade wrote:
You removed any logical path by which "-EB" would be added to the flags. Your addition of the new flag variables is fine if it works, but it should not break the big endian code. Why don't you rework your patch to not break big endian support?
I don't think I'm breaking big endian support with my patch since if you have the ld command:
OUTPUT_FORMAT("elf32-tradbigmips", "elf32-tradbigmips", "elf32-tradlittlemips")
you say to the linker that your default endian is "big", please see:
info -f /usr/local/mipsel/info/ld.info -n "Format Commands"
So if you do not specify "-EB" nor "-EL" the linker will use default (big endian in the above example).
The following patch also fixes examples compilations:
Index: examples/mips.lds =================================================================== RCS file: /home/develop/cvs_private/uboot-mips-exadron/examples/mips.lds,v retrieving revision 1.1.1.1 retrieving revision 1.2 diff -u -r1.1.1.1 -r1.2 --- a/examples/mips.lds 16 Sep 2005 12:14:21 -0000 1.1.1.1 +++ b/examples/mips.lds 16 Sep 2005 15:05:08 -0000 1.2 @@ -24,7 +24,7 @@ /* OUTPUT_FORMAT("elf32-bigmips", "elf32-bigmips", "elf32-bigmips") */ -OUTPUT_FORMAT("elf32-tradbigmips", "elf32-tradbigmips", "elf32-tradbigmips") +OUTPUT_FORMAT("elf32-tradbigmips", "elf32-tradbigmips", "elf32-tradlittlemips") OUTPUT_ARCH(mips) SECTIONS {
Ciao,
Rodolfo