From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from bear.ext.ti.com (bear.ext.ti.com [192.94.94.41]) by yocto-www.yoctoproject.org (Postfix) with ESMTP id CB060E013B0 for ; Tue, 9 Jul 2013 12:07:57 -0700 (PDT) Received: from dlelxv90.itg.ti.com ([172.17.2.17]) by bear.ext.ti.com (8.13.7/8.13.7) with ESMTP id r69J7vfo001343 for ; Tue, 9 Jul 2013 14:07:57 -0500 Received: from DFLE72.ent.ti.com (dfle72.ent.ti.com [128.247.5.109]) by dlelxv90.itg.ti.com (8.14.3/8.13.8) with ESMTP id r69J7v2D000643 for ; Tue, 9 Jul 2013 14:07:57 -0500 Received: from dlelxv22.itg.ti.com (172.17.1.197) by DFLE72.ent.ti.com (128.247.5.109) with Microsoft SMTP Server id 14.2.342.3; Tue, 9 Jul 2013 14:07:56 -0500 Received: from localhost (gtudedge.gt.design.ti.com [158.218.102.158]) by dlelxv22.itg.ti.com (8.13.8/8.13.8) with ESMTP id r69J7ut5018897; Tue, 9 Jul 2013 14:07:56 -0500 Date: Tue, 9 Jul 2013 15:07:56 -0400 From: Denys Dmytriyenko To: "Cooper Jr., Franklin" Message-ID: <20130709190756.GO14185@edge> References: <1372783680-32302-1-git-send-email-fcooper@ti.com> <1372783680-32302-2-git-send-email-fcooper@ti.com> <7D46E86EC0A8354091174257B2FED101596306DD@DLEE11.ent.ti.com> <20130709165347.GL14185@edge> <8F29D6B095ED194EA1980491A5E029710C4F177C@DFLE08.ent.ti.com> MIME-Version: 1.0 In-Reply-To: <8F29D6B095ED194EA1980491A5E029710C4F177C@DFLE08.ent.ti.com> User-Agent: Mutt/1.5.20 (2009-06-14) Cc: "meta-ti@yoctoproject.org" Subject: Re: [master & danny][PATCH 2/2] u-boot: Add memory alignment fix for armv7 for am37x and am3517 X-BeenThere: meta-ti@yoctoproject.org X-Mailman-Version: 2.1.13 Precedence: list List-Id: Usage and development list for the meta-ti layer List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 09 Jul 2013 19:07:58 -0000 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline On Tue, Jul 09, 2013 at 03:04:57PM -0400, Cooper Jr., Franklin wrote: > > > > -----Original Message----- > > From: Dmytriyenko, Denys > > Sent: Tuesday, July 09, 2013 11:54 AM > > To: Maupin, Chase > > Cc: Cooper Jr., Franklin; meta-ti@yoctoproject.org > > Subject: Re: [meta-ti] [master & danny][PATCH 2/2] u-boot: Add memory > > alignment fix for armv7 for am37x and am3517 > > > > On Tue, Jul 02, 2013 at 07:09:54PM +0000, Maupin, Chase wrote: > > > Couldn't you only use one copy of the patch file and place it in a > > > more generic directory? Not a big issue but just asking. > > > > Franklin, > > > > Any comment for the above question? > Sent a v2 addressing Chase's comment. Thanks! > > Also, please remind me in 2 days to apply your patches. I'm not sure I will have > > time today or tomorrow. Thanks. > Will do! > > > > -- > > Denys > > > > > > > >-----Original Message----- > > > >From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti- > > > >bounces@yoctoproject.org] On Behalf Of Cooper Jr., Franklin > > > >Sent: Tuesday, July 02, 2013 11:48 AM > > > >To: meta-ti@yoctoproject.org > > > >Subject: [meta-ti] [master & danny][PATCH 2/2] u-boot: Add memory > > > >alignment fix for armv7 for am37x and am3517 > > > > > > > >* On am335x a memory alignment issue was found when using the > > > >Linaro toolchain. > > > >* A similar patch was applied for am335x that fixed this well > > > >documented issue. > > > >* Add a similar fix to am37x and am3517 since this is a generic > > > >armv7 issue. > > > > > > > >Signed-off-by: Franklin S. Cooper Jr > > > >--- > > > > .../0001-armv7-Unaligned-access-fix.patch | 30 > > > >++++++++++++++++++++ > > > > recipes-bsp/u-boot/u-boot-am3517_2011.09.bb | 5 ++- > > > > .../0001-armv7-Unaligned-access-fix.patch | 30 > > > >++++++++++++++++++++ > > > > recipes-bsp/u-boot/u-boot-am37x_2012.04.01.bb | 3 ++ > > > > 4 files changed, 66 insertions(+), 2 deletions(-) > > > > create mode 100644 recipes-bsp/u-boot/u-boot-am3517-2011.09/0001- > > > >armv7-Unaligned-access-fix.patch > > > > create mode 100644 recipes-bsp/u-boot/u-boot-am37x- > > > >2012.04.01/0001-armv7-Unaligned-access-fix.patch > > > > > > > >diff --git a/recipes-bsp/u-boot/u-boot-am3517-2011.09/0001-armv7- > > > >Unaligned-access-fix.patch b/recipes-bsp/u-boot/u-boot-am3517- > > > >2011.09/0001-armv7-Unaligned-access-fix.patch > > > >new file mode 100644 > > > >index 0000000..35307ac > > > >--- /dev/null > > > >+++ b/recipes-bsp/u-boot/u-boot-am3517-2011.09/0001-armv7- > > > >Unaligned-access-fix.patch > > > >@@ -0,0 +1,30 @@ > > > >+From 40f1e7db785f477a2aa145a3b47bd498c48ad52d Mon Sep 17 > > 00:00:00 > > > >2001 > > > >+From: Steve Kipisz > > > >+Date: Fri, 14 Jun 2013 05:16:52 -0500 > > > >+Subject: [PATCH] armv7:Unaligned access fix > > > >+ > > > >+This fixes an unaligned access data abort introduced when using > > > >gcc 4.7 > > > >+or higher. The Linaro toolchain uses gcc 4.7.3. This is > > > >documented in u-boot > > > >+doc/README.arm-unaligned-accesses. > > > >+ > > > >+Signed-off-by: Steve Kipisz > > > >+--- > > > >+ arch/arm/cpu/armv7/config.mk | 2 +- > > > >+ 1 file changed, 1 insertion(+), 1 deletion(-) > > > >+ > > > >+diff --git a/arch/arm/cpu/armv7/config.mk > > > >b/arch/arm/cpu/armv7/config.mk > > > >+index 9c3e2f3..9736da8 100644 > > > >+--- a/arch/arm/cpu/armv7/config.mk > > > >++++ b/arch/arm/cpu/armv7/config.mk > > > >+@@ -20,7 +20,7 @@ > > > >+ # Foundation, Inc., 59 Temple Place, Suite 330, Boston, > > > >+ # MA 02111-1307 USA > > > >+ # > > > >+-PLATFORM_RELFLAGS += -fno-common -ffixed-r8 -msoft-float > > > >++PLATFORM_RELFLAGS += -fno-common -ffixed-r8 -msoft-float -mno- > > > >unaligned-access > > > >+ > > > >+ # If armv7-a is not supported by GCC fall-back to armv5, which > > > >is > > > >+ # supported by more tool-chains > > > >+-- > > > >+1.7.9.5 > > > >+ > > > >diff --git a/recipes-bsp/u-boot/u-boot-am3517_2011.09.bb > > > >b/recipes-bsp/u-boot/u-boot-am3517_2011.09.bb > > > >index 6321406..02019e9 100644 > > > >--- a/recipes-bsp/u-boot/u-boot-am3517_2011.09.bb > > > >+++ b/recipes-bsp/u-boot/u-boot-am3517_2011.09.bb > > > >@@ -6,9 +6,10 @@ COMPATIBLE_MACHINE = "omap3" > > > > > > > > DEFAULT_PREFERENCE = "-1" > > > > > > > >-PR = "r0+gitr${SRCPV}" > > > >+PR = "r1+gitr${SRCPV}" > > > > > > > >-SRC_URI = "git://arago-project.org/git/projects/u-boot- > > > >am33x.git;protocol=git;branch=${BRANCH}" > > > >+SRC_URI = "git://arago-project.org/git/projects/u-boot- > > > >am33x.git;protocol=git;branch=${BRANCH} \ > > > >+ file://0001-armv7-Unaligned-access-fix.patch" > > > > > > > > BRANCH = "AM335XPSP_04.06.00.08" > > > > > > > >diff --git a/recipes-bsp/u-boot/u-boot-am37x-2012.04.01/0001- > > > >armv7-Unaligned-access-fix.patch b/recipes-bsp/u-boot/u-boot- > > > >am37x-2012.04.01/0001-armv7-Unaligned-access-fix.patch > > > >new file mode 100644 > > > >index 0000000..35307ac > > > >--- /dev/null > > > >+++ b/recipes-bsp/u-boot/u-boot-am37x-2012.04.01/0001-armv7- > > > >Unaligned-access-fix.patch > > > >@@ -0,0 +1,30 @@ > > > >+From 40f1e7db785f477a2aa145a3b47bd498c48ad52d Mon Sep 17 > > 00:00:00 > > > >2001 > > > >+From: Steve Kipisz > > > >+Date: Fri, 14 Jun 2013 05:16:52 -0500 > > > >+Subject: [PATCH] armv7:Unaligned access fix > > > >+ > > > >+This fixes an unaligned access data abort introduced when using > > > >gcc 4.7 > > > >+or higher. The Linaro toolchain uses gcc 4.7.3. This is > > > >documented in u-boot > > > >+doc/README.arm-unaligned-accesses. > > > >+ > > > >+Signed-off-by: Steve Kipisz > > > >+--- > > > >+ arch/arm/cpu/armv7/config.mk | 2 +- > > > >+ 1 file changed, 1 insertion(+), 1 deletion(-) > > > >+ > > > >+diff --git a/arch/arm/cpu/armv7/config.mk > > > >b/arch/arm/cpu/armv7/config.mk > > > >+index 9c3e2f3..9736da8 100644 > > > >+--- a/arch/arm/cpu/armv7/config.mk > > > >++++ b/arch/arm/cpu/armv7/config.mk > > > >+@@ -20,7 +20,7 @@ > > > >+ # Foundation, Inc., 59 Temple Place, Suite 330, Boston, > > > >+ # MA 02111-1307 USA > > > >+ # > > > >+-PLATFORM_RELFLAGS += -fno-common -ffixed-r8 -msoft-float > > > >++PLATFORM_RELFLAGS += -fno-common -ffixed-r8 -msoft-float -mno- > > > >unaligned-access > > > >+ > > > >+ # If armv7-a is not supported by GCC fall-back to armv5, which > > > >is > > > >+ # supported by more tool-chains > > > >+-- > > > >+1.7.9.5 > > > >+ > > > >diff --git a/recipes-bsp/u-boot/u-boot-am37x_2012.04.01.bb > > > >b/recipes-bsp/u-boot/u-boot-am37x_2012.04.01.bb > > > >index c25cf1e..1cd96b6 100644 > > > >--- a/recipes-bsp/u-boot/u-boot-am37x_2012.04.01.bb > > > >+++ b/recipes-bsp/u-boot/u-boot-am37x_2012.04.01.bb > > > >@@ -6,6 +6,8 @@ COMPATIBLE_MACHINE = "am37x-evm|beagleboard" > > > > > > > > BRANCH ?= "master" > > > > > > > >+PR = "r1+gitr${SRCPV}" > > > >+ > > > > # v2012.04.01 > > > > SRCREV = "415d386877df49eb051b85ef74fa59a16dc17c7d" > > > > > > > >@@ -33,6 +35,7 @@ SRC_URI += "file://0001-omap3_beagle-add- > > > >usbethaddr-setting-to-enable-networ.pat > > > > file://0002-omap3evm-Make-the-board-start-at- > > > >800MHz.patch \ > > > > file://0003-beagleboard-Make-xM-rev-C-go-to- > > > >800MHz.patch \ > > > > file://0001-ARM-omap3-Set-SPL-stack-size-to-8KB- > > > >image-to-54KB.patch \ > > > >+ file://0001-armv7-Unaligned-access-fix.patch \ > > > > " > > > > > > > > SPL_BINARY = "MLO" > > > >-- > > > >1.7.0.4 > > > > > > > >_______________________________________________ > > > >meta-ti mailing list > > > >meta-ti@yoctoproject.org > > > >https://lists.yoctoproject.org/listinfo/meta-ti > > > _______________________________________________ > > > meta-ti mailing list > > > meta-ti@yoctoproject.org > > > https://lists.yoctoproject.org/listinfo/meta-ti