* [Buildroot] [PATCH] u-boot: Fix building for ARC700
@ 2016-09-23 12:07 Alexey Brodkin
2016-09-23 21:08 ` Arnout Vandecappelle
2016-09-25 21:30 ` Thomas Petazzoni
0 siblings, 2 replies; 3+ messages in thread
From: Alexey Brodkin @ 2016-09-23 12:07 UTC (permalink / raw)
To: buildroot
With newer ARC toolchain obsolete -marcXXX were finally deperecated
and so compiler throws errors now about unknown option.
Solution is as simple as switching to more generic -mcu-XXX
options. Which we do. Unfortunately that change in upstream U-Boot [1]
happened right after v2016.09 was released an so we need to have that fix
for v2016.07 which is mentioned in axs10x defconfigs and for the latest
U-Boot release v2016.09 (which is selected by default if no defconfig is
used).
Once we deprecate either U-Boot version in U-Boot corresponding patch
should be removed essentially.
[1] http://git.denx.de/?p=u-boot.git;a=commit;h=7c8d81605302e7d7fdd3e7d8eb69302bddc64a2c
Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Peter Korsgaard <peter@korsgaard.com>
---
...-Use-mcpu-XXX-instead-of-obsolete-marcXXX.patch | 39 ++++++++++++++++++++++
...-Use-mcpu-XXX-instead-of-obsolete-marcXXX.patch | 39 ++++++++++++++++++++++
2 files changed, 78 insertions(+)
create mode 100644 boot/uboot/2016.07/0001-arc-Use-mcpu-XXX-instead-of-obsolete-marcXXX.patch
create mode 100644 boot/uboot/2016.09/0001-arc-Use-mcpu-XXX-instead-of-obsolete-marcXXX.patch
diff --git a/boot/uboot/2016.07/0001-arc-Use-mcpu-XXX-instead-of-obsolete-marcXXX.patch b/boot/uboot/2016.07/0001-arc-Use-mcpu-XXX-instead-of-obsolete-marcXXX.patch
new file mode 100644
index 0000000..eded9a3
--- /dev/null
+++ b/boot/uboot/2016.07/0001-arc-Use-mcpu-XXX-instead-of-obsolete-marcXXX.patch
@@ -0,0 +1,39 @@
+From 7c8d81605302e7d7fdd3e7d8eb69302bddc64a2c Mon Sep 17 00:00:00 2001
+From: Alexey Brodkin <abrodkin@synopsys.com>
+Date: Fri, 16 Sep 2016 12:12:26 +0300
+Subject: [PATCH] arc: Use -mcpu=XXX instead of obsolete -marcXXX
+
+With newer ARC tools old way of CPU specification gets obsolete,
+so we're switching to newer and more common way of setting "-mcpu".
+
+Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>
+---
+ arch/arc/config.mk | 6 +++---
+ 1 file changed, 3 insertions(+), 3 deletions(-)
+
+diff --git a/arch/arc/config.mk b/arch/arc/config.mk
+index 7c974f0..13676bd 100644
+--- a/arch/arc/config.mk
++++ b/arch/arc/config.mk
+@@ -31,15 +31,15 @@ CONFIG_MMU = 1
+ endif
+
+ ifdef CONFIG_CPU_ARC750D
+-PLATFORM_CPPFLAGS += -marc700
++PLATFORM_CPPFLAGS += -mcpu=arc700
+ endif
+
+ ifdef CONFIG_CPU_ARC770D
+-PLATFORM_CPPFLAGS += -marc700 -mlock -mswape
++PLATFORM_CPPFLAGS += -mcpu=arc700 -mlock -mswape
+ endif
+
+ ifdef CONFIG_CPU_ARCEM6
+-PLATFORM_CPPFLAGS += -marcem
++PLATFORM_CPPFLAGS += -mcpu=arcem
+ endif
+
+ ifdef CONFIG_CPU_ARCHS34
+--
+2.7.4
+
diff --git a/boot/uboot/2016.09/0001-arc-Use-mcpu-XXX-instead-of-obsolete-marcXXX.patch b/boot/uboot/2016.09/0001-arc-Use-mcpu-XXX-instead-of-obsolete-marcXXX.patch
new file mode 100644
index 0000000..eded9a3
--- /dev/null
+++ b/boot/uboot/2016.09/0001-arc-Use-mcpu-XXX-instead-of-obsolete-marcXXX.patch
@@ -0,0 +1,39 @@
+From 7c8d81605302e7d7fdd3e7d8eb69302bddc64a2c Mon Sep 17 00:00:00 2001
+From: Alexey Brodkin <abrodkin@synopsys.com>
+Date: Fri, 16 Sep 2016 12:12:26 +0300
+Subject: [PATCH] arc: Use -mcpu=XXX instead of obsolete -marcXXX
+
+With newer ARC tools old way of CPU specification gets obsolete,
+so we're switching to newer and more common way of setting "-mcpu".
+
+Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>
+---
+ arch/arc/config.mk | 6 +++---
+ 1 file changed, 3 insertions(+), 3 deletions(-)
+
+diff --git a/arch/arc/config.mk b/arch/arc/config.mk
+index 7c974f0..13676bd 100644
+--- a/arch/arc/config.mk
++++ b/arch/arc/config.mk
+@@ -31,15 +31,15 @@ CONFIG_MMU = 1
+ endif
+
+ ifdef CONFIG_CPU_ARC750D
+-PLATFORM_CPPFLAGS += -marc700
++PLATFORM_CPPFLAGS += -mcpu=arc700
+ endif
+
+ ifdef CONFIG_CPU_ARC770D
+-PLATFORM_CPPFLAGS += -marc700 -mlock -mswape
++PLATFORM_CPPFLAGS += -mcpu=arc700 -mlock -mswape
+ endif
+
+ ifdef CONFIG_CPU_ARCEM6
+-PLATFORM_CPPFLAGS += -marcem
++PLATFORM_CPPFLAGS += -mcpu=arcem
+ endif
+
+ ifdef CONFIG_CPU_ARCHS34
+--
+2.7.4
+
--
2.6.3
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH] u-boot: Fix building for ARC700
2016-09-23 12:07 [Buildroot] [PATCH] u-boot: Fix building for ARC700 Alexey Brodkin
@ 2016-09-23 21:08 ` Arnout Vandecappelle
2016-09-25 21:30 ` Thomas Petazzoni
1 sibling, 0 replies; 3+ messages in thread
From: Arnout Vandecappelle @ 2016-09-23 21:08 UTC (permalink / raw)
To: buildroot
On 23-09-16 14:07, Alexey Brodkin wrote:
> With newer ARC toolchain obsolete -marcXXX were finally deperecated
deprecated
> and so compiler throws errors now about unknown option.
>
> Solution is as simple as switching to more generic -mcu-XXX
-mcpu=arcXXX
> options. Which we do. Unfortunately that change in upstream U-Boot [1]
> happened right after v2016.09 was released an so we need to have that fix
> for v2016.07 which is mentioned in axs10x defconfigs and for the latest
> U-Boot release v2016.09 (which is selected by default if no defconfig is
> used).
It's unfortunate that the deprecation period was so short, and that U-Boot
didn't fix it sooner.
I really don't like adding patches to U-Boot, because you never know what
version will be used. In this case, however, it's just for ARC configs which is
pretty controlled, and in the likely case that someone has a custom U-Boot fork,
they can easily apply the patch themselves. So
Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Regards,
Arnout
> Once we deprecate either U-Boot version in U-Boot corresponding patch
> should be removed essentially.
>
> [1] http://git.denx.de/?p=u-boot.git;a=commit;h=7c8d81605302e7d7fdd3e7d8eb69302bddc64a2c
>
> Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>
> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> Cc: Peter Korsgaard <peter@korsgaard.com>
> ---
> ...-Use-mcpu-XXX-instead-of-obsolete-marcXXX.patch | 39 ++++++++++++++++++++++
> ...-Use-mcpu-XXX-instead-of-obsolete-marcXXX.patch | 39 ++++++++++++++++++++++
> 2 files changed, 78 insertions(+)
> create mode 100644 boot/uboot/2016.07/0001-arc-Use-mcpu-XXX-instead-of-obsolete-marcXXX.patch
> create mode 100644 boot/uboot/2016.09/0001-arc-Use-mcpu-XXX-instead-of-obsolete-marcXXX.patch
>
> diff --git a/boot/uboot/2016.07/0001-arc-Use-mcpu-XXX-instead-of-obsolete-marcXXX.patch b/boot/uboot/2016.07/0001-arc-Use-mcpu-XXX-instead-of-obsolete-marcXXX.patch
> new file mode 100644
> index 0000000..eded9a3
> --- /dev/null
> +++ b/boot/uboot/2016.07/0001-arc-Use-mcpu-XXX-instead-of-obsolete-marcXXX.patch
> @@ -0,0 +1,39 @@
> +From 7c8d81605302e7d7fdd3e7d8eb69302bddc64a2c Mon Sep 17 00:00:00 2001
> +From: Alexey Brodkin <abrodkin@synopsys.com>
> +Date: Fri, 16 Sep 2016 12:12:26 +0300
> +Subject: [PATCH] arc: Use -mcpu=XXX instead of obsolete -marcXXX
> +
> +With newer ARC tools old way of CPU specification gets obsolete,
> +so we're switching to newer and more common way of setting "-mcpu".
> +
> +Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>
> +---
> + arch/arc/config.mk | 6 +++---
> + 1 file changed, 3 insertions(+), 3 deletions(-)
> +
> +diff --git a/arch/arc/config.mk b/arch/arc/config.mk
> +index 7c974f0..13676bd 100644
> +--- a/arch/arc/config.mk
> ++++ b/arch/arc/config.mk
> +@@ -31,15 +31,15 @@ CONFIG_MMU = 1
> + endif
> +
> + ifdef CONFIG_CPU_ARC750D
> +-PLATFORM_CPPFLAGS += -marc700
> ++PLATFORM_CPPFLAGS += -mcpu=arc700
> + endif
> +
> + ifdef CONFIG_CPU_ARC770D
> +-PLATFORM_CPPFLAGS += -marc700 -mlock -mswape
> ++PLATFORM_CPPFLAGS += -mcpu=arc700 -mlock -mswape
> + endif
> +
> + ifdef CONFIG_CPU_ARCEM6
> +-PLATFORM_CPPFLAGS += -marcem
> ++PLATFORM_CPPFLAGS += -mcpu=arcem
> + endif
> +
> + ifdef CONFIG_CPU_ARCHS34
> +--
> +2.7.4
> +
> diff --git a/boot/uboot/2016.09/0001-arc-Use-mcpu-XXX-instead-of-obsolete-marcXXX.patch b/boot/uboot/2016.09/0001-arc-Use-mcpu-XXX-instead-of-obsolete-marcXXX.patch
> new file mode 100644
> index 0000000..eded9a3
> --- /dev/null
> +++ b/boot/uboot/2016.09/0001-arc-Use-mcpu-XXX-instead-of-obsolete-marcXXX.patch
> @@ -0,0 +1,39 @@
> +From 7c8d81605302e7d7fdd3e7d8eb69302bddc64a2c Mon Sep 17 00:00:00 2001
> +From: Alexey Brodkin <abrodkin@synopsys.com>
> +Date: Fri, 16 Sep 2016 12:12:26 +0300
> +Subject: [PATCH] arc: Use -mcpu=XXX instead of obsolete -marcXXX
> +
> +With newer ARC tools old way of CPU specification gets obsolete,
> +so we're switching to newer and more common way of setting "-mcpu".
> +
> +Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>
> +---
> + arch/arc/config.mk | 6 +++---
> + 1 file changed, 3 insertions(+), 3 deletions(-)
> +
> +diff --git a/arch/arc/config.mk b/arch/arc/config.mk
> +index 7c974f0..13676bd 100644
> +--- a/arch/arc/config.mk
> ++++ b/arch/arc/config.mk
> +@@ -31,15 +31,15 @@ CONFIG_MMU = 1
> + endif
> +
> + ifdef CONFIG_CPU_ARC750D
> +-PLATFORM_CPPFLAGS += -marc700
> ++PLATFORM_CPPFLAGS += -mcpu=arc700
> + endif
> +
> + ifdef CONFIG_CPU_ARC770D
> +-PLATFORM_CPPFLAGS += -marc700 -mlock -mswape
> ++PLATFORM_CPPFLAGS += -mcpu=arc700 -mlock -mswape
> + endif
> +
> + ifdef CONFIG_CPU_ARCEM6
> +-PLATFORM_CPPFLAGS += -marcem
> ++PLATFORM_CPPFLAGS += -mcpu=arcem
> + endif
> +
> + ifdef CONFIG_CPU_ARCHS34
> +--
> +2.7.4
> +
>
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH] u-boot: Fix building for ARC700
2016-09-23 12:07 [Buildroot] [PATCH] u-boot: Fix building for ARC700 Alexey Brodkin
2016-09-23 21:08 ` Arnout Vandecappelle
@ 2016-09-25 21:30 ` Thomas Petazzoni
1 sibling, 0 replies; 3+ messages in thread
From: Thomas Petazzoni @ 2016-09-25 21:30 UTC (permalink / raw)
To: buildroot
Hello,
On Fri, 23 Sep 2016 15:07:53 +0300, Alexey Brodkin wrote:
> With newer ARC toolchain obsolete -marcXXX were finally deperecated
> and so compiler throws errors now about unknown option.
>
> Solution is as simple as switching to more generic -mcu-XXX
> options. Which we do. Unfortunately that change in upstream U-Boot [1]
> happened right after v2016.09 was released an so we need to have that fix
> for v2016.07 which is mentioned in axs10x defconfigs and for the latest
> U-Boot release v2016.09 (which is selected by default if no defconfig is
> used).
>
> Once we deprecate either U-Boot version in U-Boot corresponding patch
> should be removed essentially.
>
> [1] http://git.denx.de/?p=u-boot.git;a=commit;h=7c8d81605302e7d7fdd3e7d8eb69302bddc64a2c
>
> Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>
> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> Cc: Peter Korsgaard <peter@korsgaard.com>
> ---
> ...-Use-mcpu-XXX-instead-of-obsolete-marcXXX.patch | 39 ++++++++++++++++++++++
> ...-Use-mcpu-XXX-instead-of-obsolete-marcXXX.patch | 39 ++++++++++++++++++++++
> 2 files changed, 78 insertions(+)
> create mode 100644 boot/uboot/2016.07/0001-arc-Use-mcpu-XXX-instead-of-obsolete-marcXXX.patch
> create mode 100644 boot/uboot/2016.09/0001-arc-Use-mcpu-XXX-instead-of-obsolete-marcXXX.patch
Applied to master. I've fixed the patch so that it works fine after the
U-Boot bump to 2016.09.01.
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-09-25 21:30 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-09-23 12:07 [Buildroot] [PATCH] u-boot: Fix building for ARC700 Alexey Brodkin
2016-09-23 21:08 ` Arnout Vandecappelle
2016-09-25 21:30 ` Thomas Petazzoni
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox