[PATCH] powerpc: add powerpc64le support

[PATCH] powerpc: add powerpc64le support

[Cody P Schafer]

# HG changeset patch
# User Cody P Schafer <dev@codyps.com>
# Date 1399801832 25200
#      Sun May 11 02:50:32 2014 -0700
# Node ID 64bfbbced9dd8f62e0d64936c54b46653a87dd12
# Parent  53aa6fe55269fd11e7cee8ad6d13cb5ee0a7c3de
powerpc: add powerpc64le support

Technically, I don't forbid powerpcle support either, but I'm not sure that
there is any library/compiler support for that at the moment (though the hw
technically makes it possible).

powerpc64le needs glibc 2.19 and gcc 4.9. I haven't looked into the support
tools, but at least gdb 7.5 is too old (7.7.1 definitely has support).

Also make powerpc64 non-experimental. It's practically old at this point.

Signed-of-by: Cody P Schafer <dev@codyps.com>

diff --git a/config/arch/powerpc.in b/config/arch/powerpc.in
--- a/config/arch/powerpc.in
+++ b/config/arch/powerpc.in
@@ -1,8 +1,10 @@
 # powerpc specific configuration file
 
 ## select ARCH_SUPPORTS_32
-## select ARCH_SUPPORTS_64 if EXPERIMENTAL
+## select ARCH_SUPPORTS_64
 ## select ARCH_DEFAULT_32
+## select ARCH_SUPPORTS_BOTH_ENDIAN
+## select ARCH_DEFAULT_BE
 ## select ARCH_USE_MMU
 ## select ARCH_SUPPORTS_WITH_ABI
 ## select ARCH_SUPPORTS_WITH_CPU
diff --git a/scripts/build/arch/powerpc.sh b/scripts/build/arch/powerpc.sh
--- a/scripts/build/arch/powerpc.sh
+++ b/scripts/build/arch/powerpc.sh
@@ -3,9 +3,19 @@
 CT_DoArchTupleValues () {
     # The architecture part of the tuple, override only for 64-bit
     if [ "${CT_ARCH_64}" = "y" ]; then
-        CT_TARGET_ARCH="powerpc64${CT_ARCH_SUFFIX}"
+        _64="64"
+    else
+        _64=""
     fi
 
+    if [ "${CT_ARCH_ENDIAN}" = big ] ; then
+        _le=""
+    else
+        _le="le"
+    fi
+
+    CT_TARGET_ARCH="powerpc${_64}${_le}${CT_ARCH_SUFFIX}"
+
     # Only override values when ABI is not the default
     case "${CT_ARCH_powerpc_ABI}" in
         eabi)

--
For unsubscribe information see http://sourceware.org/lists.html#faq

Re: [PATCH] powerpc: add powerpc64le support

[Yann E. MORIN]

Cody, All,

On 2014-05-11 02:52 -0700, Cody P Schafer spake thusly:
> # HG changeset patch
> # User Cody P Schafer <dev@codyps.com>
> # Date 1399801832 25200
> #      Sun May 11 02:50:32 2014 -0700
> # Node ID 64bfbbced9dd8f62e0d64936c54b46653a87dd12
> # Parent  53aa6fe55269fd11e7cee8ad6d13cb5ee0a7c3de
> powerpc: add powerpc64le support
> 
> Technically, I don't forbid powerpcle support either, but I'm not sure that
> there is any library/compiler support for that at the moment (though the hw
> technically makes it possible).
> 
> powerpc64le needs glibc 2.19 and gcc 4.9. I haven't looked into the support
> tools, but at least gdb 7.5 is too old (7.7.1 definitely has support).
> 
> Also make powerpc64 non-experimental. It's practically old at this point.

That's not PPC64 that is experimental; it is the support thereof in
crosstool-NG that is experimental. If it works for you, I am happy to
remove it, of course! :-)

> Signed-of-by: Cody P Schafer <dev@codyps.com>
> 
> diff --git a/config/arch/powerpc.in b/config/arch/powerpc.in
> --- a/config/arch/powerpc.in
> +++ b/config/arch/powerpc.in
> @@ -1,8 +1,10 @@
>  # powerpc specific configuration file
>  
>  ## select ARCH_SUPPORTS_32
> -## select ARCH_SUPPORTS_64 if EXPERIMENTAL
> +## select ARCH_SUPPORTS_64
>  ## select ARCH_DEFAULT_32
> +## select ARCH_SUPPORTS_BOTH_ENDIAN
> +## select ARCH_DEFAULT_BE
>  ## select ARCH_USE_MMU
>  ## select ARCH_SUPPORTS_WITH_ABI
>  ## select ARCH_SUPPORTS_WITH_CPU
> diff --git a/scripts/build/arch/powerpc.sh b/scripts/build/arch/powerpc.sh
> --- a/scripts/build/arch/powerpc.sh
> +++ b/scripts/build/arch/powerpc.sh
> @@ -3,9 +3,19 @@
>  CT_DoArchTupleValues () {
>      # The architecture part of the tuple, override only for 64-bit
>      if [ "${CT_ARCH_64}" = "y" ]; then
> -        CT_TARGET_ARCH="powerpc64${CT_ARCH_SUFFIX}"
> +        _64="64"
> +    else
> +        _64=""

You forgot to declare that variable. Fixed.

>      fi
>  
> +    if [ "${CT_ARCH_ENDIAN}" = big ] ; then
> +        _le=""
> +    else
> +        _le="le"

We already have ${target_endian_el} for that. We're missing
${target_endian_le} which I've added and used. Fixed.

Thanks!

Regards,
Yann E. MORIN.

> +    fi
> +
> +    CT_TARGET_ARCH="powerpc${_64}${_le}${CT_ARCH_SUFFIX}"
> +
>      # Only override values when ABI is not the default
>      case "${CT_ARCH_powerpc_ABI}" in
>          eabi)
> 
> --
> For unsubscribe information see http://sourceware.org/lists.html#faq
> 

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

--
For unsubscribe information see http://sourceware.org/lists.html#faq

arch/powerpc: add powerpc64le support

[Yann E. MORIN]

Cody, All,

Your patch:
    arch/powerpc: add powerpc64le support

has been applied as: #eb13867a034c
    http://crosstool-ng.org/hg/crosstool-ng/rev/eb13867a034c

Thank you!

Regards,
Yann E. MORIN.



--
For unsubscribe information see http://sourceware.org/lists.html#faq