[meta-freescale] [meta-fsl-arm][PATCH 08/33] imx-base: Add mx7 support to U-Boot, MACHINE filter and DEFAULT_TUNE

[Max Krummenacher]

Hi Lauren



2015-07-12 19:25 GMT+02:00 Lauren Post <Lauren.Post at freescale.com>:
> This is the tune feature we tested with for mx7.  I can investigate changing it going forward but it will not align to what was tested in this release (and in the upcoming beta for 7d).  I prefer to change it once we have a test cycle with the change.
>
> Lauren
>

Whatever fits your QS process best.

I don't think the change is intrusive as long as no package is
actually built using thumb but on never knows until one actually did
the build.

Regards
Max

> -----Original Message-----
> From: Max Krummenacher [mailto:max.oss.09 at gmail.com]
> Sent: Sunday, July 12, 2015 5:23 AM
> To: Post Lauren-RAA013
> Cc: meta-freescale at yoctoproject.org
> Subject: Re: [meta-freescale] [meta-fsl-arm][PATCH 08/33] imx-base: Add mx7 support to U-Boot, MACHINE filter and DEFAULT_TUNE
>
> Hi Lauren
>
> Am Samstag, den 11.07.2015, 09:46 -0500 schrieb Lauren Post:
>> Add the mx7 settings for U-boot, MACHINE and DEFAULT_TUNE.
>>
>> MX7 is a new i.MX SoC Family created with the 3.14.28-7D_alpha release
>> and upstreamed with the 3.14.38-6QP_beta release.
>>
>> i.MX 7Dual is a CortexA7 ARM core so requires different DEFAULT_TUNE
>> however it does build with the cortexa9 but not as optimized as it is
>> with the cortexa7 toolchain.
>>
>> Signed-off-by: Lauren Post <lauren.post at freescale.com>
>> ---
>>  conf/machine/include/imx-base.inc |    3 +++
>>  1 file changed, 3 insertions(+)
>>
>> diff --git a/conf/machine/include/imx-base.inc
>> b/conf/machine/include/imx-base.inc
>> index f64ba6b..83b771b 100644
>> --- a/conf/machine/include/imx-base.inc
>> +++ b/conf/machine/include/imx-base.inc
>> @@ -40,6 +41,7 @@ MACHINE_EXTRA_RRECOMMENDS = "kernel-modules"
>>  # DEFAULTTUNE_mx6 ?= "cortexa9t-neon"
>>  # handled by hardware
>>  DEFAULTTUNE_mx6 ?= "cortexa9thf-neon"
>> +DEFAULTTUNE_mx7 ?= "cortexa7hf-neon"
>>
>
> Is there a reason to disallow thumb?
> the t in "cortexa9thf-neon" has recently be added.
> https://lists.yoctoproject.org/pipermail/meta-freescale/2015-May/014042.html
>
>
> Regards
> Max
>