Subject: kern/2617: some MUSTEKs fill with 0xff instead of 0x20
To: None <gnats-bugs@NetBSD.ORG>
From: Ignatios Souvatzis <is@beverly.rhein.de>
List: netbsd-bugs
Date: 07/11/1996 15:47:05
>Number:         2617
>Category:       kern
>Synopsis:       some MUSTEKs fill with 0xff instead of 0x20
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    kern-bug-people (Kernel Bug People)
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Thu Jul 11 10:20:01 1996
>Last-Modified:
>Originator:     Ignatios Souvatzis
>Organization:
computer science department, university of Bonn, Germany
>Release:        1.2_BETA
>Environment:
	
System: NetBSD cosinus.cs.uni-bonn.de 1.2_ALPHA NetBSD 1.2_ALPHA (COSINUS) #12: Mon Jun 17 13:46:38 MET DST 1996 ignatios@cosinus.cs.uni-bonn.de:/usr/src/sys/arch/i386/compile/COSINUS i386

Original report from Neil Brewitt, with a (for this discussion) 
NetBSD-1.2_ALPHA running on Amiga 3000

>Description:
	Some MUSTEK MFS-6000CX scanners fill the vendor part of the
id string with 0xFF instead of 0x20, making the match function fail.
You see:
ss0 at scsibus0 targ 3 lun 0: <MUSTEK\\377\\377, 
	MFS-06000CX\\377\\377\\377\\377\\377, 2.61> SCSI0 6/scanner removable
and immediately afterwards the probe for the next target id.
>How-To-Repeat:
	Boot Neils machine.
>Fix:
This patch to src/sys/scsi/ss.c should fix the problem.

--- ss.c.original	Thu Jul 11 15:23:00 1996
+++ ss.c	Thu Jul 11 15:23:35 1996
@@ -140,7 +140,7 @@
 	 * and install functions for special handling
 	 */
 	SC_DEBUG(sc_link, SDEV_DB2, ("ssattach:\n"));
-	if (!bcmp(sa->sa_inqbuf->vendor, "MUSTEK  ", 8))
+	if (!bcmp(sa->sa_inqbuf->vendor, "MUSTEK", 6))
 		mustek_attach(ss, sa);
 	if (!bcmp(sa->sa_inqbuf->vendor, "HP      ", 8))
 		scanjet_attach(ss, sa);
>Audit-Trail:
>Unformatted: