From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Michael Jones <michael.jones@matrix-vision.de>
Cc: jean-philippe francois <jp.francois@cynove.com>,
Sakari Ailus <sakari.ailus@iki.fi>,
linux-media@vger.kernel.org
Subject: Re: Lockup on second streamon with omap3-isp
Date: Fri, 09 Mar 2012 13:58:43 +0100 [thread overview]
Message-ID: <2038085.aDq2jrhkOM@avalon> (raw)
In-Reply-To: <4F59FD87.4030506@matrix-vision.de>
Hi Michael,
On Friday 09 March 2012 13:54:31 Michael Jones wrote:
> On 03/09/2012 11:42 AM, Laurent Pinchart wrote:
> > Hi Jean-Philippe,
>
> [snip]
>
> > From my experience, the ISP doesn't handle free-running sensors very
> > well.
> >
> > There are other things it doesn't handle well, such as sensors stopping in
> > the middle of the frame. I would consider this as limitations.
>
> Considering choking on sensors which stop in the middle of the frame- is
> this just a limitation of the driver, or is it really a limitation of
> the ISP hardware itself?
It's a limitation of the hardware. Various OMAP3 ISP blocks can't be stopped
before they have processed a complete frame once they have been started. The
work around is to reset the whole ISP, which we will do in v3.4, but that
won't solve the problem completely if one application uses the resizer in
memory-to-memory mode and another application uses the rest of the ISP. In
that case the driver won't be able to reset the ISP as long as the first
application uses it.
> It is at least a limitation of the driver because we rely on the VD1 and VD0
> interrupts, so we'll of course have problems if we never get to the last
> line. But isn't it conceivable to use HS_VS to do our end-of-frame stuff
> instead of VD0? Maybe then the ISP would be OK with frames that ended
> early, as long as they had reached VD1. Then of course, you could move VD1
> to an even earlier line, even to the first line.
>
> Do you think that's possible?
Unfortunately not. HS_VS could be used as a fallback to detect the end of a
frame in case something bad occurs, but that hardware will still be stuck
waiting for the end of the frame. The real problem here is a lack of feature
on the hardware side, the ISP modules should either support being stopped in
the middle of a frame, or support per-module reset.
--
Regards,
Laurent Pinchart
next prev parent reply other threads:[~2012-03-09 12:58 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-06 17:08 Lockup on second streamon with omap3-isp jean-philippe francois
2012-03-07 14:24 ` jean-philippe francois
2012-03-08 17:22 ` Sakari Ailus
2012-03-08 23:28 ` Laurent Pinchart
2012-03-09 7:30 ` jean-philippe francois
2012-03-09 10:42 ` Laurent Pinchart
2012-03-09 12:54 ` Michael Jones
2012-03-09 12:58 ` Laurent Pinchart [this message]
2012-03-09 15:55 ` jean-philippe francois
2012-12-13 14:14 ` jean-philippe francois
2012-12-14 14:18 ` Julien BERAUD
2012-12-17 9:32 ` Laurent Pinchart
2012-12-17 15:41 ` jean-philippe francois
2012-12-17 16:09 ` Laurent Pinchart
2012-12-18 10:13 ` Julien BERAUD
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=2038085.aDq2jrhkOM@avalon \
--to=laurent.pinchart@ideasonboard.com \
--cc=jp.francois@cynove.com \
--cc=linux-media@vger.kernel.org \
--cc=michael.jones@matrix-vision.de \
--cc=sakari.ailus@iki.fi \
/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