Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 30 Apr 2021 20:16:09 GMT
From:      Vladimir Kondratyev <wulf@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: ad711fa497d5 - stable/13 - ng_ubt: Do not clear stall before receiving of HCI command response.
Message-ID:  <202104302016.13UKG9lo004450@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch stable/13 has been updated by wulf:

URL: https://cgit.FreeBSD.org/src/commit/?id=ad711fa497d564f7bf9fd1a2e9e01efcb4de102f

commit ad711fa497d564f7bf9fd1a2e9e01efcb4de102f
Author:     Vladimir Kondratyev <wulf@FreeBSD.org>
AuthorDate: 2021-04-15 14:25:00 +0000
Commit:     Vladimir Kondratyev <wulf@FreeBSD.org>
CommitDate: 2021-04-30 20:12:41 +0000

    ng_ubt: Do not clear stall before receiving of HCI command response.
    
    Unconditional execution of "clear feature" request at SETUP stage was
    workaround for probe failures on ng_ubt.ko re-kldloading which is
    unnecessary now.
    
    Reviewed by:    hselasky
    Differential revision:  https://reviews.freebsd.org/D29775
---
 sys/netgraph/bluetooth/drivers/ubt/ng_ubt.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/sys/netgraph/bluetooth/drivers/ubt/ng_ubt.c b/sys/netgraph/bluetooth/drivers/ubt/ng_ubt.c
index 30a012702c8f..8b93687c6688 100644
--- a/sys/netgraph/bluetooth/drivers/ubt/ng_ubt.c
+++ b/sys/netgraph/bluetooth/drivers/ubt/ng_ubt.c
@@ -825,8 +825,6 @@ ubt_probe_intr_callback(struct usb_xfer *xfer, usb_error_t error)
 
         case USB_ST_SETUP:
 submit_next:
-		/* Try clear stall first */
-		usbd_xfer_set_stall(xfer);
 		usbd_xfer_set_frame_len(xfer, 0, usbd_xfer_max_len(xfer));
 		usbd_transfer_submit(xfer);
 		break;
@@ -835,6 +833,8 @@ submit_next:
 		if (error != USB_ERR_CANCELLED) {
 			printf("ng_ubt: interrupt transfer failed: %s\n",
 				usbd_errstr(error));
+			/* Try clear stall first */
+			usbd_xfer_set_stall(xfer);
 			goto submit_next;
 		}
 		break;



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