[PATCH v7] power: introduce library for device-specific OPPs

Rafael J. Wysocki rjw at sisk.pl
Sat Oct 9 18:59:20 EDT 2010


On Friday, October 08, 2010, Nishanth Menon wrote:
> SoCs have a standard set of tuples consisting of frequency and
> voltage pairs that the device will support per voltage domain. These
> are called Operating Performance Points or OPPs. The actual
> definitions of OPP varies over silicon versions. For a specific domain,
> we can have a set of {frequency, voltage} pairs. As the kernel boots
> and more information is available, a default set of these are activated
> based on the precise nature of device. Further on operation, based on
> conditions prevailing in the system (such as temperature), some OPP
> availability may be temporarily controlled by the SoC frameworks.
> 
> To implement an OPP, some sort of power management support is necessary
> hence this library depends on CONFIG_PM.
> 
> Contributions include:
> Sanjeev Premi for the initial concept:
> 	http://patchwork.kernel.org/patch/50998/
> Kevin Hilman for converting original design to device-based
> Kevin Hilman and Paul Walmsey for cleaning up many of the function
> abstractions, improvements and data structure handling
> Romit Dasgupta for using enums instead of opp pointers
> Thara Gopinath, Eduardo Valentin and Vishwanath BS for fixes and
> cleanups.
> Linus Walleij for recommending this layer be made generic for usage
> in other architectures beyond OMAP and ARM.
> Mark Brown, Andrew Morton, Rafael J Wysocki, Paul E McKenney for valuable
> improvements.
> 
> Discussions and comments from:
> http://marc.info/?l=linux-omap&m=126033945313269&w=2
> http://marc.info/?l=linux-omap&m=125482970102327&w=2
> http://marc.info/?t=125809247500002&r=1&w=2
> http://marc.info/?l=linux-omap&m=126025973426007&w=2
> http://marc.info/?t=128152609200064&r=1&w=2
> http://marc.info/?t=128468723000002&r=1&w=2
> incorporated.
> 
> Cc: Benoit Cousson <b-cousson at ti.com>
> Cc: Madhusudhan Chikkature Rajashekar <madhu.cr at ti.com>
> Cc: Phil Carmody <ext-phil.2.carmody at nokia.com>
> Cc: Roberto Granados Dorado <x0095451 at ti.com>
> Cc: Santosh Shilimkar <santosh.shilimkar at ti.com>
> Cc: Sergio Alberto Aguirre Rodriguez <saaguirre at ti.com>
> Cc: Tero Kristo <Tero.Kristo at nokia.com>
> Cc: Eduardo Valentin <eduardo.valentin at nokia.com>
> Cc: Paul Walmsley <paul at pwsan.com>
> Cc: Sanjeev Premi <premi at ti.com>
> Cc: Thara Gopinath <thara at ti.com>
> Cc: Vishwanath BS <vishwanath.bs at ti.com>
> Cc: Linus Walleij <linus.walleij at stericsson.com>
> Cc: Mark Brown <broonie at opensource.wolfsonmicro.com>
> Cc: Andrew Morton <akpm at linux-foundation.org>
> Cc: Rafael J. Wysocki <rjw at sisk.pl>
> Cc: Paul E. McKenney <paulmck at linux.vnet.ibm.com>
> 
> Signed-off-by: Nishanth Menon <nm at ti.com>

OK

Your error messages are a bit inconsistent (e.g. some of them print the
error code while others don't), but I guess I can fix that up.

Still, to apply the patch I need a copyright notice for the doc too.

> Signed-off-by: Kevin Hilman <khilman at deeprootsystems.com>

Kevin, your sign-off here means you endorse the patch as the maintainer.
Is that correct?

Thanks,
Rafael



More information about the linux-arm-kernel mailing list