All of lore.kernel.org
 help / color / mirror / Atom feed
From: sujith h <sujith.h@gmail.com>
To: "Burton, Ross" <ross.burton@intel.com>
Cc: OE-core <openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH] x264: add textrel to INSANE_SKIP for cyclone5 machine
Date: Thu, 15 Sep 2016 13:30:06 +0530	[thread overview]
Message-ID: <CADyYWLnjSuHC2JHfRcWX0XM5F3S8g3Km8SSxXXN_3=zfJ9E7zA@mail.gmail.com> (raw)
In-Reply-To: <CADyYWL=VezFuDFy=ZV8y4XDyzTP20ErxgpfXjLh5uCoCWDrdHQ@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 2027 bytes --]

On Thu, Sep 15, 2016 at 12:15 PM, sujith h <sujith.h@gmail.com> wrote:

>
>
> On Thu, Sep 15, 2016 at 10:13 AM, sujith h <sujith.h@gmail.com> wrote:
>
>>
>>
>> On Thu, Sep 15, 2016 at 4:41 AM, Burton, Ross <ross.burton@intel.com>
>> wrote:
>>
>>>
>>> On 12 September 2016 at 07:13, Sujith H <sujith.h@gmail.com> 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
>> <Project>Contributor to KDE project
>> <Project>Contributor to Yocto project
>> http://fci.wikia.com/wiki/Anti-DRM-Campaign
>> <Blog> http://sujithh.info
>> C-x C-c
>>
>
>
>
> --
> സുജിത് ഹരിദാസന്
> Bangalore
> <Project>Contributor to KDE project
> <Project>Contributor to Yocto project
> http://fci.wikia.com/wiki/Anti-DRM-Campaign
> <Blog> http://sujithh.info
> C-x C-c
>



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

[-- Attachment #2: Type: text/html, Size: 5127 bytes --]

      reply	other threads:[~2016-09-15  8:00 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-12  6:13 [PATCH] x264: add textrel to INSANE_SKIP for cyclone5 machine Sujith H
2016-09-14 23:11 ` Burton, Ross
2016-09-15  4:43   ` sujith h
2016-09-15  6:45     ` sujith h
2016-09-15  8:00       ` sujith h [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CADyYWLnjSuHC2JHfRcWX0XM5F3S8g3Km8SSxXXN_3=zfJ9E7zA@mail.gmail.com' \
    --to=sujith.h@gmail.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=ross.burton@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.