On Tuesday 01 November 2011 13:15:33 Pawel Jakub Dawidek wrote:
> 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?

The old code did not try the synchronize command, so as long as you use 
msdosfs nothing will show up broken. If you format your memory stick with ZFS, 
then it will throw out warnings.

--HPS
_______________________________________________
freebsd-usb@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-usb
To unsubscribe, send any mail to "freebsd-usb-unsubscr...@freebsd.org"

Reply via email to