From: Gary Thomas <gary@mlbassoc.com>
To: "linux-media@vger.kernel.org" <linux-media@vger.kernel.org>
Subject: OMAP3 ISP - interlaced data incorrect
Date: Fri, 14 Oct 2011 17:07:07 -0600 [thread overview]
Message-ID: <4E98C09B.2060800@mlbassoc.com> (raw)
For days, I've been chasing ghosts :-) I know they are still there,
but I think they are more a function of the source than the ISP setup.
So, I went looking for a better source, NTSC in my case. My choice is
is a DVD player with known good video (I'm convinced that my cheap NTSC
camera produces crap, especially when there is a lot of motion in the
frames). Looking at this on an analogue TV (yes, they still exist!),
the picture is not bad, so I think it's a good choice, at least when
trying to understand what's happening with the OMAP3 ISP.
Look at these two pictures:
http://www.mlbassoc.com/misc/nemo-00001.png
http://www.mlbassoc.com/misc/nemo-swapped-00001.png
These represent one frame of data captured via my OMAP3 ISP + TVP5150
from a DVD (sorry, Disney). The first is a raw conversion of the
frame using ffmpeg. As you can see, there seem to be lines swapped,
so I wrote a little program to swap the lines even/odd. The second
(nemo-swapped) shows what this looks like. Obviously, the data is
not being stored in memory correctly. Does anyone know how to adjust
the ISP to make this work the right way around? Currently in ispccdc.c, we have:
ccdc_config_outlineoffset(ccdc, pix.bytesperline, EVENEVEN, 1);
ccdc_config_outlineoffset(ccdc, pix.bytesperline, EVENODD, 1);
ccdc_config_outlineoffset(ccdc, pix.bytesperline, ODDEVEN, 1);
ccdc_config_outlineoffset(ccdc, pix.bytesperline, ODDODD, 1);
I tried this:
ccdc_config_outlineoffset(ccdc, pix.bytesperline, EVENEVEN, 2);
ccdc_config_outlineoffset(ccdc, pix.bytesperline, EVENODD, 0);
ccdc_config_outlineoffset(ccdc, pix.bytesperline, ODDEVEN, 2);
ccdc_config_outlineoffset(ccdc, pix.bytesperline, ODDODD, 0);
but this lead to a kernel panic :-(
Somehow, we need to be storing the data something like this:
EE EE EE EE ...
EO EO EO EO ...
OE OE OE OE ...
OO OO OO OO ...
but the current layout is ccdc_config_outlineoffset(ccdc, pix.bytesperline, EVENEVEN, 1);
ccdc_config_outlineoffset(ccdc, pix.bytesperline, EVENODD, 1);
ccdc_config_outlineoffset(ccdc, pix.bytesperline, ODDEVEN, 1);
ccdc_config_outlineoffset(ccdc, pix.bytesperline, ODDODD, 1);
EO EO EO EO ...
EE EE EE EE ...
OO OO OO OO ...
OE OE OE OE ...
First, I need to get the data into memory in the correct order :-)
Note: these results are consistent, i.e. if I stop things and do another
grab, they are incorrect in the same [wrong] order.
Note 2: I think I have explanations for much of the ghosting that has been
observed:
* Frame to frame "tearing" results from the fact that the frames are interlaced
and the actual data changes from one half of the frame to the other. This would
not be observed with traditional media, e.g. film, where nothing can move from
one frame to the next because of the speed of shutters. In the purely digital
capture case, every pixel has the opportunity to change constantly and to find
some that change with the 33ms (NTSC) window (1 frame) is quite probable.
You can see some of this in the [otherwise quite good] sequence of images
http://www.mlbassoc.com/misc/nemo-swapped-00001.png
...
http://www.mlbassoc.com/misc/nemo-swapped-00062.png
* Frames being skipped &/or very stale data being reused - I think this is a [user]
software problem. The ISP driver assumes that it always has an empty buffer to
move captured data into. Depending on the [user] program which is consuming the
data, this may or not be true. In the case of ffmpeg, if I capture raw images,
ffmpeg can almost always keep up and there is always a free buffer. However, if
I have ffmpeg turn the raw frames into compressed video (mp4), nearly 1/2 of
the time, the ISP will run dry on buffers. I think I know how to fix this (untested)
but it shows that some of the issues may be with the userland code we rely on.
I've not done any recent tests with the gstreamer modules and the TI DSP code,
but I will shortly. We'll see how well that does.
Note 3: The image viewer found at http://djv.sourceforge.net/ works great for these
analyses. Just run 'djv_view' and point at the first image in a sequence :-)
n.b. I'm not associated with that project, but I really like it!
--
------------------------------------------------------------
Gary Thomas | Consulting for the
MLB Associates | Embedded world
------------------------------------------------------------
next reply other threads:[~2011-10-14 23:07 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-10-14 23:07 Gary Thomas [this message]
2011-10-14 23:23 ` OMAP3 ISP - interlaced data incorrect Gary Thomas
2011-10-14 23:53 ` Enrico
2011-10-14 23:43 ` Enrico
2011-10-15 0:25 ` Gary Thomas
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=4E98C09B.2060800@mlbassoc.com \
--to=gary@mlbassoc.com \
--cc=linux-media@vger.kernel.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 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.