On Monday, September 06, 2010, Mark Brown wrote: > On Mon, Sep 06, 2010 at 03:32:09PM +0300, Ohad Ben-Cohen wrote: > > > That may result in some drivers defining nop handlers, just to return > > 0 (in case there's nothing else they need to do). > > > Do we want that ? > > Funnily enough I was about to report this issue too - it doesn't look > great in the driver code. My use case is using runtime PM in an MFD to > communicate status to the parent devices. The subdevices are just > indicating that they are idle to the parent and have no reason to do > anything in a suspend or resume callback. > > > Alternatively, we may want to allow drivers to enable Runtime PM (by > > taking the appropriate action for their subsystem, e.g. calling > > put_noidle in probe and get_noresume in remove), but still not define > > any runtime pm handlers (implicitly always returning a success), with > > something like: > > This would be my preferred solution. OK Please submit the patch with appropriate changelog and I'll apply it. Thanks, Rafael -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
