[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/d20d621df159
branches:  trunk
changeset: 368988:d20d621df159
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 f744bfa290a8 -r d20d621df159 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
@@ -1108,6 +1108,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++) {
@@ -1118,7 +1119,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;
        }
 
@@ -1147,7 +1147,6 @@
        } else {
                urtwn_write_region(sc, R92C_HMEBOX(fwcur), cp, len);
        }
-       mutex_exit(&sc->sc_fwcmd_mtx);
 
        return 0;
 }