From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Yuqing Zhu <b54851@freescale.com>,
openembedded-core@lists.openembedded.org
Subject: Re: [poky][PATCH v2 0/7] Add patches for gstreamer 1.6
Date: Thu, 21 Jan 2016 14:39:09 +0000 [thread overview]
Message-ID: <1453387149.27999.171.camel@linuxfoundation.org> (raw)
In-Reply-To: <1453283501-3982-1-git-send-email-b54851@freescale.com>
On Wed, 2016-01-20 at 17:51 +0800, Yuqing Zhu wrote:
> ---gstreamer1.0
> Correct the setting for FLAG_DISCONT when push to adapter,
> in case audio decoder subclass can not get enough data.
>
> ---gstreamer1.0-plugins-base
> Send taglist if no valid frame is decoded util EOS.
> Add video crop supporting when convert frame.
>
> ---gstreamer1.0-plugins-bad
> Need to set right pid if pid is 0 when create streams.
> Add detecting picture coding type when drain.
> Fix memory leak of navigation thread.
The questions asked about the original patches still stand. Why this
particular set of patches rather than some of the other fixes in the
gstreamer master branch?
Would we be better off with a git recipe for gstreamer that some BSPs
could use?
This does look like the result of some particular product focused work
and looks more BSP focused than OE-Core specific. Backports might be ok
but I particularly worry about patches that have not been accepted
upstream.
I think we need more explanation about the intent here, currently there
has been none.
Cheers,
Richard
prev parent reply other threads:[~2016-01-21 14:39 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-20 9:51 [poky][PATCH v2 0/7] Add patches for gstreamer 1.6 Yuqing Zhu
2016-01-20 9:51 ` [poky][PATCH v2 1/7] gstreamer1.0: [baseparse] unset FLAG_DISCONT when push to adapter Yuqing Zhu
2016-01-20 9:51 ` [poky][PATCH v2 2/7] gstreamer1.0-plugins-base: Fix taglist not being sent to down stream if all the frame corrupted Yuqing Zhu
2016-01-20 9:51 ` [poky][PATCH v2 3/7] gstreamer1.0-plugins-base: Add video crop supporting when convert frame Yuqing Zhu
2016-01-20 9:51 ` [poky][PATCH v2 4/7] gstreamer1.0-plugins-bad: Need to set right pid if pid is 0 when create streams Yuqing Zhu
2016-01-20 9:51 ` [poky][PATCH v2 5/7] gstreamer1.0-plugins-bad: Add detecting picture coding type when drain Yuqing Zhu
2016-01-20 9:51 ` [poky][PATCH v2 6/7] " Yuqing Zhu
2016-01-20 9:51 ` [poky][PATCH v2 7/7] gstreamer1.0-plugins-bad: Fix memory leak of navigation thread Yuqing Zhu
2016-01-21 14:39 ` Richard Purdie [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=1453387149.27999.171.camel@linuxfoundation.org \
--to=richard.purdie@linuxfoundation.org \
--cc=b54851@freescale.com \
--cc=openembedded-core@lists.openembedded.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox