diff mbox series

net: wan: wanxl: define CROSS_COMPILE_M68K

Message ID 20210912212321.10982-1-kilobyte@angband.pl
State New
Headers show
Series net: wan: wanxl: define CROSS_COMPILE_M68K | expand

Commit Message

Adam Borowski Sept. 12, 2021, 9:23 p.m. UTC
It was used but never set.  The hardcoded value from before the dawn of
time was non-standard; the usual name for cross-tools is $TRIPLET-$TOOL

Signed-off-by: Adam Borowski <kilobyte@angband.pl>
---
 This is neither the host nor target arch, thus it's very unlikely to be
 set by the user.  With this patch, it works out of the box on Debian
 and Fedora.

 drivers/net/wan/Makefile | 2 ++
 1 file changed, 2 insertions(+)

Comments

Jakub Kicinski Sept. 15, 2021, 3:07 a.m. UTC | #1
On Sun, 12 Sep 2021 23:23:21 +0200 Adam Borowski wrote:
> It was used but never set.  The hardcoded value from before the dawn of

> time was non-standard; the usual name for cross-tools is $TRIPLET-$TOOL

> 

> Signed-off-by: Adam Borowski <kilobyte@angband.pl>

> ---

>  This is neither the host nor target arch, thus it's very unlikely to be

>  set by the user.  With this patch, it works out of the box on Debian

>  and Fedora.

> 

>  drivers/net/wan/Makefile | 2 ++

>  1 file changed, 2 insertions(+)

> 

> diff --git a/drivers/net/wan/Makefile b/drivers/net/wan/Makefile

> index f6b92efffc94..480bcd1f6c1c 100644

> --- a/drivers/net/wan/Makefile

> +++ b/drivers/net/wan/Makefile

> @@ -34,6 +34,8 @@ obj-$(CONFIG_SLIC_DS26522)	+= slic_ds26522.o

>  clean-files := wanxlfw.inc

>  $(obj)/wanxl.o:	$(obj)/wanxlfw.inc

>  

> +CROSS_COMPILE_M68K = m68k-linux-gnu-


This will not overwrite the parameter if set from command line, right?
My Makefile-foo is getting rusty.

>  ifeq ($(CONFIG_WANXL_BUILD_FIRMWARE),y)

>  ifeq ($(ARCH),m68k)

>    M68KCC = $(CC)
patchwork-bot+netdevbpf@kernel.org Sept. 16, 2021, 1:10 p.m. UTC | #2
Hello:

This patch was applied to netdev/net.git (refs/heads/master):

On Sun, 12 Sep 2021 23:23:21 +0200 you wrote:
> It was used but never set.  The hardcoded value from before the dawn of

> time was non-standard; the usual name for cross-tools is $TRIPLET-$TOOL

> 

> Signed-off-by: Adam Borowski <kilobyte@angband.pl>

> ---

>  This is neither the host nor target arch, thus it's very unlikely to be

>  set by the user.  With this patch, it works out of the box on Debian

>  and Fedora.

> 

> [...]


Here is the summary with links:
  - net: wan: wanxl: define CROSS_COMPILE_M68K
    https://git.kernel.org/netdev/net/c/84fb7dfc7463

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
diff mbox series

Patch

diff --git a/drivers/net/wan/Makefile b/drivers/net/wan/Makefile
index f6b92efffc94..480bcd1f6c1c 100644
--- a/drivers/net/wan/Makefile
+++ b/drivers/net/wan/Makefile
@@ -34,6 +34,8 @@  obj-$(CONFIG_SLIC_DS26522)	+= slic_ds26522.o
 clean-files := wanxlfw.inc
 $(obj)/wanxl.o:	$(obj)/wanxlfw.inc
 
+CROSS_COMPILE_M68K = m68k-linux-gnu-
+
 ifeq ($(CONFIG_WANXL_BUILD_FIRMWARE),y)
 ifeq ($(ARCH),m68k)
   M68KCC = $(CC)