public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
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


  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