* [PATCH 0/2] fixes for binutils 2.21
@ 2011-01-24 14:48 Víctor Manuel Jáquez Leal
2011-01-24 14:48 ` [PATCH 1/2] u-boot-sakoman: fix break caused by the new binutils Víctor Manuel Jáquez Leal
2011-01-24 14:48 ` [PATCH 2/2] linux-omap4: " Víctor Manuel Jáquez Leal
0 siblings, 2 replies; 7+ messages in thread
From: Víctor Manuel Jáquez Leal @ 2011-01-24 14:48 UTC (permalink / raw)
To: openembedded-devel
These patches focus the same problem: with the binutils upgrade to
2.21 it is required specify the +sec to the march parameter.
The kernel patch is just grabbed from the ubuntu's repository. For the
u-boot I just came up with a translation of those patches.
vmjl
Víctor Manuel Jáquez Leal (2):
u-boot-sakoman: fix break caused by the new binutils
linux-omap4: fix break caused by the new binutils
...TU-Config-Fix-FTBS-caused-by-new-binutils.patch | 38 ++++++++++++++++++++
...0002-Add-AFLAGS-for-sleep44xx-to-fix-FTBS.patch | 26 +++++++++++++
recipes/linux/linux-omap4_2.6.35.3.bb | 2 +
.../fix-break-caused-by-new-binutils.patch | 31 ++++++++++++++++
recipes/u-boot/u-boot-sakoman_git.bb | 1 +
5 files changed, 98 insertions(+), 0 deletions(-)
create mode 100644 recipes/linux/linux-omap4/0001-UBUNTU-Config-Fix-FTBS-caused-by-new-binutils.patch
create mode 100644 recipes/linux/linux-omap4/0002-Add-AFLAGS-for-sleep44xx-to-fix-FTBS.patch
create mode 100644 recipes/u-boot/u-boot-sakoman/fix-break-caused-by-new-binutils.patch
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/2] u-boot-sakoman: fix break caused by the new binutils
2011-01-24 14:48 [PATCH 0/2] fixes for binutils 2.21 Víctor Manuel Jáquez Leal
@ 2011-01-24 14:48 ` Víctor Manuel Jáquez Leal
2011-01-24 15:47 ` Khem Raj
[not found] ` <1295882416.1999.8.camel@quadra>
2011-01-24 14:48 ` [PATCH 2/2] linux-omap4: " Víctor Manuel Jáquez Leal
1 sibling, 2 replies; 7+ messages in thread
From: Víctor Manuel Jáquez Leal @ 2011-01-24 14:48 UTC (permalink / raw)
To: openembedded-devel; +Cc: Koen Kooi, Steve Sakoman
New assemblers need -march=armv7-a+sec on command line or .arch_extension sec
inline to enable use of the smc instruction.
This patch adds as-instr to check the latter to conditionally enable the
former in AFLAGS for files that use smc.
Signed-off-by: Víctor Manuel Jáquez Leal <vjaquez@igalia.com>
---
.../fix-break-caused-by-new-binutils.patch | 31 ++++++++++++++++++++
recipes/u-boot/u-boot-sakoman_git.bb | 1 +
2 files changed, 32 insertions(+), 0 deletions(-)
create mode 100644 recipes/u-boot/u-boot-sakoman/fix-break-caused-by-new-binutils.patch
diff --git a/recipes/u-boot/u-boot-sakoman/fix-break-caused-by-new-binutils.patch b/recipes/u-boot/u-boot-sakoman/fix-break-caused-by-new-binutils.patch
new file mode 100644
index 0000000..16bc980
--- /dev/null
+++ b/recipes/u-boot/u-boot-sakoman/fix-break-caused-by-new-binutils.patch
@@ -0,0 +1,31 @@
+Index: git/arch/arm/cpu/armv7/omap4/Makefile
+===================================================================
+--- git.orig/arch/arm/cpu/armv7/omap4/Makefile
++++ git/arch/arm/cpu/armv7/omap4/Makefile
+@@ -26,6 +26,8 @@ include $(TOPDIR)/config.mk
+ LIB = $(obj)lib$(SOC).o
+
+ SOBJS += lowlevel_init.o
++plus_sec := $(call as-instr,.arch_extension sec,+sec)
++lowlevel_init.o: AFLAGS += -Wa,-march=armv7-a$(plus_sec)
+
+ COBJS += board.o
+ COBJS += mem.o
+Index: git/config.mk
+===================================================================
+--- git.orig/config.mk
++++ git/config.mk
+@@ -266,3 +266,13 @@ cmd_link_o_target = $(if $(strip $1),\
+ rm -f $@; $(AR) rcs $@ )
+
+ #########################################################################
++
++# Tries to compile an assembly instruction
++as-instr = $(shell if echo -e "$(1)" | \
++ $(CC) $(AFLAGS) -c -xassembler - \
++ -o $(TMPOUT)astest$$$$.out > /dev/null 2>&1; \
++ then rm $(TMPOUT)astest$$$$.out; echo "$(2)"; \
++ else echo "$(3)"; fi)
++
++#########################################################################
++
diff --git a/recipes/u-boot/u-boot-sakoman_git.bb b/recipes/u-boot/u-boot-sakoman_git.bb
index 5b9ff11..8dfe713 100644
--- a/recipes/u-boot/u-boot-sakoman_git.bb
+++ b/recipes/u-boot/u-boot-sakoman_git.bb
@@ -7,6 +7,7 @@ SRCREV = "261733408a27d14590cf3ec6b596461808050e32"
PV = "2010.12+${PR}+git${SRCREV}"
SRC_URI = "git://www.sakoman.com/git/u-boot.git;branch=omap4-exp;protocol=git \
+ file://fix-break-caused-by-new-binutils.patch \
"
S = "${WORKDIR}/git"
--
1.7.0.2
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 2/2] linux-omap4: fix break caused by the new binutils
2011-01-24 14:48 [PATCH 0/2] fixes for binutils 2.21 Víctor Manuel Jáquez Leal
2011-01-24 14:48 ` [PATCH 1/2] u-boot-sakoman: fix break caused by the new binutils Víctor Manuel Jáquez Leal
@ 2011-01-24 14:48 ` Víctor Manuel Jáquez Leal
2011-01-24 15:48 ` Khem Raj
1 sibling, 1 reply; 7+ messages in thread
From: Víctor Manuel Jáquez Leal @ 2011-01-24 14:48 UTC (permalink / raw)
To: openembedded-devel; +Cc: Koen Kooi, Koen Kooi
According to this discussion [1], a couple patches where required by the
kernel using the latest binutils (2.21).
This patch add those patches, which were posted in the Ubuntu's kernel
repository.
1. http://comments.gmane.org/gmane.comp.handhelds.openembedded/40650
Signed-off-by: Víctor Manuel Jáquez Leal <vjaquez@igalia.com>
---
...TU-Config-Fix-FTBS-caused-by-new-binutils.patch | 38 ++++++++++++++++++++
...0002-Add-AFLAGS-for-sleep44xx-to-fix-FTBS.patch | 26 +++++++++++++
recipes/linux/linux-omap4_2.6.35.3.bb | 2 +
3 files changed, 66 insertions(+), 0 deletions(-)
create mode 100644 recipes/linux/linux-omap4/0001-UBUNTU-Config-Fix-FTBS-caused-by-new-binutils.patch
create mode 100644 recipes/linux/linux-omap4/0002-Add-AFLAGS-for-sleep44xx-to-fix-FTBS.patch
diff --git a/recipes/linux/linux-omap4/0001-UBUNTU-Config-Fix-FTBS-caused-by-new-binutils.patch b/recipes/linux/linux-omap4/0001-UBUNTU-Config-Fix-FTBS-caused-by-new-binutils.patch
new file mode 100644
index 0000000..e83a3e2
--- /dev/null
+++ b/recipes/linux/linux-omap4/0001-UBUNTU-Config-Fix-FTBS-caused-by-new-binutils.patch
@@ -0,0 +1,38 @@
+From 7699704e011db864c56f333bce9936a36522fb79 Mon Sep 17 00:00:00 2001
+From: Tim Gardner <tim.gardner@canonical.com>
+Date: Thu, 9 Dec 2010 08:16:52 -0700
+Subject: [PATCH 1/2] UBUNTU: [Config] Fix FTBS caused by new binutils
+
+New assemblers need -march=armv7-a+sec on command line or
+.arch_extension sec inline to enable use of the smc instruction.
+
+This patch uses as-instr to check the latter to conditionally
+enable the former in AFLAGS for files that use smc.
+
+Checked on both old and new binutils to verify that it does
+not break old versions.
+
+Signed-off-by: John Rigby <john.rigby@linaro.org>
+Signed-off-by: Tim Gardner <tim.gardner@canonical.com>
+---
+ arch/arm/mach-omap2/Makefile | 4 +++-
+ 1 files changed, 3 insertions(+), 1 deletions(-)
+
+diff --git a/arch/arm/mach-omap2/Makefile b/arch/arm/mach-omap2/Makefile
+index 75251b4..0241b4d 100644
+--- a/arch/arm/mach-omap2/Makefile
++++ b/arch/arm/mach-omap2/Makefile
+@@ -29,7 +29,9 @@ obj-$(CONFIG_HOTPLUG_CPU) += omap-hotplug.o
+ obj-$(CONFIG_ARCH_OMAP4) += omap44xx-smc.o omap4-common.o \
+ omap4-wakeupgen.o
+
+-AFLAGS_omap44xx-smc.o :=-Wa,-march=armv7-a
++plus_sec := $(call as-instr,.arch_extension sec,+sec)
++AFLAGS_omap-headsmp.o :=-Wa,-march=armv7-a$(plus_sec)
++AFLAGS_omap44xx-smc.o :=-Wa,-march=armv7-a$(plus_sec)
+
+ # Functions loaded to SRAM
+ obj-$(CONFIG_ARCH_OMAP2420) += sram242x.o
+--
+1.7.0.2
+
diff --git a/recipes/linux/linux-omap4/0002-Add-AFLAGS-for-sleep44xx-to-fix-FTBS.patch b/recipes/linux/linux-omap4/0002-Add-AFLAGS-for-sleep44xx-to-fix-FTBS.patch
new file mode 100644
index 0000000..ea16ab1
--- /dev/null
+++ b/recipes/linux/linux-omap4/0002-Add-AFLAGS-for-sleep44xx-to-fix-FTBS.patch
@@ -0,0 +1,26 @@
+From da58d36fb6b13899249bf1efbf25f7385d048bb3 Mon Sep 17 00:00:00 2001
+From: Tim Gardner <tim.gardner@canonical.com>
+Date: Tue, 14 Dec 2010 06:56:34 -0700
+Subject: [PATCH 2/2] Add AFLAGS for sleep44xx to fix FTBS
+
+Signed-off-by: Tim Gardner <tim.gardner@canonical.com>
+---
+ arch/arm/mach-omap2/Makefile | 2 ++
+ 1 files changed, 2 insertions(+), 0 deletions(-)
+
+diff --git a/arch/arm/mach-omap2/Makefile b/arch/arm/mach-omap2/Makefile
+index 0241b4d..865a3e2 100644
+--- a/arch/arm/mach-omap2/Makefile
++++ b/arch/arm/mach-omap2/Makefile
+@@ -59,6 +59,8 @@ obj-$(CONFIG_ARCH_OMAP4) += pm44xx.o pm_bus.o cpuidle44xx.o \
+ omap4-mpuss-lowpower.o sleep44xx.o \
+ voltage.o opp44xx_data.o
+
++AFLAGS_sleep44xx.o :=-Wa,-march=armv7-a$(plus_sec)
++
+ obj-$(CONFIG_PM_DEBUG) += pm-debug.o
+ obj-$(CONFIG_OMAP_SMARTREFLEX) += sr_device.o smartreflex.o
+ obj-$(CONFIG_OMAP_SMARTREFLEX_CLASS3) += smartreflex-class3.o
+--
+1.7.0.2
+
diff --git a/recipes/linux/linux-omap4_2.6.35.3.bb b/recipes/linux/linux-omap4_2.6.35.3.bb
index 4db0b28..f5a11f4 100644
--- a/recipes/linux/linux-omap4_2.6.35.3.bb
+++ b/recipes/linux/linux-omap4_2.6.35.3.bb
@@ -15,6 +15,8 @@ SRC_URI = "git://dev.omapzoom.org/pub/scm/integration/kernel-ubuntu.git;protocol
file://0006-ARM-Add-option-to-allow-userspace-access-to-performa.patch \
file://0007-OMAP4-do-not-force-select-options-which-are-not-requ.patch \
file://0008-omap4-pandaboard-add-support-for-DVI-output.patch \
+ file://0001-UBUNTU-Config-Fix-FTBS-caused-by-new-binutils.patch \
+ file://0002-Add-AFLAGS-for-sleep44xx-to-fix-FTBS.patch \
file://defconfig"
S = "${WORKDIR}/git"
--
1.7.0.2
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] u-boot-sakoman: fix break caused by the new binutils
2011-01-24 14:48 ` [PATCH 1/2] u-boot-sakoman: fix break caused by the new binutils Víctor Manuel Jáquez Leal
@ 2011-01-24 15:47 ` Khem Raj
[not found] ` <1295882416.1999.8.camel@quadra>
1 sibling, 0 replies; 7+ messages in thread
From: Khem Raj @ 2011-01-24 15:47 UTC (permalink / raw)
To: openembedded-devel; +Cc: Koen Kooi, Steve Sakoman
2011/1/24 Víctor Manuel Jáquez Leal <vjaquez@igalia.com>:
> New assemblers need -march=armv7-a+sec on command line or .arch_extension sec
> inline to enable use of the smc instruction.
>
> This patch adds as-instr to check the latter to conditionally enable the
> former in AFLAGS for files that use smc.
>
> Signed-off-by: Víctor Manuel Jáquez Leal <vjaquez@igalia.com>
Acked-by: Khem Raj <raj.khem@gmail.com>
> ---
> .../fix-break-caused-by-new-binutils.patch | 31 ++++++++++++++++++++
> recipes/u-boot/u-boot-sakoman_git.bb | 1 +
> 2 files changed, 32 insertions(+), 0 deletions(-)
> create mode 100644 recipes/u-boot/u-boot-sakoman/fix-break-caused-by-new-binutils.patch
>
> diff --git a/recipes/u-boot/u-boot-sakoman/fix-break-caused-by-new-binutils.patch b/recipes/u-boot/u-boot-sakoman/fix-break-caused-by-new-binutils.patch
> new file mode 100644
> index 0000000..16bc980
> --- /dev/null
> +++ b/recipes/u-boot/u-boot-sakoman/fix-break-caused-by-new-binutils.patch
> @@ -0,0 +1,31 @@
> +Index: git/arch/arm/cpu/armv7/omap4/Makefile
> +===================================================================
> +--- git.orig/arch/arm/cpu/armv7/omap4/Makefile
> ++++ git/arch/arm/cpu/armv7/omap4/Makefile
> +@@ -26,6 +26,8 @@ include $(TOPDIR)/config.mk
> + LIB = $(obj)lib$(SOC).o
> +
> + SOBJS += lowlevel_init.o
> ++plus_sec := $(call as-instr,.arch_extension sec,+sec)
> ++lowlevel_init.o: AFLAGS += -Wa,-march=armv7-a$(plus_sec)
> +
> + COBJS += board.o
> + COBJS += mem.o
> +Index: git/config.mk
> +===================================================================
> +--- git.orig/config.mk
> ++++ git/config.mk
> +@@ -266,3 +266,13 @@ cmd_link_o_target = $(if $(strip $1),\
> + rm -f $@; $(AR) rcs $@ )
> +
> + #########################################################################
> ++
> ++# Tries to compile an assembly instruction
> ++as-instr = $(shell if echo -e "$(1)" | \
> ++ $(CC) $(AFLAGS) -c -xassembler - \
> ++ -o $(TMPOUT)astest$$$$.out > /dev/null 2>&1; \
> ++ then rm $(TMPOUT)astest$$$$.out; echo "$(2)"; \
> ++ else echo "$(3)"; fi)
> ++
> ++#########################################################################
> ++
> diff --git a/recipes/u-boot/u-boot-sakoman_git.bb b/recipes/u-boot/u-boot-sakoman_git.bb
> index 5b9ff11..8dfe713 100644
> --- a/recipes/u-boot/u-boot-sakoman_git.bb
> +++ b/recipes/u-boot/u-boot-sakoman_git.bb
> @@ -7,6 +7,7 @@ SRCREV = "261733408a27d14590cf3ec6b596461808050e32"
> PV = "2010.12+${PR}+git${SRCREV}"
>
> SRC_URI = "git://www.sakoman.com/git/u-boot.git;branch=omap4-exp;protocol=git \
> + file://fix-break-caused-by-new-binutils.patch \
> "
> S = "${WORKDIR}/git"
>
> --
> 1.7.0.2
>
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] linux-omap4: fix break caused by the new binutils
2011-01-24 14:48 ` [PATCH 2/2] linux-omap4: " Víctor Manuel Jáquez Leal
@ 2011-01-24 15:48 ` Khem Raj
0 siblings, 0 replies; 7+ messages in thread
From: Khem Raj @ 2011-01-24 15:48 UTC (permalink / raw)
To: openembedded-devel; +Cc: Koen Kooi, Koen Kooi
2011/1/24 Víctor Manuel Jáquez Leal <vjaquez@igalia.com>:
> According to this discussion [1], a couple patches where required by the
> kernel using the latest binutils (2.21).
>
> This patch add those patches, which were posted in the Ubuntu's kernel
> repository.
>
> 1. http://comments.gmane.org/gmane.comp.handhelds.openembedded/40650
>
> Signed-off-by: Víctor Manuel Jáquez Leal <vjaquez@igalia.com>
Acked-by: Khem Raj <raj.khem@gmail.com>
> ---
> ...TU-Config-Fix-FTBS-caused-by-new-binutils.patch | 38 ++++++++++++++++++++
> ...0002-Add-AFLAGS-for-sleep44xx-to-fix-FTBS.patch | 26 +++++++++++++
> recipes/linux/linux-omap4_2.6.35.3.bb | 2 +
> 3 files changed, 66 insertions(+), 0 deletions(-)
> create mode 100644 recipes/linux/linux-omap4/0001-UBUNTU-Config-Fix-FTBS-caused-by-new-binutils.patch
> create mode 100644 recipes/linux/linux-omap4/0002-Add-AFLAGS-for-sleep44xx-to-fix-FTBS.patch
>
> diff --git a/recipes/linux/linux-omap4/0001-UBUNTU-Config-Fix-FTBS-caused-by-new-binutils.patch b/recipes/linux/linux-omap4/0001-UBUNTU-Config-Fix-FTBS-caused-by-new-binutils.patch
> new file mode 100644
> index 0000000..e83a3e2
> --- /dev/null
> +++ b/recipes/linux/linux-omap4/0001-UBUNTU-Config-Fix-FTBS-caused-by-new-binutils.patch
> @@ -0,0 +1,38 @@
> +From 7699704e011db864c56f333bce9936a36522fb79 Mon Sep 17 00:00:00 2001
> +From: Tim Gardner <tim.gardner@canonical.com>
> +Date: Thu, 9 Dec 2010 08:16:52 -0700
> +Subject: [PATCH 1/2] UBUNTU: [Config] Fix FTBS caused by new binutils
> +
> +New assemblers need -march=armv7-a+sec on command line or
> +.arch_extension sec inline to enable use of the smc instruction.
> +
> +This patch uses as-instr to check the latter to conditionally
> +enable the former in AFLAGS for files that use smc.
> +
> +Checked on both old and new binutils to verify that it does
> +not break old versions.
> +
> +Signed-off-by: John Rigby <john.rigby@linaro.org>
> +Signed-off-by: Tim Gardner <tim.gardner@canonical.com>
> +---
> + arch/arm/mach-omap2/Makefile | 4 +++-
> + 1 files changed, 3 insertions(+), 1 deletions(-)
> +
> +diff --git a/arch/arm/mach-omap2/Makefile b/arch/arm/mach-omap2/Makefile
> +index 75251b4..0241b4d 100644
> +--- a/arch/arm/mach-omap2/Makefile
> ++++ b/arch/arm/mach-omap2/Makefile
> +@@ -29,7 +29,9 @@ obj-$(CONFIG_HOTPLUG_CPU) += omap-hotplug.o
> + obj-$(CONFIG_ARCH_OMAP4) += omap44xx-smc.o omap4-common.o \
> + omap4-wakeupgen.o
> +
> +-AFLAGS_omap44xx-smc.o :=-Wa,-march=armv7-a
> ++plus_sec := $(call as-instr,.arch_extension sec,+sec)
> ++AFLAGS_omap-headsmp.o :=-Wa,-march=armv7-a$(plus_sec)
> ++AFLAGS_omap44xx-smc.o :=-Wa,-march=armv7-a$(plus_sec)
> +
> + # Functions loaded to SRAM
> + obj-$(CONFIG_ARCH_OMAP2420) += sram242x.o
> +--
> +1.7.0.2
> +
> diff --git a/recipes/linux/linux-omap4/0002-Add-AFLAGS-for-sleep44xx-to-fix-FTBS.patch b/recipes/linux/linux-omap4/0002-Add-AFLAGS-for-sleep44xx-to-fix-FTBS.patch
> new file mode 100644
> index 0000000..ea16ab1
> --- /dev/null
> +++ b/recipes/linux/linux-omap4/0002-Add-AFLAGS-for-sleep44xx-to-fix-FTBS.patch
> @@ -0,0 +1,26 @@
> +From da58d36fb6b13899249bf1efbf25f7385d048bb3 Mon Sep 17 00:00:00 2001
> +From: Tim Gardner <tim.gardner@canonical.com>
> +Date: Tue, 14 Dec 2010 06:56:34 -0700
> +Subject: [PATCH 2/2] Add AFLAGS for sleep44xx to fix FTBS
> +
> +Signed-off-by: Tim Gardner <tim.gardner@canonical.com>
> +---
> + arch/arm/mach-omap2/Makefile | 2 ++
> + 1 files changed, 2 insertions(+), 0 deletions(-)
> +
> +diff --git a/arch/arm/mach-omap2/Makefile b/arch/arm/mach-omap2/Makefile
> +index 0241b4d..865a3e2 100644
> +--- a/arch/arm/mach-omap2/Makefile
> ++++ b/arch/arm/mach-omap2/Makefile
> +@@ -59,6 +59,8 @@ obj-$(CONFIG_ARCH_OMAP4) += pm44xx.o pm_bus.o cpuidle44xx.o \
> + omap4-mpuss-lowpower.o sleep44xx.o \
> + voltage.o opp44xx_data.o
> +
> ++AFLAGS_sleep44xx.o :=-Wa,-march=armv7-a$(plus_sec)
> ++
> + obj-$(CONFIG_PM_DEBUG) += pm-debug.o
> + obj-$(CONFIG_OMAP_SMARTREFLEX) += sr_device.o smartreflex.o
> + obj-$(CONFIG_OMAP_SMARTREFLEX_CLASS3) += smartreflex-class3.o
> +--
> +1.7.0.2
> +
> diff --git a/recipes/linux/linux-omap4_2.6.35.3.bb b/recipes/linux/linux-omap4_2.6.35.3.bb
> index 4db0b28..f5a11f4 100644
> --- a/recipes/linux/linux-omap4_2.6.35.3.bb
> +++ b/recipes/linux/linux-omap4_2.6.35.3.bb
> @@ -15,6 +15,8 @@ SRC_URI = "git://dev.omapzoom.org/pub/scm/integration/kernel-ubuntu.git;protocol
> file://0006-ARM-Add-option-to-allow-userspace-access-to-performa.patch \
> file://0007-OMAP4-do-not-force-select-options-which-are-not-requ.patch \
> file://0008-omap4-pandaboard-add-support-for-DVI-output.patch \
> + file://0001-UBUNTU-Config-Fix-FTBS-caused-by-new-binutils.patch \
> + file://0002-Add-AFLAGS-for-sleep44xx-to-fix-FTBS.patch \
> file://defconfig"
>
> S = "${WORKDIR}/git"
> --
> 1.7.0.2
>
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] u-boot-sakoman: fix break caused by the new binutils
[not found] ` <1295882416.1999.8.camel@quadra>
@ 2011-01-24 17:18 ` Víctor M. Jáquez L.
[not found] ` <1295890383.1999.11.camel@quadra>
0 siblings, 1 reply; 7+ messages in thread
From: Víctor M. Jáquez L. @ 2011-01-24 17:18 UTC (permalink / raw)
To: Steve Sakoman; +Cc: openembedded-devel
On Mon, Jan 24, 2011 at 07:20:16AM -0800, Steve Sakoman wrote:
> On Mon, 2011-01-24 at 15:48 +0100, Víctor Manuel Jáquez Leal wrote:
> > New assemblers need -march=armv7-a+sec on command line or .arch_extension sec
> > inline to enable use of the smc instruction.
> >
> > This patch adds as-instr to check the latter to conditionally enable the
> > former in AFLAGS for files that use smc.
>
> Since this is a generic u-boot issue, it would be best to submit the
> Makefile patch to the u-boot mailing list.
>
These patches depend on these ones:
http://www.sakoman.com/cgi-bin/gitweb.cgi?p=u-boot.git;a=commitdiff;h=2a9a9da5530f5bc9cb09ffa2fe8e32494d1a8b0f
and
http://www.sakoman.com/cgi-bin/gitweb.cgi?p=u-boot.git;a=commitdiff;h=268fff79ea11b2bb6f174c3c97c104768b5cd7e3
but neither are in the denx's git repository, neither in its patchwork.
So, I'm not sure what should I do. Any advise?
vmjl
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] u-boot-sakoman: fix break caused by the new binutils
[not found] ` <1295890383.1999.11.camel@quadra>
@ 2011-01-24 18:37 ` Víctor M. Jáquez L.
0 siblings, 0 replies; 7+ messages in thread
From: Víctor M. Jáquez L. @ 2011-01-24 18:37 UTC (permalink / raw)
To: Steve Sakoman; +Cc: openembedded-devel
On Mon, Jan 24, 2011 at 09:33:03AM -0800, Steve Sakoman wrote:
> On Mon, 2011-01-24 at 18:18 +0100, Víctor M. Jáquez L. wrote:
> > On Mon, Jan 24, 2011 at 07:20:16AM -0800, Steve Sakoman wrote:
> > > On Mon, 2011-01-24 at 15:48 +0100, Víctor Manuel Jáquez Leal wrote:
> > > > New assemblers need -march=armv7-a+sec on command line or .arch_extension sec
> > > > inline to enable use of the smc instruction.
> > > >
> > > > This patch adds as-instr to check the latter to conditionally enable the
> > > > former in AFLAGS for files that use smc.
> > >
> > > Since this is a generic u-boot issue, it would be best to submit the
> > > Makefile patch to the u-boot mailing list.
> > >
> >
> > These patches depend on these ones:
> >
> > http://www.sakoman.com/cgi-bin/gitweb.cgi?p=u-boot.git;a=commitdiff;h=2a9a9da5530f5bc9cb09ffa2fe8e32494d1a8b0f
> >
> > and
> >
> > http://www.sakoman.com/cgi-bin/gitweb.cgi?p=u-boot.git;a=commitdiff;h=268fff79ea11b2bb6f174c3c97c104768b5cd7e3
> >
> > but neither are in the denx's git repository, neither in its patchwork.
> >
> > So, I'm not sure what should I do. Any advise?
>
> Yeah, Aneesh is still revising those patches to respond to list
> feedback.
>
> You could submit your patch in a RFC form with a note that it depends on
> the cache maintenance patches that Aneesh is revising.
>
> That way you can get some comments on the basic idea while waiting for
> Aneesh's patches to be accepted. At that point you could submit a final
> patch for approval.
Done. Thanks a lot! :)
vmjl
>
> Steve
>
>
>
>
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2011-01-24 18:38 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-01-24 14:48 [PATCH 0/2] fixes for binutils 2.21 Víctor Manuel Jáquez Leal
2011-01-24 14:48 ` [PATCH 1/2] u-boot-sakoman: fix break caused by the new binutils Víctor Manuel Jáquez Leal
2011-01-24 15:47 ` Khem Raj
[not found] ` <1295882416.1999.8.camel@quadra>
2011-01-24 17:18 ` Víctor M. Jáquez L.
[not found] ` <1295890383.1999.11.camel@quadra>
2011-01-24 18:37 ` Víctor M. Jáquez L.
2011-01-24 14:48 ` [PATCH 2/2] linux-omap4: " Víctor Manuel Jáquez Leal
2011-01-24 15:48 ` Khem Raj
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.