public inbox for linux-omap@vger.kernel.org
 help / color / mirror / Atom feed
From: Tomi Valkeinen <tomi.valkeinen@ti.com>
To: Paul Walmsley <paul@pwsan.com>
Cc: Benoit Cousson <b-cousson@ti.com>,
	linux-omap mailing list <linux-omap@vger.kernel.org>
Subject: Re: Setting OMAP DSS fck
Date: Fri, 02 Dec 2011 09:39:01 +0200	[thread overview]
Message-ID: <1322811541.1867.29.camel@deskari> (raw)
In-Reply-To: <alpine.DEB.2.00.1112011121220.6289@utopia.booyaka.com>

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

On Thu, 2011-12-01 at 11:23 -0700, Paul Walmsley wrote:
> On Thu, 1 Dec 2011, Tomi Valkeinen wrote:
> 
> > Why is it that the rate of DSS functional clock (dss_dss_clk on OMAP4)
> > cannot be set, but we need to get the parent of the fck, and set the
> > rate of that? The same is on OMAP3.
> > 
> > From driver's perspective I think this only makes things more complex,
> > as the driver is not interested in the parent, only about the dss fck.
> 
> Yeah, I agree.  We've talked about implementing rate changes that 
> percolate up to some higher point in the clock tree, but have never gotten 
> around to it due to other, higher priorities.  And now the common clock 
> discussion has reduced the desire to do much OMAP-specific implementation 
> of this stuff.
> 
> Another (related) problem is that the driver probably needs to know the 
> ranges of the possible values that can be set.

That's true. The DSS driver has knowledge of the possible divider ranges
that the parent clock can use. Not very neat.

And note that the DSS driver needs to know about the possible dividers,
not the clock freq range. We need to get quite exact pixel clocks,
derived via some dividers, and we iterate through the dividers trying to
find a divider set that produces a pixel clock that is close to the
required one.

 Tomi


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

      reply	other threads:[~2011-12-02  7:39 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-12-01  8:16 Setting OMAP DSS fck Tomi Valkeinen
2011-12-01  8:58 ` Cousson, Benoit
2011-12-01 17:00   ` Turquette, Mike
2011-12-01 18:23 ` Paul Walmsley
2011-12-02  7:39   ` Tomi Valkeinen [this message]

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=1322811541.1867.29.camel@deskari \
    --to=tomi.valkeinen@ti.com \
    --cc=b-cousson@ti.com \
    --cc=linux-omap@vger.kernel.org \
    --cc=paul@pwsan.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