[Buildroot] [PATCH] valgrind: allow building on more ARM systems

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Wed Oct 1 17:19:33 UTC 2014


Dear Thomas De Schampheleire,

On Wed, 01 Oct 2014 18:30:54 +0200, Thomas De Schampheleire wrote:

> >diff --git a/package/valgrind/valgrind.mk b/package/valgrind/valgrind.mk
> >index 24d0290..8bea031 100644
> >--- a/package/valgrind/valgrind.mk
> >+++ b/package/valgrind/valgrind.mk
> >@@ -17,7 +17,7 @@ VALGRIND_AUTORECONF = YES
> > # not. Therefore, we adjust the host tuple to specify we're on
> > # ARMv7. The valgrind package is guaranteed, through Config.in, to
> > # only be selected on Cortex A8 and Cortex A9 platforms.
> >-ifeq ($(BR2_cortex_a8)$(BR2_cortex_a9),y)
> >+ifeq ($(BR2_cortex_a5)$(BR2_cortex_a7)$(BR2_cortex_a8)$(BR2_cortex_a9)$(BR2_cortex_a12)$(BR2_cortex_a15),y)
> > VALGRIND_CONF_OPT += \
> > 	--host=$(patsubst arm-%,armv7-%,$(GNU_TARGET_NAME))
> > endif
> 
> In fact, mips64+n32 abi is not supported, so I think
>  this should be reflected too. I guess the same applies
>  to the the le variant...

Right, but it's not an issue introduced by this new patch.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com


More information about the buildroot mailing list