[src-draft/trunk]: src/sys/dev/usb Backed out changeset 4652edef3061 (may be ...

[Martin Husemann <martin%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src-all/rev/9676b3521bfa
branches:  trunk
changeset: 1025586:9676b3521bfa
user:      Martin Husemann <martin%NetBSD.org@localhost>
date:      Mon Sep 20 17:01:52 2021 +0200

description:
Backed out changeset 4652edef3061 (may be redone differently later)

diffstat:

 sys/dev/usb/if_urtwn.c |  3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)

diffs (27 lines):

diff -r 0de692d4de86 -r 9676b3521bfa sys/dev/usb/if_urtwn.c
--- a/sys/dev/usb/if_urtwn.c    Mon Sep 20 17:01:29 2021 +0200
+++ b/sys/dev/usb/if_urtwn.c    Mon Sep 20 17:01:52 2021 +0200
@@ -1107,6 +1107,7 @@
        mutex_enter(&sc->sc_fwcmd_mtx);
        fwcur = sc->fwcur;
        sc->fwcur = (sc->fwcur + 1) % R92C_H2C_NBOX;
+       mutex_exit(&sc->sc_fwcmd_mtx);
 
        /* Wait for current FW box to be empty. */
        for (ntries = 0; ntries < 100; ntries++) {
@@ -1117,7 +1118,6 @@
        if (ntries == 100) {
                aprint_error_dev(sc->sc_dev,
                    "could not send firmware command %d\n", id);
-               mutex_exit(&sc->sc_fwcmd_mtx);
                return ETIMEDOUT;
        }
 
@@ -1146,7 +1146,6 @@
        } else {
                urtwn_write_region(sc, R92C_HMEBOX(fwcur), cp, len);
        }
-       mutex_exit(&sc->sc_fwcmd_mtx);
 
        return 0;
 }