Re: [OE-core] [PATCH] x264: add textrel to INSANE_SKIP for cyclone5 machine

2016-09-15 Thread sujith h 
On Thu, Sep 15, 2016 at 12:15 PM, sujith h  wrote:

>
>
> On Thu, Sep 15, 2016 at 10:13 AM, sujith h  wrote:
>
>>
>>
>> On Thu, Sep 15, 2016 at 4:41 AM, Burton, Ross 
>> wrote:
>>
>>>
>>> On 12 September 2016 at 07:13, Sujith H  wrote:
>>>
 +# PIC can't be enabled for 32-bit x86 and cyclone5
  INSANE_SKIP_${PN}_append_x86 = " textrel"
 +INSANE_SKIP_${PN}_append_cyclone5 = " textrel"

>>>
>>> Can this be generalised or is it absolutely specific to cyclone5?  If it
>>> can't be generalised then I feel that this should belong in the cyclone5
>>> BSP.
>>>
>>
>> We found this with cyclone5 BSP. I will try to figure out how if we can
>> see it with qemuarm or other BSP's too.
>>
>
> I tried with qemuarm, it wasn't reproducible. But with imx6qsabresd from
> meta-fsl-arm, it was reproducible. So I believe we can generalize this
> patch. I will send the updated one shortly. Thanks for the pointer Ross.
>

I forgot to mention that I posted this change in meta-altera for review. I
got the feedback from meta-altera to post the change for review here:
https://github.com/kraj/meta-altera/pull/37. Sharing this for the reference.


>
>>
>>>
>>> Ross
>>>
>>
>>
>>
>> --
>> സുജിത് ഹരിദാസന്
>> Bangalore
>> Contributor to KDE project
>> Contributor to Yocto project
>> http://fci.wikia.com/wiki/Anti-DRM-Campaign
>>  http://sujithh.info
>> C-x C-c
>>
>
>
>
> --
> സുജിത് ഹരിദാസന്
> Bangalore
> Contributor to KDE project
> Contributor to Yocto project
> http://fci.wikia.com/wiki/Anti-DRM-Campaign
>  http://sujithh.info
> C-x C-c
>



-- 
സുജിത് ഹരിദാസന്
Bangalore
Contributor to KDE project
Contributor to Yocto project
http://fci.wikia.com/wiki/Anti-DRM-Campaign
 http://sujithh.info
C-x C-c
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] x264: add textrel to INSANE_SKIP for cyclone5 machine

2016-09-15 Thread sujith h 
On Thu, Sep 15, 2016 at 10:13 AM, sujith h  wrote:

>
>
> On Thu, Sep 15, 2016 at 4:41 AM, Burton, Ross 
> wrote:
>
>>
>> On 12 September 2016 at 07:13, Sujith H  wrote:
>>
>>> +# PIC can't be enabled for 32-bit x86 and cyclone5
>>>  INSANE_SKIP_${PN}_append_x86 = " textrel"
>>> +INSANE_SKIP_${PN}_append_cyclone5 = " textrel"
>>>
>>
>> Can this be generalised or is it absolutely specific to cyclone5?  If it
>> can't be generalised then I feel that this should belong in the cyclone5
>> BSP.
>>
>
> We found this with cyclone5 BSP. I will try to figure out how if we can
> see it with qemuarm or other BSP's too.
>

I tried with qemuarm, it wasn't reproducible. But with imx6qsabresd from
meta-fsl-arm, it was reproducible. So I believe we can generalize this
patch. I will send the updated one shortly. Thanks for the pointer Ross.


>
>>
>> Ross
>>
>
>
>
> --
> സുജിത് ഹരിദാസന്
> Bangalore
> Contributor to KDE project
> Contributor to Yocto project
> http://fci.wikia.com/wiki/Anti-DRM-Campaign
>  http://sujithh.info
> C-x C-c
>



-- 
സുജിത് ഹരിദാസന്
Bangalore
Contributor to KDE project
Contributor to Yocto project
http://fci.wikia.com/wiki/Anti-DRM-Campaign
 http://sujithh.info
C-x C-c
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] x264: add textrel to INSANE_SKIP for cyclone5 machine

2016-09-14 Thread sujith h 
On Thu, Sep 15, 2016 at 4:41 AM, Burton, Ross  wrote:

>
> On 12 September 2016 at 07:13, Sujith H  wrote:
>
>> +# PIC can't be enabled for 32-bit x86 and cyclone5
>>  INSANE_SKIP_${PN}_append_x86 = " textrel"
>> +INSANE_SKIP_${PN}_append_cyclone5 = " textrel"
>>
>
> Can this be generalised or is it absolutely specific to cyclone5?  If it
> can't be generalised then I feel that this should belong in the cyclone5
> BSP.
>

We found this with cyclone5 BSP. I will try to figure out how if we can see
it with qemuarm or other BSP's too.


>
> Ross
>



-- 
സുജിത് ഹരിദാസന്
Bangalore
Contributor to KDE project
Contributor to Yocto project
http://fci.wikia.com/wiki/Anti-DRM-Campaign
 http://sujithh.info
C-x C-c
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] x264: add textrel to INSANE_SKIP for cyclone5 machine

2016-09-14 Thread Burton, Ross 
On 12 September 2016 at 07:13, Sujith H  wrote:

> +# PIC can't be enabled for 32-bit x86 and cyclone5
>  INSANE_SKIP_${PN}_append_x86 = " textrel"
> +INSANE_SKIP_${PN}_append_cyclone5 = " textrel"
>

Can this be generalised or is it absolutely specific to cyclone5?  If it
can't be generalised then I feel that this should belong in the cyclone5
BSP.

Ross
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core