Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 23 Jan 2007 10:41:54 -0800
From:      Nate Lawson <nate@root.org>
To:        "M. Warner Losh" <imp@bsdimp.com>
Cc:        cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/cam/scsi scsi_da.c
Message-ID:  <45B656F2.9010501@root.org>
In-Reply-To: <20070123.102738.1244031887.imp@bsdimp.com>
References:  <45B5AE3B.6010005@root.org>	<20070123.012823.1341031192.imp@bsdimp.com>	<45B6410B.2090107@root.org> <20070123.102738.1244031887.imp@bsdimp.com>

next in thread | previous in thread | raw e-mail | index | archive | help
M. Warner Losh wrote:
> In message: <45B6410B.2090107@root.org>
>             Nate Lawson <nate@root.org> writes:
> : M. Warner Losh wrote:
> : > In message: <45B5AE3B.6010005@root.org>
> : >             Nate Lawson <nate@root.org> writes:
> : > : Warner Losh wrote:
> : > : > imp         2007-01-22 04:34:03 UTC
> : > : > 
> : > : >   FreeBSD src repository
> : > : > 
> : > : >   Modified files:
> : > : >     sys/cam/scsi         scsi_da.c 
> : > : >   Log:
> : > : >   Add quirk for EasyMP3 EM732X usb 2.0 flash mp3 player.
> : > : >   (It appears that the quirk proceedures link has disappeared and that
> : > : >   this PR complied with it, if there's a problem, please contact me).
> : > : >   
> : > : >   PR: usb/96546
> : > : >   
> : > : >   Revision  Changes    Path
> : > : >   1.201     +8 -0      src/sys/cam/scsi/scsi_da.c
> : > : > 
> : > : > 
> : > : > Index: src/sys/cam/scsi/scsi_da.c
> : > : > diff -u src/sys/cam/scsi/scsi_da.c:1.200 src/sys/cam/scsi/scsi_da.c:1.201
> : > : > --- src/sys/cam/scsi/scsi_da.c:1.200	Tue Dec  5 07:45:28 2006
> : > : > +++ src/sys/cam/scsi/scsi_da.c	Mon Jan 22 04:34:03 2007
> : > : > @@ -466,6 +466,14 @@
> : > : >  		{T_DIRECT, SIP_MEDIA_REMOVABLE, "X-Micro", "Flash Disk",
> : > : >  		"*"}, /*quirks*/ DA_Q_NO_SYNC_CACHE
> : > : >  	},
> : > : > +	{
> : > : > +		/*
> : > : > +		 * EasyMP3 EM732X USB 2.0 Flash MP3 Player
> : > : > +		 * PR: usb/96546
> : > : > +		 */
> : > : > +		{T_DIRECT, SIP_MEDIA_REMOVABLE, "*", "MP3 Player*",
> : > : > +		"1.0"}, /*quirks*/ DA_Q_NO_SYNC_CACHE
> : > : > +	},
> : > : >  };
> : > : >  
> : > : >  static	disk_strategy_t	dastrategy;
> : > : 
> : > : Um, this matches every device that has "MP3 Player" somewhere in it. 
> : > : Are you sure it doesn't have a vendor string?
> : > 
> : > See the PR for details and suggest something better.
> : > 
> : > Warner
> : 
> : According to the submitter:
> : pass0: <EM732X MP3 Player 1.00> Removable Direct Access SCSI-0 device
> : 
> : So the vendor string should be set to "EM732X" and retested.  Blank 
> : ("*") vendor or device strings should not be committed.
> 
> Any chance you could get your quirks how-to listed in scsi_da back
> online?
> 
> Warner

I don't understand.  It's been at the same location for years:
http://www.root.org/~nate/freebsd/scsi/quirks.html


-- 
Nate



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?45B656F2.9010501>