Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 9 Jun 2008 10:44:17 GMT
From:      Weongyo Jeong <weongyo@FreeBSD.org>
To:        Perforce Change Reviews <perforce@FreeBSD.org>
Subject:   PERFORCE change 143196 for review
Message-ID:  <200806091044.m59AiHao024391@repoman.freebsd.org>

next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=143196

Change 143196 by weongyo@weongyo_ws on 2008/06/09 10:43:52

	fix a bug that it missed to set the status value of URB that it causes
	problems failing to attach at some NDIS drivers.

Affected files ...

.. //depot/projects/ndisusb/sys/compat/ndis/subr_usbd.c#3 edit

Differences ...

==== //depot/projects/ndisusb/sys/compat/ndis/subr_usbd.c#3 (text+ko) ====

@@ -303,6 +303,11 @@
 	union usbd_urb *urb;
 
 	urb = usbd_geturb(ip);
+	/*
+	 * In a case of URB_FUNCTION_BULK_OR_INTERRUPT_TRANSFER,
+	 * USBD_URB_STATUS(urb) would be set at callback functions like
+	 * usbd_intr() or usbd_xfereof().
+	 */
 	switch (urb->uu_hdr.uuh_func) {
 	case URB_FUNCTION_BULK_OR_INTERRUPT_TRANSFER:
 		status = usbd_func_bulkintr(ip);
@@ -316,6 +321,7 @@
 	case URB_FUNCTION_CLASS_ENDPOINT:
 	case URB_FUNCTION_CLASS_OTHER:
 		status = usbd_func_vendorclass(ip);
+		USBD_URB_STATUS(urb) = status;
 		break;
 	case URB_FUNCTION_SELECT_CONFIGURATION:
 		status = usbd_func_selconf(ip);



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