]> pilppa.com Git - linux-2.6-omap-h63xx.git/commitdiff
ieee1394: sbp2: wrap two functions into one
authorStefan Richter <stefanr@s5r6.in-berlin.de>
Sat, 4 Nov 2006 08:55:33 +0000 (09:55 +0100)
committerStefan Richter <stefanr@s5r6.in-berlin.de>
Thu, 7 Dec 2006 22:04:03 +0000 (23:04 +0100)
Move the body of a function into its only caller.

Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
drivers/ieee1394/sbp2.c

index 187e26f0485ddd7003ab4498c15eee7cdc602be2..5d3b96e0a9512d025036e295fe547db1c1ad2ff3 100644 (file)
@@ -591,18 +591,14 @@ static struct sbp2_command_info *sbp2util_allocate_command_orb(
        return cmd;
 }
 
-static void sbp2util_free_command_dma(struct sbp2_command_info *cmd)
+/*
+ * Unmaps the DMAs of a command and moves the command to the completed ORB list.
+ * Must be called with lu->cmd_orb_lock held.
+ */
+static void sbp2util_mark_command_completed(struct sbp2_lu *lu,
+                                           struct sbp2_command_info *cmd)
 {
-       struct sbp2_lu *lu = (struct sbp2_lu *)
-                       cmd->Current_SCpnt->device->host->hostdata[0];
-       struct hpsb_host *host;
-
-       if (!lu) {
-               SBP2_ERR("%s: lu == NULL", __FUNCTION__);
-               return;
-       }
-
-       host = lu->ud->ne->host;
+       struct hpsb_host *host = lu->ud->ne->host;
 
        if (cmd->cmd_dma) {
                if (cmd->dma_type == CMD_DMA_SINGLE)
@@ -615,23 +611,11 @@ static void sbp2util_free_command_dma(struct sbp2_command_info *cmd)
                cmd->dma_type = CMD_DMA_NONE;
                cmd->cmd_dma = 0;
        }
-
        if (cmd->sge_buffer) {
                pci_unmap_sg(host->pdev, cmd->sge_buffer,
                             cmd->dma_size, cmd->dma_dir);
                cmd->sge_buffer = NULL;
        }
-}
-
-/*
- * This function moves a command to the completed orb list.
- * Must be called with lu->cmd_orb_lock held.
- */
-static void sbp2util_mark_command_completed(
-               struct sbp2_lu *lu,
-               struct sbp2_command_info *cmd)
-{
-       sbp2util_free_command_dma(cmd);
        list_move_tail(&cmd->list, &lu->cmd_orb_completed);
 }