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

Rafael J. Wysocki
2010-07-03T19:29:24Z

On Saturday, July 03, 2010, Alan Stern wrote:
> 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.

Well, whatever. I'm fine with that either way.

Rafael