From: Rafael J. Wysocki Date: Wed, 15 Aug 2012 19:31:45 +0000 (+0200) Subject: PM / Runtime: Fix rpm_resume() return value for power.no_callbacks set X-Git-Tag: v3.6-rc3~33^2~5 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=7f321c26c04807834fef4c524d2b21573423fc74;p=profile%2Fivi%2Fkernel-x86-ivi.git PM / Runtime: Fix rpm_resume() return value for power.no_callbacks set For devices whose power.no_callbacks flag is set, rpm_resume() should return 1 if the device's parent is already active, so that the callers of pm_runtime_get() don't think that they have to wait for the device to resume (asynchronously) in that case (the core won't queue up an asynchronous resume in that case, so there's nothing to wait for anyway). Modify the code accordingly (and make sure that an idle notification will be queued up on success, even if 1 is to be returned). Signed-off-by: Rafael J. Wysocki Acked-by: Alan Stern Cc: stable@vger.kernel.org --- diff --git a/drivers/base/power/runtime.c b/drivers/base/power/runtime.c index 5989487..9891a85 100644 --- a/drivers/base/power/runtime.c +++ b/drivers/base/power/runtime.c @@ -584,6 +584,7 @@ static int rpm_resume(struct device *dev, int rpmflags) || dev->parent->power.runtime_status == RPM_ACTIVE) { atomic_inc(&dev->parent->power.child_count); spin_unlock(&dev->parent->power.lock); + retval = 1; goto no_callback; /* Assume success. */ } spin_unlock(&dev->parent->power.lock); @@ -664,7 +665,7 @@ static int rpm_resume(struct device *dev, int rpmflags) } wake_up_all(&dev->power.wait_queue); - if (!retval) + if (retval >= 0) rpm_idle(dev, RPM_ASYNC); out: