linux-omap.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Tomi Valkeinen <tomi.valkeinen@ti.com>
To: Rob Clark <rob.clark@linaro.org>
Cc: Archit Taneja <archit@ti.com>,
	dri-devel@lists.freedesktop.org, linux-omap@vger.kernel.org,
	patches@linaro.org, Greg KH <greg@kroah.com>,
	Andy Gross <andy.gross@ti.com>
Subject: Re: [RFC 0/3] solving omapdrm/omapdss layering issues
Date: Wed, 01 Aug 2012 20:38:24 +0300	[thread overview]
Message-ID: <1343842704.2645.94.camel@deskari> (raw)
In-Reply-To: <CAF6AEGvjKCgPh=sTL922J48N9eqMGD_kNrYx_bU-zBF-bXv=Ww@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 1268 bytes --]

On Wed, 2012-08-01 at 11:53 -0500, Rob Clark wrote:

> Ok.. this would help.  I'll take a look.  I do request that
> interfaces/panels don't set any mgr/timing related registers.  I had
> to comment all this stuff out in my prototype.  Really we want to set
> the timings separately on the crtc (mgr) / encoder (interface) /
> connector (panel.. not sure if it is needed, though?).  KMS will take
> care of propagating the timings through the pipeline.

If we only had auto-update displays, and only the video timings were
shadow register, it'd work. Unfortunately we have other registers as
shadow registers also, like DISPC_CONTROL1, DISPC_CONFIG1 and
DISPC_DIVISOR1.

But we should think if this could be somehow be changed, so that all the
shadow register info would come from one place. I do find it a bit
unlikely with a quick thought, though.

Well, hmm. Perhaps... Omapdrm (or omapfb etc) doesn't really need to
know about the values of those registers, it just needs to control the
GO bit. So perhaps if we had some method to inform omapdrm that these
things have changed, and omapdrm would then set the GO bit as soon as
possible.

But there are some tricky stuff, like the divisors... Well, we need to
think about this.

 Tomi


[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

  parent reply	other threads:[~2012-08-01 17:38 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-28  1:07 [RFC 0/3] solving omapdrm/omapdss layering issues Rob Clark
2012-07-28  1:07 ` [RFC 1/3] OMAPDSS: expose dispc for use from omapdrm Rob Clark
2012-07-28  1:07 ` [RFC 2/3] omap2+: use dss_dispc hwmod for omapdrm Rob Clark
2012-07-28  1:07 ` [RFC 3/3] drm/omap: use dispc directly Rob Clark
2012-07-31 13:40 ` [RFC 0/3] solving omapdrm/omapdss layering issues Tomi Valkeinen
2012-07-31 14:45   ` Rob Clark
2012-08-01  9:21     ` Tomi Valkeinen
2012-08-01 14:25       ` Rob Clark
2012-08-01 16:46         ` Archit Taneja
2012-08-01 16:53           ` Rob Clark
2012-08-01 17:03             ` Rob Clark
2012-08-01 17:38             ` Tomi Valkeinen [this message]
2012-08-01 22:41               ` Rob Clark
2012-08-02  7:13         ` Tomi Valkeinen
2012-08-02 12:45           ` Rob Clark
2012-08-02 13:15             ` Tomi Valkeinen
2012-08-02 14:16               ` Rob Clark
2012-08-03  6:01                 ` Semwal, Sumit
2012-08-03 12:22                   ` Rob Clark

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=1343842704.2645.94.camel@deskari \
    --to=tomi.valkeinen@ti.com \
    --cc=andy.gross@ti.com \
    --cc=archit@ti.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=greg@kroah.com \
    --cc=linux-omap@vger.kernel.org \
    --cc=patches@linaro.org \
    --cc=rob.clark@linaro.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).