diff options
author | Christoph Hellwig <hch@lst.de> | 2014-11-24 09:36:20 -0500 |
---|---|---|
committer | Christoph Hellwig <hch@lst.de> | 2014-12-04 03:55:46 -0500 |
commit | 17ea01267c75c5a2e00f3603037715d0d2ce4a58 (patch) | |
tree | ac78360f95ee125cd1ee6f0238e2473f948c6377 | |
parent | b70870c38ede2bb5b0a738f2f69de145a1931a00 (diff) |
scsi: remove scsi_get_tag_type
Both remaining users are better of just checking sdev->simple_tags
directly.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Bart Van Assche <bvanassche@acm.org>
Reviewed-by: Martin K. Petersen <martin.petersen@oracle.com
Reviewed-by: Hannes Reinecke <hare@suse.de>
-rw-r--r-- | drivers/scsi/53c700.c | 4 | ||||
-rw-r--r-- | drivers/scsi/ipr.c | 2 | ||||
-rw-r--r-- | include/scsi/scsi_tcq.h | 13 |
3 files changed, 3 insertions, 16 deletions
diff --git a/drivers/scsi/53c700.c b/drivers/scsi/53c700.c index e7229114b52d..2c87a7ac1d86 100644 --- a/drivers/scsi/53c700.c +++ b/drivers/scsi/53c700.c | |||
@@ -1816,8 +1816,8 @@ NCR_700_queuecommand_lck(struct scsi_cmnd *SCp, void (*done)(struct scsi_cmnd *) | |||
1816 | hostdata->tag_negotiated &= ~(1<<scmd_id(SCp)); | 1816 | hostdata->tag_negotiated &= ~(1<<scmd_id(SCp)); |
1817 | } | 1817 | } |
1818 | 1818 | ||
1819 | if((hostdata->tag_negotiated &(1<<scmd_id(SCp))) | 1819 | if ((hostdata->tag_negotiated & (1<<scmd_id(SCp))) && |
1820 | && scsi_get_tag_type(SCp->device)) { | 1820 | SCp->device->simple_tags) { |
1821 | slot->tag = SCp->request->tag; | 1821 | slot->tag = SCp->request->tag; |
1822 | CDEBUG(KERN_DEBUG, SCp, "sending out tag %d, slot %p\n", | 1822 | CDEBUG(KERN_DEBUG, SCp, "sending out tag %d, slot %p\n", |
1823 | slot->tag, slot); | 1823 | slot->tag, slot); |
diff --git a/drivers/scsi/ipr.c b/drivers/scsi/ipr.c index c35ef5f01e92..c91c7c5da569 100644 --- a/drivers/scsi/ipr.c +++ b/drivers/scsi/ipr.c | |||
@@ -5755,7 +5755,7 @@ static void ipr_erp_cancel_all(struct ipr_cmnd *ipr_cmd) | |||
5755 | 5755 | ||
5756 | ipr_reinit_ipr_cmnd_for_erp(ipr_cmd); | 5756 | ipr_reinit_ipr_cmnd_for_erp(ipr_cmd); |
5757 | 5757 | ||
5758 | if (!scsi_get_tag_type(scsi_cmd->device)) { | 5758 | if (!scsi_cmd->device->simple_tags) { |
5759 | ipr_erp_request_sense(ipr_cmd); | 5759 | ipr_erp_request_sense(ipr_cmd); |
5760 | return; | 5760 | return; |
5761 | } | 5761 | } |
diff --git a/include/scsi/scsi_tcq.h b/include/scsi/scsi_tcq.h index 59578beac053..681e010c2c48 100644 --- a/include/scsi/scsi_tcq.h +++ b/include/scsi/scsi_tcq.h | |||
@@ -15,19 +15,6 @@ | |||
15 | 15 | ||
16 | 16 | ||
17 | #ifdef CONFIG_BLOCK | 17 | #ifdef CONFIG_BLOCK |
18 | /** | ||
19 | * scsi_get_tag_type - get the type of tag the device supports | ||
20 | * @sdev: the scsi device | ||
21 | */ | ||
22 | static inline int scsi_get_tag_type(struct scsi_device *sdev) | ||
23 | { | ||
24 | if (!sdev->tagged_supported) | ||
25 | return 0; | ||
26 | if (sdev->simple_tags) | ||
27 | return MSG_SIMPLE_TAG; | ||
28 | return 0; | ||
29 | } | ||
30 | |||
31 | static inline void scsi_set_tag_type(struct scsi_device *sdev, int tag) | 18 | static inline void scsi_set_tag_type(struct scsi_device *sdev, int tag) |
32 | { | 19 | { |
33 | switch (tag) { | 20 | switch (tag) { |