diff options
author | Bart Van Assche <bvanassche@acm.org> | 2018-10-18 18:45:46 -0400 |
---|---|---|
committer | Martin K. Petersen <martin.petersen@oracle.com> | 2018-10-23 21:32:34 -0400 |
commit | 2c309aeed62c25661eb2c7d4e4510613a1c7ffc2 (patch) | |
tree | 666ec2f6d2ef308ae33ca779620ea0df6a64b871 /drivers | |
parent | 5b0af4777b1bf397787f03336f0db34f185ca565 (diff) |
scsi: qla2xxx: Remove two arguments from qlafx00_error_entry()
Move a debug statement from qlafx00_error_entry() into its caller. Remove
one unused argument from that function. This patch does not change the
behavior of the qla2xxx driver.
Cc: Himanshu Madhani <himanshu.madhani@cavium.com>
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
Acked-by: Himanshu Madhani <himanshu.madhani@cavium.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/scsi/qla2xxx/qla_mr.c | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/drivers/scsi/qla2xxx/qla_mr.c b/drivers/scsi/qla2xxx/qla_mr.c index b8f967e61891..60f964c53c01 100644 --- a/drivers/scsi/qla2xxx/qla_mr.c +++ b/drivers/scsi/qla2xxx/qla_mr.c | |||
@@ -2681,12 +2681,10 @@ qlafx00_multistatus_entry(struct scsi_qla_host *vha, | |||
2681 | * @vha: SCSI driver HA context | 2681 | * @vha: SCSI driver HA context |
2682 | * @rsp: response queue | 2682 | * @rsp: response queue |
2683 | * @pkt: Entry pointer | 2683 | * @pkt: Entry pointer |
2684 | * @estatus: | ||
2685 | * @etype: | ||
2686 | */ | 2684 | */ |
2687 | static void | 2685 | static void |
2688 | qlafx00_error_entry(scsi_qla_host_t *vha, struct rsp_que *rsp, | 2686 | qlafx00_error_entry(scsi_qla_host_t *vha, struct rsp_que *rsp, |
2689 | struct sts_entry_fx00 *pkt, uint8_t estatus, uint8_t etype) | 2687 | struct sts_entry_fx00 *pkt) |
2690 | { | 2688 | { |
2691 | srb_t *sp; | 2689 | srb_t *sp; |
2692 | struct qla_hw_data *ha = vha->hw; | 2690 | struct qla_hw_data *ha = vha->hw; |
@@ -2695,9 +2693,6 @@ qlafx00_error_entry(scsi_qla_host_t *vha, struct rsp_que *rsp, | |||
2695 | struct req_que *req = NULL; | 2693 | struct req_que *req = NULL; |
2696 | int res = DID_ERROR << 16; | 2694 | int res = DID_ERROR << 16; |
2697 | 2695 | ||
2698 | ql_dbg(ql_dbg_async, vha, 0x507f, | ||
2699 | "type of error status in response: 0x%x\n", estatus); | ||
2700 | |||
2701 | req = ha->req_q_map[que]; | 2696 | req = ha->req_q_map[que]; |
2702 | 2697 | ||
2703 | sp = qla2x00_get_sp_from_handle(vha, func, req, pkt); | 2698 | sp = qla2x00_get_sp_from_handle(vha, func, req, pkt); |
@@ -2745,9 +2740,11 @@ qlafx00_process_response_queue(struct scsi_qla_host *vha, | |||
2745 | 2740 | ||
2746 | if (pkt->entry_status != 0 && | 2741 | if (pkt->entry_status != 0 && |
2747 | pkt->entry_type != IOCTL_IOSB_TYPE_FX00) { | 2742 | pkt->entry_type != IOCTL_IOSB_TYPE_FX00) { |
2743 | ql_dbg(ql_dbg_async, vha, 0x507f, | ||
2744 | "type of error status in response: 0x%x\n", | ||
2745 | pkt->entry_status); | ||
2748 | qlafx00_error_entry(vha, rsp, | 2746 | qlafx00_error_entry(vha, rsp, |
2749 | (struct sts_entry_fx00 *)pkt, pkt->entry_status, | 2747 | (struct sts_entry_fx00 *)pkt); |
2750 | pkt->entry_type); | ||
2751 | continue; | 2748 | continue; |
2752 | } | 2749 | } |
2753 | 2750 | ||