Re: [PATCH v11 0/4] Allow USB devices to remain runtime-suspended when sleeping

2016-01-05 Thread Rafael J. Wysocki
On Monday, January 04, 2016 06:27:18 PM Derek Basehore wrote: > On Mon, Nov 02, 2015 at 02:50:40AM +0100, Rafael J. Wysocki wrote: > > > > I've queued up this series for the second half of the v4.4 merge window. > &g

Re: [PATCH v11 0/4] Allow USB devices to remain runtime-suspended when sleeping

2015-11-01 Thread Rafael J. Wysocki
On Tuesday, October 27, 2015 03:38:47 PM Tomeu Vizoso wrote: > Hi, > > this is v11 of an attempt to make it easier for devices to remain in > runtime PM when the system goes to sleep, mainly to reduce the time > spent resuming devices. > > For this, we interpret the absence of all PM callback imp

Re: [PATCH v10 2/4] PM / Domains: add setter for dev.pm_domain

2015-10-25 Thread Rafael J. Wysocki
On Wednesday, October 21, 2015 05:34:12 PM Tomeu Vizoso wrote: > Adds a function that sets the pointer to dev_pm_domain in struct device > and that warns if the device has already finished probing. The reason > why we want to enforce that is because in the general case that can > cause problems and

Re: [PATCH v3 1/2] Trace: PM: promote event 'power_domain_target' to generic power domains.

2015-10-14 Thread Rafael J. Wysocki
On Wednesday, October 14, 2015 10:18:27 AM Marc Titinger wrote: > On 14/10/2015 02:55, Rafael J. Wysocki wrote: > > On Monday, September 28, 2015 03:20:44 PM Marc Titinger wrote: > >> - change arg3 to a state name string: we got the current CPU rom the trace > >>

Re: [PATCH v3 1/2] Trace: PM: promote event 'power_domain_target' to generic power domains.

2015-10-13 Thread Rafael J. Wysocki
On Monday, September 28, 2015 03:20:44 PM Marc Titinger wrote: > - change arg3 to a state name string: we got the current CPU rom the trace > backend already. This also prepares for multiple/named states in the power > domain, consistent with idle-states. states in the domain may match given > CPU

Re: [PATCH v9 0/4] Allow USB devices to remain runtime-suspended when sleeping

2015-10-05 Thread Rafael J. Wysocki
On Monday, October 05, 2015 04:45:28 PM Tomeu Vizoso wrote: > Hi, > > this is v9 of an attempt to make it easier for devices to remain in > runtime PM when the system goes to sleep, mainly to reduce the time > spent resuming devices. > > For this, we interpret the absence of all PM callback imple

Re: [PATCH] base: power: wakeirq: don't leak dev->power.wakeirq

2015-07-07 Thread Rafael J. Wysocki
On Tue, Jul 7, 2015 at 10:11 AM, Felipe Balbi wrote: > On Tue, Jul 07, 2015 at 12:40:53AM -0700, Tony Lindgren wrote: >> * Rafael J. Wysocki [150706 15:49]: >> > On Monday, July 06, 2015 01:01:18 PM Felipe Balbi wrote: >> > > on a first call to dev_pm_attach_wake

Re: [PATCH] base: power: wakeirq: don't leak dev->power.wakeirq

2015-07-06 Thread Rafael J. Wysocki
On Monday, July 06, 2015 01:01:18 PM Felipe Balbi wrote: > on a first call to dev_pm_attach_wake_irq(), if it > fails, it will leave dev->power.wakeirq set to a > dangling pointer. Instead, let's clear it to make > sure a subsequent call to dev_pm_attach_wake_irq() > has chance to succeed. > > Cc:

Re: [PATCH 2/2] cpufreq: dt: allow driver to boot automatically

2015-06-16 Thread Rafael J. Wysocki
On Tue, Jun 16, 2015 at 11:40 PM, Felipe Balbi wrote: > On Fri, May 08, 2015 at 02:57:30PM -0500, Felipe Balbi wrote: >> by adding the missing MODULE_ALIAS(), cpufreq-dt >> can be autoloaded by udev/systemd. >> >> Signed-off-by: Felipe Balbi > > looks like this wasn't applied anywhere. Viresh, ca

Re: [PATCH] PM / Wakeirq: Fix typo in prototype for dev_pm_set_dedicated_wake_irq

2015-05-29 Thread Rafael J. Wysocki
NFIG_MMC_OMAP_HS are selected. > >=20 > > Cc: Stephen Rothwell > > Cc: Ulf Hansson > > Reported-by: Jim Davis > > Signed-off-by: Tony Lindgren > > cool: > > Reviewed-by: Felipe Balbi Applied, thanks! -- I speak only for myself. Rafael J. Wysocki, Inte

Re: [PATCH 5/5] mmc: omap_hsmmc: Change wake-up interrupt to use generic wakeirq

2015-05-27 Thread Rafael J. Wysocki
On 5/25/2015 10:38 AM, Ulf Hansson wrote: On 14 May 2015 at 01:36, Tony Lindgren wrote: We can now use generic wakeirq handling and remove the custom handling for the wake-up interrupts. Signed-off-by: Tony Lindgren Rafael, if you are fine with it, please take this one through your linux-pm

Re: [PATCH 2/5] PM / Wakeirq: Add automated device wake IRQ handling

2015-05-20 Thread Rafael J. Wysocki
On Tuesday, May 19, 2015 07:10:57 PM Tony Lindgren wrote: > * Rafael J. Wysocki [150519 17:01]: > > On Tuesday, May 19, 2015 04:27:56 PM Tony Lindgren wrote: > > > * Rafael J. Wysocki [150519 16:07]: > > > > On Wednesday, May 20, 2015 12:41:06 AM Thomas Gleixner w

Re: [PATCH 2/5] PM / Wakeirq: Add automated device wake IRQ handling

2015-05-19 Thread Rafael J. Wysocki
On Wednesday, May 20, 2015 12:41:06 AM Thomas Gleixner wrote: > On Wed, 20 May 2015, Rafael J. Wysocki wrote: > > This one looks really good. :-) > > > > If it doesn't depend on anything, I can apply it right away, so please let > > me > > know. > > C

Re: [PATCH 0/5] PM / clock_ops: provide default runtime ops and cleanup users

2015-04-24 Thread Rafael J. Wysocki
apply this series, but I can do that if people don't have problems with that. -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH 1/4] PM / Wakeirq: Add minimal device wakeirq helper functions

2015-03-09 Thread Rafael J. Wysocki
On Sunday, March 08, 2015 11:43:34 AM Alan Stern wrote: > On Sat, 7 Mar 2015, Rafael J. Wysocki wrote: > > > But this is part of a bigger picture. Namely, if a separete wakeup > > interrupt > > is required for a device, the device's power.can_wakeup flag cannot be

Re: [PATCH 1/4] PM / Wakeirq: Add minimal device wakeirq helper functions

2015-03-06 Thread Rafael J. Wysocki
On Friday, March 06, 2015 03:05:40 PM Tony Lindgren wrote: > * Alan Stern [150306 11:05]: > > On Fri, 6 Mar 2015, Tony Lindgren wrote: > > > > > > > + struct wakeirq_source *wirq = _wirq; > > > > > + irqreturn_t ret = IRQ_NONE; > > > > > + > > > > > + /* We don't want RPM_ASYNC or RPM

Re: [PATCH 1/4] PM / Wakeirq: Add minimal device wakeirq helper functions

2015-03-06 Thread Rafael J. Wysocki
On Friday, March 06, 2015 02:05:43 PM Alan Stern wrote: > On Fri, 6 Mar 2015, Tony Lindgren wrote: > > > > > + struct wakeirq_source *wirq = _wirq; > > > > + irqreturn_t ret = IRQ_NONE; > > > > + > > > > + /* We don't want RPM_ASYNC or RPM_NOWAIT here */ > > > > + if (pm_ru

Re: [PATCH 1/4] PM / Wakeirq: Add minimal device wakeirq helper functions

2015-03-06 Thread Rafael J. Wysocki
On Fri, Mar 6, 2015 at 3:02 AM, Rafael J. Wysocki wrote: > Please always CC linux-pm on CC patches. Doh. That was supposed to say "Please always CC linux-pm on PM patches". I really should not reply to email when I'm too tired ... -- To unsubscribe from this list: send the

Re: [PATCH 1/4] PM / Wakeirq: Add minimal device wakeirq helper functions

2015-03-05 Thread Rafael J. Wysocki
gt; +#else/* !CONFIG_PM_WAKEIRQ */ > + > +static inline int dev_pm_wakeirq_request(struct device *dev, > + int wakeirq, > + irq_handler_t handler, > + unsigned long ir

Re: [PATCH] treewide: Convert clockevents_notify to use int cpu

2014-12-20 Thread Rafael J. Wysocki
> Add pr_fmt and convert one printk(KERN_ERR to pr_err That is fine by me, so Acked-by: Rafael J. Wysocki for the cpuidle and ACPI changes. > Signed-off-by: Joe Perches > --- > arch/arm/mach-omap2/cpuidle44xx.c | 7 +++ > arch/arm/mach-tegra/cpuidle-tegra114.c | 4 +

[PATCH] ARM / PM: Replace CONFIG_PM_RUNTIME with CONFIG_PM

2014-12-05 Thread Rafael J. Wysocki
From: Rafael J. Wysocki After commit b2b49ccbdd54 (PM: Kconfig: Set PM_RUNTIME if PM_SLEEP is selected) PM_RUNTIME is always set if PM is set, so #ifdef blocks depending on CONFIG_PM_RUNTIME may now be changed to depend on CONFIG_PM. Replace CONFIG_PM_RUNTIME with CONFIG_PM everywhere in the

Re: [PATCH] hsi / OMAP / PM: Replace CONFIG_PM_RUNTIME with CONFIG_PM

2014-12-03 Thread Rafael J. Wysocki
On Wednesday, December 03, 2014 08:49:12 PM Arnd Bergmann wrote: > On Wednesday 03 December 2014 03:02:24 Rafael J. Wysocki wrote: > > From: Rafael J. Wysocki > > > > After commit b2b49ccbdd54 (PM: Kconfig: Set PM_RUNTIME if PM_SLEEP is > > selected) PM_RUNTIME is

[PATCH] hsi / OMAP / PM: Replace CONFIG_PM_RUNTIME with CONFIG_PM

2014-12-02 Thread Rafael J. Wysocki
From: Rafael J. Wysocki After commit b2b49ccbdd54 (PM: Kconfig: Set PM_RUNTIME if PM_SLEEP is selected) PM_RUNTIME is always set if PM is set, so #ifdef blocks depending on CONFIG_PM_RUNTIME may now be changed to depend on CONFIG_PM. Do that for the omap_ssi driver. Signed-off-by: Rafael J

[PATCH] i2c-omap / PM: Drop CONFIG_PM_RUNTIME from i2c-omap.c

2014-12-02 Thread Rafael J. Wysocki
From: Rafael J. Wysocki After commit b2b49ccbdd54 (PM: Kconfig: Set PM_RUNTIME if PM_SLEEP is selected) PM_RUNTIME is always set if PM is set, so some #ifdef blocks depending on CONFIG_PM_RUNTIME may be dropped now. Do that in drivers/i2c/busses/i2c-omap.c. Signed-off-by: Rafael J. Wysocki

[PATCH] gpio / PM: Replace CONFIG_PM_RUNTIME with CONFIG_PM

2014-12-02 Thread Rafael J. Wysocki
From: Rafael J. Wysocki After commit b2b49ccbdd54 (PM: Kconfig: Set PM_RUNTIME if PM_SLEEP is selected) PM_RUNTIME is always set if PM is set, so #ifdef blocks depending on CONFIG_PM_RUNTIME may now be changed to depend on CONFIG_PM. Replace CONFIG_PM_RUNTIME with CONFIG_PM in drivers/gpio/gpio

Re: [PATCH 00/27] add pm_runtime_last_busy_and_autosuspend() helper

2014-09-25 Thread Rafael J. Wysocki
s. If in fu= > ture > > > there are no users (today we have quite a few), then we can remove the = > dead > > > macro, no harm. But that is not the situation today. > >=20 > > as I said, a commit that's bound to be useless. It's not like you're &

Re: [PATCH 00/27] add pm_runtime_last_busy_and_autosuspend() helper

2014-09-24 Thread Rafael J. Wysocki
+-- > drivers/usb/musb/omap2430.c |6 +-- > drivers/video/fbdev/auo_k190x.c | 9 +--- > include/linux/pm_runtime.h |6 +++ > 31 files changed, 97 insertions(+), 177 deletions(-) OK, I guess this is as

Re: [PATCH] PM / OPP: Remove ARCH_HAS_OPP

2014-06-09 Thread Rafael J. Wysocki
On Monday, June 09, 2014 11:51:42 PM Mark Brown wrote: > > --fYgRtaZIy+F1uhp1 > Content-Type: text/plain; charset=us-ascii > Content-Disposition: inline > > On Tue, Jun 10, 2014 at 12:22:07AM +0200, Rafael J. Wysocki wrote: > > On Monday, June 09, 2014 08:49:17 PM Mark B

Re: [PATCH] PM / OPP: Remove ARCH_HAS_OPP

2014-06-09 Thread Rafael J. Wysocki
On Monday, June 09, 2014 08:49:17 PM Mark Brown wrote: > On Fri, Jun 06, 2014 at 11:15:06PM +0200, Rafael J. Wysocki wrote: > > On Friday, June 06, 2014 02:08:50 PM Mark Brown wrote: > > > > Yes, the conversion to make ARCH_HAS_OPP unused is in Raphael's tree

Re: [PATCH] PM / OPP: Remove ARCH_HAS_OPP

2014-06-06 Thread Rafael J. Wysocki
On Friday, June 06, 2014 02:08:50 PM Mark Brown wrote: > > --cU9XODsizZBnwgll > Content-Type: text/plain; charset=us-ascii > Content-Disposition: inline > > On Fri, Jun 06, 2014 at 09:50:06PM +0900, Simon Horman wrote: > > On Fri, Jun 06, 2014 at 09:14:01PM +0900, Magnus Damm wrote: > > > > I'm

Re: Ethernet controller not starting

2014-03-03 Thread Rafael J. Wysocki
ue I'm having to the following commit: > > > 45f0a85c8258741d11bda25c0a5669c06267204a is the first bad commit > > commit 45f0a85c8258741d11bda25c0a5669c06267204a > > Author: Rafael J. Wysocki > > Date: Mon Jun 3 21:49:52 2013 +0200 > > > &

Re: pm_runtime functions and IS_ERR_VALUE (was Re: [PATCH v6] ARM: omap: edma: add suspend suspend/resume hooks)

2013-11-28 Thread Rafael J. Wysocki
0 > +... > +) > + > +//-- > +// For org and report mode > +//-- > + > +@r depends on runtime_bad_err_handle exists@ > +position p1, p2; > +identifier pm_run

Re: [GIT PULL] PM / AVS: SmartReflex: misc. cleanups for v3.11

2013-06-15 Thread Rafael J. Wysocki
> the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH] cpufreq: OMAP: instantiate omap-cpufreq as a platform_driver

2013-04-10 Thread Rafael J. Wysocki
(&omap_driver); > } > > +static struct platform_driver omap_cpufreq_platdrv = { > + .driver = { > + .name = "omap-cpufreq", > + .owner = THIS_MODULE, > + }, > + .probe = omap_cpufreq_probe, > + .remove = o

Re: [GIT PULL] CPUidle: OMAP cleanups for v3.10

2013-04-10 Thread Rafael J. Wysocki
2:07 +0200) > > are available in the git repository at: > > > git://git.kernel.org/pub/scm/linux/kernel/git/khilman/linux-omap-pm.git > tags/omap-pm-v3.10/cleanup/cpuidle-v2 Pulled into my bleeding-edge branch. I'll move it to linux-next tomorrow. Thanks, Rafael -- I

Re: [PATCH 0/4] OPP usage fixes for RCU locking

2013-01-21 Thread Rafael J. Wysocki
On Monday, January 21, 2013 02:45:32 PM MyungJoo Ham wrote: > On Sat, Jan 19, 2013 at 7:28 AM, Rafael J. Wysocki wrote: > > On Friday, January 18, 2013 01:52:31 PM Nishanth Menon wrote: > >> Hi, > >> Despite being documented in function documentation and in > >&g

Re: [PATCH 0/4] OPP usage fixes for RCU locking

2013-01-18 Thread Rafael J. Wysocki
5 +++ > drivers/devfreq/exynos4_bus.c | 94 > > 4 files changed, 80 insertions(+), 27 deletions(-) > > [1] http://www.beagleboard.org/irclogs/index.php?date=2013-01-18#T14:14:07 > [2] > https://github.com/nmenon/linux-2.6-playgr

Re: [RFC PATCH 4/5] arm: omap2: support port power on lan95xx devices

2012-12-05 Thread Rafael J. Wysocki
s for the handling of system suspend/resume in such a way that it is consistent with runtime PM. For USB ports I'd recommend to use something simpler. :-) Thanks, Rafael -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: OMAP baseline test results for v3.7-rc1

2012-10-31 Thread Rafael J. Wysocki
gt; > constraint remove. > I do not think this is correct. Using disable_hlt is a too radical > solution and will prevent the idle completely, this is not what we > want. > > Rafael, what do you think? Well, I agree. Thanks, Rafael -- I speak only for myself. Rafael J. Wysoc

Re: [PATCHv2] Input: omap4-keypad: Add pinctrl support

2012-10-30 Thread Rafael J. Wysocki
> and pins. Should it be centralized or not? If it's to be centralized it > needs to become a large piece of infrastructure refactoring and > needs the attention of Linaro and the like to happen. Well, I personally think it should be centralized somehow. I'm not quite sure how to a

Re: [PATCH 0/4] cpufreq: OMAP: fixes for v3.7-rc2

2012-10-07 Thread Rafael J. Wysocki
kernel.org during that time. So I think it would be better to merge this material through the arm-soc tree, if that's not a problem. If you decide to do so, please feel free to add my ACK to the patches. Thanks, Rafael -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Techno

Re: [PATCH v2] PM / Runtime: let rpm_resume() succeed if RPM_ACTIVE, even when disabled

2012-09-22 Thread Rafael J. Wysocki
On Saturday, September 22, 2012, Alan Stern wrote: > On Sat, 22 Sep 2012, Rafael J. Wysocki wrote: > > > On Saturday, September 22, 2012, Kevin Hilman wrote: > > > OK, this looks good to me, thanks! > > > > Alan, what do you think? > > > > Rafae

Re: [PATCH v2] PM / Runtime: let rpm_resume() succeed if RPM_ACTIVE, even when disabled

2012-09-22 Thread Rafael J. Wysocki
gle-xM where a wakeup IRQ firing during the late > suspend phase triggers runtime PM activity in the I2C driver since the > wakeup IRQ is on an I2C-connected PMIC. > > Cc: Rafael J. Wysocki > Cc: Alan Stern > Signed-off-by: Kevin Hilman > --- > v2: > - major changelog re

Re: [PATCH RESEND] PM / Runtime: let rpm_resume() succeed if RPM_ACTIVE, even when disabled

2012-09-21 Thread Rafael J. Wysocki
On Friday, September 21, 2012, Alan Stern wrote: > On Thu, 20 Sep 2012, Rafael J. Wysocki wrote: > > > On Thursday, September 20, 2012, Kevin Hilman wrote: > > > From: Kevin Hilman > > > > > > When runtime PM is disabled, what we want is for callba

Re: [PATCH RESEND] PM / Runtime: let rpm_resume() succeed if RPM_ACTIVE, even when disabled

2012-09-20 Thread Rafael J. Wysocki
e simplified slightly. > To permit the usage described above, add a check to rpm_resume() so > that success is returned in the case where a driver is suspended (it's > ->suspend callback has been called) but is still RPM_ACTIVE. > > This patch was developed in close collaboration

Re: [PATCH] PM / Runtime: let rpm_resume() succeed if RPM_ACTIVE, even when disabled

2012-09-20 Thread Rafael J. Wysocki
here a driver is suspended (it's > ->suspend callback has been called) but is still RPM_ACTIVE. > > This patch was developed in close collaboration with Rafael J. Wysocki > > > Tested on AM3730/Beagle-xM where wakeup IRQ firing during the late > suspend phase triggers run

Re: [PATCH] CPUFreq: OMAP: remove unnecessary plat/ includes

2012-09-19 Thread Rafael J. Wysocki
On Thursday, September 20, 2012, Paul Walmsley wrote: > Hi > > On Fri, 14 Sep 2012, Rafael J. Wysocki wrote: > > > On Wednesday, September 12, 2012, Paul Walmsley wrote: > > > > > > Remove some unnecessary plat/ includes that are interfering with &

Re: [PATCH] cpufreq: OMAP: Check IS_ERR() instead of NULL for omap_device_get_by_hwmod_name

2012-09-19 Thread Rafael J. Wysocki
On Wednesday, September 19, 2012, Kevin Hilman wrote: > "Rafael J. Wysocki" writes: > > > On Tuesday, September 18, 2012, Axel Lin wrote: > >> omap_device_get_by_hwmod_name() returns ERR_PTR on error. > >> > >> Signed-off-by: Axel Lin &g

Re: [PATCH] cpufreq: OMAP: Check IS_ERR() instead of NULL for omap_device_get_by_hwmod_name

2012-09-18 Thread Rafael J. Wysocki
On Tuesday, September 18, 2012, Axel Lin wrote: > omap_device_get_by_hwmod_name() returns ERR_PTR on error. > > Signed-off-by: Axel Lin Kevin, should I take this? Rafael > --- > drivers/cpufreq/omap-cpufreq.c |4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/dr

Re: [PATCH] CPUFreq: OMAP: remove unnecessary plat/ includes

2012-09-14 Thread Rafael J. Wysocki
On Wednesday, September 12, 2012, Paul Walmsley wrote: > > Remove some unnecessary plat/ includes that are interfering with multi-subarch > ARM kernels. > > Signed-off-by: Paul Walmsley > Cc: Kevin Hilman > Cc: Rafael J. Wysocki > Acked-by: Kevin Hilman Ack

Re: [PATCH v2 1/2] cpufreq: OMAP: Handle missing frequency table on SMP systems

2012-08-09 Thread Rafael J. Wysocki
On Thursday, August 09, 2012, Kevin Hilman wrote: > "Rafael J. Wysocki" writes: > > > On Thursday, August 09, 2012, Rajendra Nayak wrote: > >> On OMAP4, if the first CPU fails to get a valid frequency table (this > >> could happen if the platfo

Re: [PATCH v2 1/2] cpufreq: OMAP: Handle missing frequency table on SMP systems

2012-08-09 Thread Rafael J. Wysocki
On Thursday, August 09, 2012, Rajendra Nayak wrote: > On OMAP4, if the first CPU fails to get a valid frequency table (this > could happen if the platform does not register any OPP table), the > subsequent CPU instances end up dealing with a NULL freq_table and > crash. > > Check for an already ex

Re: [PATCH] MFD: twl-core: move device_init_wakeup to after platform_device_add.

2012-07-07 Thread Rafael J. Wysocki
t; calls dev_set_name. > > So the call to device_init_wakeup() must be after the call to > platform_device_add(). > Making this change causes correct names to appear in the > wakeup_sources file. > > > Signed-off-by: NeilBrown Acked-by: Rafael J. Wysocki > > diff --gi

[PATCH] omap-rng: Use struct dev_pm_ops for power management

2012-07-04 Thread Rafael J. Wysocki
From: Rafael J. Wysocki Make the omap-rng driver define its PM callbacks through a struct dev_pm_ops object rather than by using legacy PM hooks in struct platform_driver. Signed-off-by: Rafael J. Wysocki --- drivers/char/hw_random/omap-rng.c | 13 +++-- 1 file changed, 7 insertions

Re: [PATCH v2 0/9] PM: Create the AVS class of drivers

2012-04-18 Thread Rafael J. Wysocki
On Wednesday, April 18, 2012, Kevin Hilman wrote: > Rafael, > > jean.pi...@newoldbits.com writes: > > > From: Jean Pihet > > > > AVS is a power management technique which controls the operating > > voltage of a device in order to optimize (i.e. reduce) its power > > consumption. The voltage is a

Re: [PATCH v8 0/8] Consolidate cpuidle functionality

2012-03-20 Thread Rafael J. Wysocki
On Wednesday, March 21, 2012, Amit Kucheria wrote: > On Wed, Mar 21, 2012 at 12:48 AM, Kevin Hilman wrote: > > Arnd Bergmann writes: > > > >> On Tuesday 20 March 2012, Robert Lee wrote: > >>> This patch series moves various functionality duplicated in platform > >>> cpuidle drivers to the core cp

Re: [PATCH 10/10] ARM: OMAP: SmartReflex: Move smartreflex driver to drivers/

2012-03-15 Thread Rafael J. Wysocki
On Thursday, March 15, 2012, jean.pi...@newoldbits.com wrote: > From: Jean Pihet > > After a clean-up of the interfaces the OMAP IP driver and class > support code is now a generic driver. > Move it to drivers/power/avs/. > > The build is controlled by the following Kconfig options: > . CONFIG_

Re: [PATCH RFC 2/2] mtd : Make the mtd_suspend return 0 if the suspend is not implemented

2012-02-02 Thread Rafael J. Wysocki
On Thursday, February 02, 2012, Russell King - ARM Linux wrote: > On Thu, Feb 02, 2012 at 07:40:26PM +0100, Rafael J. Wysocki wrote: > > On Thursday, February 02, 2012, Artem Bityutskiy wrote: > > > On Mon, 2012-01-30 at 23:58 +0100, Rafael J. Wysocki wrote: > > > >

Re: [PATCH RFC 2/2] mtd : Make the mtd_suspend return 0 if the suspend is not implemented

2012-02-02 Thread Rafael J. Wysocki
On Thursday, February 02, 2012, Artem Bityutskiy wrote: > On Mon, 2012-01-30 at 23:58 +0100, Rafael J. Wysocki wrote: > > Thanks, but is anyone actually going to push it to Linus any time soon? > > I agree, but I am not the maintainer so cannot answer. David Woodhouse > is aware

Re: [PATCH RFC 2/2] mtd : Make the mtd_suspend return 0 if the suspend is not implemented

2012-01-30 Thread Rafael J. Wysocki
; > 1. When the '->suspend' method is not present, we return -EOPNOTSUPP, > but >the callers of 'mtd_suspend()' expects 0 instead. > 2. Checking of the 'mtd' parameter against NULL has been incorrectly > removed >in 'mtd_cls_suspend()&

Re: PM / OPP: Fix build when CONFIG_PM_OPP is not set

2011-11-02 Thread Rafael J. Wysocki
gt; include/linux/opp.h:103: multiple definition of `opp_get_notifier' > include/linux/opp.h:103: first defined here > > Also fix incorrect comment. > > Cc: MyungJoo Ham > Cc: Kyungmin Park > Cc: Mike Turquette > Cc: Kevin Hilman > Cc: Rafael J. Wysocki > Sig

Re: [PATCH] PM QoS: update Documentation for the pm_qos and dev_pm_qos frameworks

2011-10-04 Thread Rafael J. Wysocki
On Monday, October 03, 2011, jean.pi...@newoldbits.com wrote: > From: Jean Pihet > > Update the documentation for the recently updated pm_qos API, kernel > and user space. > Add the documentation for the per-device PM QoS (dev_pm_qos) framework > API. > > Signed-off-by: Jean Pihet Applied to l

Re: [RFC/PATCH] PM QoS: main memory throughput constraints

2011-09-30 Thread Rafael J. Wysocki
Hi, On Friday, September 30, 2011, Jean Pihet wrote: > Hi, > > Here is a patch which adds a PM_QOS_MEMORY_THROUGHPUT class to the PM > QoS framework. The idea is to provide a memory or SDMA throughput > constraints class, which can be applied to the low level platform code > using the callback no

Re: [PATCH] OMAP: omap_device: fix !CONFIG_SUSPEND case in _noirq handlers

2011-09-01 Thread Rafael J. Wysocki
On Thursday, September 01, 2011, Arnd Bergmann wrote: > On Thursday 01 September 2011 11:12:02 Kevin Hilman wrote: > > The suspend/resume _noirq handlers were #ifdef'd out in the > > !CONFIG_SUSPEND case, but were still assigned to the dev_pm_ops > > struct. Fix by defining them to NULL in the !CO

[GIT PULL] Power management fixes for 3.1

2011-08-28 Thread Rafael J. Wysocki
not normal suspend/resume Magnus Damm (2): ARM: mach-shmobile: sh7372 LCDC1 suspend fix ARM: mach-shmobile: sh7372 LCDC1 suspend fix V2 (incremental) Rafael J. Wysocki (3): PM: Use spinlock instead of mutex in clock management functions sh-sci / PM: Use power.irq_safe PM

Re: [linux-pm] [PATCH 02/11] PM: extend PM QoS with per-device wake-up constraints

2011-08-26 Thread Rafael J. Wysocki
On Friday, August 26, 2011, mark gross wrote: > On Fri, Aug 26, 2011 at 11:25:13AM +0900, MyungJoo Ham wrote: > > On Fri, Jul 1, 2011 at 12:11 AM, wrote: > > > @@ -129,19 +146,19 @@ static const struct file_operations > > > pm_qos_power_fops = { > > > /* unlocked internal variant */ > > > stat

Re: Issue: Runtime API usage in wake-up device irq_handler during wakeup from system-wide-suspend.

2011-08-26 Thread Rafael J. Wysocki
On Friday, August 26, 2011, Govindraj.R wrote: > Hello, > > During system_wide_suspend pm runtime is disabled. > I.e. __pm_runtime_disable is called from __device_suspend. > Now, if a wakeup interrupt is triggered and the wakeup device irq handler > is called even before device_resume and pm_runti

Re: [linux-pm] [PATCH 02/11] PM: extend PM QoS with per-device wake-up constraints

2011-08-25 Thread Rafael J. Wysocki
On Thursday, August 25, 2011, Jean Pihet wrote: > On Thu, Aug 25, 2011 at 4:17 PM, Rafael J. Wysocki wrote: > > On Thursday, August 25, 2011, Mark Brown wrote: > >> On Tue, Aug 23, 2011 at 11:31:54PM +0200, Rafael J. Wysocki wrote: > >> > >> > Perhaps. Sti

Re: [linux-pm] [PATCH 02/11] PM: extend PM QoS with per-device wake-up constraints

2011-08-25 Thread Rafael J. Wysocki
On Thursday, August 25, 2011, Mark Brown wrote: > On Tue, Aug 23, 2011 at 11:31:54PM +0200, Rafael J. Wysocki wrote: > > > Perhaps. Still, that requires some policy to be put into drivers, > > which I don't think is entirely correct. > > So, I don't really

Re: [PATCH 1/1] OMAP: omap_device: only override _noirq methods, not normal suspend/resume

2011-08-25 Thread Rafael J. Wysocki
On Thursday, August 25, 2011, Kevin Hilman wrote: > commit c03f007a8bf0e092caeb6856a5c8a850df10b974 (OMAP: PM: > omap_device: add system PM methods for PM domain handling) mistakenly > used SET_SYSTEM_SLEEP_PM_OPS() when trying to configure custom methods > for the PM domains noirq methods. Fix th

Re: [linux-pm] [PATCH 02/11] PM: extend PM QoS with per-device wake-up constraints

2011-08-23 Thread Rafael J. Wysocki
On Tuesday, August 23, 2011, Mark Brown wrote: > On Sun, Aug 21, 2011 at 08:05:53PM +0200, Rafael J. Wysocki wrote: > > On Sunday, August 21, 2011, Mark Brown wrote: > > > > I don't understand why the driver would need to know what situation it's > > > in.

Re: [PATCH v6 0/7] PM QoS: add a per-device latency constraints framework

2011-08-22 Thread Rafael J. Wysocki
On Monday, August 22, 2011, Kevin Hilman wrote: > jean.pi...@newoldbits.com writes: > > > From: Jean Pihet > > > > High level implementation: > > > > 1. Preparation of the PM QoS for the addition of a device PM QoS constraints > >framework: > > . rename and move of the PM QoS implementation

Re: [linux-pm] [PATCH 02/11] PM: extend PM QoS with per-device wake-up constraints

2011-08-21 Thread Rafael J. Wysocki
On Sunday, August 21, 2011, Mark Brown wrote: > On Sat, Aug 20, 2011 at 09:14:37PM +0200, Rafael J. Wysocki wrote: > > > I guess you mean the driver here and I'm not really sure it can. > > For instance, the driver may not know what configuration it works in, > > e.g.

Re: [linux-pm] [PATCH 02/11] PM: extend PM QoS with per-device wake-up constraints

2011-08-20 Thread Rafael J. Wysocki
On Saturday, August 20, 2011, Mark Brown wrote: > On Sat, Aug 20, 2011 at 06:51:42PM +0200, Rafael J. Wysocki wrote: > > On Saturday, August 20, 2011, Mark Brown wrote: > > > > Coming at this from the embedded perspective modifying the kernel just > > > isn't

Re: [linux-pm] [PATCH 02/11] PM: extend PM QoS with per-device wake-up constraints

2011-08-20 Thread Rafael J. Wysocki
On Saturday, August 20, 2011, Mark Brown wrote: > On Sat, Aug 20, 2011 at 06:34:34PM +0200, Rafael J. Wysocki wrote: > > On Saturday, August 20, 2011, Mark Brown wrote: > > > > Any sort of media streaming would be an obvious example - the > > > application picks the

Re: [linux-pm] [PATCH 02/11] PM: extend PM QoS with per-device wake-up constraints

2011-08-20 Thread Rafael J. Wysocki
On Saturday, August 20, 2011, Mark Brown wrote: > On Sat, Aug 20, 2011 at 11:35:58AM +0200, Rafael J. Wysocki wrote: > > > I'm not sure what you mean by "exposing per-device wakeup latency > > constraints > > to user space". I simply thought it might be us

Re: [linux-pm] [PATCH 02/11] PM: extend PM QoS with per-device wake-up constraints

2011-08-20 Thread Rafael J. Wysocki
On Saturday, August 20, 2011, Mark Brown wrote: > On Sat, Aug 20, 2011 at 09:48:25AM -0400, Alan Stern wrote: > > > No, I as wasking about driver- and subsystem-specific interfaces to > > userspace that translate into things users are already doing. Kevin's > > example was a touchscreen (although

Re: [linux-pm] [PATCH 02/11] PM: extend PM QoS with per-device wake-up constraints

2011-08-20 Thread Rafael J. Wysocki
On Saturday, August 20, 2011, Mark Brown wrote: > On Fri, Aug 19, 2011 at 10:42:20PM +0200, Rafael J. Wysocki wrote: > > > I really wouldn't like the discussion to go in circles. > > > First, please tell me what in particular you are objecting to, > > because

Re: [linux-pm] [PATCH 02/11] PM: extend PM QoS with per-device wake-up constraints

2011-08-19 Thread Rafael J. Wysocki
Hi, I really wouldn't like the discussion to go in circles. First, please tell me what in particular you are objecting to, because I don't think that's any of the patches that have been sent to the linux-pm list to date. Thanks, Rafael -- To unsubscribe from this list: send the line "unsubscribe

Re: [PATCH v6 0/7] PM QoS: add a per-device latency constraints framework

2011-08-18 Thread Rafael J. Wysocki
Hi, On Wednesday, August 17, 2011, jean.pi...@newoldbits.com wrote: > From: Jean Pihet > > High level implementation: > > 1. Preparation of the PM QoS for the addition of a device PM QoS constraints >framework: > . rename and move of the PM QoS implementation files to kernel/power/qos.c >

Re: [PATCH 07/15] PM QoS: add a global notification mechanism for the device constraints

2011-08-16 Thread Rafael J. Wysocki
Hi, On Tuesday, August 16, 2011, jean.pi...@newoldbits.com wrote: > From: Jean Pihet > > Add a global notification chain that gets called upon changes to the > aggregated constraint value for any device. > The notification callbacks are passing the full constraint request data > in order for the

Re: [PATCH 06/15] PM QoS: implement the per-device PM QoS constraints

2011-08-16 Thread Rafael J. Wysocki
Hi, On Tuesday, August 16, 2011, jean.pi...@newoldbits.com wrote: > From: Jean Pihet > > Implement the per-device PM QoS constraints by creating a device > PM QoS API, which calls the PM QoS constraints management core code. > > The per-device latency constraints data strctures are stored > in

Re: [PATCH 05/15] PM QoS: generalize and export the constraints management code

2011-08-16 Thread Rafael J. Wysocki
On Tuesday, August 16, 2011, mark gross wrote: > On Tue, Aug 16, 2011 at 08:44:19AM +0200, Jean Pihet wrote: > > On Tue, Aug 16, 2011 at 6:08 AM, mark gross wrote: > > > On Sun, Aug 14, 2011 at 03:37:43PM +0200, Rafael J. Wysocki wrote: > > >> On Sunday, Aug

Re: [PATCH 07/15] PM QoS: add a global notification mechanism for the device constraints

2011-08-14 Thread Rafael J. Wysocki
Hi, There is some code duplication in this patch that should better be avoided (details below). On Thursday, August 11, 2011, jean.pi...@newoldbits.com wrote: > From: Jean Pihet > > Add a global notification chain that gets called upon changes to the > aggregated constraint value for any device

Re: [PATCH v4 00/15] PM QoS: add a per-device latency constraints class

2011-08-14 Thread Rafael J. Wysocki
On Sunday, August 14, 2011, Jean Pihet wrote: > Rafael, > > On Fri, Aug 12, 2011 at 11:56 PM, Rafael J. Wysocki wrote: > > Hi, > > > > On Friday, August 12, 2011, Jean Pihet wrote: > >> Hi Rafael, > >> > >> 2011/8/12 Rafael J.

Re: [PATCH 06/15] PM QoS: implement the per-device PM QoS constraints

2011-08-14 Thread Rafael J. Wysocki
Hi, On Sunday, August 14, 2011, Jean Pihet wrote: ... > >> + > >> + if (dev_pm_qos_request_active(req)) { > >> + WARN(1, KERN_ERR "dev_pm_qos_add_request() called for > >> already " > >> + "added request\n"); > >> + return; > >> + } > >> + r

Re: [PATCH 05/15] PM QoS: generalize and export the constraints management code

2011-08-14 Thread Rafael J. Wysocki
On Sunday, August 14, 2011, Jean Pihet wrote: > Hi Rafael, Mark, > > On Sat, Aug 13, 2011 at 10:34 PM, Rafael J. Wysocki wrote: > > On Saturday, August 13, 2011, mark gross wrote: > >> On Thu, Aug 11, 2011 at 05:06:42PM +0200, jean.pi...@newoldbits.com wrote:

Re: [PATCH 04/15] PM QoS: re-organize data structs

2011-08-14 Thread Rafael J. Wysocki
On Sunday, August 14, 2011, Jean Pihet wrote: > Rafael, Mark, > > On Sat, Aug 13, 2011 at 10:58 PM, Rafael J. Wysocki wrote: > > On Saturday, August 13, 2011, mark gross wrote: > >> On Thu, Aug 11, 2011 at 05:06:41PM +0200, jean.pi...@newoldbits.com wrote:

Re: [PATCH 06/15] PM QoS: implement the per-device PM QoS constraints

2011-08-13 Thread Rafael J. Wysocki
Hi, Well, it looks like I should have reviewed this one more carefully. On Thursday, August 11, 2011, jean.pi...@newoldbits.com wrote: > From: Jean Pihet > > Implement the per-device PM QoS constraints by creating a device > PM QoS API, which calls the PM QoS constraints management core code. >

Re: [PATCH 04/15] PM QoS: re-organize data structs

2011-08-13 Thread Rafael J. Wysocki
On Saturday, August 13, 2011, mark gross wrote: > On Thu, Aug 11, 2011 at 05:06:41PM +0200, jean.pi...@newoldbits.com wrote: > > From: Jean Pihet > > > > In preparation for the per-device constratins support, re-organize > > the data strctures: > > - add a struct pm_qos_constraints which contains

Re: [PATCH 05/15] PM QoS: generalize and export the constraints management code

2011-08-13 Thread Rafael J. Wysocki
On Saturday, August 13, 2011, mark gross wrote: > On Thu, Aug 11, 2011 at 05:06:42PM +0200, jean.pi...@newoldbits.com wrote: > > From: Jean Pihet > > > > In preparation for the per-device constratins support: > > - rename update_target to pm_qos_update_target > > - generalize and export pm_qos_up

Re: [PATCH v4 00/15] PM QoS: add a per-device latency constraints class

2011-08-12 Thread Rafael J. Wysocki
Hi, On Friday, August 12, 2011, Jean Pihet wrote: > Hi Rafael, > > 2011/8/12 Rafael J. Wysocki : > > On Thursday, August 11, 2011, jean.pi...@newoldbits.com wrote: > >> From: Jean Pihet > >> > >> This patch set is in an RFC state, for review and com

Re: [PATCH v4 00/15] PM QoS: add a per-device latency constraints class

2011-08-12 Thread Rafael J. Wysocki
On Thursday, August 11, 2011, jean.pi...@newoldbits.com wrote: > From: Jean Pihet > > This patch set is in an RFC state, for review and comments. > > High level implementation: > > 1. Preparation of the PM QoS for the addition of a device PM QoS constraints >framework: > . rename and move

Re: [linux-pm] [PATCH 02/11] PM: extend PM QoS with per-device wake-up constraints

2011-08-08 Thread Rafael J. Wysocki
On Sunday, August 07, 2011, Mark Brown wrote: > On Sat, Aug 06, 2011 at 09:46:20PM +0200, Rafael J. Wysocki wrote: > > On Saturday, August 06, 2011, Mark Brown wrote: > > > > I wouldn't say we've got to rely on userspace here - it seems like we > > > oug

Re: [linux-pm] [PATCH 02/11] PM: extend PM QoS with per-device wake-up constraints

2011-08-06 Thread Rafael J. Wysocki
On Saturday, August 06, 2011, Mark Brown wrote: > On Fri, Aug 05, 2011 at 09:37:36PM +0200, Rafael J. Wysocki wrote: > > On Friday, August 05, 2011, Mark Brown wrote: > > > > Do you have any examples of this that aren't better expressed in device > > > specific

Re: [linux-pm] [PATCH 02/11] PM: extend PM QoS with per-device wake-up constraints

2011-08-05 Thread Rafael J. Wysocki
On Friday, August 05, 2011, Mark Brown wrote: > On Thu, Aug 04, 2011 at 09:15:30PM +0200, Rafael J. Wysocki wrote: > > On Thursday, August 04, 2011, Mark Brown wrote: > > > > On the one hand that's true. On the other hand that just seems like > > > going do

Re: [linux-pm] [RFC/PATCH v2] PM / Runtime: allow _put_sync() from interrupts-disabled context

2011-08-05 Thread Rafael J. Wysocki
On Friday, August 05, 2011, Kevin Hilman wrote: > "Rafael J. Wysocki" writes: > > > On Friday, July 22, 2011, Kevin Hilman wrote: > >> Currently the use of pm_runtime_put_sync() is not safe from > >> interrupts-disabled context because rpm_idle() w

Re: [linux-pm] [PATCH 02/11] PM: extend PM QoS with per-device wake-up constraints

2011-08-04 Thread Rafael J. Wysocki
On Thursday, August 04, 2011, Mark Brown wrote: > On Wed, Aug 03, 2011 at 12:16:17AM +0200, Rafael J. Wysocki wrote: > > On Tuesday, August 02, 2011, Kevin Hilman wrote: > > > > I disagree and think that both are quite realistic (mainly because they > > > exist to

Re: [PATCH 02/11] PM: extend PM QoS with per-device wake-up constraints

2011-08-02 Thread Rafael J. Wysocki
On Tuesday, August 02, 2011, Kevin Hilman wrote: > [adding Mark Brown as we discussed similar topics a couple plumbers ago] > > "Rafael J. Wysocki" writes: > > [...] > > >> >> The new class is only available from kernel drivers and so is

Re: [PATCH 04/13] PM: QoS: implement the per-device latency constraints

2011-08-02 Thread Rafael J. Wysocki
Hi, On Tuesday, August 02, 2011, Jean Pihet wrote: ... > >> -static inline void pm_qos_set_value(struct pm_qos_object *o, s32 value) > >> +static inline void pm_qos_set_value(struct pm_qos_constraints *c, s32 > >> value) > >> { > >> - o->target_value = value; > >> + c->target_value = va

  1   2   3   4   >