All of lore.kernel.org
 help / color / mirror / Atom feed
From: jouni.hogander@nokia.com (Högander Jouni)
To: ext Paul Walmsley <paul@pwsan.com>
Cc: ext Tony Lindgren <tony@atomide.com>,
	Igor Stoppa <igor.stoppa@nokia.com>,
	Felipe Balbi <me@felipebalbi.com>,
	David Brownell <david-b@pacbell.net>,
	linux-omap@vger.kernel.org, Felipe Balbi <felipe.balbi@nokia.com>,
	r-woodruff2@ti.com
Subject: Re: [PATCH 1/3] clk: introduce omap_clk_associate
Date: Wed, 15 Oct 2008 16:03:42 +0300	[thread overview]
Message-ID: <87wsga9fip.fsf@trdhcp146196.ntc.nokia.com> (raw)
In-Reply-To: <alpine.DEB.2.00.0810150106370.8581@utopia.booyaka.com> (ext Paul Walmsley's message of "Wed, 15 Oct 2008 01:49:16 -0600 (MDT)")

"ext Paul Walmsley" <paul@pwsan.com> writes:

> Hello Jouni,
>
> On Wed, 15 Oct 2008, Högander Jouni wrote:
>
>> "ext Tony Lindgren" <tony@atomide.com> writes:
>> 
>> > * Igor Stoppa <igor.stoppa@nokia.com> [081014 14:12]:
>> >> On Tue, 2008-10-14 at 13:59 -0700, ext Tony Lindgren wrote:
>> >> 
>> >> > And that we can use same naming in the driver no matter which omap :)
>> >> 
>> >> Wasn't that one of the main features of the clock FW, when it was
>> >> designed?
>> >
>> > Yes. But we adding separate ick and fck during omap2420 confused
>> > that.
>> 
>> Why this was originally done? I mean we have possibility to autogate
>> interface clocks. AFAIK those could be just fine be enabled on boot and let
>> the hw to take care of them.
>
> Richard is really the best person to ask about this, so he's been added to 
> the cc.  My current understanding is below.  Perhaps he can respond with 
> more detail and correct any errors.
>
> In terms of original motivation, you might want to look at 34xx TRM 
> 4.12.2.4.3.  I don't think this actually works in practice on OMAP3.
>
> Current practical use is different.  Autogating only takes effect during 
> sleep & wakeup transitions for the entire CORE_Lx clockdomain (34xx TRM 
> 4.12.2.4.4).  So even if a module's functional clocks are disabled, the 
> PRCM won't autogate the module's interface clock until the entire CORE_Lx 
> clockdomain transitions to inactive.  The theory here is that this wastes 
> power and that manually disabling the interface clock when the device is 
> known to be inactive should save some power.

L3 and L4 are anyway on when not in wfi, because of sleep/wkdep, so no
power save. This is the case at least with OMAP3 not sure about OMAP2.

>
> There also was some discussion last year that modules like GPTIMER can 
> have their functional clocks enabled, but their interface clocks disabled 
> when register access is not needed.  Wakeups from the module are still 
> sent asynchronously to the PRCM.  Again the motivation is to save power.  
> I don't think the current code tries to do this.
>

In many modules there is also possibility to gate ick by HW when not
in wfi (see AUTOIDLE in SYCONFIG). I'm not sure about this but
extremely also fcks could be taken care by HW (see CLOCKACTIVITY in
SYSCONFIG).

>
> - Paul

-- 
Jouni Högander

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  parent reply	other threads:[~2008-10-15 13:01 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-06 22:47 [PATCH 0/3] omap_clk_associate Felipe Balbi
2008-10-06 22:47 ` [PATCH 1/3] clk: introduce omap_clk_associate Felipe Balbi
2008-10-06 22:47   ` [PATCH 2/3] clk: use clk_associate for musb driver Felipe Balbi
2008-10-06 22:47     ` [PATCH 3/3] clk: use clk_associate on watchdog driver Felipe Balbi
2008-10-14 17:08   ` [PATCH 1/3] clk: introduce omap_clk_associate Paul Walmsley
2008-10-14 17:29     ` Felipe Balbi
2008-10-14 17:47       ` David Brownell
2008-10-14 18:06         ` Felipe Balbi
2008-10-14 20:59           ` Tony Lindgren
2008-10-14 21:09             ` Igor Stoppa
2008-10-14 21:24               ` Tony Lindgren
2008-10-14 21:28                 ` Felipe Balbi
2008-10-15  6:21                 ` Högander Jouni
2008-10-15  7:49                   ` Paul Walmsley
2008-10-15 12:56                     ` Woodruff, Richard
2008-10-15 13:03                     ` Högander Jouni [this message]
2008-10-15 13:08                       ` Paul Walmsley
2008-10-15 22:42                       ` Woodruff, Richard
2008-10-15 12:45                   ` Woodruff, Richard
2008-10-16  9:02       ` Paul Walmsley
  -- strict thread matches above, loose matches on Subject: below --
2008-10-25 19:55 Felipe Balbi
2008-10-25 20:03 ` Felipe Balbi
2008-10-28  1:28   ` Felipe Balbi
2008-11-06  0:53   ` Felipe Balbi
2008-10-25 19:58 Felipe Balbi

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=87wsga9fip.fsf@trdhcp146196.ntc.nokia.com \
    --to=jouni.hogander@nokia.com \
    --cc=david-b@pacbell.net \
    --cc=felipe.balbi@nokia.com \
    --cc=igor.stoppa@nokia.com \
    --cc=linux-omap@vger.kernel.org \
    --cc=me@felipebalbi.com \
    --cc=paul@pwsan.com \
    --cc=r-woodruff2@ti.com \
    --cc=tony@atomide.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 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.