Date: Wed, 28 Mar 2007 08:00:00 +0200 From: Alexander Leidinger <Alexander@Leidinger.net> To: grem <freebsdusb@bindone.de> Cc: freebsd-usb@freebsd.org Subject: Re: Support for new device, important fix and enhancement to umass.c Message-ID: <20070328080000.bhakk1rou88ww8ks@webmail.leidinger.net> In-Reply-To: <4609D885.8070505@bindone.de> References: <4609D885.8070505@bindone.de>
next in thread | previous in thread | raw e-mail | index | archive | help
Quoting grem <freebsdusb@bindone.de> (from Wed, 28 Mar 2007 04:52:53 +0200): [analysis of the problem] > Any feedback is welcome, since I'm not an expert in how USB works/is =20 > implemented in FreeBSD. Please submit this as a problem report. Quirks have to be registered =20 in GNATS before we can commit them so that we are able to reevaluate =20 them if the need arises. > @@ -1665,6 +1673,8 @@ > USETDW(sc->csw.dCSWSignature, CSWSIGNATURE= ); > } > > + if (sc->quirks & IGNORE_RESIDUE) > + USETDW(sc->csw.dCSWDataResidue, 0); > int Residue; > Residue =3D UGETDW(sc->csw.dCSWDataResidue); > if (Residue =3D=3D 0 && Wrong indent for the USETDW line. I don't know much about the USB =20 code. If the residue is not used somewhere else, wouldn't it be better =20 to do "if quirk set the Residue variable to 0 else get it from the =20 device" instead of setting it? Bye, Alexander. --=20 BOFH excuse #71: The file system is full of it http://www.Leidinger.net Alexander @ Leidinger.net: PGP ID =3D B0063FE7 http://www.FreeBSD.org netchild @ FreeBSD.org : PGP ID =3D 72077137
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20070328080000.bhakk1rou88ww8ks>