Re: [RFC] Add "rpm_not_supported" flag

Rafael J. Wysocki
2014-07-01T23:00:18Z

On Monday, June 30, 2014 10:42:19 AM Alan Stern wrote:
> On Mon, 30 Jun 2014, Rafael J. Wysocki wrote:
>
> > > > Do you know of any tools that actually look at these files?
> > >
> > > I don't. Of course, that doesn't mean much.
> >
> > The only tool I'm aware of that may be looking at them is powertop, so
> > if the change doesn't affect powertop adversely, it should be generally
> > safe.
> >
> > > > If there isn't any, then we can try to change it and see who screams :)
> > >
> > > It'll be a learning experience...
> >
> > Yes, it will. :-)
>
> Then you have no other objections to the patch?

My concern still is that it will be confusing, because people won't read the
documentation carefully enough and will confuse "runtime PM never used" with
"hardware can't do PM". I'm not sure how to make that more clear, though.

Also we have the no_callbacks flag and I wonder if/how it is related to the
new one. Do we still need both?

In addition to that, I think that "hardware can't do PM" should apply to the
handling of system suspend resume too.

Rafael