From: Ivaylo Dimitrov <ivo.g.dimitrov.75@gmail.com>
To: cmahapatra@ti.com
Cc: Tomi Valkeinen <tomi.valkeinen@ti.com>,
"pali.rohar@gmail.com" <pali.rohar@gmail.com>,
"pavel@ucw.cz" <pavel@ucw.cz>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: OMAPDSS: DISPC: horizontal timing too tight errors
Date: Mon, 06 Jan 2014 11:43:23 +0200 [thread overview]
Message-ID: <52CA7ABB.4010401@gmail.com> (raw)
Hi,
commit 7faa92339bbb1e6b9a80983b206642517327eb75 "OMAPDSS: DISPC: Handle
synclost errors in OMAP3" introduces some limits check to prevent
SYNCLOST errors on OMAP3 in a specific usecase. The problem I see here
(on Nokia N900, Maemo 5, linux 3.13-rc6, DSP accel video decoding) is
that those checks effectively prevent fullscreen video playback of
anything above lets say 640x350 with "horizontal timing too tight"
errors spit in dmesg log. If I hack check_horiz_timing_omap3 function to
always return true, I can happily play videos up to (and including) 720p
resolutions, with no SYNCLOST errors.
So, a couple of questions:
Where do the values in static const u8 limits[3] come from? Are those
documented somewhere?
Commit message says "This code is written based on code written by Ville
Syrjälä <ville.syrjala@nokia.com> in Linux OMAP kernel.", is that code
publicly available and where (if it is).
Besides compiling DSS driver with DEBUG enabled and providing the log
(yeah, I know I should've done it already and have the logs included in
this mail, but... :) ), is there anything else I can do to find the
culprit for those errors.
Regards,
Ivo
next reply other threads:[~2014-01-06 9:43 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-01-06 9:43 Ivaylo Dimitrov [this message]
2014-01-07 12:35 ` OMAPDSS: DISPC: horizontal timing too tight errors Tomi Valkeinen
2014-01-07 18:05 ` Pavel Machek
2014-01-07 18:30 ` Ivaylo Dimitrov
2014-01-07 20:25 ` Ivaylo Dimitrov
2014-01-07 21:28 ` Pavel Machek
2014-01-08 14:03 ` Tomi Valkeinen
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=52CA7ABB.4010401@gmail.com \
--to=ivo.g.dimitrov.75@gmail.com \
--cc=cmahapatra@ti.com \
--cc=linux-kernel@vger.kernel.org \
--cc=pali.rohar@gmail.com \
--cc=pavel@ucw.cz \
--cc=tomi.valkeinen@ti.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