On Tue, Feb 04, 2014 at 10:09:26AM +0100, Maxime Ripard wrote: > On Tue, Feb 04, 2014 at 12:21:10AM +0000, Mark Brown wrote: > > It isn't awesome, no. Ideally the runtime PM code would do this but > > then you couldn't ifdef the operations which as far as I can tell is the > > main thing people want from disabling it and it gets complicated for > > devices that genuinely do power up on startup so here we are. > We discussed it with Kevin on IRC, and he suggested that we move that > pm_runtime initialization to the SPI core, but I guess that would also > mean that all drivers shouldn't ifdef the operations, so that the core > can call the runtime_resume callback directly. No, that's not going to be robust - it means drivers can't do any power sequencing of their own. > However, I don't really get why any driver should be doing so, since > you still need these functions to at least to the device > suspend/resume in the probe/remove, and you don't really want to > duplicate the code. I don't think it's particularly useful to support disabling runtime PM in the first place but some drivers do different things when doing runtime management to those they do on first init - for example there may be additional steps that only need to be done during first power up.
Attachment:
signature.asc
Description: Digital signature