Ping. Max
2016-07-27 20:10 GMT+02:00 Khem Raj <raj.k...@gmail.com>: > >> On Jul 27, 2016, at 10:55 AM, Max Krummenacher <max.oss...@gmail.com> wrote: >> >> _armv4t, _armv5te are not (no longer?) set as MACHINEOVERRIDES if an armv4t >> or armv5te tune is set. >> Thus the default ARM_INSTRUCTION_SET ??= "thumb" kicks in. >> >> Set the override to _armv4, _armv5 to get the expected ARM_INSTRUCTION_SET. >> >> While at it cleanup comment to keep up with changes. >> >> Build tested with a build for a _armv5te machine. > > this is ok. > >> >> Signed-off-by: Max Krummenacher <max.krummenac...@toradex.com> >> --- >> conf/distro/include/angstrom.inc | 19 +++++++++---------- >> 1 file changed, 9 insertions(+), 10 deletions(-) >> >> diff --git a/conf/distro/include/angstrom.inc >> b/conf/distro/include/angstrom.inc >> index 778ab10..09f6121 100644 >> --- a/conf/distro/include/angstrom.inc >> +++ b/conf/distro/include/angstrom.inc >> @@ -46,16 +46,6 @@ require conf/distro/include/angstrom-${TCLIBC}.inc >> require conf/distro/include/angstrom-java.inc >> require conf/distro/include/angstrom-jalimo.conf >> >> -# ARM920T and up can use thumb mode to decrease binary size at the expense >> of speed >> -# (the complete story is a bit more nuanced due to cache starvation) >> -# Angstrom turns on thumb for armv4t machine according to this RFC: >> -# >> http://lists.linuxtogo.org/pipermail/angstrom-distro-devel/2008-October/002714.html >> - >> -ARM_INSTRUCTION_SET ??= "thumb" >> -ARM_INSTRUCTION_SET_armv4t ?= "arm" >> -ARM_INSTRUCTION_SET_armv5te ?= "arm" >> -ARM_INSTRUCTION_SET_armv6 ?= "arm" >> - >> # "arm" "thumb" >> # The instruction set the compiler should use when generating application >> # code. The kernel is always compiled with arm code at present. arm code >> @@ -63,6 +53,15 @@ ARM_INSTRUCTION_SET_armv6 ?= "arm" >> # encoded RISC sub-set. Thumb code is smaller (maybe 70% of the ARM size) >> # but requires more instructions (140% for 70% smaller code) so may be >> # slower. >> +# thumb 2 is an evolution of the original thumb instruction set which >> strives >> +# to keep the code size advantages without the performance penality. >> +# When setting thumb the compiler will use thumb or thumb2 depending on >> what >> +# the selected arm architecture. >> + >> +ARM_INSTRUCTION_SET ??= "thumb" >> +ARM_INSTRUCTION_SET_armv4 ?= "arm" >> +ARM_INSTRUCTION_SET_armv5 ?= "arm" >> +ARM_INSTRUCTION_SET_armv6 ?= "arm" >> >> #Use this variable in feeds and other parts that need a URI >> ANGSTROM_URI ?= "http://feeds.angstrom-distribution.org" >> -- >> 2.5.5 >> >> >> _______________________________________________ >> Angstrom-distro-devel mailing list >> Angstrom-distro-devel@linuxtogo.org >> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/angstrom-distro-devel > > > _______________________________________________ > Angstrom-distro-devel mailing list > Angstrom-distro-devel@linuxtogo.org > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/angstrom-distro-devel > _______________________________________________ Angstrom-distro-devel mailing list Angstrom-distro-devel@linuxtogo.org http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/angstrom-distro-devel