diff options
| author | Kenneth R Westerback <krw@cvs.openbsd.org> | 2005-12-03 16:53:17 +0000 |
|---|---|---|
| committer | Kenneth R Westerback <krw@cvs.openbsd.org> | 2005-12-03 16:53:17 +0000 |
| commit | 59413aaf9936ee396905effecb9206e3e5d9e01f (patch) | |
| tree | d820a69874e8e867417c8c87fff410362b921e48 /sys/dev/ic/dpt.c | |
| parent | 734d8d431c424e5467db4143f21b9249f5900059 (diff) | |
The first thing done when XXX_scsi_cmd() returns TRY_AGAIN_LATER is
to set xs->error to XS_BUSY. So it is pointless and misleading to
set xs->error to XS_TIMEOUT, XS_DRIVER_STUFFUP or XS_BUSY just before
returning TRY_AGAIN_LATER.
No functional change.
Diffstat (limited to 'sys/dev/ic/dpt.c')
| -rw-r--r-- | sys/dev/ic/dpt.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/sys/dev/ic/dpt.c b/sys/dev/ic/dpt.c index e63c407bade..15056140ff6 100644 --- a/sys/dev/ic/dpt.c +++ b/sys/dev/ic/dpt.c @@ -1,4 +1,4 @@ -/* $OpenBSD: dpt.c,v 1.10 2004/12/26 21:22:13 miod Exp $ */ +/* $OpenBSD: dpt.c,v 1.11 2005/12/03 16:53:16 krw Exp $ */ /* $NetBSD: dpt.c,v 1.12 1999/10/23 16:26:33 ad Exp $ */ /*- @@ -1030,7 +1030,6 @@ dpt_scsi_cmd(xs) */ if (dontqueue) { splx(s); - xs->error = XS_DRIVER_STUFFUP; return (TRY_AGAIN_LATER); } @@ -1058,7 +1057,6 @@ dpt_scsi_cmd(xs) /* If we can't queue, we lose */ if (dontqueue) { splx(s); - xs->error = XS_DRIVER_STUFFUP; return (TRY_AGAIN_LATER); } @@ -1219,7 +1217,6 @@ dpt_scsi_cmd(xs) if (dpt_cmd(sc, &ccb->ccb_eata_cp, ccb->ccb_ccbpa, CP_DMA_CMD, 0)) { printf("%s: dpt_cmd failed\n", sc->sc_dv.dv_xname); - xs->error = XS_DRIVER_STUFFUP; dpt_free_ccb(sc, ccb); return (TRY_AGAIN_LATER); } |