[src/trunk]: src/sys/dev/scsipi revert the only part of rev. 1.177 which made...

[drochner <drochner%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/8373131fca2d
branches:  trunk
changeset: 580487:8373131fca2d
user:      drochner <drochner%NetBSD.org@localhost>
date:      Mon Apr 25 17:08:46 2005 +0000

description:
revert the only part of rev. 1.177 which made a functional
change -- it broke error reporting because st->asc(q) are
not set
(actually, I believe that SSD_RCODE_VALID is misnamed)

diffstat:

 sys/dev/scsipi/st.c |  9 ++++-----
 1 files changed, 4 insertions(+), 5 deletions(-)

diffs (30 lines):

diff -r 6e329435d98b -r 8373131fca2d sys/dev/scsipi/st.c
--- a/sys/dev/scsipi/st.c       Mon Apr 25 16:30:29 2005 +0000
+++ b/sys/dev/scsipi/st.c       Mon Apr 25 17:08:46 2005 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: st.c,v 1.179 2005/03/31 11:28:53 yamt Exp $ */
+/*     $NetBSD: st.c,v 1.180 2005/04/25 17:08:46 drochner Exp $ */
 
 /*-
  * Copyright (c) 1998, 2004 The NetBSD Foundation, Inc.
@@ -57,7 +57,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: st.c,v 1.179 2005/03/31 11:28:53 yamt Exp $");
+__KERNEL_RCSID(0, "$NetBSD: st.c,v 1.180 2005/04/25 17:08:46 drochner Exp $");
 
 #include "opt_scsi.h"
 
@@ -2068,9 +2068,8 @@
         * If it isn't a extended or extended/deferred error, let
         * the generic code handle it.
         */
-       if ((sense->response_code & SSD_RCODE_VALID) == 0 ||
-           (SSD_RCODE(sense->response_code) != SSD_RCODE_CURRENT &&
-            SSD_RCODE(sense->response_code) != SSD_RCODE_DEFERRED))
+       if (SSD_RCODE(sense->response_code) != SSD_RCODE_CURRENT &&
+           SSD_RCODE(sense->response_code) != SSD_RCODE_DEFERRED)
                return (retval);
 
        if (sense->response_code & SSD_RCODE_VALID)