Date: Tue, 23 Jan 2007 11:57:49 -0700 (MST) From: "M. Warner Losh" <imp@bsdimp.com> To: nate@root.org 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: <20070123.115749.-1303479476.imp@bsdimp.com> In-Reply-To: <45B656F2.9010501@root.org> References: <45B6410B.2090107@root.org> <20070123.102738.1244031887.imp@bsdimp.com> <45B656F2.9010501@root.org>
index | next in thread | previous in thread | raw e-mail
In message: <45B656F2.9010501@root.org>
Nate Lawson <nate@root.org> writes:
: 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
Weird. When I was looking into this bug, I got a 404 on the above
link. It is working now. Sorry for the hassles. Lemme know if the
corrected entry is right or not. Thanks for the feedback.
Warner
home |
help
Want to link to this message? Use this
URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20070123.115749.-1303479476.imp>