From: Gary Thomas <gary@mlbassoc.com>
To: Javier Martinez Canillas <martinez.javier@gmail.com>
Cc: Enrico <ebutera@users.berlios.de>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Deepthy Ravi <deepthy.ravi@ti.com>,
Adam Pledger <a.pledger@thermoteknix.com>,
linux-media@vger.kernel.org
Subject: Re: omap3-isp status
Date: Thu, 06 Oct 2011 09:47:05 -0600 [thread overview]
Message-ID: <4E8DCD79.3060507@mlbassoc.com> (raw)
In-Reply-To: <CAAwP0s0tOHmdG6eWuY_QDZ6ReVFXg9S6-MSbX7s4GNEX60U2mQ@mail.gmail.com>
On 2011-10-06 08:50, Javier Martinez Canillas wrote:
> On Thu, Oct 6, 2011 at 4:29 PM, Javier Martinez Canillas
> <martinez.javier@gmail.com> wrote:
>> On Thu, Oct 6, 2011 at 4:00 PM, Gary Thomas<gary@mlbassoc.com> wrote:
>>> On 2011-10-06 01:51, Javier Martinez Canillas wrote:
>>>>
>>>> On Wed, Oct 5, 2011 at 7:43 PM, Javier Martinez Canillas
>>>> <martinez.javier@gmail.com> wrote:
>>>>>
>>>>> On Wed, Oct 5, 2011 at 6:28 PM, Enrico<ebutera@users.berlios.de> wrote:
>>>>>>
>>>>>> Hi all,
>>>>>>
>>>>>> since we are all interested in this driver (and tvp5150) i'll try to
>>>>>> make a summary of the current situation and understand what is needed
>>>>>> to finally get it into the main tree instead of having to apply a
>>>>>> dozen patches manually.
>>>>>>
>>>>>> The current status of git repositories/branches is:
>>>>>>
>>>>>> - main tree: working (i suppose) but no support for bt656 input
>>>>>>
>>>>>> - pinchartl/media:
>>>>>> * omap3isp-omap3isp-next: i think it's in sync with linuxtv master
>>>>>> (for the omap3-isp parts)
>>>>>> * omap3isp-omap3isp-yuv: like ..next but with some additional format
>>>>>> patches
>>>>>>
>>>>>> "Floating" patches:
>>>>>>
>>>>>> - Deepthy: sent patches (against mainline) to add bt656 support
>>>>>>
>>>>>> Laurent made some comments, i haven't seen a v2 to be applied
>>>>>>
>>>>>> - Javier: sent patches for tvp5150, currently discussed on
>>>>>> linux-media; possible patches/fixes for omap3-isp
>>>>>>
>>>>
>>>> Hello,
>>>>
>>>> Since the patches are not against mainline I can't post for reviewing
>>>> but can be found in one of our development trees [1]. Comments are
>>>> highly appreciated.
>>>>
>>>> The tree is a 2.6.37 that already contain Deepthy patch. I rebased my
>>>> changes on top of that to correctly support both BT656 an non-BT656
>>>> video data processing.
>>>>
>>>> [1]:
>>>> http://git.igep.es/?p=pub/scm/linux-omap-2.6.git;a=shortlog;h=refs/heads/linux-2.6.37.y-next
>>>
>>> Any chance of rebasing these against a more up to date kernel, e.g.
>>> 3.2-working
>>> with the patches Laurent sent today?
>>>
>>
>> Sure, but I won't have time to do it neither today nor tomorrow. But
>> will do it during the weekend.
>>
>>>>>
>>>>> I will find some free time slots to resolve the issues called out by
>>>>> Sakari, Hans and Mauro and resend the patch-set for the tvp5151.
>>>>>
>>>>> Also I can send the patches of the modifications I made to the ISP
>>>>> driver. Right now I'm working on top of Deepthy patches.
>>>>>
>>>>> I can either send on top of that patch or rebase to mainline, whatever
>>>>> you think is better for reviewing.
>>>>>
>>>>>> Now what can we all do to converge to a final solution? I think this
>>>>>> is also blocking the possible development/test of missing features,
>>>>>> like the recently-discussed resizer and cropping ones.
>>>>>>
>>>>>> Enrico
>>>>>>
>>>>>
>>>>> Right now I have a working the tvp5151 with the ISP. I can capture
>>>>> ITU-R BT656 video both in PAL-M and NTSC standard. Also, the whole
>>>>> pipeline is configured automatically with the video standard detected
>>>>> by the tvp5151. Also, I'm using the CCDC to crop the frames and only
>>>>> capture the active lines for each standard (576 for PAL and 480 for
>>>>> NTSC) using the CCDC to crop the image.
>>>>>
>>>>
>>>> As I told you before video capturing is working for both PAL and NTSC
>>>> using standard V4L2 application (i.e: gstreamer) but the video still
>>>> shows some motion artifacts. Capturing YUV frames and looking at them
>>>> I realized that there does exist a pattern, the sequence 2 frames
>>>> correct and 3 frames with interlacing effects always repeats.
>>>
>>> I think I've seen this as well. Could you provide a short video
>>> which shows the artefacts?
>>>
>>
>> Yes, I've attached a 16-frame video file. It is a PAL-M video
>> (720x576) in YUV 4:22 data format. Please let me know if it is OK for
>> you.
>>
>
> Sorry, I didn't notice the size of the image (13 MB) and got a lot of
> rejects from your MTAs. I uploaded the file to my personal github
> account [1].
>
> [1]: https://github.com/martinezjavier/omap3isp_tvp5151/blob/master/pal.yuv
Very interesting. What was your source (camera type, etc)?
How are you looking (or extracting) individual frames for analysis?
I see much the same sort of artefacts as you are. An example is at
http://www.mlbassoc.com/misc/untitled.m2t
This is a little example I put together using kdenlive. The first segment
is the raw video from my camera, imported via USB. The second is roughly
the same video captured using my OMAP board and converted to MP4 on the fly
by this command:
ffmpeg -r 30/1 -pix_fmt uyvy422 -s 720x524 -f video4linux2 -i /dev/video2 -qscale 1 -f mp4 test1.mp4
I think there are some aspect ratio issues with these but what bothers me
the most is how much the captured data tears whenever there is a lot of
motion in the video.
--
------------------------------------------------------------
Gary Thomas | Consulting for the
MLB Associates | Embedded world
------------------------------------------------------------
next prev parent reply other threads:[~2011-10-06 15:47 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-10-05 16:28 omap3-isp status Enrico
2011-10-05 17:43 ` Javier Martinez Canillas
2011-10-06 7:51 ` Javier Martinez Canillas
2011-10-06 14:00 ` Gary Thomas
[not found] ` <CAAwP0s0ddOYAnC7rknLVzcN10iKAwnuOawznpKy9z6B2yWRdCg@mail.gmail.com>
2011-10-06 14:50 ` Javier Martinez Canillas
2011-10-06 15:47 ` Gary Thomas [this message]
2011-10-06 16:11 ` Javier Martinez Canillas
2011-10-07 9:34 ` Gary Thomas
2011-10-07 10:08 ` Javier Martinez Canillas
2011-10-07 10:22 ` Gary Thomas
2011-10-07 10:36 ` Enrico
2011-10-07 11:02 ` Javier Martinez Canillas
2011-10-07 11:39 ` Gary Thomas
2011-10-07 11:50 ` Javier Martinez Canillas
2011-10-06 15:25 ` Enrico
2011-10-06 16:05 ` Javier Martinez Canillas
2011-10-07 8:54 ` Enrico
2011-10-07 9:31 ` Javier Martinez Canillas
2011-10-08 15:51 ` Laurent Pinchart
2011-10-08 16:11 ` Javier Martinez Canillas
2011-10-09 22:35 ` Enrico
2011-10-09 23:00 ` Javier Martinez Canillas
2011-10-10 8:54 ` Enrico
2011-10-10 9:02 ` Javier Martinez Canillas
2011-10-10 10:06 ` Enrico
2011-10-10 10:07 ` Enrico
2011-10-10 10:33 ` Javier Martinez Canillas
2011-10-10 12:46 ` Enrico
2011-10-10 14:17 ` Enrico
2011-10-10 16:34 ` Enrico
2011-10-10 16:53 ` Javier Martinez Canillas
2011-10-10 17:09 ` Enrico
2011-10-10 18:18 ` Javier Martinez Canillas
2011-10-11 10:29 ` Enrico
2011-10-11 10:50 ` Javier Martinez Canillas
2011-10-10 12:54 ` Enrico
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=4E8DCD79.3060507@mlbassoc.com \
--to=gary@mlbassoc.com \
--cc=a.pledger@thermoteknix.com \
--cc=deepthy.ravi@ti.com \
--cc=ebutera@users.berlios.de \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-media@vger.kernel.org \
--cc=martinez.javier@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox