From: Tomi Valkeinen <tomi.valkeinen@ti.com>
To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Florian Neuhaus <florian.neuhaus@reberinformatik.ch>,
"Taneja, Archit" <archit@ti.com>,
"linux-media@vger.kernel.org" <linux-media@vger.kernel.org>
Subject: Re: AW: omapdss/omap3isp/omapfb: Picture from omap3isp can't recover after a blank/unblank (or overlay disables after resuming)
Date: Thu, 14 Feb 2013 13:15:19 +0200 [thread overview]
Message-ID: <511CC747.4060508@ti.com> (raw)
In-Reply-To: <4202523.mOtkCksGpI@avalon>
[-- Attachment #1: Type: text/plain, Size: 1111 bytes --]
On 2013-02-14 13:07, Laurent Pinchart wrote:
>> In many cases underflows are rather hard to debug and solve. There are
>> things in the DSS hardware like FIFO thresholds and prefetch, and VRFB
>> tile sizes, which can be changed (although unfortunately only by
>> modifying the drivers). How they should be changed if a difficult
>> question, though, and whether it'll help is also a question mark.
>
> Naive question here, instead of killing the overlay completely when an
> underflow happens, couldn't the DSS driver somehow recover from that condition
> by restarting whatever needs to be restarted ?
Yes. Killing the overlay is just the safest choice. Presumably if an
underflow happens, the problem is still there, and it'll just happen
again if you re-enable the overlay. Obviously this is not always the
case, as this problem at hand shows.
There's much to improve with the DSS driver's error handling, though. I
think first step would be to remove it totally from DSS, and move it to
omapfb/omapdrm. It's a bit difficult to handle the errors at the lowest
level.
Tomi
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 899 bytes --]
next prev parent reply other threads:[~2013-02-14 11:15 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <6EE9CD707FBED24483D4CB0162E85467245822C8@AMSPRD0711MB532.eurprd07.prod.outlook.com>
[not found] ` <2253226.r6AZgSrtcE@avalon>
2013-01-31 13:06 ` AW: omapdss/omap3isp/omapfb: Picture from omap3isp can't recover after a blank/unblank (or overlay disables after resuming) Florian Neuhaus
2013-02-01 22:14 ` Laurent Pinchart
[not found] ` <51138BCA.4010701@ti.com>
2013-02-14 9:30 ` Florian Neuhaus
2013-02-14 10:08 ` Tomi Valkeinen
2013-02-14 11:07 ` Laurent Pinchart
2013-02-14 11:15 ` Tomi Valkeinen [this message]
2013-02-14 12:09 ` Archit Taneja
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=511CC747.4060508@ti.com \
--to=tomi.valkeinen@ti.com \
--cc=archit@ti.com \
--cc=florian.neuhaus@reberinformatik.ch \
--cc=laurent.pinchart@ideasonboard.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.