On Sun, Oct 30, 2011 at 09:15:40PM +0100, Hans Petter Selasky wrote: > On Saturday 29 October 2011 20:38:07 Pawel Jakub Dawidek wrote: > > > + err = usbd_do_request(udev, NULL, &req, &buf); > > > + if (err) > > > + buf = 0; > > > + > > > + return (buf); > > > +} > > > > You could drop 'err' variable and just check function return directly. > > There is only one extra empty line at the begining of the function. > > No, because there is &buf.
How about:
if (usbd_do_request(udev, NULL, &req, &buf) != 0)
buf = 0;
return (buf);
> I'll add your device to the device quirk list.
Ok. How come did it work in the past? I don't think that my device was
removed from the quirk list by accident? May other, similar devices not
work as well with the current code?
--
Pawel Jakub Dawidek http://www.wheelsystems.com
FreeBSD committer http://www.FreeBSD.org
Am I Evil? Yes, I Am! http://yomoli.com
pgpkBaIV9mvb1.pgp
Description: PGP signature
