[PATCH] [MIPS] Add mips*el-rtems stubs

[PATCH] [MIPS] Add mips*el-rtems stubs

[Jia Liu]

Hi all,

I'm working on several MIPS64-based processors, RTEMS is a part of my SDK.
When I build a mips64el toolchain for RTEMS, I found that mips*el support
has beed removed, so, I try to take it back.
This patch is tested, and the RTEMS patch will summitted by a friend at ICT.

Regards,
Jia.

ChangeLog
2012-10-28  Jia Liu  <proljc@gmail.com>

        bfd/
        * config.bfd: Add mips*el-*-rtems* support.

        gas/
        * configure.tgt: Change mips-*-elf* | mips-*-rtems*
        into mips*-*-elf* | mips*-*-rtems*.

        ld/
        * configure.tgt: Add mips*el-*-rtems*.


Signed-off-by: Jia Liu <proljc@gmail.com>
---
 bfd/config.bfd    |    3 +--
 gas/configure.tgt |    2 +-
 ld/configure.tgt  |    1 +
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/bfd/config.bfd b/bfd/config.bfd
index 7def642..1272693 100644
--- a/bfd/config.bfd
+++ b/bfd/config.bfd
@@ -57,7 +57,6 @@ case $targ in
  hppa*-*-rtems* | \
  *-go32-rtems* | \
  i[3-7]86*-*-rtemscoff* | \
- mips*el-*-rtems* | \
  powerpcle-*-rtems* | \
  sparc*-*-rtemsaout* | \
  null)
@@ -989,7 +988,7 @@ case "${targ}" in
     targ_selvecs="bfd_elf32_tradlittlemips_vec bfd_elf32_ntradbigmips_vec bfd_elf32_ntradlittlemips_vec bfd_elf64_tradbigmips_vec bfd_elf64_tradlittlemips_vec"
     want64=true
     ;;
-  mips*el-*-elf* | mips*el-*-vxworks* | mips*-*-chorus*)
+  mips*el-*-elf* | mips*el-*-rtems* | mips*el-*-vxworks* | mips*-*-chorus*)
     targ_defvec=bfd_elf32_littlemips_vec
     targ_selvecs="bfd_elf32_bigmips_vec bfd_elf64_bigmips_vec bfd_elf64_littlemips_vec"
     ;;
diff --git a/gas/configure.tgt b/gas/configure.tgt
index ff4cb3f..9bc8bee 100644
--- a/gas/configure.tgt
+++ b/gas/configure.tgt
@@ -310,7 +310,7 @@ case ${generic_target} in
   mips-*-sysv4*MP* | mips-*-gnu*)	fmt=elf em=tmips ;;
   mips*-sde-elf* | mips*-mti-elf*)	fmt=elf em=tmips ;;
   mips-*-sysv*)				fmt=ecoff ;;
-  mips-*-elf* | mips-*-rtems*)		fmt=elf ;;
+  mips*-*-elf* | mips*-*-rtems*)	fmt=elf ;;
   mips-*-netbsd*)			fmt=elf em=tmips ;;
   mips-*-openbsd*)			fmt=elf em=tmips ;;
 
diff --git a/ld/configure.tgt b/ld/configure.tgt
index 8d4c349..bdf3fa0 100644
--- a/ld/configure.tgt
+++ b/ld/configure.tgt
@@ -429,6 +429,7 @@ mips*-sde-elf* | mips*-mti-elf*)
 			targ_extra_emuls="elf32ltsmip elf32btsmipn32 elf64btsmip elf32ltsmipn32 elf64ltsmip" ;;
 mips*el-*-elf*)		targ_emul=elf32elmip ;;
 mips*-*-elf*)		targ_emul=elf32ebmip ;;
+mips*el-*-rtems*)	targ_emul=elf32elmip ;;
 mips*-*-rtems*)		targ_emul=elf32ebmip ;;
 mips*el-*-vxworks*)	targ_emul=elf32elmipvxworks
 			targ_extra_emuls="elf32ebmipvxworks" ;;
-- 
1.7.10.2 (Apple Git-33)

Re: [PATCH] [MIPS] Add mips*el-rtems stubs

[Richard Sandiford]

Jia Liu <proljc@gmail.com> writes:
> Hi all,
>
> I'm working on several MIPS64-based processors, RTEMS is a part of my SDK.
> When I build a mips64el toolchain for RTEMS, I found that mips*el support
> has beed removed, so, I try to take it back.
> This patch is tested, and the RTEMS patch will summitted by a friend at ICT.

This looks good in principle.  It was Ralf (cc:ed) who removed the port
originally, so once he's happy that the port has indeed been resurrected
from an RTEMS POV, I'll go ahead and apply it.

Richard

Re: [PATCH] [MIPS] Add mips*el-rtems stubs

[Ralf Corsepius]

On 10/30/2012 12:21 PM, Richard Sandiford wrote:
> Jia Liu <proljc@gmail.com> writes:
>> Hi all,
>>
>> I'm working on several MIPS64-based processors, RTEMS is a part of my SDK.
>> When I build a mips64el toolchain for RTEMS, I found that mips*el support
>> has beed removed, so, I try to take it back.
 >>
>> This patch is tested, and the RTEMS patch will summitted by a friend at ICT.
>
> This looks good in principle.  It was Ralf (cc:ed) who removed the port
> originally,

I don't recall me having done so ;) It probably was removed, because 
there currently is no mips*el port in RTEMS

> so once he's happy that the port has indeed been resurrected
> from an RTEMS POV,
Well, I don't have any problem in a mips64el target being added to RTEMS 
and its toolchains, but so far, this does not apply.

> I'll go ahead and apply it.
I am uncertain on how to proceed. Without the missing other components 
(RTEMS, newlib, gcc (+ gdb)) also being submitted, adding a mips64el to 
binutils doesn't make much sense.

Ralf

Re: [PATCH] [MIPS] Add mips*el-rtems stubs

[Devin]

Hi,all.
According to  Liu's Changelog, I change same content in 1,/bfd/config.bfd 
2,/gas/configure.tgt  3/ld/configure.tgt  ,just like what Liu changed.
and then , I have not changed anything. I build the toolchain like
"mipsel-rtems-gcc",but when I configure RTEMS. It does not appear
information like :
/target architecture: mips.
available BSPs:  hurricane.
'gmake all' will build the following BSPs: hurricane.
other BSPs can be built with 'gmake RTEMS_BSP="bsp1 bsp2 ..."'/
my configure command  is 
"../rtems-4.10.2/configure  --target=mipsel-rtems --quiet --disable-posix
--disable-networking --disable-cxx --disable-itron
--enable-rtemsbsp="hurricane" --prefix=/opt/mipsel "
what happend about this?
thanks a lot !
Devin




--
View this message in context: http://sourceware-org.1504.n7.nabble.com/PATCH-MIPS-Add-mips-el-rtems-stubs-tp211233p217630.html
Sent from the Sourceware - binutils list mailing list archive at Nabble.com.

Re: [PATCH] [MIPS] Add mips*el-rtems stubs

[Ralf Corsepius]

On 12/31/2012 10:47 AM, Devin wrote:
> Hi,all.
> According to  Liu's Changelog, I change same content in 1,/bfd/config.bfd
> 2,/gas/configure.tgt  3/ld/configure.tgt  ,just like what Liu changed.
> and then , I have not changed anything. I build the toolchain like
> "mipsel-rtems-gcc",but when I configure RTEMS. It does not appear
> information like :
> /target architecture: mips.
> available BSPs:  hurricane.
> 'gmake all' will build the following BSPs: hurricane.
> other BSPs can be built with 'gmake RTEMS_BSP="bsp1 bsp2 ..."'/
> my configure command  is
> "../rtems-4.10.2/configure  --target=mipsel-rtems --quiet --disable-posix
> --disable-networking --disable-cxx --disable-itron
> --enable-rtemsbsp="hurricane" --prefix=/opt/mipsel "
> what happend about this?
> thanks a lot !

This question of off-topic for this list.

To answer your question: mipsel-rtems is not a valid target tuple.

a) The RTEMS source tree doesn't support a "mipsel" cpu.
It expects little endian mips as a multilib-variant of the "mips" cpu.

b) The OS name is supposed to carry a version number (-rtems4.10, 
rtems4.11 etc.). Non-versioned OS names are not supported.

Ralf

Re: [PATCH] [MIPS] Add mips*el-rtems stubs

[Devin]

I am sorry about that my questions don't relate to this topic list.

It support little endian through using multilib-bariant form。and then ，I
find one folder named “el” in
“/INSTALL_PATH/lib/gcc/mips-rtems/4.4.6/mip3//” , saddly, don't find folder
"el" in "/INSTALL_PATH/lib/gcc/mips-rtems/4.4.6/mip32//".
So , when I configure RTEMS adding parameter "-mips32" and "-EL" in the same
time .Some errors occur.

*error*:
//mips-rtems4.10/lib/crt0.o: compiled for a big endian system and target is
little endian
/opt/rtems/lib/gcc/mips-rtems4.10/4.4.6/../../../../mips-rtems4.10/bin/ld:
/opt/rtems/lib/gcc/mips-rtems4.10/4.4.6/../../../../mips-rtems4.10/lib/crt0.o:
endianness incompatible with that of the selected emulation/

I think I should change something in binutils/gcc to make sure it can
compile RTEMS successfully.
Thanks.
Devin



--
View this message in context: http://sourceware-org.1504.n7.nabble.com/PATCH-MIPS-Add-mips-el-rtems-stubs-tp211233p217967.html
Sent from the Sourceware - binutils list mailing list archive at Nabble.com.

Re: [PATCH] [MIPS] Add mips*el-rtems stubs

[Jia Liu]

Hi Joel, Ralf

On Tue, Oct 30, 2012 at 8:24 PM, Joel Sherrill
<Joel.Sherrill@oarcorp.com> wrote:
> I agree with Ralf.
>
> Jai. Post the entire tool chain patch set and RTEMS changes to the rtems-devel mailing list. For something like this, we just want to make sure all the bits are there and line up.
>

I've sent the RTEMS patch to rtems-devel@rtems.org, but, somehow, it didn't
appeared in the rtems-devel Archives, what's wrong with me or my friend?
May you have a check? The mail subject is  [PATCH] mips*el support .

> Then once that review is done, it can go upstream.
>
> --joel
>
> Ralf Corsepius <ralf.corsepius@googlemail.com> wrote:
>
>
> On 10/30/2012 12:21 PM, Richard Sandiford wrote:
>> Jia Liu <proljc@gmail.com> writes:
>>> Hi all,
>>>
>>> I'm working on several MIPS64-based processors, RTEMS is a part of my SDK.
>>> When I build a mips64el toolchain for RTEMS, I found that mips*el support
>>> has beed removed, so, I try to take it back.
>  >>
>>> This patch is tested, and the RTEMS patch will summitted by a friend at ICT.
>>
>> This looks good in principle.  It was Ralf (cc:ed) who removed the port
>> originally,
>
> I don't recall me having done so ;) It probably was removed, because
> there currently is no mips*el port in RTEMS
>
>> so once he's happy that the port has indeed been resurrected
>> from an RTEMS POV,
> Well, I don't have any problem in a mips64el target being added to RTEMS
> and its toolchains, but so far, this does not apply.
>
>> I'll go ahead and apply it.
> I am uncertain on how to proceed. Without the missing other components
> (RTEMS, newlib, gcc (+ gdb)) also being submitted, adding a mips64el to
> binutils doesn't make much sense.
>
> Ralf
>

Regards,
Jia.

Re: [PATCH] [MIPS] Add mips*el-rtems stubs

[Jia Liu]

Hi Joel Ralf and Richard,

On Tue, Oct 30, 2012 at 8:24 PM, Joel Sherrill
<Joel.Sherrill@oarcorp.com> wrote:
> I agree with Ralf.
>
> Jai. Post the entire tool chain patch set and RTEMS changes to the rtems-devel mailing list. For something like this, we just want to make sure all the bits are there and line up.
>
> Then once that review is done, it can go upstream.
>

This is a little difficult to me, this one the entire toolchain patch,
this binutils one is all to change.

but the RTEMS is not what I can handle, I know almost nothing about
RTOS/kernel, so I asked a friend to working on it, he tested my
toolchain is OK, but RTEMS patch set is not completed by him, so...

> --joel
>
> Ralf Corsepius <ralf.corsepius@googlemail.com> wrote:
>
>
> On 10/30/2012 12:21 PM, Richard Sandiford wrote:
>> Jia Liu <proljc@gmail.com> writes:
>>> Hi all,
>>>
>>> I'm working on several MIPS64-based processors, RTEMS is a part of my SDK.
>>> When I build a mips64el toolchain for RTEMS, I found that mips*el support
>>> has beed removed, so, I try to take it back.
>  >>
>>> This patch is tested, and the RTEMS patch will summitted by a friend at ICT.
>>
>> This looks good in principle.  It was Ralf (cc:ed) who removed the port
>> originally,
>
> I don't recall me having done so ;) It probably was removed, because
> there currently is no mips*el port in RTEMS
>
>> so once he's happy that the port has indeed been resurrected
>> from an RTEMS POV,
> Well, I don't have any problem in a mips64el target being added to RTEMS
> and its toolchains, but so far, this does not apply.
>
>> I'll go ahead and apply it.
> I am uncertain on how to proceed. Without the missing other components
> (RTEMS, newlib, gcc (+ gdb)) also being submitted, adding a mips64el to
> binutils doesn't make much sense.
>
> Ralf
>

Regards,
Jia.

Re: [PATCH] [MIPS] Add mips*el-rtems stubs

[Joel Sherrill]

I agree with Ralf.

Jai. Post the entire tool chain patch set and RTEMS changes to the rtems-devel mailing list. For something like this, we just want to make sure all the bits are there and line up.

Then once that review is done, it can go upstream.

--joel

Ralf Corsepius <ralf.corsepius@googlemail.com> wrote:


On 10/30/2012 12:21 PM, Richard Sandiford wrote:
> Jia Liu <proljc@gmail.com> writes:
>> Hi all,
>>
>> I'm working on several MIPS64-based processors, RTEMS is a part of my SDK.
>> When I build a mips64el toolchain for RTEMS, I found that mips*el support
>> has beed removed, so, I try to take it back.
 >>
>> This patch is tested, and the RTEMS patch will summitted by a friend at ICT.
>
> This looks good in principle.  It was Ralf (cc:ed) who removed the port
> originally,

I don't recall me having done so ;) It probably was removed, because
there currently is no mips*el port in RTEMS

> so once he's happy that the port has indeed been resurrected
> from an RTEMS POV,
Well, I don't have any problem in a mips64el target being added to RTEMS
and its toolchains, but so far, this does not apply.

> I'll go ahead and apply it.
I am uncertain on how to proceed. Without the missing other components
(RTEMS, newlib, gcc (+ gdb)) also being submitted, adding a mips64el to
binutils doesn't make much sense.

Ralf