[meta-ti] [master & danny][PATCH 2/2] u-boot: Add memory alignment fix for armv7 for am37x and am3517

Denys Dmytriyenko denys at ti.com
Tue Jul 9 12:07:56 PDT 2013 

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 at 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 at yoctoproject.org [mailto:meta-ti-
> > > >bounces at yoctoproject.org] On Behalf Of Cooper Jr., Franklin
> > > >Sent: Tuesday, July 02, 2013 11:48 AM
> > > >To: meta-ti at 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 <fcooper at ti.com>
> > > >---
> > > > .../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 <s-kipisz2 at ti.com>
> > > >+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 <s-kipisz2 at ti.com>
> > > >+---
> > > >+ 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 <s-kipisz2 at ti.com>
> > > >+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 <s-kipisz2 at ti.com>
> > > >+---
> > > >+ 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 at yoctoproject.org
> > > >https://lists.yoctoproject.org/listinfo/meta-ti
> > > _______________________________________________
> > > meta-ti mailing list
> > > meta-ti at yoctoproject.org
> > > https://lists.yoctoproject.org/listinfo/meta-ti

More information about the meta-ti mailing list