From: Eric Nelson <eric.nelson@boundarydevices.com>
To: Otavio Salvador <otavio@ossystems.com.br>,
Sandoval Gonzalez Leonardo-B42214 <B42214@freescale.com>
Cc: "meta-freescale@yoctoproject.org" <meta-freescale@yoctoproject.org>
Subject: Re: gstreamer strange distorted image.
Date: Mon, 16 Sep 2013 17:37:20 -0700 [thread overview]
Message-ID: <5237A440.5020903@boundarydevices.com> (raw)
In-Reply-To: <5237A0E9.2060006@boundarydevices.com>
On 09/16/2013 05:23 PM, Eric Nelson wrote:
> On 09/16/2013 02:19 PM, Otavio Salvador wrote:
>> On Mon, Sep 16, 2013 at 5:34 PM, Dennis Han <jshan@live.co.kr> wrote:
>>> I tested you patch more detail and found a problem.
>>>
>>> In brief.
>>>
>>> 1. With original value
>>> IC_RSZ_MAX_RESIZE_RATIO = 0x00004000
>>>>> it's OK
>>>
>>>
>>> 2. With a new value
>>> IC_RSZ_MAX_RESIZE_RATIO = 0x00002000
>>>>> center pixels are overlapping
>>>
>>>
>>> I think this patch has to be ignored.
>>
>> Good for testing and letting us know about the result. Eric, does it
>> also fails for you?
>>
>
> The re-sizing patch did nothing for me, while our patch
> fixed the issue playing the file in both qtmediaplayer
> and gst-launch/playbin2.
>
> I'm not testing with qt-in-use-image, but another qt4e
> image with a patched-up 4.0.0 kernel.
>
> This makes sense because the video is 1280x720 and the
> display is also 1280x720.
>
I meant to add some details about what we're seeing.
It's very clear when playing full-screen that the
issue is one of a missing color-space conversion.
Playing Dennis' video clearly shows that the data
is in NV12 (YUV planar) format, with two instances
of the video across the top of the screen, followed
by a very distorted bottom 1/3 or so.
Our patch circumvents code elsewhere in the V4L2
driver that appears to be an optimization for the
case where the YUV plane can be output directly and
the display can somehow do the conversion.
https://github.com/boundarydevices/linux-imx6/commit/c4eb189e1cae98c5535c0a26e859a010b0c70510
Also interesting is that when qmediaplayer starts up,
it seems to tell V4L2 to start up a 1280x720 plane,
but not at offset 0,0. In other words, it's not full-screen,
so the "bypass CSC" logic seems to be missing another
component in determining whether the YUV plane is "full screen".
Oh, and the Qt4 Phonon layer should probably be patched
to shrink the overlay size if the initial window position
doesn't allow the overlay to fit on the screen.
Regards,
Eric
next prev parent reply other threads:[~2013-09-17 0:37 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-09-14 21:56 gstreamer strange distorted image Dennis Han
2013-09-15 4:52 ` Sandoval Gonzalez Leonardo-B42214
2013-09-15 23:55 ` Otavio Salvador
2013-09-16 4:19 ` Sandoval Gonzalez Leonardo-B42214
[not found] ` <000401ceb2a4$5d8a2bc0$189e8340$@freescale.com>
2013-09-16 8:49 ` Dennis Han
2013-09-16 11:07 ` Otavio Salvador
[not found] ` <000101ceb2cd$cfea6d50$6fbf47f0$@ossystems.com.br>
2013-09-16 14:05 ` Dennis Han
2013-09-16 20:34 ` Dennis Han
2013-09-16 21:19 ` Otavio Salvador
2013-09-17 0:23 ` Eric Nelson
2013-09-17 0:37 ` Eric Nelson [this message]
2013-09-17 1:47 ` Otavio Salvador
2013-09-17 13:31 ` Eric Nelson
[not found] ` <000301ceb34d$5c62b6f0$152824d0$@boundarydevices.com>
2013-09-17 5:17 ` Dennis Han
2013-09-16 14:13 ` Eric Nelson
2013-09-16 16:25 ` Otavio Salvador
[not found] ` <000a01ceb1d2$4d58d940$e80a8bc0$@freescale.com>
2013-09-15 5:40 ` Dennis Han
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=5237A440.5020903@boundarydevices.com \
--to=eric.nelson@boundarydevices.com \
--cc=B42214@freescale.com \
--cc=meta-freescale@yoctoproject.org \
--cc=otavio@ossystems.com.br \
/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.