Re: [PATCH] PM: add dpm_use_runtime_{suspend, resume} helper functions

Alan Stern
2010-02-26T22:06:59Z

On Fri, 26 Feb 2010, Rafael J. Wysocki wrote:

> I have one problem with the design. Namely, dpm_invoke_runtime_*() can
> run a callback from another subsystem. Say you are a device class and you
> decide to use dpm_invoke_runtime_*(), but the device's bus type implements
> the runtime PM callbacks, so they will be run as device class suspend and
> resume callbacks. That doesn't look particularly clean to me.

That is a valid point. I suppose there could be separate bus-type,
device-type, and device-class versions of these functions, but that
seems like excessive complication with little real benefit.

Mark and Kevin, do you have any thoughts on this?

Alan Stern