public inbox for linux-omap@vger.kernel.org
 help / color / mirror / Atom feed
From: Kevin Hilman <khilman@deeprootsystems.com>
To: Andrew Murray <amurray@mpc-data.co.uk>
Cc: Nishanth Menon <nm@ti.com>, linux-omap@vger.kernel.org
Subject: Re: PM: VDD2 OPPs
Date: Tue, 26 Jan 2010 14:40:53 -0800	[thread overview]
Message-ID: <87636o1ocq.fsf@deeprootsystems.com> (raw)
In-Reply-To: <0C997DD87CC88A4F86C897F52650B52B02807703@MOLSON.mpc-data.co.uk> (Andrew Murray's message of "Tue\, 26 Jan 2010 21\:46\:15 -0000")

"Andrew Murray" <amurray@mpc-data.co.uk> writes:

[...]

> I've noticed that other than using the debugfs there is no way for a
> user to configure sleep_when_idle, enable_off_mode,
> voltage_off_when_idle. Do you think it would be worthwhile to add
> these options to the KConfigs? I'd be happy to make these
> modifications if so.  :)

Hi Andrew,

I'm curious what benefit you to having them as compile-time options
instead of run-time?

In general, these options are all under debugfs for debug during PM
development, but they should not be considered as PM knobs for a
production system.

For a production system, the assumption is that the kernel is *always*
trying to sleep-while-idle and enter off-mode.  Preventing low-power
states is intended to be directed by drivers using constraints
(latency, throughput, etc.)  When using constraints, the deeper sleep
states are prevented while the constraints are in place and then
(re)allowed after the constraints are removed.

Kevin



>
> Many Thanks,
>
> Andrew Murray
> 	
> -----Original Message-----
> From: Nishanth Menon [mailto:nm@ti.com] 
> Sent: 26 January 2010 20:41
> To: Andrew Murray
> Cc: linux-omap@vger.kernel.org
> Subject: Re: PM: VDD2 OPPs
>
> Andrew Murray had written, on 01/26/2010 02:34 PM, the following:
>> Hello,
>> 
>> With regards to the OMAP2 (or at least the 3530 EVM) -the TRM and
>> various whitepapers suggest that they are 3 OPP levels available for
>> VDD2 (L3). However, from looking at the sources (linux-omap-pm / pm
>> branch) it seems that only 2 OPP levels are supported (@166Mhz and
>> @83Mhz) and used. I also notice that these rates are different to
> those
>> in a whitepaper (166, 100 and 41.5). Is there any particular reason
> why
> on OMAP34/35xx, I believe it should be s/100/83/.
>> only 2 OPPs are used? 
> to my knowledge 41.5Mhz is not known to provide any performance 
> benefits. you can also see [1] and add 41.5 (pm-wip-opp is the new 
> branch where we are introducing opp layer.
>
>> 
>> I understand that the OPP level of VDD2 may be set by changes to the
> OPP
>> level of VDD1 (i.e. resource34xx.c:set_opp) - and modifying VDD1's OPP
>> via cpufreq seems to be the only way to adjust the VDD2 OPP from
>> user-land - is this correct?
> The old /sys/power/vdd2_[opp|lock] was deprecated out. currently the 
> control is for vdd1 OPP using cpufreq and indirect dependency for VDD2, 
> is there a need for direct control of VDD2 freq?
> -- 
> Regards,
> Nishanth Menon
>
> Ref:
> [1] 
> http://git.kernel.org/?p=linux/kernel/git/khilman/linux-omap-pm.git;a=bl
> ob;f=arch/arm/mach-omap2/cpufreq34xx.c;h=07873e87ffc0fef97b4554efc3f17dc
> 696cb25e3;hb=4f54a09e0ed9b2ee8e1adfe1716297792310d1c6#l46
>
>
> No virus found in this incoming message.
> Checked by AVG - www.avg.com 
> Version: 8.5.432 / Virus Database: 271.1.1/2641 - Release Date: 01/25/10
> 19:36:00
> --
> 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:[~2010-01-26 22:40 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-01-26 20:34 PM: VDD2 OPPs Andrew Murray
2010-01-26 20:41 ` Nishanth Menon
2010-01-26 21:46   ` Andrew Murray
2010-01-26 22:02     ` Cousson, Benoit
2010-01-26 23:06       ` Andrew Murray
2010-01-26 22:40     ` Kevin Hilman [this message]
2010-01-26 23:22       ` Andrew Murray
2010-01-26 23:47         ` Kevin Hilman

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=87636o1ocq.fsf@deeprootsystems.com \
    --to=khilman@deeprootsystems.com \
    --cc=amurray@mpc-data.co.uk \
    --cc=linux-omap@vger.kernel.org \
    --cc=nm@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