[src/trunk]: src/sys/dev/usb Use more-semantically-correct types in xhci allo...

[jakllsch <jakllsch%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/968e4651f6ed
branches:  trunk
changeset: 933168:968e4651f6ed
user:      jakllsch <jakllsch%NetBSD.org@localhost>
date:      Wed May 20 17:32:27 2020 +0000

description:
Use more-semantically-correct types in xhci allocx/freex functions.

diffstat:

 sys/dev/usb/xhci.c |  19 ++++++++++---------
 1 files changed, 10 insertions(+), 9 deletions(-)

diffs (60 lines):

diff -r dfd9b4ee3f3a -r 968e4651f6ed sys/dev/usb/xhci.c
--- a/sys/dev/usb/xhci.c        Wed May 20 17:06:15 2020 +0000
+++ b/sys/dev/usb/xhci.c        Wed May 20 17:32:27 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: xhci.c,v 1.124 2020/04/05 20:59:38 skrll Exp $ */
+/*     $NetBSD: xhci.c,v 1.125 2020/05/20 17:32:27 jakllsch Exp $      */
 
 /*
  * Copyright (c) 2013 Jonathan A. Kollasch
@@ -34,7 +34,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: xhci.c,v 1.124 2020/04/05 20:59:38 skrll Exp $");
+__KERNEL_RCSID(0, "$NetBSD: xhci.c,v 1.125 2020/05/20 17:32:27 jakllsch Exp $");
 
 #ifdef _KERNEL_OPT
 #include "opt_usb.h"
@@ -2239,25 +2239,26 @@
 xhci_allocx(struct usbd_bus *bus, unsigned int nframes)
 {
        struct xhci_softc * const sc = XHCI_BUS2SC(bus);
-       struct usbd_xfer *xfer;
+       struct xhci_xfer *xx;
 
        XHCIHIST_FUNC(); XHCIHIST_CALLED();
 
-       xfer = pool_cache_get(sc->sc_xferpool, PR_WAITOK);
-       if (xfer != NULL) {
-               memset(xfer, 0, sizeof(struct xhci_xfer));
+       xx = pool_cache_get(sc->sc_xferpool, PR_WAITOK);
+       if (xx != NULL) {
+               memset(xx, 0, sizeof(*xx));
 #ifdef DIAGNOSTIC
-               xfer->ux_state = XFER_BUSY;
+               xx->xx_xfer.ux_state = XFER_BUSY;
 #endif
        }
 
-       return xfer;
+       return &xx->xx_xfer;
 }
 
 static void
 xhci_freex(struct usbd_bus *bus, struct usbd_xfer *xfer)
 {
        struct xhci_softc * const sc = XHCI_BUS2SC(bus);
+       struct xhci_xfer * const xx = XHCI_XFER2XXFER(xfer);
 
        XHCIHIST_FUNC(); XHCIHIST_CALLED();
 
@@ -2269,7 +2270,7 @@
        }
        xfer->ux_state = XFER_FREE;
 #endif
-       pool_cache_put(sc->sc_xferpool, xfer);
+       pool_cache_put(sc->sc_xferpool, xx);
 }
 
 static bool