diff mbox

[U-Boot,v2,2/7] moveconfig: Add a mapping for the arc cross-compiler

Message ID 1431364998-2905-2-git-send-email-joe.hershberger@ni.com
State Superseded
Delegated to: Tom Rini
Headers show

Commit Message

Joe Hershberger May 11, 2015, 5:23 p.m. UTC
The arc architecture is supported by U-Boot, so add a mapping here
for it as well.

Signed-off-by: Joe Hershberger <joe.hershberger@ni.com>
---

Changes in v2: None

 tools/moveconfig.py | 1 +
 1 file changed, 1 insertion(+)

Comments

Masahiro Yamada May 13, 2015, 2:14 a.m. UTC | #1
Hi Joe.

2015-05-12 2:23 GMT+09:00 Joe Hershberger <joe.hershberger@ni.com>:
> The arc architecture is supported by U-Boot, so add a mapping here
> for it as well.
>
> Signed-off-by: Joe Hershberger <joe.hershberger@ni.com>
> ---
>
> Changes in v2: None
>
>  tools/moveconfig.py | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/tools/moveconfig.py b/tools/moveconfig.py
> index 30dc4f6..c81f32c 100755
> --- a/tools/moveconfig.py
> +++ b/tools/moveconfig.py
> @@ -26,6 +26,7 @@ SLEEP_TIME=0.03
>
>  CROSS_COMPILE = {
>      'aarch64': 'aarch64-linux-',
> +    'arc': 'arc-linux-',
>      'arm': 'arm-unknown-linux-gnueabi-',
>      'avr32': 'avr32-linux-',
>      'blackfin': 'bfin-elf-',


IIRC, I intentionally did not add it
because the default ARC CROSS_COMPILE is defined by arch/arc/config.mk
(Looks like ARC wants to choose little/big endian variants
depending on the configuration, although it might not be an issue for
this tool.)

Better to override it?
Joe Hershberger May 13, 2015, 3:34 p.m. UTC | #2
Hi Masahiro-san,

On Tue, May 12, 2015 at 9:14 PM, Masahiro Yamada
<yamada.masahiro@socionext.com> wrote:
> Hi Joe.
>
> 2015-05-12 2:23 GMT+09:00 Joe Hershberger <joe.hershberger@ni.com>:
>> The arc architecture is supported by U-Boot, so add a mapping here
>> for it as well.
>>
>> Signed-off-by: Joe Hershberger <joe.hershberger@ni.com>
>> ---
>>
>> Changes in v2: None
>>
>>  tools/moveconfig.py | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/tools/moveconfig.py b/tools/moveconfig.py
>> index 30dc4f6..c81f32c 100755
>> --- a/tools/moveconfig.py
>> +++ b/tools/moveconfig.py
>> @@ -26,6 +26,7 @@ SLEEP_TIME=0.03
>>
>>  CROSS_COMPILE = {
>>      'aarch64': 'aarch64-linux-',
>> +    'arc': 'arc-linux-',
>>      'arm': 'arm-unknown-linux-gnueabi-',
>>      'avr32': 'avr32-linux-',
>>      'blackfin': 'bfin-elf-',
>
>
> IIRC, I intentionally did not add it
> because the default ARC CROSS_COMPILE is defined by arch/arc/config.mk
> (Looks like ARC wants to choose little/big endian variants
> depending on the configuration, although it might not be an issue for
> this tool.)

Thanks for pointing that out.

> Better to override it?

I am successfully building, at least to get the config output, without
switching to a big-endian compiler. I can build the other compiler too
if you prefer not to map this.

Cheers,
-Joe
diff mbox

Patch

diff --git a/tools/moveconfig.py b/tools/moveconfig.py
index 30dc4f6..c81f32c 100755
--- a/tools/moveconfig.py
+++ b/tools/moveconfig.py
@@ -26,6 +26,7 @@  SLEEP_TIME=0.03
 
 CROSS_COMPILE = {
     'aarch64': 'aarch64-linux-',
+    'arc': 'arc-linux-',
     'arm': 'arm-unknown-linux-gnueabi-',
     'avr32': 'avr32-linux-',
     'blackfin': 'bfin-elf-',