[OE-core] [PATCH] goarch.bbclass: Replace logic for setting GOARM

Will Newton will.newton at gmail.com
Fri Sep 8 08:53:25 UTC 2017


On Thu, Sep 7, 2017 at 7:17 PM, Andre McCurdy <armccurdy at gmail.com> wrote:
> On Thu, Sep 7, 2017 at 5:31 AM, Will Newton <will.newton at gmail.com> wrote:
>> The previous logic applied a regex to TUNE_FEATURES which could
>> set the GOARM value to 7 incorrectly, for example when dealing
>> with an arm1176 core. Simplify to check for the presence of
>> "armv7a" instead. At the same time add a check for "armv6" and
>> set GOARM to 6 in that case.
>>
>> Signed-off-by: Will Newton <willn at resin.io>
>> ---
>>  meta/classes/goarch.bbclass | 7 +++++--
>>  1 file changed, 5 insertions(+), 2 deletions(-)
>>
>> diff --git a/meta/classes/goarch.bbclass b/meta/classes/goarch.bbclass
>> index 4a5b2ec..35f2987 100644
>> --- a/meta/classes/goarch.bbclass
>> +++ b/meta/classes/goarch.bbclass
>> @@ -38,8 +38,11 @@ def go_map_arch(a, d):
>>
>>  def go_map_arm(a, f, d):
>>      import re
>> -    if re.match('arm.*', a) and re.match('arm.*7.*', f):
>> -        return '7'
>> +    if re.match('arm.*', a):
>> +        if 'armv7a' in f:
>
> You need to test for both armv7a (Cortex A9, etc) and armv7ve (Cortex
> A7, A15, etc).

Thanks, I've done a v2 that should take care of that.



More information about the Openembedded-core mailing list