[OE-core] [PATCH] arch-powerpc.inc: use default value of TUNE_PKGARCH

[McClintock Matthew-B29882]

On Fri, Mar 9, 2012 at 10:45 AM, Mark Hatle <mark.hatle at windriver.com> wrote:
> On 3/8/12 2:15 PM, Richard Purdie wrote:
>>
>> On Tue, 2012-03-06 at 17:20 -0600, Matthew McClintock wrote:
>>>
>>> We can use the default value for TUNE_PKGARCH, and now we just
>>> append "-nf" if TARGET_FPU is fpu-soft
>>>
>>> Signed-off-by: Matthew McClintock<msm at freescale.com>
>>> ---
>>>  meta/conf/machine/include/powerpc/arch-powerpc.inc |    6 ++----
>>>  1 files changed, 2 insertions(+), 4 deletions(-)
>>>
>>> diff --git a/meta/conf/machine/include/powerpc/arch-powerpc.inc
>>> b/meta/conf/machine/include/powerpc/arch-powerpc.inc
>>> index c9b2829..abd72ed 100644
>>> --- a/meta/conf/machine/include/powerpc/arch-powerpc.inc
>>> +++ b/meta/conf/machine/include/powerpc/arch-powerpc.inc
>>> @@ -18,10 +18,8 @@ TARGET_FPU .= "${@bb.utils.contains("TUNE_FEATURES",
>>> "fpu-soft", "soft", "", d)}
>>>
>>>  ABIEXTENSION = "${@['','spe'][d.getVar('TARGET_FPU', True) in
>>> ['ppc-efd', 'ppc-efs']]}"
>>>
>>> -PPCPKGSFX_FPU = "${@bb.utils.contains("TUNE_FEATURES", "fpu-hard" , "",
>>> "-nf", d)}"
>>> -
>>> -PPCPKGARCH = "${TUNE_ARCH}${PPCPKGSFX_FPU}"
>>> -TUNE_PKGARCH ?= "${PPCPKGARCH}"
>>> +PPCPKGSFX_FPU = "${@['', '-nf'][bb.data.getVar('TARGET_FPU',d,1) in
>>> ['fpu-soft']]}"
>>> +TUNE_PKGARCH_append = "${PPCPKGSFX_FPU}"
>>
>>
>> Can we please do d.getVar(xxx, True) and not bb.data and ,1 :)
>
>
> This was originally done with a check for fpu-hard and not fpu-soft to
> ensure that fpu-soft became the default value when one or the other wasn't
> set.
>
> I don't know if it really matters which is used, but that is why it was done
> that way.

That logic was not quite working for us - since we don't set fpu-hard,
but rather ppc-efd and ppc-efs... so I had to invert this...

-M