[PATCH] h8300: Don't set CROSS_COMPILE unconditionally

[PATCH] h8300: Don't set CROSS_COMPILE unconditionally

[Javier Martinez Canillas]

The h8300 architecture Makefile sets the CROSS_COMPILE variable to
"h8300-unknown-linux-" unconditionally but doing that prevents the
variable to be used if a toolchain with a different binary name is
installed in the machine.

So only set CROSS_COMPILE to that default if the variable has not
been set by the user before.

Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>

---

Hello,

After $SUBJECT, I'm able to build using the toolchain installed by
my Linux distribution but the linker gives me an error about an
unrecognised emulation mode:

h8300-linux-gnu-ld: unrecognised emulation mode: h8300helf_linux

I see that h8300helf_linux and 8300self_linux are hardcoded in the
Makefile as well and these are not listed as supported emulations:

Supported emulations: h8300elf h8300helf h8300self h8300hnelf h8300snelf h8300sxelf h8300sxnelf

I tried changing -mh8300helf_linux for -mh8300helf that is listed
as a supported emulation but then I got the following error:

h8300-linux-gnu-ld: cannot open linker script file ldscripts/h8300helf.xr: No such file or directory

Best regards,
Javier

 arch/h8300/Makefile | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/h8300/Makefile b/arch/h8300/Makefile
index 0d2d96e52d9f..e1c02ca230cb 100644
--- a/arch/h8300/Makefile
+++ b/arch/h8300/Makefile
@@ -22,7 +22,9 @@ KBUILD_CFLAGS += -DUTS_SYSNAME=\"uClinux\"
 KBUILD_AFLAGS += $(aflags-y)
 LDFLAGS += $(ldflags-y)
 
+ifeq ($(CROSS_COMPILE),)
 CROSS_COMPILE := h8300-unknown-linux-
+endif
 
 core-y	+= arch/$(ARCH)/kernel/ arch/$(ARCH)/mm/
 ifneq '$(CONFIG_H8300_BUILTIN_DTB)' '""'
-- 
2.4.3

Re: [PATCH] h8300: Don't set CROSS_COMPILE unconditionally

[Yoshinori Sato]

On Fri, 16 Oct 2015 20:31:27 +0900,
Javier Martinez Canillas wrote:
> 
> The h8300 architecture Makefile sets the CROSS_COMPILE variable to
> "h8300-unknown-linux-" unconditionally but doing that prevents the
> variable to be used if a toolchain with a different binary name is
> installed in the machine.
> 
> So only set CROSS_COMPILE to that default if the variable has not
> been set by the user before.
> 
> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
> 
> ---
> 
> Hello,
> 
> After $SUBJECT, I'm able to build using the toolchain installed by
> my Linux distribution but the linker gives me an error about an
> unrecognised emulation mode:
> 
> h8300-linux-gnu-ld: unrecognised emulation mode: h8300helf_linux
> 
> I see that h8300helf_linux and 8300self_linux are hardcoded in the
> Makefile as well and these are not listed as supported emulations:
>
> Supported emulations: h8300elf h8300helf h8300self h8300hnelf h8300snelf h8300sxelf h8300sxnelf
> 
> I tried changing -mh8300helf_linux for -mh8300helf that is listed
> as a supported emulation but then I got the following error:
> 
> h8300-linux-gnu-ld: cannot open linker script file ldscripts/h8300helf.xr: No such file or directory
>

h8300-linux is alias of h8300-*-elf.
kernel building toolchain is h8300-*-linux.

h8300-*-linux toolchain is already merged for trunk.
But not include latest release.
Backported patch is https://osdn.jp/projects/uclinux-h8/releases/p14625

> Best regards,
> Javier
> 
>  arch/h8300/Makefile | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/arch/h8300/Makefile b/arch/h8300/Makefile
> index 0d2d96e52d9f..e1c02ca230cb 100644
> --- a/arch/h8300/Makefile
> +++ b/arch/h8300/Makefile
> @@ -22,7 +22,9 @@ KBUILD_CFLAGS += -DUTS_SYSNAME=\"uClinux\"
>  KBUILD_AFLAGS += $(aflags-y)
>  LDFLAGS += $(ldflags-y)
>  
> +ifeq ($(CROSS_COMPILE),)
>  CROSS_COMPILE := h8300-unknown-linux-
> +endif
>  
>  core-y	+= arch/$(ARCH)/kernel/ arch/$(ARCH)/mm/
>  ifneq '$(CONFIG_H8300_BUILTIN_DTB)' '""'
> -- 
> 2.4.3
> 

Applied.
Thanks.

-- 
Yoshinori Sato
<ysato@users.sourceforge.jp>

Re: [PATCH] h8300: Don't set CROSS_COMPILE unconditionally

[Javier Martinez Canillas]

Hello Yoshinori, 

On 10/21/2015 04:55 PM, Yoshinori Sato wrote:
> On Fri, 16 Oct 2015 20:31:27 +0900,
> Javier Martinez Canillas wrote:
>>
>> The h8300 architecture Makefile sets the CROSS_COMPILE variable to
>> "h8300-unknown-linux-" unconditionally but doing that prevents the
>> variable to be used if a toolchain with a different binary name is
>> installed in the machine.
>>
>> So only set CROSS_COMPILE to that default if the variable has not
>> been set by the user before.
>>
>> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
>>
>> ---
>>
>> Hello,
>>
>> After $SUBJECT, I'm able to build using the toolchain installed by
>> my Linux distribution but the linker gives me an error about an
>> unrecognised emulation mode:
>>
>> h8300-linux-gnu-ld: unrecognised emulation mode: h8300helf_linux
>>
>> I see that h8300helf_linux and 8300self_linux are hardcoded in the
>> Makefile as well and these are not listed as supported emulations:
>>
>> Supported emulations: h8300elf h8300helf h8300self h8300hnelf h8300snelf h8300sxelf h8300sxnelf
>>
>> I tried changing -mh8300helf_linux for -mh8300helf that is listed
>> as a supported emulation but then I got the following error:
>>
>> h8300-linux-gnu-ld: cannot open linker script file ldscripts/h8300helf.xr: No such file or directory
>>
> 
> h8300-linux is alias of h8300-*-elf.
> kernel building toolchain is h8300-*-linux.
> 
> h8300-*-linux toolchain is already merged for trunk.
> But not include latest release.
> Backported patch is https://osdn.jp/projects/uclinux-h8/releases/p14625
>

I see, thanks a lot for the information.
 
>> Best regards,
>> Javier
>>
>>  arch/h8300/Makefile | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/arch/h8300/Makefile b/arch/h8300/Makefile
>> index 0d2d96e52d9f..e1c02ca230cb 100644
>> --- a/arch/h8300/Makefile
>> +++ b/arch/h8300/Makefile
>> @@ -22,7 +22,9 @@ KBUILD_CFLAGS += -DUTS_SYSNAME=\"uClinux\"
>>  KBUILD_AFLAGS += $(aflags-y)
>>  LDFLAGS += $(ldflags-y)
>>  
>> +ifeq ($(CROSS_COMPILE),)
>>  CROSS_COMPILE := h8300-unknown-linux-
>> +endif
>>  
>>  core-y	+= arch/$(ARCH)/kernel/ arch/$(ARCH)/mm/
>>  ifneq '$(CONFIG_H8300_BUILTIN_DTB)' '""'
>> -- 
>> 2.4.3
>>
> 
> Applied.
> Thanks.
> 

Best regards,
-- 
Javier Martinez Canillas
Open Source Group
Samsung Research America