lkml.org 
[lkml]   [2014]   [Jun]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 1/1] PM / Runtime: let rpm_resume fail if rpm disabled and device suspended.
    On Sun, 22 Jun 2014, Rafael J. Wysocki wrote:

    > > How would you treat them specially? Add a "runtime_pm_not_supported"
    > > flag?
    >
    > I thought about a "runtime PM has been enabled at least once" flag rather
    > that would be set by pm_runtime_enable() every time it is called and never
    > cleared. That would allow the core to distinguish between "runtime PM
    > disabled temporarily" and "runtime PM not used" which turn out to be
    > sufficiently different cases.

    Interesting idea, but it can't tell the difference between "runtime PM
    not supported" and "runtime PM not enabled yet". I think a simple "not
    supported" flag will be more straightforward.

    > Yes. The core definitely needs to be able to distinguish between the
    > "runtime PM disabled temporarily" and "runtime PM not supported/not used"
    > situations.

    Let me work out a patch, and we'll see what you think. For the time
    being we can stick with our "runtime PM must be disabled (or in error)
    when the status is changed" approach.

    Alan Stern



    \
     
     \ /
      Last update: 2014-06-22 19:21    [W:6.750 / U:0.016 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site