Skip to content

Commit

Permalink
mtip32xx: remove mtip_get_int_command
Browse files Browse the repository at this point in the history
Merging this function into the only callers makes the code flow easier.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
  • Loading branch information
Christoph Hellwig authored and Jens Axboe committed Nov 9, 2018
1 parent 7bbf118 commit 55c7bc3
Showing 1 changed file with 7 additions and 17 deletions.
24 changes: 7 additions & 17 deletions drivers/block/mtip32xx/mtip32xx.c
Original file line number Diff line number Diff line change
Expand Up @@ -168,20 +168,6 @@ static bool mtip_check_surprise_removal(struct pci_dev *pdev)
return false; /* device present */
}

static struct mtip_cmd *mtip_get_int_command(struct driver_data *dd)
{
struct request *rq;

if (mtip_check_surprise_removal(dd->pdev))
return NULL;

rq = blk_mq_alloc_request(dd->queue, REQ_OP_DRV_IN, BLK_MQ_REQ_RESERVED);
if (IS_ERR(rq))
return NULL;

return blk_mq_rq_to_pdu(rq);
}

static struct mtip_cmd *mtip_cmd_from_tag(struct driver_data *dd,
unsigned int tag)
{
Expand Down Expand Up @@ -1002,12 +988,15 @@ static int mtip_exec_internal_command(struct mtip_port *port,
return -EFAULT;
}

int_cmd = mtip_get_int_command(dd);
if (!int_cmd) {
if (mtip_check_surprise_removal(dd->pdev))
return -EFAULT;

rq = blk_mq_alloc_request(dd->queue, REQ_OP_DRV_IN, BLK_MQ_REQ_RESERVED);
if (IS_ERR(rq)) {
dbg_printk(MTIP_DRV_NAME "Unable to allocate tag for PIO cmd\n");
return -EFAULT;
}
rq = blk_mq_rq_from_pdu(int_cmd);

rq->special = &icmd;

set_bit(MTIP_PF_IC_ACTIVE_BIT, &port->flags);
Expand All @@ -1029,6 +1018,7 @@ static int mtip_exec_internal_command(struct mtip_port *port,
}

/* Copy the command to the command table */
int_cmd = blk_mq_rq_to_pdu(rq);
memcpy(int_cmd->command, fis, fis_len*4);

rq->timeout = timeout;
Expand Down

0 comments on commit 55c7bc3

Please sign in to comment.