Re: [oe-commits] Andy Voltz : tune-cortexa.inc: create a common include for cortex-a armv7a tuning

[Andy Voltz <andy.voltz@timesys.com>]

Hi Martin,

> On Sat, Jun 08, 2013 at 11:10:20AM +0200, Martin Jansa wrote:

> > Something is wrong with this one:
> > 
> > ERROR:  OE-core's config sanity checker detected a potential misconfiguration.
> >     Either fix the cause of this error or at your own risk disable the checker (see sanity.conf).
> >     Following is the list of potential problems / advisories:
> > 
> >     Error, the PACKAGE_ARCHS variable does not contain TUNE_PKGARCH (${ARMPKGARCH_tune-cortexa8t-neon}).Toolchain tunings invalid:
> > Tuning 'cortexa8t-neon' has no defined features, and cannot be used.
> > 
> > ERROR: Execution of event handler 'check_sanity_eventhandler' failed
> > 
> > Please use tests like
> > http://git.openembedded.org/openembedded-core-contrib/log/?h=jansa/tune2-test
> > to check that all tunes are still valid after such change.
> 

I was not aware of this test. I'll look into this.

> Also it was using -mtune and documentation talks about -mtune, but
> actual tune-cortexa.inc is using -mcpu, this change wasn't mentioned in
> commit message and I don't think it's improvement.
> 

*facepalm*

This is actually my mistake when I squashed my cortex-a5 file into a general
cortexa file. I meant to keep the -mtune in place. I started out using mcpu
before realizing mtune was also valid for cortex-a5.

I have a patch to revert that change / go back to -mtune. I will send it after
I rebuild for cortex-a5.

Sorry about that. It didn't come out because the build was happy with -mcpu=cortex-a5
-- 
Andy Voltz
Timesys Corporation