ide: complete power step in ide_complete_pm_request()

* Complete power step in ide_complete_pm_request().

* Rename ide_complete_pm_request() to ide_complete_pm_rq().

There should be no functional changes caused by this patch.

Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>

2 files changed, 9 insertions, 8 deletions

diff --git a/drivers/ide/ide-io.c b/drivers/ide/ide-io.c
index 7007c48e27ae..49b098de367c 100644
--- a/drivers/ide/ide-io.c
+++ b/drivers/ide/ide-io.c
@@ -178,11 +178,7 @@ void ide_end_drive_cmd (ide_drive_t *drive, u8 stat, u8 err)
                                 kfree(task);
                 }
         } else if (blk_pm_request(rq)) {
-                struct request_pm_state *pm = rq->data;
-
-                ide_complete_power_step(drive, rq);
-                if (pm->pm_step == IDE_PM_COMPLETED)
-                        ide_complete_pm_request(drive, rq);
+                ide_complete_pm_rq(drive, rq);
                 return;
         }
 
@@ -438,7 +434,7 @@ static ide_startstop_t start_request (ide_drive_t *drive, struct request *rq)
                         startstop = ide_start_power_step(drive, rq);
                         if (startstop == ide_stopped &&
                             pm->pm_step == IDE_PM_COMPLETED)
-                                ide_complete_pm_request(drive, rq);
+                                ide_complete_pm_rq(drive, rq);
                         return startstop;
                 } else if (!rq->rq_disk && blk_special_request(rq))
                         /*
diff --git a/drivers/ide/ide-pm.c b/drivers/ide/ide-pm.c
index 60538d9c84ee..74c7c2bbe0fd 100644
--- a/drivers/ide/ide-pm.c
+++ b/drivers/ide/ide-pm.c
@@ -169,18 +169,23 @@ out_do_tf:
 }
 
 /**
- *      ide_complete_pm_request - end the current Power Management request
+ *      ide_complete_pm_rq - end the current Power Management request
  *      @drive: target drive
  *      @rq: request
  *
  *      This function cleans up the current PM request and stops the queue
  *      if necessary.
  */
-void ide_complete_pm_request(ide_drive_t *drive, struct request *rq)
+void ide_complete_pm_rq(ide_drive_t *drive, struct request *rq)
 {
         struct request_queue *q = drive->queue;
+        struct request_pm_state *pm = rq->data;
         unsigned long flags;
 
+        ide_complete_power_step(drive, rq);
+        if (pm->pm_step != IDE_PM_COMPLETED)
+                return;
+
 #ifdef DEBUG_PM
         printk("%s: completing PM request, %s\n", drive->name,
                blk_pm_suspend_request(rq) ? "suspend" : "resume");