All of lore.kernel.org
 help / color / mirror / Atom feed
From: Khem Raj <raj.khem@gmail.com>
To: openembedded-devel@lists.openembedded.org
Cc: Koen Kooi <koen@openembedded.org>, Koen Kooi <k-kooi@ti.com>
Subject: Re: [PATCH 2/2] linux-omap4: fix break caused by the new binutils
Date: Mon, 24 Jan 2011 07:48:09 -0800	[thread overview]
Message-ID: <AANLkTinMDTBOJD6WE09HaqXedWau8r_PURYUMxhNM5WN@mail.gmail.com> (raw)
In-Reply-To: <d51a8cfd5ef81f649c82d490a2b9359c47735250.1295880053.git.vjaquez@igalia.com>

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
>



      reply	other threads:[~2011-01-24 15:49 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
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 message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=AANLkTinMDTBOJD6WE09HaqXedWau8r_PURYUMxhNM5WN@mail.gmail.com \
    --to=raj.khem@gmail.com \
    --cc=k-kooi@ti.com \
    --cc=koen@openembedded.org \
    --cc=openembedded-devel@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.