[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/7928e6e865e0
branches:  trunk
changeset: 989164:7928e6e865e0
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 |  7 +++----
 1 files changed, 3 insertions(+), 4 deletions(-)

diffs (42 lines):

diff -r 51df260fd19a -r 7928e6e865e0 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
@@ -1,4 +1,4 @@
-/*     $NetBSD: if_urtwn.c,v 1.98 2021/09/17 12:58:31 nat Exp $        */
+/*     $NetBSD: if_urtwn.c,v 1.97 2021/09/17 12:55:10 nat Exp $        */
 /*     $OpenBSD: if_urtwn.c,v 1.42 2015/02/10 23:25:46 mpi Exp $       */
 
 /*-
@@ -25,7 +25,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if_urtwn.c,v 1.98 2021/09/17 12:58:31 nat Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if_urtwn.c,v 1.97 2021/09/17 12:55:10 nat Exp $");
 
 #ifdef _KERNEL_OPT
 #include "opt_inet.h"
@@ -1106,6 +1106,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++) {
@@ -1116,7 +1117,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;
        }
 
@@ -1145,7 +1145,6 @@
        } else {
                urtwn_write_region(sc, R92C_HMEBOX(fwcur), cp, len);
        }
-       mutex_exit(&sc->sc_fwcmd_mtx);
 
        return 0;
 }