Re: [PATCH] PM: make runtime_status attribute not debug-only

Alan Stern
2010-07-03T00:03:27Z

On Fri, 2 Jul 2010, Rafael J. Wysocki wrote:

> On Friday, July 02, 2010, Alan Stern wrote:
> > This patch (as1404) makes the runtime_status sysfs attribute available
> > even in the absence of CONFIG_PM_ADVANCED_DEBUG, and it changes the
> > routine to display "unknown" when runtime PM is disabled for a device.
> >
> > Signed-off-by: Alan Stern <stern@rowland.harvard.edu>
> >
> > ---
> >
> > Maybe "unsupported" would be a little better than "unknown". What do
> > you think?
>
> I thought about that. "Unsupported" seems to suggest that the device/driver
> doesn't support runtime PM, whiile in fact it may be supported, although it's
> not enabled at the moment.

Drivers that support runtime PM generally don't leave it disabled for
very long. IMO this wouldn't be terribly misleading.

You can change the patch, or if you prefer, I'll submit a revised
version in a few days.

Alan Stern