From: Santosh Shilimkar <santosh.shilimkar@ti.com>
To: Benoit Cousson <b-cousson@ti.com>,
Tomi Valkeinen <tomi.valkeinen@ti.com>,
Rajendra Nayak <rnayak@ti.com>,
Sumit Semwal <sumit.semwal@ti.com>
Cc: Paul Walmsley <paul@pwsan.com>, Kevin Hilman <khilman@ti.com>,
linux-omap <linux-omap@vger.kernel.org>,
Archit Taneja <archit@ti.com>
Subject: RE: Problem with DSS clocks & accessing registers
Date: Mon, 7 Mar 2011 19:35:13 +0530 [thread overview]
Message-ID: <90dc32aa0eb6d68d2a56424de664653d@mail.gmail.com> (raw)
In-Reply-To: <4D74E2D5.2000705@ti.com>
> -----Original Message-----
> From: linux-omap-owner@vger.kernel.org [mailto:linux-omap-
> owner@vger.kernel.org] On Behalf Of Cousson, Benoit
> Sent: Monday, March 07, 2011 7:21 PM
> To: Valkeinen, Tomi; Nayak, Rajendra
> Cc: Paul Walmsley; Hilman, Kevin; linux-omap; Taneja, Archit;
> Semwal, Sumit
> Subject: Re: Problem with DSS clocks & accessing registers
>
> + Rajendra
>
> Hi Tomi,
>
> On 3/7/2011 9:22 AM, Valkeinen, Tomi wrote:
> > Hi Kevin, Paul,
> >
> > We currently have a small problem with OMAP4 DSS. When we enable
> the DSS
> > clocks, it seems that the DSS registers are not always accessible
> right
> > after the clock enable.
>
> What clocks are you talking about? As you know, the DSS has a bunch
> of functional clocks available depending of the use case.
>
> > I understood that on OMAP4 the clock framework doesn't guarantee
> that
> > the registers are accessible after enabling clocks, and pm_runtime
> will handle this. Is this correct?
>
This seems to the question about a so called optional clock (DSS_CLK)
which really not an optional clock :)
This clock needs to be enabled and disabled along with interface and
functional clock for the registers access to work.
If you look internals of the clock routing in DSS, Figure 10-4
this clock is not really optional. You can call it optional because
there can be a option to use PLL clock instead of this one. But the
fact is either DSS_CLK or the PLL clock has to be running for the
IP accesses to work.
Sumit,
Please add more points about our discussion on this one.
Regards,
Santosh
next prev parent reply other threads:[~2011-03-07 14:05 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-03-07 8:22 Problem with DSS clocks & accessing registers Tomi Valkeinen
2011-03-07 13:51 ` Cousson, Benoit
2011-03-07 14:05 ` Santosh Shilimkar [this message]
2011-03-07 14:52 ` Cousson, Benoit
2011-03-07 15:03 ` Tomi Valkeinen
2011-03-08 10:22 ` Cousson, Benoit
2011-03-10 15:07 ` Paul Walmsley
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=90dc32aa0eb6d68d2a56424de664653d@mail.gmail.com \
--to=santosh.shilimkar@ti.com \
--cc=archit@ti.com \
--cc=b-cousson@ti.com \
--cc=khilman@ti.com \
--cc=linux-omap@vger.kernel.org \
--cc=paul@pwsan.com \
--cc=rnayak@ti.com \
--cc=sumit.semwal@ti.com \
--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