diff options
| author | Matthew Wilcox <matthew@wil.cx> | 2007-10-05 15:55:01 -0400 |
|---|---|---|
| committer | James Bottomley <jejb@mulgrave.localdomain> | 2007-10-23 15:09:04 -0400 |
| commit | 71c222dc45ddf01b0dabb8a4d3e232e13e1452b5 (patch) | |
| tree | 932f363121f69d8a07fdbf06e80989423b84c548 /drivers/scsi/sym53c8xx_2 | |
| parent | 39c05d1e3c85c725e140ded1281bbb2303dfe5d3 (diff) | |
[SCSI] sym53c8xx: Remove unnecessary check in queuecommand
The midlayer won't scan the host ID, so we don't need to check.
This is the only caller of sym_xpt_done2, so remove that too.
Signed-off-by: Matthew Wilcox <willy@linux.intel.com>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Diffstat (limited to 'drivers/scsi/sym53c8xx_2')
| -rw-r--r-- | drivers/scsi/sym53c8xx_2/sym_glue.c | 18 |
1 files changed, 1 insertions, 17 deletions
diff --git a/drivers/scsi/sym53c8xx_2/sym_glue.c b/drivers/scsi/sym53c8xx_2/sym_glue.c index cfce00ba2b6e..9fc2ae836257 100644 --- a/drivers/scsi/sym53c8xx_2/sym_glue.c +++ b/drivers/scsi/sym53c8xx_2/sym_glue.c | |||
| @@ -151,13 +151,6 @@ void sym_xpt_done(struct sym_hcb *np, struct scsi_cmnd *cmd) | |||
| 151 | cmd->scsi_done(cmd); | 151 | cmd->scsi_done(cmd); |
| 152 | } | 152 | } |
| 153 | 153 | ||
| 154 | static void sym_xpt_done2(struct sym_hcb *np, struct scsi_cmnd *cmd, int cam_status) | ||
| 155 | { | ||
| 156 | sym_set_cam_status(cmd, cam_status); | ||
| 157 | sym_xpt_done(np, cmd); | ||
| 158 | } | ||
| 159 | |||
| 160 | |||
| 161 | /* | 154 | /* |
| 162 | * Tell the SCSI layer about a BUS RESET. | 155 | * Tell the SCSI layer about a BUS RESET. |
| 163 | */ | 156 | */ |
| @@ -324,15 +317,6 @@ static int sym_queue_command(struct sym_hcb *np, struct scsi_cmnd *cmd) | |||
| 324 | int order; | 317 | int order; |
| 325 | 318 | ||
| 326 | /* | 319 | /* |
| 327 | * Minimal checkings, so that we will not | ||
| 328 | * go outside our tables. | ||
| 329 | */ | ||
| 330 | if (sdev->id == np->myaddr) { | ||
| 331 | sym_xpt_done2(np, cmd, DID_NO_CONNECT); | ||
| 332 | return 0; | ||
| 333 | } | ||
| 334 | |||
| 335 | /* | ||
| 336 | * Retrieve the target descriptor. | 320 | * Retrieve the target descriptor. |
| 337 | */ | 321 | */ |
| 338 | tp = &np->target[sdev->id]; | 322 | tp = &np->target[sdev->id]; |
| @@ -537,7 +521,7 @@ static int sym53c8xx_queue_command(struct scsi_cmnd *cmd, | |||
| 537 | struct sym_ucmd *ucp = SYM_UCMD_PTR(cmd); | 521 | struct sym_ucmd *ucp = SYM_UCMD_PTR(cmd); |
| 538 | int sts = 0; | 522 | int sts = 0; |
| 539 | 523 | ||
| 540 | cmd->scsi_done = done; | 524 | cmd->scsi_done = done; |
| 541 | memset(ucp, 0, sizeof(*ucp)); | 525 | memset(ucp, 0, sizeof(*ucp)); |
| 542 | 526 | ||
| 543 | /* | 527 | /* |
