Skip to contentSkip to conversation

am33x-cm3: Enforce correct -march option

PATCH New Thomas Petazzoni 2 participants 2 messages Jun 9, 2013

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 am33x-cm3: Enforce correct -march option

Buildroot uses a normal ARM compiler to build the AM33X CM3 firmware (which runs on a Cortex-M3 processor), but Buildroot will have a default -march value that doesn't necessarily match the one needed for Cortex-M3, leading to build failures (gcc complains that the -mcpu=cortex-m3 option being passed is not compatible with the selected -march).

Fix this by explicitly indicating -march=armv7-m.

Fixes:

http://autobuild.buildroot.net/results/db1/db13623b1701bfe678c302e8f88c7473cb2345f9/build-end.log

---
 .../am33x-cm3/am33x-cm3-0001-fix-makefile.patch    | 26 ++++++++++++++++++++++
 1 file changed, 26 insertions(+)
 create mode 100644 package/am33x-cm3/am33x-cm3-0001-fix-makefile.patch
package/am33x-cm3/am33x-cm3-0001-fix-makefile.patch new file +26 -0
new file mode 100644
index 0000000..77ba11c
@@ -0,0 +1,26 @@
+Enforce correct -march option
+
+Buildroot uses a normal ARM compiler to build the AM33X CM3 firmware
+(which runs on a Cortex-M3 processor), but Buildroot will have a
+default -march value that doesn't necessarily match the one needed for
+Cortex-M3, leading to build failures (gcc complains that the
+-mcpu=cortex-m3 option being passed is not compatible with the
+selected -march).
+
+Fix this by explicitly indicating -march=armv7-m.
+
+Signed-off-by: Thomas Petazzoni <[email protected]>
+
+Index: b/Makefile
+===================================================================
+--- a/Makefile
++++ b/Makefile
+@@ -13,7 +13,7 @@
+ BINDIR = bin
+ 
+ INCLUDES = $(SRCDIR)/include
+-CFLAGS =-mcpu=cortex-m3 -mthumb -nostdlib -Wall -g -I$(INCLUDES)
++CFLAGS =-march=armv7-m -mcpu=cortex-m3 -mthumb -nostdlib -Wall -g -I$(INCLUDES)
+ LDFLAGS =-nostartfiles -fno-exceptions -Tfirmware.ld
+ 
+ EXECUTABLE=am335x-pm-firmware.elf
Signature · 1 line
1.8.1.2
#1 lore raw

PATCH 1/1 am33x-cm3: Enforce correct -march option

Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Thomas" == Thomas Petazzoni <[email protected]> writes:

Thomas> Buildroot uses a normal ARM compiler to build the AM33X CM3 firmware Thomas> (which runs on a Cortex-M3 processor), but Buildroot will have a Thomas> default -march value that doesn't necessarily match the one needed for Thomas> Cortex-M3, leading to build failures (gcc complains that the Thomas> -mcpu=cortex-m3 option being passed is not compatible with the Thomas> selected -march).

Thomas> Fix this by explicitly indicating -march=armv7-m.

Committed, thanks.

Signature · 1 line
Bye, Peter Korsgaard