Author: hselasky
Date: Thu Aug 20 17:45:54 2020
New Revision: 364433
URL: https://svnweb.freebsd.org/changeset/base/364433
Log:
Fix regression after r364379.
The AMD's Ryzen 3 3200g XHCI controllers apparently need the evaluate
control endpoint context command, but we don't need to issue this
command when the bMaxPacketSize is received after the read of the USB
device descriptor, because this part should be handled automatically.
PR: 248784
Tested by: emaste, hselasky
MFC after: 1 week
Sponsored by: Mellanox Technologies
Modified:
head/sys/dev/usb/controller/xhci.c
Modified: head/sys/dev/usb/controller/xhci.c
==============================================================================
--- head/sys/dev/usb/controller/xhci.c Thu Aug 20 17:35:47 2020
(r364432)
+++ head/sys/dev/usb/controller/xhci.c Thu Aug 20 17:45:54 2020
(r364433)
@@ -3904,10 +3904,8 @@ xhci_configure_reset_endpoint(struct usb_xfer *xfer)
if (!(sc->sc_hw.devs[index].ep_configured & mask)) {
sc->sc_hw.devs[index].ep_configured |= mask;
err = xhci_cmd_configure_ep(sc, buf_inp.physaddr, 0, index);
- } else if (epno != 1) {
- err = xhci_cmd_evaluate_ctx(sc, buf_inp.physaddr, index);
} else {
- err = 0;
+ err = xhci_cmd_evaluate_ctx(sc, buf_inp.physaddr, index);
}
if (err != 0) {
_______________________________________________
[email protected] mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "[email protected]"
