On Friday 12 of October 2012 11:07:18 Alan Stern wrote: > On Fri, 12 Oct 2012, Aaron Lu wrote: > > > v3: > > Only patch 4 is modified: > > Remove the special case for system freeze in scsi_bus_suspend_common > > as pointed out by Alan Stern; > > Updated some comments; > > Removed the use of typedef (*pm_callback_t)(struct device *). > > > > v2: > > Change the runtime suspend behaviour of sd driver by putting the device > > into stopped power state. > > Revert 2 patches which are no longer needed as pointed out by Alan Stern. > > Find out device callbacks in bus callbacks as suggested by Alan Stern. > > > > Due to these changes, patch number grows from 2 -> 5. > > > > v1: > > The 2 patches will migrate SCSI drivers to use the pm callbacks defined > > in dev_pm_ops as pm_message is deprecated and should not be used by driver. > > Bus level callback is changed to use callbacks defined in dev_pm_ops when > > needed and sd's pm callback is updated to use what are defined in dev_pm_ops. > > > > Aaron Lu (5): > > sd: put to stopped power state when runtime suspend > > Revert "[SCSI] scsi_pm: set device runtime state before parent > > suspended" > > Revert "[SCSI] runtime resume parent for child's system-resume" > > pm: use callbacks from dev_pm_ops for scsi devices > > sd: update sd to use the new pm callbacks > > > > drivers/scsi/scsi_pm.c | 97 +++++++++++++++++++++++++++----------------------- > > drivers/scsi/sd.c | 18 +++++++--- > > 2 files changed, 66 insertions(+), 49 deletions(-) > > Remember to run all the patches through checkpatch.pl before submitting > them. Aside from that, for the whole series: > > Acked-by: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> >From me too: Acked-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx> And please do the checkpatch.pl thing as Alan said. Thanks, Rafael -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html