 @@ -1,4717 +1,4694 @@
-/*	$NetBSD: ehci.c,v 1.234.2.34 2015/03/01 08:26:55 skrll Exp $ */
+/*	$NetBSD: ehci.c,v 1.234.2.35 2015/03/02 21:52:02 skrll Exp $ */
 /*
  * Copyright (c) 2004-2012 The NetBSD Foundation, Inc.
  * All rights reserved.
+ *
  * This code is derived from software contributed to The NetBSD Foundation
  * by Lennart Augustsson (lennart@augustsson.net), Charles M. Hannum,
  * Jeremy Morse (jeremy.morse@gmail.com), Jared D. McNeill
  * (jmcneill@invisible.ca) and Matthew R. Green (mrg@eterna.com.au).
+ *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
+ *
  * THIS SOFTWARE IS PROVIDED BY THE NETBSD FOUNDATION, INC. AND CONTRIBUTORS
  * ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED
  * TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
  * PURPOSE ARE DISCLAIMED.  IN NO EVENT SHALL THE FOUNDATION OR CONTRIBUTORS
  * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
  * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
  * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
  * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
  * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
  * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
  * POSSIBILITY OF SUCH DAMAGE.
  */
 /*
  * USB Enhanced Host Controller Driver, a.k.a. USB 2.0 controller.
+ *
  * The EHCI 1.0 spec can be found at
  * http://www.intel.com/technology/usb/spec.htm
  * and the USB 2.0 spec at
  * http://www.usb.org/developers/docs/
+ *
  */
 /*
  * TODO:
  * 1) hold off explorations by companion controllers until ehci has started.
+ *
  * 2) The hub driver needs to handle and schedule the transaction translator,
  *    to assign place in frame where different devices get to go. See chapter
  *    on hubs in USB 2.0 for details.
+ *
  * 3) Command failures are not recovered correctly.
  */
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: ehci.c,v 1.234.2.34 2015/03/01 08:26:55 skrll Exp $");
+__KERNEL_RCSID(0, "$NetBSD: ehci.c,v 1.234.2.35 2015/03/02 21:52:02 skrll Exp $");
 #include "ohci.h"
 #include "uhci.h"
 #include "opt_usb.h"
 #include <sys/param.h>
 #include <sys/bus.h>
 #include <sys/cpu.h>
 #include <sys/device.h>
 #include <sys/kernel.h>
 #include <sys/kmem.h>
 #include <sys/mutex.h>
 #include <sys/proc.h>
 #include <sys/queue.h>
 #include <sys/select.h>
 #include <sys/sysctl.h>
 #include <sys/systm.h>
 #include <machine/endian.h>
 #include <dev/usb/usb.h>
 #include <dev/usb/usbdi.h>
 #include <dev/usb/usbdivar.h>
 #include <dev/usb/usbhist.h>
 #include <dev/usb/usb_mem.h>
 #include <dev/usb/usb_quirks.h>
 #include <dev/usb/ehcireg.h>
 #include <dev/usb/ehcivar.h>
 #include <dev/usb/usbroothub.h>
 #ifdef USB_DEBUG
 #ifndef EHCI_DEBUG
 #define ehcidebug 0
 #else
 static int ehcidebug = 0;
 SYSCTL_SETUP(sysctl_hw_ehci_setup, "sysctl hw.ehci setup")
+{
 	int err;
 	const struct sysctlnode *rnode;
 	const struct sysctlnode *cnode;
 	err = sysctl_createv(clog, 0, NULL, &rnode,
 	    CTLFLAG_PERMANENT, CTLTYPE_NODE, "ehci",
 	    SYSCTL_DESCR("ehci global controls"),
 	    NULL, 0, NULL, 0, CTL_HW, CTL_CREATE, CTL_EOL);
 	if (err)
 		goto fail;
 	/* control debugging printfs */
 	err = sysctl_createv(clog, 0, &rnode, &cnode,
 	    CTLFLAG_PERMANENT|CTLFLAG_READWRITE, CTLTYPE_INT,
 	    "debug", SYSCTL_DESCR("Enable debugging output"),
 	    NULL, 0, &ehcidebug, sizeof(ehcidebug), CTL_CREATE, CTL_EOL);
 	if (err)
 		goto fail;
 	return;
 fail:
 	aprint_error("%s: sysctl_createv failed (err = %d)\n", __func__, err);
+}
 #endif /* EHCI_DEBUG */
 #endif /* USB_DEBUG */
 struct ehci_pipe {
 	struct usbd_pipe pipe;
 	int nexttoggle;
 	ehci_soft_qh_t *sqh;
 	union {
 		ehci_soft_qtd_t *qtd;
 		/* ehci_soft_itd_t *itd; */
 		/* ehci_soft_sitd_t *sitd; */
 	} tail;
 	union {
 		/* Control pipe */
 		struct {
 			usb_dma_t reqdma;
 		} ctl;
 		/* Interrupt pipe */
 		struct {
 			u_int length;
 		} intr;
 		/* Bulk pipe */
 		struct {
 			u_int length;
 		} bulk;
 		/* Iso pipe */
 		struct {
 			u_int next_frame;
 			u_int cur_xfers;
 		} isoc;
 	} u;
 };
 Static usbd_status	ehci_open(usbd_pipe_handle);
 Static void		ehci_poll(struct usbd_bus *);
 Static void		ehci_softintr(void *);
 Static int		ehci_intr1(ehci_softc_t *);
 Static void		ehci_waitintr(ehci_softc_t *, usbd_xfer_handle);
 Static void		ehci_check_intr(ehci_softc_t *, struct ehci_xfer *);
 Static void		ehci_check_qh_intr(ehci_softc_t *, struct ehci_xfer *);
 Static void		ehci_check_itd_intr(ehci_softc_t *, struct ehci_xfer *);
 Static void		ehci_check_sitd_intr(ehci_softc_t *, struct ehci_xfer *);
 Static void		ehci_idone(struct ehci_xfer *);
 Static void		ehci_timeout(void *);
 Static void		ehci_timeout_task(void *);
 Static void		ehci_intrlist_timeout(void *);
 Static void		ehci_doorbell(void *);
 Static void		ehci_pcd(void *);
 Static usbd_xfer_handle	ehci_allocx(struct usbd_bus *);
 Static void		ehci_freex(struct usbd_bus *, usbd_xfer_handle);
 Static void		ehci_get_lock(struct usbd_bus *, kmutex_t **);
 Static int		ehci_roothub_ctrl(struct usbd_bus *,
     usb_device_request_t *, void *, int);
 Static usbd_status	ehci_root_intr_transfer(usbd_xfer_handle);
 Static usbd_status	ehci_root_intr_start(usbd_xfer_handle);
 Static void		ehci_root_intr_abort(usbd_xfer_handle);
 Static void		ehci_root_intr_close(usbd_pipe_handle);
 Static void		ehci_root_intr_done(usbd_xfer_handle);
 Static usbd_status	ehci_device_ctrl_transfer(usbd_xfer_handle);
 Static usbd_status	ehci_device_ctrl_start(usbd_xfer_handle);
 Static void		ehci_device_ctrl_abort(usbd_xfer_handle);
 Static void		ehci_device_ctrl_close(usbd_pipe_handle);
 Static void		ehci_device_ctrl_done(usbd_xfer_handle);
 Static usbd_status	ehci_device_bulk_transfer(usbd_xfer_handle);
 Static usbd_status	ehci_device_bulk_start(usbd_xfer_handle);
 Static void		ehci_device_bulk_abort(usbd_xfer_handle);
 Static void		ehci_device_bulk_close(usbd_pipe_handle);
 Static void		ehci_device_bulk_done(usbd_xfer_handle);
 Static usbd_status	ehci_device_intr_transfer(usbd_xfer_handle);
 Static usbd_status	ehci_device_intr_start(usbd_xfer_handle);
 Static void		ehci_device_intr_abort(usbd_xfer_handle);
 Static void		ehci_device_intr_close(usbd_pipe_handle);
 Static void		ehci_device_intr_done(usbd_xfer_handle);
 Static usbd_status	ehci_device_isoc_transfer(usbd_xfer_handle);
 Static usbd_status	ehci_device_isoc_start(usbd_xfer_handle);
 Static void		ehci_device_isoc_abort(usbd_xfer_handle);
 Static void		ehci_device_isoc_close(usbd_pipe_handle);
 Static void		ehci_device_isoc_done(usbd_xfer_handle);
 Static usbd_status	ehci_device_fs_isoc_transfer(usbd_xfer_handle);
 Static usbd_status	ehci_device_fs_isoc_start(usbd_xfer_handle);
 Static void		ehci_device_fs_isoc_abort(usbd_xfer_handle);
 Static void		ehci_device_fs_isoc_close(usbd_pipe_handle);
 Static void		ehci_device_fs_isoc_done(usbd_xfer_handle);
 Static void		ehci_device_clear_toggle(usbd_pipe_handle);
 Static void		ehci_noop(usbd_pipe_handle);
 Static void		ehci_disown(ehci_softc_t *, int, int);
 Static ehci_soft_qh_t  *ehci_alloc_sqh(ehci_softc_t *);
 Static void		ehci_free_sqh(ehci_softc_t *, ehci_soft_qh_t *);
 Static ehci_soft_qtd_t  *ehci_alloc_sqtd(ehci_softc_t *);
 Static void		ehci_free_sqtd(ehci_softc_t *, ehci_soft_qtd_t *);
 Static usbd_status	ehci_alloc_sqtd_chain(struct ehci_pipe *,
 			    ehci_softc_t *, int, int, usbd_xfer_handle,
 			    ehci_soft_qtd_t **, ehci_soft_qtd_t **);
 Static void		ehci_free_sqtd_chain(ehci_softc_t *, ehci_soft_qtd_t *,
 					    ehci_soft_qtd_t *);
 Static ehci_soft_itd_t	*ehci_alloc_itd(ehci_softc_t *);
 Static ehci_soft_sitd_t *ehci_alloc_sitd(ehci_softc_t *);
 Static void		ehci_free_itd(ehci_softc_t *, ehci_soft_itd_t *);
 Static void		ehci_free_sitd(ehci_softc_t *, ehci_soft_sitd_t *);
 Static void 		ehci_rem_free_itd_chain(ehci_softc_t *,
 						struct ehci_xfer *);
 Static void		ehci_rem_free_sitd_chain(ehci_softc_t *,
 						 struct ehci_xfer *);
 Static void 		ehci_abort_isoc_xfer(usbd_xfer_handle,
 						usbd_status);
 Static usbd_status	ehci_device_request(usbd_xfer_handle);
 Static usbd_status	ehci_device_setintr(ehci_softc_t *, ehci_soft_qh_t *,
 			    int);
 Static void		ehci_add_qh(ehci_softc_t *, ehci_soft_qh_t *,
 				    ehci_soft_qh_t *);
 Static void		ehci_rem_qh(ehci_softc_t *, ehci_soft_qh_t *,
 				    ehci_soft_qh_t *);
 Static void		ehci_set_qh_qtd(ehci_soft_qh_t *, ehci_soft_qtd_t *);
 Static void		ehci_sync_hc(ehci_softc_t *);
 Static void		ehci_close_pipe(usbd_pipe_handle, ehci_soft_qh_t *);
 Static void		ehci_abort_xfer(usbd_xfer_handle, usbd_status);
 #ifdef EHCI_DEBUG
 Static ehci_softc_t 	*theehci;
 void			ehci_dump(void);
 #endif
 #ifdef EHCI_DEBUG
 Static void		ehci_dump_regs(ehci_softc_t *);
 Static void		ehci_dump_sqtds(ehci_soft_qtd_t *);
 Static void		ehci_dump_sqtd(ehci_soft_qtd_t *);
 Static void		ehci_dump_qtd(ehci_qtd_t *);
 Static void		ehci_dump_sqh(ehci_soft_qh_t *);
 Static void		ehci_dump_sitd(struct ehci_soft_itd *);
 Static void		ehci_dump_itd(struct ehci_soft_itd *);
 Static void		ehci_dump_exfer(struct ehci_xfer *);
 #endif
 #define EHCI_NULL htole32(EHCI_LINK_TERMINATE)
 #define ehci_add_intr_list(sc, ex) \
 	TAILQ_INSERT_TAIL(&(sc)->sc_intrhead, (ex), ex_next);
 #define ehci_del_intr_list(sc, ex) \
 	do { \
 		TAILQ_REMOVE(&sc->sc_intrhead, (ex), ex_next); \
 		(ex)->ex_next.tqe_prev = NULL; \
 	} while (0)
 #define ehci_active_intr_list(ex) ((ex)->ex_next.tqe_prev != NULL)
 Static const struct usbd_bus_methods ehci_bus_methods = {
 	.ubm_open =	ehci_open,
 	.ubm_softint =	ehci_softintr,
 	.ubm_dopoll =	ehci_poll,
 	.ubm_allocx =	ehci_allocx,
 	.ubm_freex =	ehci_freex,
 	.ubm_getlock =	ehci_get_lock,
 	.ubm_rhctrl =	ehci_roothub_ctrl,
 };
 Static const struct usbd_pipe_methods ehci_root_intr_methods = {
 	.upm_transfer =	ehci_root_intr_transfer,
 	.upm_start =	ehci_root_intr_start,
 	.upm_abort =	ehci_root_intr_abort,
 	.upm_close =	ehci_root_intr_close,
 	.upm_cleartoggle =	ehci_noop,
 	.upm_done =	ehci_root_intr_done,
 };
 Static const struct usbd_pipe_methods ehci_device_ctrl_methods = {
 	.upm_transfer =	ehci_device_ctrl_transfer,
 	.upm_start =	ehci_device_ctrl_start,
 	.upm_abort =	ehci_device_ctrl_abort,
 	.upm_close =	ehci_device_ctrl_close,
 	.upm_cleartoggle =	ehci_noop,
 	.upm_done =	ehci_device_ctrl_done,
 };
 Static const struct usbd_pipe_methods ehci_device_intr_methods = {
 	.upm_transfer =	ehci_device_intr_transfer,
 	.upm_start =	ehci_device_intr_start,
 	.upm_abort =	ehci_device_intr_abort,
 	.upm_close =	ehci_device_intr_close,
 	.upm_cleartoggle =	ehci_device_clear_toggle,
 	.upm_done =	ehci_device_intr_done,
 };
 Static const struct usbd_pipe_methods ehci_device_bulk_methods = {
 	.upm_transfer =	ehci_device_bulk_transfer,
 	.upm_start =	ehci_device_bulk_start,
 	.upm_abort =	ehci_device_bulk_abort,
 	.upm_close =	ehci_device_bulk_close,
 	.upm_cleartoggle =	ehci_device_clear_toggle,
 	.upm_done =	ehci_device_bulk_done,
 };
 Static const struct usbd_pipe_methods ehci_device_isoc_methods = {
 	.upm_transfer =	ehci_device_isoc_transfer,
 	.upm_start =	ehci_device_isoc_start,
 	.upm_abort =	ehci_device_isoc_abort,
 	.upm_close =	ehci_device_isoc_close,
 	.upm_cleartoggle =	ehci_noop,
 	.upm_done =	ehci_device_isoc_done,
 };
 Static const struct usbd_pipe_methods ehci_device_fs_isoc_methods = {
 	.upm_transfer =	ehci_device_fs_isoc_transfer,
 	.upm_start =	ehci_device_fs_isoc_start,
 	.upm_abort =	ehci_device_fs_isoc_abort,
 	.upm_close =	ehci_device_fs_isoc_close,
 	.upm_cleartoggle = ehci_noop,
 	.upm_done =	ehci_device_fs_isoc_done,
 };
 static const uint8_t revbits[EHCI_MAX_POLLRATE] = {
 x00,0x40,0x20,0x60,0x10,0x50,0x30,0x70,0x08,0x48,0x28,0x68,0x18,0x58,0x38,0x78,
 x04,0x44,0x24,0x64,0x14,0x54,0x34,0x74,0x0c,0x4c,0x2c,0x6c,0x1c,0x5c,0x3c,0x7c,
 x02,0x42,0x22,0x62,0x12,0x52,0x32,0x72,0x0a,0x4a,0x2a,0x6a,0x1a,0x5a,0x3a,0x7a,
 x06,0x46,0x26,0x66,0x16,0x56,0x36,0x76,0x0e,0x4e,0x2e,0x6e,0x1e,0x5e,0x3e,0x7e,
 x01,0x41,0x21,0x61,0x11,0x51,0x31,0x71,0x09,0x49,0x29,0x69,0x19,0x59,0x39,0x79,
 x05,0x45,0x25,0x65,0x15,0x55,0x35,0x75,0x0d,0x4d,0x2d,0x6d,0x1d,0x5d,0x3d,0x7d,
 x03,0x43,0x23,0x63,0x13,0x53,0x33,0x73,0x0b,0x4b,0x2b,0x6b,0x1b,0x5b,0x3b,0x7b,
 x07,0x47,0x27,0x67,0x17,0x57,0x37,0x77,0x0f,0x4f,0x2f,0x6f,0x1f,0x5f,0x3f,0x7f,
 };
 int
 ehci_init(ehci_softc_t *sc)
+{
 	uint32_t vers, sparams, cparams, hcr;
 	u_int i;
 	usbd_status err;
 	ehci_soft_qh_t *sqh;
 	u_int ncomp;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 #ifdef EHCI_DEBUG
 	theehci = sc;
 #endif
 	mutex_init(&sc->sc_lock, MUTEX_DEFAULT, IPL_SOFTUSB);
 	mutex_init(&sc->sc_intr_lock, MUTEX_DEFAULT, IPL_SCHED);
 	cv_init(&sc->sc_softwake_cv, "ehciab");
 	cv_init(&sc->sc_doorbell, "ehcidi");
 	sc->sc_xferpool = pool_cache_init(sizeof(struct ehci_xfer), 0, 0, 0,
 	    "ehcixfer", NULL, IPL_USB, NULL, NULL, NULL);
 	sc->sc_doorbell_si = softint_establish(SOFTINT_NET | SOFTINT_MPSAFE,
 	    ehci_doorbell, sc);
 	KASSERT(sc->sc_doorbell_si != NULL);
 	sc->sc_pcd_si = softint_establish(SOFTINT_NET | SOFTINT_MPSAFE,
 	    ehci_pcd, sc);
 	KASSERT(sc->sc_pcd_si != NULL);
 	sc->sc_offs = EREAD1(sc, EHCI_CAPLENGTH);
 	vers = EREAD2(sc, EHCI_HCIVERSION);
 	aprint_verbose("%s: EHCI version %x.%x\n", device_xname(sc->sc_dev),
 	       vers >> 8, vers & 0xff);
 	sparams = EREAD4(sc, EHCI_HCSPARAMS);
 	USBHIST_LOG(ehcidebug, "sparams=%#x", sparams, 0, 0, 0);
 	sc->sc_npcomp = EHCI_HCS_N_PCC(sparams);
 	ncomp = EHCI_HCS_N_CC(sparams);
 	if (ncomp != sc->sc_ncomp) {
 		aprint_verbose("%s: wrong number of companions (%d != %d)\n",
 			       device_xname(sc->sc_dev), ncomp, sc->sc_ncomp);
 #if NOHCI == 0 || NUHCI == 0
 		aprint_error("%s: ohci or uhci probably not configured\n",
 			     device_xname(sc->sc_dev));
 #endif
 		if (ncomp < sc->sc_ncomp)
 			sc->sc_ncomp = ncomp;
+	}
 	if (sc->sc_ncomp > 0) {
 		KASSERT(!(sc->sc_flags & EHCIF_ETTF));
 		aprint_normal("%s: companion controller%s, %d port%s each:",
 		    device_xname(sc->sc_dev), sc->sc_ncomp!=1 ? "s" : "",
 		    EHCI_HCS_N_PCC(sparams),
 		    EHCI_HCS_N_PCC(sparams)!=1 ? "s" : "");
 		for (i = 0; i < sc->sc_ncomp; i++)
 			aprint_normal(" %s", device_xname(sc->sc_comps[i]));
 		aprint_normal("\n");
+	}
 	sc->sc_noport = EHCI_HCS_N_PORTS(sparams);
 	cparams = EREAD4(sc, EHCI_HCCPARAMS);
 	USBHIST_LOG(ehcidebug, "cparams=%#x", cparams, 0, 0, 0);
 	sc->sc_hasppc = EHCI_HCS_PPC(sparams);
 	if (EHCI_HCC_64BIT(cparams)) {
 		/* MUST clear segment register if 64 bit capable. */
 		EWRITE4(sc, EHCI_CTRLDSSEGMENT, 0);
+	}
 	sc->sc_bus.ub_revision = USBREV_2_0;
 	sc->sc_bus.ub_usedma = true;
 	sc->sc_bus.ub_dmaflags = USBMALLOC_MULTISEG;
 	/* Reset the controller */
 	USBHIST_LOG(ehcidebug, "resetting", 0, 0, 0, 0);
 	EOWRITE4(sc, EHCI_USBCMD, 0);	/* Halt controller */
 	usb_delay_ms(&sc->sc_bus, 1);
 	EOWRITE4(sc, EHCI_USBCMD, EHCI_CMD_HCRESET);
 	for (i = 0; i < 100; i++) {
 		usb_delay_ms(&sc->sc_bus, 1);
 		hcr = EOREAD4(sc, EHCI_USBCMD) & EHCI_CMD_HCRESET;
 		if (!hcr)
 			break;
+	}
 	if (hcr) {
 		aprint_error("%s: reset timeout\n", device_xname(sc->sc_dev));
 		return EIO;
+	}
 	if (sc->sc_vendor_init)
 		sc->sc_vendor_init(sc);
 	/*
 	 * If we are doing embedded transaction translation function, force
 	 * the controller to host mode.
 	 */
 	if (sc->sc_flags & EHCIF_ETTF) {
 		uint32_t usbmode = EREAD4(sc, EHCI_USBMODE);
 		usbmode &= ~EHCI_USBMODE_CM;
 		usbmode |= EHCI_USBMODE_CM_HOST;
 		EWRITE4(sc, EHCI_USBMODE, usbmode);
+	}
 	/* XXX need proper intr scheduling */
 	sc->sc_rand = 96;
 	/* frame list size at default, read back what we got and use that */
 	switch (EHCI_CMD_FLS(EOREAD4(sc, EHCI_USBCMD))) {
 	case 0: sc->sc_flsize = 1024; break;
 	case 1: sc->sc_flsize = 512; break;
 	case 2: sc->sc_flsize = 256; break;
 	case 3: return EIO;
+	}
 	err = usb_allocmem(&sc->sc_bus, sc->sc_flsize * sizeof(ehci_link_t),
 	    EHCI_FLALIGN_ALIGN, &sc->sc_fldma);
 	if (err)
 		return err;
 	USBHIST_LOG(ehcidebug, "flsize=%d", sc->sc_flsize, 0, 0, 0);
 	sc->sc_flist = KERNADDR(&sc->sc_fldma, 0);
 	for (i = 0; i < sc->sc_flsize; i++) {
 		sc->sc_flist[i] = EHCI_NULL;
+	}
 	EOWRITE4(sc, EHCI_PERIODICLISTBASE, DMAADDR(&sc->sc_fldma, 0));
 	sc->sc_softitds = kmem_zalloc(sc->sc_flsize * sizeof(ehci_soft_itd_t *),
 				     KM_SLEEP);
 	if (sc->sc_softitds == NULL)
 		return ENOMEM;
 	LIST_INIT(&sc->sc_freeitds);
 	LIST_INIT(&sc->sc_freesitds);
 	TAILQ_INIT(&sc->sc_intrhead);
 	/* Set up the bus struct. */
 	sc->sc_bus.ub_methods = &ehci_bus_methods;
 	sc->sc_bus.ub_pipesize= sizeof(struct ehci_pipe);
 	sc->sc_eintrs = EHCI_NORMAL_INTRS;
 	/*
 	 * Allocate the interrupt dummy QHs. These are arranged to give poll
 	 * intervals that are powers of 2 times 1ms.
 	 */
 	for (i = 0; i < EHCI_INTRQHS; i++) {
 		sqh = ehci_alloc_sqh(sc);
 		if (sqh == NULL) {
 			err = ENOMEM;
 			goto bad1;
+		}
 		sc->sc_islots[i].sqh = sqh;
+	}
 	for (i = 0; i < EHCI_INTRQHS; i++) {
 		sqh = sc->sc_islots[i].sqh;
 		if (i == 0) {
 			/* The last (1ms) QH terminates. */
 			sqh->qh.qh_link = EHCI_NULL;
 			sqh->next = NULL;
 		} else {
 			/* Otherwise the next QH has half the poll interval */
 			sqh->next = sc->sc_islots[(i + 1) / 2 - 1].sqh;
 			sqh->qh.qh_link = htole32(sqh->next->physaddr |
 			    EHCI_LINK_QH);
+		}
 		sqh->qh.qh_endp = htole32(EHCI_QH_SET_EPS(EHCI_QH_SPEED_HIGH));
 		sqh->qh.qh_curqtd = EHCI_NULL;
 		sqh->next = NULL;
 		sqh->qh.qh_qtd.qtd_next = EHCI_NULL;
 		sqh->qh.qh_qtd.qtd_altnext = EHCI_NULL;
 		sqh->qh.qh_qtd.qtd_status = htole32(EHCI_QTD_HALTED);
 		sqh->sqtd = NULL;
 		usb_syncmem(&sqh->dma, sqh->offs, sizeof(sqh->qh),
 		    BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
+	}
 	/* Point the frame list at the last level (128ms). */
 	for (i = 0; i < sc->sc_flsize; i++) {
 		int j;
 		j = (i & ~(EHCI_MAX_POLLRATE-1)) |
 		    revbits[i & (EHCI_MAX_POLLRATE-1)];
 		sc->sc_flist[j] = htole32(EHCI_LINK_QH |
 		    sc->sc_islots[EHCI_IQHIDX(EHCI_IPOLLRATES - 1,
 		    i)].sqh->physaddr);
+	}
 	usb_syncmem(&sc->sc_fldma, 0, sc->sc_flsize * sizeof(ehci_link_t),
 	    BUS_DMASYNC_PREWRITE);
 	/* Allocate dummy QH that starts the async list. */
 	sqh = ehci_alloc_sqh(sc);
 	if (sqh == NULL) {
 		err = ENOMEM;
 		goto bad1;
+	}
 	/* Fill the QH */
 	sqh->qh.qh_endp =
 	    htole32(EHCI_QH_SET_EPS(EHCI_QH_SPEED_HIGH) | EHCI_QH_HRECL);
 	sqh->qh.qh_link =
 	    htole32(sqh->physaddr | EHCI_LINK_QH);
 	sqh->qh.qh_curqtd = EHCI_NULL;
 	sqh->next = NULL;
 	/* Fill the overlay qTD */
 	sqh->qh.qh_qtd.qtd_next = EHCI_NULL;
 	sqh->qh.qh_qtd.qtd_altnext = EHCI_NULL;
 	sqh->qh.qh_qtd.qtd_status = htole32(EHCI_QTD_HALTED);
 	sqh->sqtd = NULL;
 	usb_syncmem(&sqh->dma, sqh->offs, sizeof(sqh->qh),
 	    BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
 #ifdef EHCI_DEBUG
 	USBHIST_LOGN(ehcidebug, 5, "--- dump start ---", 0, 0, 0, 0);
 	ehci_dump_sqh(sqh);
 	USBHIST_LOGN(ehcidebug, 5, "--- dump end ---", 0, 0, 0, 0);
 #endif
 	/* Point to async list */
 	sc->sc_async_head = sqh;
 	EOWRITE4(sc, EHCI_ASYNCLISTADDR, sqh->physaddr | EHCI_LINK_QH);
 	callout_init(&sc->sc_tmo_intrlist, CALLOUT_MPSAFE);
 	/* Turn on controller */
 	EOWRITE4(sc, EHCI_USBCMD,
 		 EHCI_CMD_ITC_2 | /* 2 microframes interrupt delay */
 		 (EOREAD4(sc, EHCI_USBCMD) & EHCI_CMD_FLS_M) |
 		 EHCI_CMD_ASE |
 		 EHCI_CMD_PSE |
 		 EHCI_CMD_RS);
 	/* Take over port ownership */
 	EOWRITE4(sc, EHCI_CONFIGFLAG, EHCI_CONF_CF);
 	for (i = 0; i < 100; i++) {
 		usb_delay_ms(&sc->sc_bus, 1);
 		hcr = EOREAD4(sc, EHCI_USBSTS) & EHCI_STS_HCH;
 		if (!hcr)
 			break;
+	}
 	if (hcr) {
 		aprint_error("%s: run timeout\n", device_xname(sc->sc_dev));
 		return EIO;
+	}
 	/* Enable interrupts */
 	USBHIST_LOG(ehcidebug, "enabling interupts", 0, 0, 0, 0);
 	EOWRITE4(sc, EHCI_USBINTR, sc->sc_eintrs);
 	return 0;
 #if 0
  bad2:
 	ehci_free_sqh(sc, sc->sc_async_head);
 #endif
  bad1:
 	usb_freemem(&sc->sc_bus, &sc->sc_fldma);
 	return err;
+}
 int
 ehci_intr(void *v)
+{
 	ehci_softc_t *sc = v;
 	int ret = 0;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	if (sc == NULL)
 		return 0;
 	mutex_spin_enter(&sc->sc_intr_lock);
 	if (sc->sc_dying || !device_has_power(sc->sc_dev))
 		goto done;
 	/* If we get an interrupt while polling, then just ignore it. */
 	if (sc->sc_bus.ub_usepolling) {
 		uint32_t intrs = EHCI_STS_INTRS(EOREAD4(sc, EHCI_USBSTS));
 		if (intrs)
 			EOWRITE4(sc, EHCI_USBSTS, intrs); /* Acknowledge */
 		USBHIST_LOGN(ehcidebug, 16,
 		    "ignored interrupt while polling", 0, 0, 0, 0);
 		goto done;
+	}
 	ret = ehci_intr1(sc);
 done:
 	mutex_spin_exit(&sc->sc_intr_lock);
 	return ret;
+}
 Static int
 ehci_intr1(ehci_softc_t *sc)
+{
 	uint32_t intrs, eintrs;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	/* In case the interrupt occurs before initialization has completed. */
 	if (sc == NULL) {
 #ifdef DIAGNOSTIC
 		printf("ehci_intr1: sc == NULL\n");
 #endif
 		return 0;
+	}
 	KASSERT(mutex_owned(&sc->sc_intr_lock));
 	intrs = EHCI_STS_INTRS(EOREAD4(sc, EHCI_USBSTS));
 	if (!intrs)
 		return 0;
 	eintrs = intrs & sc->sc_eintrs;
 	USBHIST_LOG(ehcidebug, "sc=%p intrs=%#x(%#x) eintrs=%#x",
 	    sc, intrs, EOREAD4(sc, EHCI_USBSTS), eintrs);
 	if (!eintrs)
 		return 0;
 	EOWRITE4(sc, EHCI_USBSTS, intrs); /* Acknowledge */
 	if (eintrs & EHCI_STS_IAA) {
 		USBHIST_LOG(ehcidebug, "door bell", 0, 0, 0, 0);
 		kpreempt_disable();
 		KASSERT(sc->sc_doorbell_si != NULL);
 		softint_schedule(sc->sc_doorbell_si);
 		kpreempt_enable();
 		eintrs &= ~EHCI_STS_IAA;
+	}
 	if (eintrs & (EHCI_STS_INT | EHCI_STS_ERRINT)) {
 		USBHIST_LOG(ehcidebug, "INT=%d  ERRINT=%d",
 		    eintrs & EHCI_STS_INT ? 1 : 0,
 		    eintrs & EHCI_STS_ERRINT ? 1 : 0, 0, 0);
 		usb_schedsoftintr(&sc->sc_bus);
 		eintrs &= ~(EHCI_STS_INT | EHCI_STS_ERRINT);
+	}
 	if (eintrs & EHCI_STS_HSE) {
 		printf("%s: unrecoverable error, controller halted\n",
 		       device_xname(sc->sc_dev));
 		/* XXX what else */
+	}
 	if (eintrs & EHCI_STS_PCD) {
 		kpreempt_disable();
 		KASSERT(sc->sc_pcd_si != NULL);
 		softint_schedule(sc->sc_pcd_si);
 		kpreempt_enable();
 		eintrs &= ~EHCI_STS_PCD;
+	}
 	if (eintrs != 0) {
 		/* Block unprocessed interrupts. */
 		sc->sc_eintrs &= ~eintrs;
 		EOWRITE4(sc, EHCI_USBINTR, sc->sc_eintrs);
 		printf("%s: blocking intrs 0x%x\n",
 		       device_xname(sc->sc_dev), eintrs);
+	}
 	return 1;
+}
 Static void
 ehci_doorbell(void *addr)
+{
 	ehci_softc_t *sc = addr;
 	mutex_enter(&sc->sc_lock);
 	cv_broadcast(&sc->sc_doorbell);
 	mutex_exit(&sc->sc_lock);
+}
 Static void
 ehci_pcd(void *addr)
+{
 	ehci_softc_t *sc = addr;
 	usbd_xfer_handle xfer;
 	u_char *p;
 	int i, m;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	mutex_enter(&sc->sc_lock);
 	xfer = sc->sc_intrxfer;
 	if (xfer == NULL) {
 		/* Just ignore the change. */
 		goto done;
+	}
 	p = xfer->ux_buf;
 	m = min(sc->sc_noport, xfer->ux_length * 8 - 1);
 	memset(p, 0, xfer->ux_length);
 	for (i = 1; i <= m; i++) {
 		/* Pick out CHANGE bits from the status reg. */
 		if (EOREAD4(sc, EHCI_PORTSC(i)) & EHCI_PS_CLEAR)
 			p[i/8] |= 1 << (i%8);
 		if (i % 8 == 7)
 			USBHIST_LOG(ehcidebug, "change(%d)=0x%02x", i / 8,
 			    p[i/8], 0, 0);
+	}
 	xfer->ux_actlen = xfer->ux_length;
 	xfer->ux_status = USBD_NORMAL_COMPLETION;
 	usb_transfer_complete(xfer);
 done:
 	mutex_exit(&sc->sc_lock);
+}
 Static void
 ehci_softintr(void *v)
+{
 	struct usbd_bus *bus = v;
 	ehci_softc_t *sc = bus->ub_hcpriv;
 	struct ehci_xfer *ex, *nextex;
 	KASSERT(sc->sc_bus.ub_usepolling || mutex_owned(&sc->sc_lock));
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	/*
 	 * The only explanation I can think of for why EHCI is as brain dead
 	 * as UHCI interrupt-wise is that Intel was involved in both.
 	 * An interrupt just tells us that something is done, we have no
 	 * clue what, so we need to scan through all active transfers. :-(
 	 */
 	for (ex = TAILQ_FIRST(&sc->sc_intrhead); ex; ex = nextex) {
 		nextex = TAILQ_NEXT(ex, ex_next);
 		ehci_check_intr(sc, ex);
+	}
 	/* Schedule a callout to catch any dropped transactions. */
 	if ((sc->sc_flags & EHCIF_DROPPED_INTR_WORKAROUND) &&
 	    !TAILQ_EMPTY(&sc->sc_intrhead))
 		callout_reset(&sc->sc_tmo_intrlist,
 		    hz, ehci_intrlist_timeout, sc);
 	if (sc->sc_softwake) {
 		sc->sc_softwake = 0;
 		cv_broadcast(&sc->sc_softwake_cv);
+	}
+}
 /* Check for an interrupt. */
 Static void
 ehci_check_intr(ehci_softc_t *sc, struct ehci_xfer *ex)
+{
 	usbd_device_handle dev = ex->ex_xfer.ux_pipe->up_dev;
 	int attr;
 	USBHIST_FUNC();	USBHIST_CALLED(ehcidebug);
 	USBHIST_LOG(ehcidebug, "ex = %p", ex, 0, 0, 0);
 	KASSERT(sc->sc_bus.ub_usepolling || mutex_owned(&sc->sc_lock));
 	attr = ex->ex_xfer.ux_pipe->up_endpoint->ue_edesc->bmAttributes;
 	if (UE_GET_XFERTYPE(attr) == UE_ISOCHRONOUS) {
 		if (dev->ud_speed == USB_SPEED_HIGH)
 			ehci_check_itd_intr(sc, ex);
 		else
 			ehci_check_sitd_intr(sc, ex);
 	} else
 		ehci_check_qh_intr(sc, ex);
 	return;
+}
 Static void
 ehci_check_qh_intr(ehci_softc_t *sc, struct ehci_xfer *ex)
+{
 	ehci_soft_qtd_t *sqtd, *lsqtd;
 	uint32_t status;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	KASSERT(sc->sc_bus.ub_usepolling || mutex_owned(&sc->sc_lock));
 	if (ex->ex_sqtdstart == NULL) {
 		printf("ehci_check_qh_intr: not valid sqtd\n");
 		return;
+	}
 	lsqtd = ex->ex_sqtdend;
 #ifdef DIAGNOSTIC
 	if (lsqtd == NULL) {
 		printf("ehci_check_qh_intr: lsqtd==0\n");
 		return;
+	}
 #endif
 	/*
 	 * If the last TD is still active we need to check whether there
 	 * is an error somewhere in the middle, or whether there was a
 	 * short packet (SPD and not ACTIVE).
 	 */
 	usb_syncmem(&lsqtd->dma,
 	    lsqtd->offs + offsetof(ehci_qtd_t, qtd_status),
 	    sizeof(lsqtd->qtd.qtd_status),
 	    BUS_DMASYNC_POSTWRITE | BUS_DMASYNC_POSTREAD);
 	status = le32toh(lsqtd->qtd.qtd_status);
 	usb_syncmem(&lsqtd->dma,
 	    lsqtd->offs + offsetof(ehci_qtd_t, qtd_status),
 	    sizeof(lsqtd->qtd.qtd_status), BUS_DMASYNC_PREREAD);
 	if (status & EHCI_QTD_ACTIVE) {
 		USBHIST_LOGN(ehcidebug, 10, "active ex=%p", ex, 0, 0, 0);
 		for (sqtd = ex->ex_sqtdstart; sqtd != lsqtd;
 		     sqtd = sqtd->nextqtd) {
 			usb_syncmem(&sqtd->dma,
 			    sqtd->offs + offsetof(ehci_qtd_t, qtd_status),
 			    sizeof(sqtd->qtd.qtd_status),
 			    BUS_DMASYNC_POSTWRITE | BUS_DMASYNC_POSTREAD);
 			status = le32toh(sqtd->qtd.qtd_status);
 			usb_syncmem(&sqtd->dma,
 			    sqtd->offs + offsetof(ehci_qtd_t, qtd_status),
 			    sizeof(sqtd->qtd.qtd_status), BUS_DMASYNC_PREREAD);
 			/* If there's an active QTD the xfer isn't done. */
 			if (status & EHCI_QTD_ACTIVE)
 				break;
 			/* Any kind of error makes the xfer done. */
 			if (status & EHCI_QTD_HALTED)
 				goto done;
 			/* Handle short packets */
 			if (EHCI_QTD_GET_BYTES(status) != 0) {
 				usbd_pipe_handle pipe = ex->ex_xfer.ux_pipe;
 				usb_endpoint_descriptor_t *ed =
 				    pipe->up_endpoint->ue_edesc;
 				uint8_t xt = UE_GET_XFERTYPE(ed->bmAttributes);
 				/*
 				 * If we get here for a control transfer then
 				 * we need to let the hardware complete the
 				 * status phase.  That is, we're not done
 				 * quite yet.
+				 *
 				 * Otherwise, we're done.
 				 */
 				if (xt == UE_CONTROL) {
 					break;
+				}
 				goto done;
+			}
+		}
 		USBHIST_LOGN(ehcidebug, 10, "ex=%p std=%p still active",
 		    ex, ex->ex_sqtdstart, 0, 0);
 #ifdef EHCI_DEBUG
 		USBHIST_LOGN(ehcidebug, 5, "--- still active start ---", 0, 0, 0, 0);
 		ehci_dump_sqtds(ex->ex_sqtdstart);
 		USBHIST_LOGN(ehcidebug, 5, "--- still active end ---", 0, 0, 0, 0);
 #endif
 		return;
+	}
  done:
 	USBHIST_LOGN(ehcidebug, 10, "ex=%p done", ex, 0, 0, 0);
 	callout_stop(&ex->ex_xfer.ux_callout);
 	ehci_idone(ex);
+}
 Static void
 ehci_check_itd_intr(ehci_softc_t *sc, struct ehci_xfer *ex)
+{
 	ehci_soft_itd_t *itd;
 	int i;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	KASSERT(mutex_owned(&sc->sc_lock));
 	if (&ex->ex_xfer != SIMPLEQ_FIRST(&ex->ex_xfer.ux_pipe->up_queue))
 		return;
 	if (ex->ex_itdstart == NULL) {
 		printf("ehci_check_itd_intr: not valid itd\n");
 		return;
+	}
 	itd = ex->ex_itdend;
 #ifdef DIAGNOSTIC
 	if (itd == NULL) {
 		printf("ehci_check_itd_intr: itdend == 0\n");
 		return;
+	}
 #endif
 	/*
 	 * check no active transfers in last itd, meaning we're finished
 	 */
 	usb_syncmem(&itd->dma, itd->offs + offsetof(ehci_itd_t, itd_ctl),
 		    sizeof(itd->itd.itd_ctl), BUS_DMASYNC_POSTWRITE |
 		    BUS_DMASYNC_POSTREAD);
 	for (i = 0; i < EHCI_ITD_NUFRAMES; i++) {
 		if (le32toh(itd->itd.itd_ctl[i]) & EHCI_ITD_ACTIVE)
 			break;
+	}
 	if (i == EHCI_ITD_NUFRAMES) {
 		goto done; /* All 8 descriptors inactive, it's done */
+	}
 	USBHIST_LOGN(ehcidebug, 10, "ex %p itd %p still active", ex,
 	    ex->ex_itdstart, 0, 0);
 	return;
 done:
 	USBHIST_LOG(ehcidebug, "ex %p done", ex, 0, 0, 0);
 	callout_stop(&ex->ex_xfer.ux_callout);
 	ehci_idone(ex);
+}
 void
 ehci_check_sitd_intr(ehci_softc_t *sc, struct ehci_xfer *ex)
+{
 	ehci_soft_sitd_t *sitd;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	KASSERT(mutex_owned(&sc->sc_lock));
 	if (&ex->ex_xfer != SIMPLEQ_FIRST(&ex->ex_xfer.ux_pipe->up_queue))
 		return;
 	if (ex->ex_sitdstart == NULL) {
 		printf("ehci_check_sitd_intr: not valid sitd\n");
 		return;
+	}
 	sitd = ex->ex_sitdend;
 #ifdef DIAGNOSTIC
 	if (sitd == NULL) {
 		printf("ehci_check_sitd_intr: sitdend == 0\n");
 		return;
+	}
 #endif
 	/*
 	 * check no active transfers in last sitd, meaning we're finished
 	 */
 	usb_syncmem(&sitd->dma, sitd->offs + offsetof(ehci_sitd_t, sitd_buffer),
 		    sizeof(sitd->sitd.sitd_buffer), BUS_DMASYNC_POSTWRITE |
 		    BUS_DMASYNC_POSTREAD);
 	if (le32toh(sitd->sitd.sitd_trans) & EHCI_SITD_ACTIVE)
 		return;
 	USBHIST_LOGN(ehcidebug, 10, "ex=%p done", ex, 0, 0, 0);
 	callout_stop(&(ex->ex_xfer.ux_callout));
 	ehci_idone(ex);
+}
 Static void
 ehci_idone(struct ehci_xfer *ex)
+{
 	usbd_xfer_handle xfer = &ex->ex_xfer;
 	struct ehci_pipe *epipe = (struct ehci_pipe *)xfer->ux_pipe;
 	struct ehci_softc *sc = xfer->ux_pipe->up_dev->ud_bus->ub_hcpriv;
 	ehci_soft_qtd_t *sqtd, *lsqtd;
 	uint32_t status = 0, nstatus = 0;
 	int actlen;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	KASSERT(sc->sc_bus.ub_usepolling || mutex_owned(&sc->sc_lock));
 	USBHIST_LOG(ehcidebug, "ex=%p", ex, 0, 0, 0);
 #ifdef DIAGNOSTIC
 	if (ex->ex_isdone) {
 		printf("ehci_idone: ex=%p is done!\n", ex);
 #ifdef EHCI_DEBUG
 	if (ex->ex_isdone) {
 		USBHIST_LOGN(ehcidebug, 5, "--- dump start ---", 0, 0, 0, 0);
 		ehci_dump_exfer(ex);
 		USBHIST_LOGN(ehcidebug, 5, "--- dump end ---", 0, 0, 0, 0);
 #endif
 		return;
+	}
-	ex->ex_isdone = 1;
+#endif
 	KASSERT(!ex->ex_isdone);
 	ex->ex_isdone = true;
 #endif
 	if (xfer->ux_status == USBD_CANCELLED ||
 	    xfer->ux_status == USBD_TIMEOUT) {
 		USBHIST_LOG(ehcidebug, "aborted xfer=%p", xfer, 0, 0, 0);
 		return;
+	}
 	USBHIST_LOG(ehcidebug, "xfer=%p, pipe=%p ready", xfer, epipe, 0, 0);
 #ifdef EHCI_DEBUG
 	USBHIST_LOGN(ehcidebug, 5, "--- dump start ---", 0, 0, 0, 0);
 	ehci_dump_sqtds(ex->ex_sqtdstart);
 	USBHIST_LOGN(ehcidebug, 5, "--- dump end ---", 0, 0, 0, 0);
 #endif
 	/* The transfer is done, compute actual length and status. */
 	u_int xfertype, speed;
 	xfertype = UE_GET_XFERTYPE(xfer->ux_pipe->up_endpoint->ue_edesc->bmAttributes);
 	speed = xfer->ux_pipe->up_dev->ud_speed;
 	if (xfertype == UE_ISOCHRONOUS && speed == USB_SPEED_HIGH) {
 		/* HS isoc transfer */
 		struct ehci_soft_itd *itd;
 		int i, nframes, len, uframes;
 		nframes = 0;
 		actlen = 0;
 		i = xfer->ux_pipe->up_endpoint->ue_edesc->bInterval;
 		uframes = min(1 << (i - 1), USB_UFRAMES_PER_FRAME);
 		for (itd = ex->ex_itdstart; itd != NULL; itd = itd->xfer_next) {
 			usb_syncmem(&itd->dma,itd->offs + offsetof(ehci_itd_t,itd_ctl),
 			    sizeof(itd->itd.itd_ctl), BUS_DMASYNC_POSTWRITE |
 			    BUS_DMASYNC_POSTREAD);
 			for (i = 0; i < EHCI_ITD_NUFRAMES; i += uframes) {
 				/*
 				 * XXX - driver didn't fill in the frame full
 				 *   of uframes. This leads to scheduling
 				 *   inefficiencies, but working around
 				 *   this doubles complexity of tracking
 				 *   an xfer.
 				 */
 				if (nframes >= xfer->ux_nframes)
 					break;
 				status = le32toh(itd->itd.itd_ctl[i]);
 				len = EHCI_ITD_GET_LEN(status);
 				if (EHCI_ITD_GET_STATUS(status) != 0)
 					len = 0; /*No valid data on error*/
 				xfer->ux_frlengths[nframes++] = len;
 				actlen += len;
+			}
 			if (nframes >= xfer->ux_nframes)
 				break;
+	    	}
 		xfer->ux_actlen = actlen;
 		xfer->ux_status = USBD_NORMAL_COMPLETION;
 		goto end;
+	}
 	if (xfertype == UE_ISOCHRONOUS && speed == USB_SPEED_FULL) {
 		/* FS isoc transfer */
 		struct ehci_soft_sitd *sitd;
 		int nframes, len;
 		nframes = 0;
 		actlen = 0;
 		for (sitd = ex->ex_sitdstart; sitd != NULL; sitd = sitd->xfer_next) {
 			usb_syncmem(&sitd->dma,sitd->offs + offsetof(ehci_sitd_t, sitd_buffer),
 			    sizeof(sitd->sitd.sitd_buffer), BUS_DMASYNC_POSTWRITE |
 			    BUS_DMASYNC_POSTREAD);
 			/*
 			 * XXX - driver didn't fill in the frame full
 			 *   of uframes. This leads to scheduling
 			 *   inefficiencies, but working around
 			 *   this doubles complexity of tracking
 			 *   an xfer.
 			 */
 			if (nframes >= xfer->ux_nframes)
 				break;
 			status = le32toh(sitd->sitd.sitd_trans);
 			len = EHCI_SITD_GET_LEN(status);
 			if (status & (EHCI_SITD_ERR|EHCI_SITD_BUFERR|
 			    EHCI_SITD_BABBLE|EHCI_SITD_XACTERR|EHCI_SITD_MISS)) {
 				/* No valid data on error */
 				len = xfer->ux_frlengths[nframes];
+			}
 			/*
 			 * frlengths[i]: # of bytes to send
 			 * len: # of bytes host didn't send
 			 */
 			xfer->ux_frlengths[nframes] -= len;
 			/* frlengths[i]: # of bytes host sent */
 			actlen += xfer->ux_frlengths[nframes++];
 			if (nframes >= xfer->ux_nframes)
 				break;
+	    	}
 		xfer->ux_actlen = actlen;
 		xfer->ux_status = USBD_NORMAL_COMPLETION;
 		goto end;
+	}
 	KASSERT(xfertype != UE_ISOCHRONOUS);
 	/* Continue processing xfers using queue heads */
 	lsqtd = ex->ex_sqtdend;
 	actlen = 0;
 	for (sqtd = ex->ex_sqtdstart; sqtd != lsqtd->nextqtd;
 	     sqtd = sqtd->nextqtd) {
 		usb_syncmem(&sqtd->dma, sqtd->offs, sizeof(sqtd->qtd),
 		    BUS_DMASYNC_POSTWRITE | BUS_DMASYNC_POSTREAD);
 		nstatus = le32toh(sqtd->qtd.qtd_status);
 		if (nstatus & EHCI_QTD_ACTIVE)
 			break;
 		status = nstatus;
 		if (EHCI_QTD_GET_PID(status) != EHCI_QTD_PID_SETUP)
 			actlen += sqtd->len - EHCI_QTD_GET_BYTES(status);
+	}
 	/*
 	 * If there are left over TDs we need to update the toggle.
 	 * The default pipe doesn't need it since control transfers
 	 * start the toggle at 0 every time.
 	 * For a short transfer we need to update the toggle for the missing
 	 * packets within the qTD.
 	 */
 	if ((sqtd != lsqtd->nextqtd || EHCI_QTD_GET_BYTES(status)) &&
 	    xfer->ux_pipe->up_dev->ud_pipe0 != xfer->ux_pipe) {
 		USBHIST_LOG(ehcidebug,
 		    "toggle update status=0x%08x nstatus=0x%08x",
 		    status, nstatus, 0, 0);
 #if 0
 		ehci_dump_sqh(epipe->sqh);
 		ehci_dump_sqtds(ex->ex_sqtdstart);
 #endif
 		epipe->nexttoggle = EHCI_QTD_GET_TOGGLE(nstatus);
+	}
 	USBHIST_LOG(ehcidebug, "len=%d actlen=%d status=0x%08x", xfer->ux_length,
 	    actlen, status, 0);
 	xfer->ux_actlen = actlen;
 	if (status & EHCI_QTD_HALTED) {
 #ifdef EHCI_DEBUG
 		USBHIST_LOG(ehcidebug, "halted addr=%d endpt=0x%02x",
 		    xfer->ux_pipe->up_dev->ud_addr,
 		    xfer->ux_pipe->up_endpoint->ue_edesc->bEndpointAddress, 0, 0);
 		USBHIST_LOG(ehcidebug, "cerr=%d pid=%d",
 		    EHCI_QTD_GET_CERR(status), EHCI_QTD_GET_PID(status),
 , 0);
 		USBHIST_LOG(ehcidebug,
 		    "active =%d halted=%d buferr=%d babble=%d",
 		    status & EHCI_QTD_ACTIVE ? 1 : 0,
 		    status & EHCI_QTD_HALTED ? 1 : 0,
 		    status & EHCI_QTD_BUFERR ? 1 : 0,
 		    status & EHCI_QTD_BABBLE ? 1 : 0);
 		USBHIST_LOG(ehcidebug,
 		    "xacterr=%d missed=%d split =%d ping  =%d",
 		    status & EHCI_QTD_XACTERR ? 1 : 0,
 		    status & EHCI_QTD_MISSEDMICRO ? 1 : 0,
 		    status & EHCI_QTD_SPLITXSTATE ? 1 : 0,
 		    status & EHCI_QTD_PINGSTATE ? 1 : 0);
 		USBHIST_LOGN(ehcidebug, 5, "--- dump start ---", 0, 0, 0, 0);
 		ehci_dump_sqh(epipe->sqh);
 		ehci_dump_sqtds(ex->ex_sqtdstart);
 		USBHIST_LOGN(ehcidebug, 5, "--- dump end ---", 0, 0, 0, 0);
 #endif
 		/* low&full speed has an extra error flag */
 		if (EHCI_QH_GET_EPS(epipe->sqh->qh.qh_endp) !=
 		    EHCI_QH_SPEED_HIGH)
 			status &= EHCI_QTD_STATERRS | EHCI_QTD_PINGSTATE;
 		else
 			status &= EHCI_QTD_STATERRS;
 		if (status == 0) /* no other errors means a stall */ {
 			xfer->ux_status = USBD_STALLED;
 		} else {
 			xfer->ux_status = USBD_IOERROR; /* more info XXX */
+		}
 		/* XXX need to reset TT on missed microframe */
 		if (status & EHCI_QTD_MISSEDMICRO) {
 			printf("%s: missed microframe, TT reset not "
 			    "implemented, hub might be inoperational\n",
 			    device_xname(sc->sc_dev));
+		}
 	} else {
 		xfer->ux_status = USBD_NORMAL_COMPLETION;
+	}
     end:
 	/*
 	 * XXX transfer_complete memcpys out transfer data (for in endpoints)
 	 * during this call, before methods->done is called: dma sync required
 	 * beforehand?
 	 */
 	usb_transfer_complete(xfer);
 	USBHIST_LOG(ehcidebug, "ex=%p done", ex, 0, 0, 0);
+}
 /*
  * Wait here until controller claims to have an interrupt.
  * Then call ehci_intr and return.  Use timeout to avoid waiting
  * too long.
  */
 Static void
 ehci_waitintr(ehci_softc_t *sc, usbd_xfer_handle xfer)
+{
 	int timo;
 	uint32_t intrs;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	xfer->ux_status = USBD_IN_PROGRESS;
 	for (timo = xfer->ux_timeout; timo >= 0; timo--) {
 		usb_delay_ms(&sc->sc_bus, 1);
 		if (sc->sc_dying)
 			break;
 		intrs = EHCI_STS_INTRS(EOREAD4(sc, EHCI_USBSTS)) &
 			sc->sc_eintrs;
 		USBHIST_LOG(ehcidebug, "0x%04x", intrs, 0, 0, 0);
 #ifdef EHCI_DEBUG
 		if (ehcidebug > 15)
 			ehci_dump_regs(sc);
 #endif
 		if (intrs) {
 			mutex_spin_enter(&sc->sc_intr_lock);
 			ehci_intr1(sc);
 			mutex_spin_exit(&sc->sc_intr_lock);
 			if (xfer->ux_status != USBD_IN_PROGRESS)
 				return;
+		}
+	}
 	/* Timeout */
 	USBHIST_LOG(ehcidebug, "timeout", 0, 0, 0, 0);
 	xfer->ux_status = USBD_TIMEOUT;
 	mutex_enter(&sc->sc_lock);
 	usb_transfer_complete(xfer);
 	mutex_exit(&sc->sc_lock);
 	/* XXX should free TD */
+}
 Static void
 ehci_poll(struct usbd_bus *bus)
+{
 	ehci_softc_t *sc = bus->ub_hcpriv;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 #ifdef EHCI_DEBUG
 	static int last;
 	int new;
 	new = EHCI_STS_INTRS(EOREAD4(sc, EHCI_USBSTS));
 	if (new != last) {
 		USBHIST_LOG(ehcidebug, "intrs=0x%04x", new, 0, 0, 0);
 		last = new;
+	}
 #endif
 	if (EOREAD4(sc, EHCI_USBSTS) & sc->sc_eintrs) {
 		mutex_spin_enter(&sc->sc_intr_lock);
 		ehci_intr1(sc);
 		mutex_spin_exit(&sc->sc_intr_lock);
+	}
+}
 void
 ehci_childdet(device_t self, device_t child)
+{
 	struct ehci_softc *sc = device_private(self);
 	KASSERT(sc->sc_child == child);
 	sc->sc_child = NULL;
+}
 int
 ehci_detach(struct ehci_softc *sc, int flags)
+{
 	int rv = 0;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	if (sc->sc_child != NULL)
 		rv = config_detach(sc->sc_child, flags);
 	if (rv != 0)
 		return rv;
 	callout_halt(&sc->sc_tmo_intrlist, NULL);
 	callout_destroy(&sc->sc_tmo_intrlist);
 	/* XXX free other data structures XXX */
 	if (sc->sc_softitds)
 		kmem_free(sc->sc_softitds,
 		    sc->sc_flsize * sizeof(ehci_soft_itd_t *));
 	cv_destroy(&sc->sc_doorbell);
 	cv_destroy(&sc->sc_softwake_cv);
 #if 0
 	/* XXX destroyed in ehci_pci.c as it controls ehci_intr access */
 	softint_disestablish(sc->sc_doorbell_si);
 	softint_disestablish(sc->sc_pcd_si);
 	mutex_destroy(&sc->sc_lock);
 	mutex_destroy(&sc->sc_intr_lock);
 #endif
 	pool_cache_destroy(sc->sc_xferpool);
 	EOWRITE4(sc, EHCI_CONFIGFLAG, 0);
 	return rv;
+}
 int
 ehci_activate(device_t self, enum devact act)
+{
 	struct ehci_softc *sc = device_private(self);
 	switch (act) {
 	case DVACT_DEACTIVATE:
 		sc->sc_dying = 1;
 		return 0;
 	default:
 		return EOPNOTSUPP;
+	}
+}
 /*
  * Handle suspend/resume.
+ *
  * We need to switch to polling mode here, because this routine is
  * called from an interrupt context.  This is all right since we
  * are almost suspended anyway.
+ *
  * Note that this power handler isn't to be registered directly; the
  * bus glue needs to call out to it.
  */
 bool
 ehci_suspend(device_t dv, const pmf_qual_t *qual)
+{
 	ehci_softc_t *sc = device_private(dv);
 	int i;
 	uint32_t cmd, hcr;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	mutex_spin_enter(&sc->sc_intr_lock);
 	sc->sc_bus.ub_usepolling++;
 	mutex_spin_exit(&sc->sc_intr_lock);
 	for (i = 1; i <= sc->sc_noport; i++) {
 		cmd = EOREAD4(sc, EHCI_PORTSC(i)) & ~EHCI_PS_CLEAR;
 		if ((cmd & EHCI_PS_PO) == 0 && (cmd & EHCI_PS_PE) == EHCI_PS_PE)
 			EOWRITE4(sc, EHCI_PORTSC(i), cmd | EHCI_PS_SUSP);
+	}
 	sc->sc_cmd = EOREAD4(sc, EHCI_USBCMD);
 	cmd = sc->sc_cmd & ~(EHCI_CMD_ASE | EHCI_CMD_PSE);
 	EOWRITE4(sc, EHCI_USBCMD, cmd);
 	for (i = 0; i < 100; i++) {
 		hcr = EOREAD4(sc, EHCI_USBSTS) & (EHCI_STS_ASS | EHCI_STS_PSS);
 		if (hcr == 0)
 			break;
 		usb_delay_ms(&sc->sc_bus, 1);
+	}
 	if (hcr != 0)
 		printf("%s: reset timeout\n", device_xname(dv));
 	cmd &= ~EHCI_CMD_RS;
 	EOWRITE4(sc, EHCI_USBCMD, cmd);
 	for (i = 0; i < 100; i++) {
 		hcr = EOREAD4(sc, EHCI_USBSTS) & EHCI_STS_HCH;
 		if (hcr == EHCI_STS_HCH)
 			break;
 		usb_delay_ms(&sc->sc_bus, 1);
+	}
 	if (hcr != EHCI_STS_HCH)
 		printf("%s: config timeout\n", device_xname(dv));
 	mutex_spin_enter(&sc->sc_intr_lock);
 	sc->sc_bus.ub_usepolling--;
 	mutex_spin_exit(&sc->sc_intr_lock);
 	return true;
+}
 bool
 ehci_resume(device_t dv, const pmf_qual_t *qual)
+{
 	ehci_softc_t *sc = device_private(dv);
 	int i;
 	uint32_t cmd, hcr;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	/* restore things in case the bios sucks */
 	EOWRITE4(sc, EHCI_CTRLDSSEGMENT, 0);
 	EOWRITE4(sc, EHCI_PERIODICLISTBASE, DMAADDR(&sc->sc_fldma, 0));
 	EOWRITE4(sc, EHCI_ASYNCLISTADDR,
 	    sc->sc_async_head->physaddr | EHCI_LINK_QH);
 	EOWRITE4(sc, EHCI_USBINTR, sc->sc_eintrs & ~EHCI_INTR_PCIE);
 	EOWRITE4(sc, EHCI_USBCMD, sc->sc_cmd);
 	hcr = 0;
 	for (i = 1; i <= sc->sc_noport; i++) {
 		cmd = EOREAD4(sc, EHCI_PORTSC(i)) & ~EHCI_PS_CLEAR;
 		if ((cmd & EHCI_PS_PO) == 0 &&
 		    (cmd & EHCI_PS_SUSP) == EHCI_PS_SUSP) {
 			EOWRITE4(sc, EHCI_PORTSC(i), cmd | EHCI_PS_FPR);
 			hcr = 1;
+		}
+	}
 	if (hcr) {
 		usb_delay_ms(&sc->sc_bus, USB_RESUME_WAIT);
 		for (i = 1; i <= sc->sc_noport; i++) {
 			cmd = EOREAD4(sc, EHCI_PORTSC(i)) & ~EHCI_PS_CLEAR;
 			if ((cmd & EHCI_PS_PO) == 0 &&
 			    (cmd & EHCI_PS_SUSP) == EHCI_PS_SUSP)
 				EOWRITE4(sc, EHCI_PORTSC(i),
 				    cmd & ~EHCI_PS_FPR);
+		}
+	}
 	EOWRITE4(sc, EHCI_USBCMD, sc->sc_cmd);
 	EOWRITE4(sc, EHCI_USBINTR, sc->sc_eintrs);
 	for (i = 0; i < 100; i++) {
 		hcr = EOREAD4(sc, EHCI_USBSTS) & EHCI_STS_HCH;
 		if (hcr != EHCI_STS_HCH)
 			break;
 		usb_delay_ms(&sc->sc_bus, 1);
+	}
 	if (hcr == EHCI_STS_HCH)
 		printf("%s: config timeout\n", device_xname(dv));
 	return true;
+}
 /*
  * Shut down the controller when the system is going down.
  */
 bool
 ehci_shutdown(device_t self, int flags)
+{
 	ehci_softc_t *sc = device_private(self);
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	EOWRITE4(sc, EHCI_USBCMD, 0);	/* Halt controller */
 	EOWRITE4(sc, EHCI_USBCMD, EHCI_CMD_HCRESET);
 	return true;
+}
 Static usbd_xfer_handle
 ehci_allocx(struct usbd_bus *bus)
+{
 	struct ehci_softc *sc = bus->ub_hcpriv;
 	usbd_xfer_handle xfer;
 	xfer = pool_cache_get(sc->sc_xferpool, PR_NOWAIT);
 	if (xfer != NULL) {
 		memset(xfer, 0, sizeof(struct ehci_xfer));
 #ifdef DIAGNOSTIC
-		EXFER(xfer)->ex_isdone = 1;
+		EXFER(xfer)->ex_isdone = true;
 		xfer->ux_state = XFER_BUSY;
 #endif
+	}
 	return xfer;
+}
 Static void
 ehci_freex(struct usbd_bus *bus, usbd_xfer_handle xfer)
+{
 	struct ehci_softc *sc = bus->ub_hcpriv;
 	KASSERT(xfer->ux_state == XFER_BUSY);
 	KASSERT(EXFER(xfer)->ex_isdone);
 #ifdef DIAGNOSTIC
 	if (xfer->ux_state != XFER_BUSY) {
 		printf("ehci_freex: xfer=%p not busy, 0x%08x\n", xfer,
 		       xfer->ux_state);
 	xfer->ux_state = XFER_FREE;
 	if (!EXFER(xfer)->ex_isdone) {
 		printf("ehci_freex: !isdone\n");
 #endif
 	pool_cache_put(sc->sc_xferpool, xfer);
+}
 Static void
 ehci_get_lock(struct usbd_bus *bus, kmutex_t **lock)
+{
 	struct ehci_softc *sc = bus->ub_hcpriv;
 	*lock = &sc->sc_lock;
+}
 Static void
 ehci_device_clear_toggle(usbd_pipe_handle pipe)
+{
 	struct ehci_pipe *epipe = (struct ehci_pipe *)pipe;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	USBHIST_LOG(ehcidebug, "epipe=%p status=0x%08x",
 	    epipe, epipe->sqh->qh.qh_qtd.qtd_status, 0, 0);
 #ifdef EHCI_DEBUG
 	if (ehcidebug)
 		usbd_dump_pipe(pipe);
 #endif
 	epipe->nexttoggle = 0;
+}
 Static void
 ehci_noop(usbd_pipe_handle pipe)
+{
+}
 #ifdef EHCI_DEBUG
 /*
  * Unused function - this is meant to be called from a kernel
  * debugger.
  */
 void
 ehci_dump(void)
+{
 	ehci_softc_t *sc = theehci;
 	int i;
 	printf("cmd=0x%08x, sts=0x%08x, ien=0x%08x\n",
 	    EOREAD4(sc, EHCI_USBCMD),
 	    EOREAD4(sc, EHCI_USBSTS),
 	    EOREAD4(sc, EHCI_USBINTR));
 	printf("frindex=0x%08x ctrdsegm=0x%08x periodic=0x%08x async=0x%08x\n",
 	    EOREAD4(sc, EHCI_FRINDEX),
 	    EOREAD4(sc, EHCI_CTRLDSSEGMENT),
 	    EOREAD4(sc, EHCI_PERIODICLISTBASE),
 	    EOREAD4(sc, EHCI_ASYNCLISTADDR));
 	for (i = 1; i <= sc->sc_noport; i++)
 		printf("port %d status=0x%08x\n", i,
 		    EOREAD4(sc, EHCI_PORTSC(i)));
+}
 Static void
 ehci_dump_regs(ehci_softc_t *sc)
+{
 	int i;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	USBHIST_LOG(ehcidebug,
 	    "cmd     = 0x%08x  sts      = 0x%08x  ien      = 0x%08x",
 	    EOREAD4(sc, EHCI_USBCMD), EOREAD4(sc, EHCI_USBSTS),
 	    EOREAD4(sc, EHCI_USBINTR), 0);
 	USBHIST_LOG(ehcidebug,
 	    "frindex = 0x%08x  ctrdsegm = 0x%08x  periodic = 0x%08x  "
 	    "async   = 0x%08x",
 	    EOREAD4(sc, EHCI_FRINDEX), EOREAD4(sc, EHCI_CTRLDSSEGMENT),
 	    EOREAD4(sc, EHCI_PERIODICLISTBASE),
 	    EOREAD4(sc, EHCI_ASYNCLISTADDR));
 	for (i = 1; i <= sc->sc_noport; i += 2) {
 		if (i == sc->sc_noport) {
 			USBHIST_LOG(ehcidebug,
 			    "port %d status = 0x%08x", i,
 			    EOREAD4(sc, EHCI_PORTSC(i)), 0, 0);
 		} else {
 			USBHIST_LOG(ehcidebug,
 			    "port %d status = 0x%08x  port %d status = 0x%08x",
 			    i, EOREAD4(sc, EHCI_PORTSC(i)),
 			    i+1, EOREAD4(sc, EHCI_PORTSC(i+1)));
+		}
+	}
+}
 #ifdef EHCI_DEBUG
 #define ehci_dump_link(link, type) do {					\
 	USBHIST_LOG(ehcidebug, "    link 0x%08x (T = %d):",		\
 	    link,							\
 	    link & EHCI_LINK_TERMINATE ? 1 : 0, 0, 0);			\
 	if (type) {							\
 		USBHIST_LOG(ehcidebug,					\
 		    "        ITD  = %d  QH   = %d  SITD = %d  FSTN = %d",\
 		    EHCI_LINK_TYPE(link) == EHCI_LINK_ITD ? 1 : 0,	\
 		    EHCI_LINK_TYPE(link) == EHCI_LINK_QH ? 1 : 0,	\
 		    EHCI_LINK_TYPE(link) == EHCI_LINK_SITD ? 1 : 0,	\
 		    EHCI_LINK_TYPE(link) == EHCI_LINK_FSTN ? 1 : 0);	\
 	}								\
 } while(0)
 #else
 #define ehci_dump_link(link, type)
 #endif
 Static void
 ehci_dump_sqtds(ehci_soft_qtd_t *sqtd)
+{
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	int i;
 	uint32_t stop = 0;
 	for (i = 0; sqtd && i < 20 && !stop; sqtd = sqtd->nextqtd, i++) {
 		ehci_dump_sqtd(sqtd);
 		usb_syncmem(&sqtd->dma,
 		    sqtd->offs + offsetof(ehci_qtd_t, qtd_next),
 		    sizeof(sqtd->qtd),
 		    BUS_DMASYNC_POSTWRITE | BUS_DMASYNC_POSTREAD);
 		stop = sqtd->qtd.qtd_next & htole32(EHCI_LINK_TERMINATE);
 		usb_syncmem(&sqtd->dma,
 		    sqtd->offs + offsetof(ehci_qtd_t, qtd_next),
 		    sizeof(sqtd->qtd), BUS_DMASYNC_PREREAD);
+	}
 	if (!stop)
 		USBHIST_LOG(ehcidebug,
 		    "dump aborted, too many TDs", 0, 0, 0, 0);
+}
 Static void
 ehci_dump_sqtd(ehci_soft_qtd_t *sqtd)
+{
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	usb_syncmem(&sqtd->dma, sqtd->offs,
 	    sizeof(sqtd->qtd), BUS_DMASYNC_POSTWRITE | BUS_DMASYNC_POSTREAD);
 	USBHIST_LOGN(ehcidebug, 10,
 	    "QTD(%p) at 0x%08x:", sqtd, sqtd->physaddr, 0, 0);
 	ehci_dump_qtd(&sqtd->qtd);
 	usb_syncmem(&sqtd->dma, sqtd->offs,
 	    sizeof(sqtd->qtd), BUS_DMASYNC_PREREAD);
+}
 Static void
 ehci_dump_qtd(ehci_qtd_t *qtd)
+{
 	USBHIST_FUNC();	USBHIST_CALLED(ehcidebug);
 #ifdef USBHIST
 	uint32_t s = le32toh(qtd->qtd_status);
 #endif
 	USBHIST_LOGN(ehcidebug, 10,
 	    "     next = 0x%08x  altnext = 0x%08x  status = 0x%08x",
 	    qtd->qtd_next, qtd->qtd_altnext, s, 0);
 	USBHIST_LOGN(ehcidebug, 10,
 	    "   toggle = %d ioc = %d bytes = %#x "
 	    "c_page = %#x", EHCI_QTD_GET_TOGGLE(s), EHCI_QTD_GET_IOC(s),
 	    EHCI_QTD_GET_BYTES(s), EHCI_QTD_GET_C_PAGE(s));
 	USBHIST_LOGN(ehcidebug, 10,
 	    "     cerr = %d pid = %d stat  = %x",
 	    EHCI_QTD_GET_CERR(s), EHCI_QTD_GET_PID(s), EHCI_QTD_GET_STATUS(s),
 );
 	USBHIST_LOGN(ehcidebug, 10,
 	    "active =%d halted=%d buferr=%d babble=%d",
 	    s & EHCI_QTD_ACTIVE ? 1 : 0,
 	    s & EHCI_QTD_HALTED ? 1 : 0,
 	    s & EHCI_QTD_BUFERR ? 1 : 0,
 	    s & EHCI_QTD_BABBLE ? 1 : 0);
 	USBHIST_LOGN(ehcidebug, 10,
 	    "xacterr=%d missed=%d split =%d ping  =%d",
 	    s & EHCI_QTD_XACTERR ? 1 : 0,
 	    s & EHCI_QTD_MISSEDMICRO ? 1 : 0,
 	    s & EHCI_QTD_SPLITXSTATE ? 1 : 0,
 	    s & EHCI_QTD_PINGSTATE ? 1 : 0);
 	USBHIST_LOGN(ehcidebug, 10,
 	    "buffer[0] = %#x  buffer[1] = %#x  "
 	    "buffer[2] = %#x  buffer[3] = %#x",
 	    le32toh(qtd->qtd_buffer[0]), le32toh(qtd->qtd_buffer[1]),
 	    le32toh(qtd->qtd_buffer[2]), le32toh(qtd->qtd_buffer[3]));
 	USBHIST_LOGN(ehcidebug, 10,
 	    "buffer[4] = %#x", le32toh(qtd->qtd_buffer[4]), 0, 0, 0);
+}
 Static void
 ehci_dump_sqh(ehci_soft_qh_t *sqh)
+{
 #ifdef USBHIST
 	ehci_qh_t *qh = &sqh->qh;
 	ehci_link_t link;
 #endif
 	uint32_t endp, endphub;
 	USBHIST_FUNC();	USBHIST_CALLED(ehcidebug);
 	usb_syncmem(&sqh->dma, sqh->offs,
 	    sizeof(sqh->qh), BUS_DMASYNC_POSTWRITE | BUS_DMASYNC_POSTREAD);
 	USBHIST_LOGN(ehcidebug, 10,
 	    "QH(%p) at %#x:", sqh, sqh->physaddr, 0, 0);
 	link = le32toh(qh->qh_link);
 	ehci_dump_link(link, true);
 	endp = le32toh(qh->qh_endp);
 	USBHIST_LOGN(ehcidebug, 10,
 	    "    endp = %#x", endp, 0, 0, 0);
 	USBHIST_LOGN(ehcidebug, 10,
 	    "        addr = 0x%02x  inact = %d  endpt = %d  eps = %d",
 	    EHCI_QH_GET_ADDR(endp), EHCI_QH_GET_INACT(endp),
 	    EHCI_QH_GET_ENDPT(endp), EHCI_QH_GET_EPS(endp));
 	USBHIST_LOGN(ehcidebug, 10,
 	    "        dtc  = %d     hrecl = %d",
 	    EHCI_QH_GET_DTC(endp), EHCI_QH_GET_HRECL(endp), 0, 0);
 	USBHIST_LOGN(ehcidebug, 10,
 	    "        ctl  = %d     nrl   = %d  mpl   = %#x(%d)",
 	    EHCI_QH_GET_CTL(endp),EHCI_QH_GET_NRL(endp),
 	    EHCI_QH_GET_MPL(endp), EHCI_QH_GET_MPL(endp));
 	endphub = le32toh(qh->qh_endphub);
 	USBHIST_LOGN(ehcidebug, 10,
 	    " endphub = %#x", endphub, 0, 0, 0);
 	USBHIST_LOGN(ehcidebug, 10,
 	    "      smask = 0x%02x  cmask = 0x%02x",
 	    EHCI_QH_GET_SMASK(endphub), EHCI_QH_GET_CMASK(endphub), 1, 0);
 	USBHIST_LOGN(ehcidebug, 10,
 	    "      huba  = 0x%02x  port  = %d  mult = %d",
 	    EHCI_QH_GET_HUBA(endphub), EHCI_QH_GET_PORT(endphub),
 	    EHCI_QH_GET_MULT(endphub), 0);
 	link = le32toh(qh->qh_curqtd);
 	ehci_dump_link(link, false);
 	USBHIST_LOGN(ehcidebug, 10, "Overlay qTD:", 0, 0, 0, 0);
 	ehci_dump_qtd(&qh->qh_qtd);
 	usb_syncmem(&sqh->dma, sqh->offs,
 	    sizeof(sqh->qh), BUS_DMASYNC_PREREAD);
+}
 Static void
 ehci_dump_itd(struct ehci_soft_itd *itd)
+{
 	ehci_isoc_trans_t t;
 	ehci_isoc_bufr_ptr_t b, b2, b3;
 	int i;
 	USBHIST_FUNC();	USBHIST_CALLED(ehcidebug);
 	USBHIST_LOG(ehcidebug, "ITD: next phys = %#x", itd->itd.itd_next, 0,
 , 0);
 	for (i = 0; i < EHCI_ITD_NUFRAMES; i++) {
 		t = le32toh(itd->itd.itd_ctl[i]);
 		USBHIST_LOG(ehcidebug, "ITDctl %d: stat = %x len = %x",
 		    i, EHCI_ITD_GET_STATUS(t), EHCI_ITD_GET_LEN(t), 0);
 		USBHIST_LOG(ehcidebug, "     ioc = %x pg = %x offs = %x",
 		    EHCI_ITD_GET_IOC(t), EHCI_ITD_GET_PG(t),
 		    EHCI_ITD_GET_OFFS(t), 0);
+	}
 	USBHIST_LOG(ehcidebug, "ITDbufr: ", 0, 0, 0, 0);
 	for (i = 0; i < EHCI_ITD_NBUFFERS; i++)
 		USBHIST_LOG(ehcidebug, "      %x",
 		    EHCI_ITD_GET_BPTR(le32toh(itd->itd.itd_bufr[i])), 0, 0, 0);
 	b = le32toh(itd->itd.itd_bufr[0]);
 	b2 = le32toh(itd->itd.itd_bufr[1]);
 	b3 = le32toh(itd->itd.itd_bufr[2]);
 	USBHIST_LOG(ehcidebug, "     ep = %x daddr = %x dir = %d",
 	    EHCI_ITD_GET_EP(b), EHCI_ITD_GET_DADDR(b), EHCI_ITD_GET_DIR(b2), 0);
 	USBHIST_LOG(ehcidebug, "     maxpkt = %x multi = %x",
 	    EHCI_ITD_GET_MAXPKT(b2), EHCI_ITD_GET_MULTI(b3), 0, 0);
+}
 Static void
 ehci_dump_sitd(struct ehci_soft_itd *itd)
+{
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	USBHIST_LOG(ehcidebug, "SITD %p next = %p prev = %p",
 	    itd, itd->u.frame_list.next, itd->u.frame_list.prev, 0);
 	USBHIST_LOG(ehcidebug, "        xfernext=%p physaddr=%X slot=%d",
 	    itd->xfer_next, itd->physaddr, itd->slot, 0);
+}
 Static void
 ehci_dump_exfer(struct ehci_xfer *ex)
+{
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	USBHIST_LOG(ehcidebug, "ex = %p sqtdstart = %p end = %p",
 	    ex, ex->ex_sqtdstart, ex->ex_sqtdend, 0);
 	USBHIST_LOG(ehcidebug, "     itdstart = %p end = %p isdone = %d",
 	    ex->ex_itdstart, ex->ex_itdend, ex->ex_isdone, 0);
+}
 #endif
 Static usbd_status
 ehci_open(usbd_pipe_handle pipe)
+{
 	usbd_device_handle dev = pipe->up_dev;
 	ehci_softc_t *sc = dev->ud_bus->ub_hcpriv;
 	usb_endpoint_descriptor_t *ed = pipe->up_endpoint->ue_edesc;
 	uint8_t rhaddr = dev->ud_bus->ub_rhaddr;
 	uint8_t addr = dev->ud_addr;
 	uint8_t xfertype = UE_GET_XFERTYPE(ed->bmAttributes);
 	struct ehci_pipe *epipe = (struct ehci_pipe *)pipe;
 	ehci_soft_qh_t *sqh;
 	usbd_status err;
 	int ival, speed, naks;
 	int hshubaddr, hshubport;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	USBHIST_LOG(ehcidebug, "pipe=%p, addr=%d, endpt=%d (%d)",
 	    pipe, addr, ed->bEndpointAddress, rhaddr);
 	if (dev->ud_myhsport) {
 		/*
 		 * When directly attached FS/LS device while doing embedded
 		 * transaction translations and we are the hub, set the hub
 		 * address to 0 (us).
 		 */
 		if (!(sc->sc_flags & EHCIF_ETTF)
 		    || (dev->ud_myhsport->up_parent->ud_addr != rhaddr)) {
 			hshubaddr = dev->ud_myhsport->up_parent->ud_addr;
 		} else {
 			hshubaddr = 0;
+		}
 		hshubport = dev->ud_myhsport->up_portno;
 	} else {
 		hshubaddr = 0;
 		hshubport = 0;
+	}
 	if (sc->sc_dying)
 		return USBD_IOERROR;
 	/* toggle state needed for bulk endpoints */
 	epipe->nexttoggle = pipe->up_endpoint->ue_toggle;
 	if (addr == rhaddr) {
 		switch (ed->bEndpointAddress) {
 		case USB_CONTROL_ENDPOINT:
 			pipe->up_methods = &roothub_ctrl_methods;
 			break;
 		case UE_DIR_IN | USBROOTHUB_INTR_ENDPT:
 			pipe->up_methods = &ehci_root_intr_methods;
 			break;
 		default:
 			USBHIST_LOG(ehcidebug,
 			    "bad bEndpointAddress 0x%02x",
 			    ed->bEndpointAddress, 0, 0, 0);
 			return USBD_INVAL;
+		}
 		return USBD_NORMAL_COMPLETION;
+	}
 	/* XXX All this stuff is only valid for async. */
 	switch (dev->ud_speed) {
 	case USB_SPEED_LOW:  speed = EHCI_QH_SPEED_LOW;  break;
 	case USB_SPEED_FULL: speed = EHCI_QH_SPEED_FULL; break;
 	case USB_SPEED_HIGH: speed = EHCI_QH_SPEED_HIGH; break;
 	default: panic("ehci_open: bad device speed %d", dev->ud_speed);
+	}
 	if (speed == EHCI_QH_SPEED_LOW && xfertype == UE_ISOCHRONOUS) {
 		USBHIST_LOG(ehcidebug, "hshubaddr=%d hshubport=%d",
 			    hshubaddr, hshubport, 0, 0);
 		return USBD_INVAL;
+	}
 	/*
 	 * For interrupt transfer, nak throttling must be disabled, but for
 	 * the other transfer type, nak throttling should be enabled from the
 	 * viewpoint that avoids the memory thrashing.
 	 */
 	naks = (xfertype == UE_INTERRUPT) ? 0
 	    : ((speed == EHCI_QH_SPEED_HIGH) ? 4 : 0);
 	/* Allocate sqh for everything, save isoc xfers */
 	if (xfertype != UE_ISOCHRONOUS) {
 		sqh = ehci_alloc_sqh(sc);
 		if (sqh == NULL)
 			return USBD_NOMEM;
 		/* qh_link filled when the QH is added */
 		sqh->qh.qh_endp = htole32(
 		    EHCI_QH_SET_ADDR(addr) |
 		    EHCI_QH_SET_ENDPT(UE_GET_ADDR(ed->bEndpointAddress)) |
 		    EHCI_QH_SET_EPS(speed) |
 		    EHCI_QH_DTC |
 		    EHCI_QH_SET_MPL(UGETW(ed->wMaxPacketSize)) |
 		    (speed != EHCI_QH_SPEED_HIGH && xfertype == UE_CONTROL ?
 		     EHCI_QH_CTL : 0) |
 		    EHCI_QH_SET_NRL(naks)
 		    );
 		sqh->qh.qh_endphub = htole32(
 		    EHCI_QH_SET_MULT(1) |
 		    EHCI_QH_SET_SMASK(xfertype == UE_INTERRUPT ? 0x02 : 0)
 		    );
 		if (speed != EHCI_QH_SPEED_HIGH)
 			sqh->qh.qh_endphub |= htole32(
 			    EHCI_QH_SET_PORT(hshubport) |
 			    EHCI_QH_SET_HUBA(hshubaddr) |
 			    EHCI_QH_SET_CMASK(0x08) /* XXX */
 			);
 		sqh->qh.qh_curqtd = EHCI_NULL;
 		/* Fill the overlay qTD */
 		sqh->qh.qh_qtd.qtd_next = EHCI_NULL;
 		sqh->qh.qh_qtd.qtd_altnext = EHCI_NULL;
 		sqh->qh.qh_qtd.qtd_status = htole32(0);
 		usb_syncmem(&sqh->dma, sqh->offs, sizeof(sqh->qh),
 		    BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
 		epipe->sqh = sqh;
 	} else {
 		sqh = NULL;
 	} /*xfertype == UE_ISOC*/
 	switch (xfertype) {
 	case UE_CONTROL:
 		err = usb_allocmem(&sc->sc_bus, sizeof(usb_device_request_t),
 , &epipe->u.ctl.reqdma);
 #ifdef EHCI_DEBUG
 		if (err)
 			printf("ehci_open: usb_allocmem()=%d\n", err);
 #endif
 		if (err)
 			goto bad;
 		pipe->up_methods = &ehci_device_ctrl_methods;
 		mutex_enter(&sc->sc_lock);
 		ehci_add_qh(sc, sqh, sc->sc_async_head);
 		mutex_exit(&sc->sc_lock);
 		break;
 	case UE_BULK:
 		pipe->up_methods = &ehci_device_bulk_methods;
 		mutex_enter(&sc->sc_lock);
 		ehci_add_qh(sc, sqh, sc->sc_async_head);
 		mutex_exit(&sc->sc_lock);
 		break;
 	case UE_INTERRUPT:
 		pipe->up_methods = &ehci_device_intr_methods;
 		ival = pipe->up_interval;
 		if (ival == USBD_DEFAULT_INTERVAL) {
 			if (speed == EHCI_QH_SPEED_HIGH) {
 				if (ed->bInterval > 16) {
 					/*
 					 * illegal with high-speed, but there
 					 * were documentation bugs in the spec,
 					 * so be generous
 					 */
 					ival = 256;
 				} else
 					ival = (1 << (ed->bInterval - 1)) / 8;
 			} else
 				ival = ed->bInterval;
+		}
 		err = ehci_device_setintr(sc, sqh, ival);
 		if (err)
 			goto bad;
 		break;
 	case UE_ISOCHRONOUS:
 		if (speed == EHCI_QH_SPEED_HIGH)
 			pipe->up_methods = &ehci_device_isoc_methods;
 		else
 			pipe->up_methods = &ehci_device_fs_isoc_methods;
 		if (ed->bInterval == 0 || ed->bInterval > 16) {
 			printf("ehci: opening pipe with invalid bInterval\n");
 			err = USBD_INVAL;
 			goto bad;
+		}
 		if (UGETW(ed->wMaxPacketSize) == 0) {
 			printf("ehci: zero length endpoint open request\n");
 			err = USBD_INVAL;
 			goto bad;
+		}
 		epipe->u.isoc.next_frame = 0;
 		epipe->u.isoc.cur_xfers = 0;
 		break;
 	default:
 		USBHIST_LOG(ehcidebug, "bad xfer type %d", xfertype, 0, 0, 0);
 		err = USBD_INVAL;
 		goto bad;
+	}
 	return USBD_NORMAL_COMPLETION;
  bad:
 	if (sqh != NULL)
 		ehci_free_sqh(sc, sqh);
 	return err;
+}
 /*
  * Add an ED to the schedule.  Called with USB lock held.
  */
 Static void
 ehci_add_qh(ehci_softc_t *sc, ehci_soft_qh_t *sqh, ehci_soft_qh_t *head)
+{
 	KASSERT(mutex_owned(&sc->sc_lock));
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	usb_syncmem(&head->dma, head->offs + offsetof(ehci_qh_t, qh_link),
 	    sizeof(head->qh.qh_link), BUS_DMASYNC_POSTWRITE);
 	sqh->next = head->next;
 	sqh->qh.qh_link = head->qh.qh_link;
 	usb_syncmem(&sqh->dma, sqh->offs + offsetof(ehci_qh_t, qh_link),
 	    sizeof(sqh->qh.qh_link), BUS_DMASYNC_PREWRITE);
 	head->next = sqh;
 	head->qh.qh_link = htole32(sqh->physaddr | EHCI_LINK_QH);
 	usb_syncmem(&head->dma, head->offs + offsetof(ehci_qh_t, qh_link),
 	    sizeof(head->qh.qh_link), BUS_DMASYNC_PREWRITE);
 #ifdef EHCI_DEBUG
 	USBHIST_LOGN(ehcidebug, 5, "--- dump start ---", 0, 0, 0, 0);
 	ehci_dump_sqh(sqh);
 	USBHIST_LOGN(ehcidebug, 5, "--- dump end ---", 0, 0, 0, 0);
 #endif
+}
 /*
  * Remove an ED from the schedule.  Called with USB lock held.
  */
 Static void
 ehci_rem_qh(ehci_softc_t *sc, ehci_soft_qh_t *sqh, ehci_soft_qh_t *head)
+{
 	ehci_soft_qh_t *p;
 	KASSERT(mutex_owned(&sc->sc_lock));
 	/* XXX */
 	for (p = head; p != NULL && p->next != sqh; p = p->next)
+		;
 	if (p == NULL)
 		panic("ehci_rem_qh: ED not found");
 	usb_syncmem(&sqh->dma, sqh->offs + offsetof(ehci_qh_t, qh_link),
 	    sizeof(sqh->qh.qh_link), BUS_DMASYNC_POSTWRITE);
 	p->next = sqh->next;
 	p->qh.qh_link = sqh->qh.qh_link;
 	usb_syncmem(&p->dma, p->offs + offsetof(ehci_qh_t, qh_link),
 	    sizeof(p->qh.qh_link), BUS_DMASYNC_PREWRITE);
 	ehci_sync_hc(sc);
+}
 Static void
 ehci_set_qh_qtd(ehci_soft_qh_t *sqh, ehci_soft_qtd_t *sqtd)
+{
 	int i;
 	uint32_t status;
 	/* Save toggle bit and ping status. */
 	usb_syncmem(&sqh->dma, sqh->offs, sizeof(sqh->qh),
 	    BUS_DMASYNC_POSTWRITE | BUS_DMASYNC_POSTREAD);
 	status = sqh->qh.qh_qtd.qtd_status &
 	    htole32(EHCI_QTD_TOGGLE_MASK |
 		    EHCI_QTD_SET_STATUS(EHCI_QTD_PINGSTATE));
 	/* Set HALTED to make hw leave it alone. */
 	sqh->qh.qh_qtd.qtd_status =
 	    htole32(EHCI_QTD_SET_STATUS(EHCI_QTD_HALTED));
 	usb_syncmem(&sqh->dma,
 	    sqh->offs + offsetof(ehci_qh_t, qh_qtd.qtd_status),
 	    sizeof(sqh->qh.qh_qtd.qtd_status),
 	    BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
 	sqh->qh.qh_curqtd = 0;
 	sqh->qh.qh_qtd.qtd_next = htole32(sqtd->physaddr);
 	sqh->qh.qh_qtd.qtd_altnext = EHCI_NULL;
 	for (i = 0; i < EHCI_QTD_NBUFFERS; i++)
 		sqh->qh.qh_qtd.qtd_buffer[i] = 0;
 	sqh->sqtd = sqtd;
 	usb_syncmem(&sqh->dma, sqh->offs, sizeof(sqh->qh),
 	    BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
 	/* Set !HALTED && !ACTIVE to start execution, preserve some fields */
 	sqh->qh.qh_qtd.qtd_status = status;
 	usb_syncmem(&sqh->dma,
 	    sqh->offs + offsetof(ehci_qh_t, qh_qtd.qtd_status),
 	    sizeof(sqh->qh.qh_qtd.qtd_status),
 	    BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
+}
 /*
  * Ensure that the HC has released all references to the QH.  We do this
  * by asking for a Async Advance Doorbell interrupt and then we wait for
  * the interrupt.
  * To make this easier we first obtain exclusive use of the doorbell.
  */
 Static void
 ehci_sync_hc(ehci_softc_t *sc)
+{
 	int error __diagused;
 	KASSERT(mutex_owned(&sc->sc_lock));
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	if (sc->sc_dying) {
 		USBHIST_LOG(ehcidebug, "dying", 0, 0, 0, 0);
 		return;
+	}
 	/* ask for doorbell */
 	EOWRITE4(sc, EHCI_USBCMD, EOREAD4(sc, EHCI_USBCMD) | EHCI_CMD_IAAD);
 	USBHIST_LOG(ehcidebug, "cmd = 0x%08x sts = 0x%08x",
 		    EOREAD4(sc, EHCI_USBCMD), EOREAD4(sc, EHCI_USBSTS), 0, 0);
 	error = cv_timedwait(&sc->sc_doorbell, &sc->sc_lock, hz); /* bell wait */
 	USBHIST_LOG(ehcidebug, "cmd = 0x%08x sts = 0x%08x",
 		    EOREAD4(sc, EHCI_USBCMD), EOREAD4(sc, EHCI_USBSTS), 0, 0);
 #ifdef DIAGNOSTIC
 	if (error)
 		printf("ehci_sync_hc: cv_timedwait() = %d\n", error);
 #endif
+}
 Static void
 ehci_rem_free_itd_chain(ehci_softc_t *sc, struct ehci_xfer *exfer)
+{
 	struct ehci_soft_itd *itd, *prev;
 	prev = NULL;
 	if (exfer->ex_itdstart == NULL || exfer->ex_itdend == NULL)
 		panic("ehci isoc xfer being freed, but with no itd chain\n");
 	for (itd = exfer->ex_itdstart; itd != NULL; itd = itd->xfer_next) {
 		prev = itd->u.frame_list.prev;
 		/* Unlink itd from hardware chain, or frame array */
 		if (prev == NULL) { /* We're at the table head */
 			sc->sc_softitds[itd->slot] = itd->u.frame_list.next;
 			sc->sc_flist[itd->slot] = itd->itd.itd_next;
 			usb_syncmem(&sc->sc_fldma,
 			    sizeof(ehci_link_t) * itd->slot,
 			    sizeof(ehci_link_t),
 			    BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
 			if (itd->u.frame_list.next != NULL)
 				itd->u.frame_list.next->u.frame_list.prev = NULL;
 		} else {
 			/* XXX this part is untested... */
 			prev->itd.itd_next = itd->itd.itd_next;
 			usb_syncmem(&itd->dma,
 			    itd->offs + offsetof(ehci_itd_t, itd_next),
 			    sizeof(itd->itd.itd_next), BUS_DMASYNC_PREWRITE);
 			prev->u.frame_list.next = itd->u.frame_list.next;
 			if (itd->u.frame_list.next != NULL)
 				itd->u.frame_list.next->u.frame_list.prev = prev;
+		}
+	}
 	prev = NULL;
 	for (itd = exfer->ex_itdstart; itd != NULL; itd = itd->xfer_next) {
 		if (prev != NULL)
 			ehci_free_itd(sc, prev);
 		prev = itd;
+	}
 	if (prev)
 		ehci_free_itd(sc, prev);
 	exfer->ex_itdstart = NULL;
 	exfer->ex_itdend = NULL;
+}
 Static void
 ehci_rem_free_sitd_chain(ehci_softc_t *sc, struct ehci_xfer *exfer)
+{
 	struct ehci_soft_sitd *sitd, *prev;
 	prev = NULL;
 	if (exfer->ex_sitdstart == NULL || exfer->ex_sitdend == NULL)
 		panic("ehci isoc xfer being freed, but with no sitd chain\n");
 	for (sitd = exfer->ex_sitdstart; sitd != NULL; sitd = sitd->xfer_next) {
 		prev = sitd->u.frame_list.prev;
 		/* Unlink sitd from hardware chain, or frame array */
 		if (prev == NULL) { /* We're at the table head */
 			sc->sc_softsitds[sitd->slot] = sitd->u.frame_list.next;
 			sc->sc_flist[sitd->slot] = sitd->sitd.sitd_next;
 			usb_syncmem(&sc->sc_fldma,
 			    sizeof(ehci_link_t) * sitd->slot,
 			    sizeof(ehci_link_t),
 			    BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
 			if (sitd->u.frame_list.next != NULL)
 				sitd->u.frame_list.next->u.frame_list.prev = NULL;
 		} else {
 			/* XXX this part is untested... */
 			prev->sitd.sitd_next = sitd->sitd.sitd_next;
 			usb_syncmem(&sitd->dma,
 			    sitd->offs + offsetof(ehci_sitd_t, sitd_next),
 			    sizeof(sitd->sitd.sitd_next), BUS_DMASYNC_PREWRITE);
 			prev->u.frame_list.next = sitd->u.frame_list.next;
 			if (sitd->u.frame_list.next != NULL)
 				sitd->u.frame_list.next->u.frame_list.prev = prev;
+		}
+	}
 	prev = NULL;
 	for (sitd = exfer->ex_sitdstart; sitd != NULL; sitd = sitd->xfer_next) {
 		if (prev != NULL)
 			ehci_free_sitd(sc, prev);
 		prev = sitd;
+	}
 	if (prev)
 		ehci_free_sitd(sc, prev);
 	exfer->ex_sitdstart = NULL;
 	exfer->ex_sitdend = NULL;
+}
 /***********/
 Static int
 ehci_roothub_ctrl(struct usbd_bus *bus, usb_device_request_t *req,
     void *buf, int buflen)
+{
 	ehci_softc_t *sc = bus->ub_hcpriv;
 	usb_hub_descriptor_t hubd;
 	usb_port_status_t ps;
 	uint16_t len, value, index;
 	int l, totlen = 0;
 	int port, i;
 	uint32_t v;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	if (sc->sc_dying)
 		return -1;
 	USBHIST_LOG(ehcidebug, "type=0x%02x request=%02x",
 		    req->bmRequestType, req->bRequest, 0, 0);
 	len = UGETW(req->wLength);
 	value = UGETW(req->wValue);
 	index = UGETW(req->wIndex);
 #define C(x,y) ((x) | ((y) << 8))
 	switch (C(req->bRequest, req->bmRequestType)) {
 	case C(UR_GET_DESCRIPTOR, UT_READ_DEVICE):
 		if (len == 0)
 			break;
 		switch (value) {
 		case C(0, UDESC_DEVICE): {
 			usb_device_descriptor_t devd;
 			totlen = min(buflen, sizeof(devd));
 			memcpy(&devd, buf, totlen);
 			USETW(devd.idVendor, sc->sc_id_vendor);
 			memcpy(buf, &devd, totlen);
 			break;
+		}
 #define sd ((usb_string_descriptor_t *)buf)
 		case C(1, UDESC_STRING):
 			/* Vendor */
 			totlen = usb_makestrdesc(sd, len, sc->sc_vendor);
 			break;
 		case C(2, UDESC_STRING):
 			/* Product */
 			totlen = usb_makestrdesc(sd, len, "EHCI root hub");
 			break;
 #undef sd
 		default:
 			/* default from usbroothub */
 			return buflen;
+		}
 		break;
 	/* Hub requests */
 	case C(UR_CLEAR_FEATURE, UT_WRITE_CLASS_DEVICE):
 		break;
 	case C(UR_CLEAR_FEATURE, UT_WRITE_CLASS_OTHER):
 		USBHIST_LOG(ehcidebug,
 		    "UR_CLEAR_PORT_FEATURE port=%d feature=%d", index, value,
 , 0);
 		if (index < 1 || index > sc->sc_noport) {
 			return -1;
+		}
 		port = EHCI_PORTSC(index);
 		v = EOREAD4(sc, port);
 		USBHIST_LOG(ehcidebug, "portsc=0x%08x", v, 0, 0, 0);
 		v &= ~EHCI_PS_CLEAR;
 		switch (value) {
 		case UHF_PORT_ENABLE:
 			EOWRITE4(sc, port, v &~ EHCI_PS_PE);
 			break;
 		case UHF_PORT_SUSPEND:
 			if (!(v & EHCI_PS_SUSP)) /* not suspended */
 				break;
 			v &= ~EHCI_PS_SUSP;
 			EOWRITE4(sc, port, v | EHCI_PS_FPR);
 			/* see USB2 spec ch. 7.1.7.7 */
 			usb_delay_ms(&sc->sc_bus, 20);
 			EOWRITE4(sc, port, v);
 			usb_delay_ms(&sc->sc_bus, 2);
 #ifdef DEBUG
 			v = EOREAD4(sc, port);
 			if (v & (EHCI_PS_FPR | EHCI_PS_SUSP))
 				printf("ehci: resume failed: %x\n", v);
 #endif
 			break;
 		case UHF_PORT_POWER:
 			if (sc->sc_hasppc)
 				EOWRITE4(sc, port, v &~ EHCI_PS_PP);
 			break;
 		case UHF_PORT_TEST:
 			USBHIST_LOG(ehcidebug, "clear port test "
 				    "%d", index, 0, 0, 0);
 			break;
 		case UHF_PORT_INDICATOR:
 			USBHIST_LOG(ehcidebug, "clear port ind "
 				    "%d", index, 0, 0, 0);
 			EOWRITE4(sc, port, v &~ EHCI_PS_PIC);
 			break;
 		case UHF_C_PORT_CONNECTION:
 			EOWRITE4(sc, port, v | EHCI_PS_CSC);
 			break;
 		case UHF_C_PORT_ENABLE:
 			EOWRITE4(sc, port, v | EHCI_PS_PEC);
 			break;
 		case UHF_C_PORT_SUSPEND:
 			/* how? */
 			break;
 		case UHF_C_PORT_OVER_CURRENT:
 			EOWRITE4(sc, port, v | EHCI_PS_OCC);
 			break;
 		case UHF_C_PORT_RESET:
 			sc->sc_isreset[index] = 0;
 			break;
 		default:
 			return -1;
+		}
 #if 0
 		switch(value) {
 		case UHF_C_PORT_CONNECTION:
 		case UHF_C_PORT_ENABLE:
 		case UHF_C_PORT_SUSPEND:
 		case UHF_C_PORT_OVER_CURRENT:
 		case UHF_C_PORT_RESET:
 		default:
 			break;
+		}
 #endif
 		break;
 	case C(UR_GET_DESCRIPTOR, UT_READ_CLASS_DEVICE):
 		if (len == 0)
 			break;
 		if ((value & 0xff) != 0) {
 			return -1;
+		}
 		totlen = min(buflen, sizeof(hubd));
 		memcpy(&hubd, buf, totlen);
 		hubd.bNbrPorts = sc->sc_noport;
 		v = EOREAD4(sc, EHCI_HCSPARAMS);
 		USETW(hubd.wHubCharacteristics,
 		    EHCI_HCS_PPC(v) ? UHD_PWR_INDIVIDUAL : UHD_PWR_NO_SWITCH |
 		    EHCI_HCS_P_INDICATOR(EREAD4(sc, EHCI_HCSPARAMS))
 			? UHD_PORT_IND : 0);
 		hubd.bPwrOn2PwrGood = 200; /* XXX can't find out? */
 		for (i = 0, l = sc->sc_noport; l > 0; i++, l -= 8, v >>= 8)
 			hubd.DeviceRemovable[i++] = 0; /* XXX can't find out? */
 		hubd.bDescLength = USB_HUB_DESCRIPTOR_SIZE + i;
 		totlen = min(totlen, hubd.bDescLength);
 		memcpy(buf, &hubd, totlen);
 		break;
 	case C(UR_GET_STATUS, UT_READ_CLASS_DEVICE):
 		if (len != 4) {
 			return -1;
+		}
 		memset(buf, 0, len); /* ? XXX */
 		totlen = len;
 		break;
 	case C(UR_GET_STATUS, UT_READ_CLASS_OTHER):
 		USBHIST_LOG(ehcidebug, "get port status i=%d", index, 0, 0, 0);
 		if (index < 1 || index > sc->sc_noport) {
 			return -1;
+		}
 		if (len != 4) {
 			return -1;
+		}
 		v = EOREAD4(sc, EHCI_PORTSC(index));
 		USBHIST_LOG(ehcidebug, "port status=0x%04x", v, 0, 0, 0);
 		i = UPS_HIGH_SPEED;
 		if (sc->sc_flags & EHCIF_ETTF) {
 			/*
 			 * If we are doing embedded transaction translation,
 			 * then directly attached LS/FS devices are reset by
 			 * the EHCI controller itself.  PSPD is encoded
 			 * the same way as in USBSTATUS.
 			 */
 			i = __SHIFTOUT(v, EHCI_PS_PSPD) * UPS_LOW_SPEED;
+		}
 		if (v & EHCI_PS_CS)	i |= UPS_CURRENT_CONNECT_STATUS;
 		if (v & EHCI_PS_PE)	i |= UPS_PORT_ENABLED;
 		if (v & EHCI_PS_SUSP)	i |= UPS_SUSPEND;
 		if (v & EHCI_PS_OCA)	i |= UPS_OVERCURRENT_INDICATOR;
 		if (v & EHCI_PS_PR)	i |= UPS_RESET;
 		if (v & EHCI_PS_PP)	i |= UPS_PORT_POWER;
 		if (sc->sc_vendor_port_status)
 			i = sc->sc_vendor_port_status(sc, v, i);
 		USETW(ps.wPortStatus, i);
 		i = 0;
 		if (v & EHCI_PS_CSC)	i |= UPS_C_CONNECT_STATUS;
 		if (v & EHCI_PS_PEC)	i |= UPS_C_PORT_ENABLED;
 		if (v & EHCI_PS_OCC)	i |= UPS_C_OVERCURRENT_INDICATOR;
 		if (sc->sc_isreset[index]) i |= UPS_C_PORT_RESET;
 		USETW(ps.wPortChange, i);
 		totlen = min(len, sizeof(ps));
 		memcpy(buf, &ps, totlen);
 		break;
 	case C(UR_SET_DESCRIPTOR, UT_WRITE_CLASS_DEVICE):
 		return -1;
 	case C(UR_SET_FEATURE, UT_WRITE_CLASS_DEVICE):
 		break;
 	case C(UR_SET_FEATURE, UT_WRITE_CLASS_OTHER):
 		if (index < 1 || index > sc->sc_noport) {
 			return -1;
+		}
 		port = EHCI_PORTSC(index);
 		v = EOREAD4(sc, port);
 		USBHIST_LOG(ehcidebug, "portsc=0x%08x", v, 0, 0, 0);
 		v &= ~EHCI_PS_CLEAR;
 		switch(value) {
 		case UHF_PORT_ENABLE:
 			EOWRITE4(sc, port, v | EHCI_PS_PE);
 			break;
 		case UHF_PORT_SUSPEND:
 			EOWRITE4(sc, port, v | EHCI_PS_SUSP);
 			break;
 		case UHF_PORT_RESET:
 			USBHIST_LOG(ehcidebug, "reset port %d", index, 0, 0, 0);
 			if (EHCI_PS_IS_LOWSPEED(v)
 			    && sc->sc_ncomp > 0
 			    && !(sc->sc_flags & EHCIF_ETTF)) {
 				/*
 				 * Low speed device on non-ETTF controller or
 				 * unaccompanied controller, give up ownership.
 				 */
 				ehci_disown(sc, index, 1);
 				break;
+			}
 			/* Start reset sequence. */
 			v &= ~ (EHCI_PS_PE | EHCI_PS_PR);
 			EOWRITE4(sc, port, v | EHCI_PS_PR);
 			/* Wait for reset to complete. */
 			usb_delay_ms(&sc->sc_bus, USB_PORT_ROOT_RESET_DELAY);
 			if (sc->sc_dying) {
 				return -1;
+			}
 			/*
 			 * An embedded transaction translator will automatically
 			 * terminate the reset sequence so there's no need to
 			 * it.
 			 */
 			v = EOREAD4(sc, port);
 			if (v & EHCI_PS_PR) {
 				/* Terminate reset sequence. */
 				EOWRITE4(sc, port, v & ~EHCI_PS_PR);
 				/* Wait for HC to complete reset. */
 				usb_delay_ms(&sc->sc_bus,
 				    EHCI_PORT_RESET_COMPLETE);
 				if (sc->sc_dying) {
 					return -1;
+				}
+			}
 			v = EOREAD4(sc, port);
 			USBHIST_LOG(ehcidebug,
 			    "ehci after reset, status=0x%08x", v, 0, 0, 0);
 			if (v & EHCI_PS_PR) {
 				printf("%s: port reset timeout\n",
 				       device_xname(sc->sc_dev));
 				return USBD_TIMEOUT;
+			}
 			if (!(v & EHCI_PS_PE)) {
 				/* Not a high speed device, give up ownership.*/
 				ehci_disown(sc, index, 0);
 				break;
+			}
 			sc->sc_isreset[index] = 1;
 			USBHIST_LOG(ehcidebug,
 			    "ehci port %d reset, status = 0x%08x", index, v, 0,
 );
 			break;
 		case UHF_PORT_POWER:
 			USBHIST_LOG(ehcidebug,
 			    "set port power %d (has PPC = %d)", index,
 			    sc->sc_hasppc, 0, 0);
 			if (sc->sc_hasppc)
 				EOWRITE4(sc, port, v | EHCI_PS_PP);
 			break;
 		case UHF_PORT_TEST:
 			USBHIST_LOG(ehcidebug, "set port test %d",
 				index, 0, 0, 0);
 			break;
 		case UHF_PORT_INDICATOR:
 			USBHIST_LOG(ehcidebug, "set port ind %d",
 				index, 0, 0, 0);
 			EOWRITE4(sc, port, v | EHCI_PS_PIC);
 			break;
 		default:
 			return -1;
+		}
 		break;
 	case C(UR_CLEAR_TT_BUFFER, UT_WRITE_CLASS_OTHER):
 	case C(UR_RESET_TT, UT_WRITE_CLASS_OTHER):
 	case C(UR_GET_TT_STATE, UT_READ_CLASS_OTHER):
 	case C(UR_STOP_TT, UT_WRITE_CLASS_OTHER):
 		break;
 	default:
 		/* default from usbroothub */
 		USBHIST_LOG(ehcidebug, "returning %d (usbroothub default)",
 		    buflen, 0, 0, 0);
 		return buflen;
+	}
 	USBHIST_LOG(ehcidebug, "returning %d", totlen, 0, 0, 0);
 	return totlen;
+}
 Static void
 ehci_disown(ehci_softc_t *sc, int index, int lowspeed)
+{
 	int port;
 	uint32_t v;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	USBHIST_LOG(ehcidebug, "index=%d lowspeed=%d", index, lowspeed, 0, 0);
 #ifdef DIAGNOSTIC
 	if (sc->sc_npcomp != 0) {
 		int i = (index-1) / sc->sc_npcomp;
 		if (i >= sc->sc_ncomp)
 			printf("%s: strange port\n",
 			       device_xname(sc->sc_dev));
 		else
 			printf("%s: handing over %s speed device on "
 			       "port %d to %s\n",
 			       device_xname(sc->sc_dev),
 			       lowspeed ? "low" : "full",
 			       index, device_xname(sc->sc_comps[i]));
 	} else {
 		printf("%s: npcomp == 0\n", device_xname(sc->sc_dev));
+	}
 #endif
 	port = EHCI_PORTSC(index);
 	v = EOREAD4(sc, port) &~ EHCI_PS_CLEAR;
 	EOWRITE4(sc, port, v | EHCI_PS_PO);
+}
 Static usbd_status
 ehci_root_intr_transfer(usbd_xfer_handle xfer)
+{
 	ehci_softc_t *sc = xfer->ux_pipe->up_dev->ud_bus->ub_hcpriv;
 	usbd_status err;
 	/* Insert last in queue. */
 	mutex_enter(&sc->sc_lock);
 	err = usb_insert_transfer(xfer);
 	mutex_exit(&sc->sc_lock);
 	if (err)
 		return err;
 	/* Pipe isn't running, start first */
 	return ehci_root_intr_start(SIMPLEQ_FIRST(&xfer->ux_pipe->up_queue));
+}
 Static usbd_status
 ehci_root_intr_start(usbd_xfer_handle xfer)
+{
 	usbd_pipe_handle pipe = xfer->ux_pipe;
 	ehci_softc_t *sc = pipe->up_dev->ud_bus->ub_hcpriv;
 	if (sc->sc_dying)
 		return USBD_IOERROR;
 	mutex_enter(&sc->sc_lock);
 	sc->sc_intrxfer = xfer;
 	mutex_exit(&sc->sc_lock);
 	return USBD_IN_PROGRESS;
+}
 /* Abort a root interrupt request. */
 Static void
 ehci_root_intr_abort(usbd_xfer_handle xfer)
+{
 	ehci_softc_t *sc = xfer->ux_pipe->up_dev->ud_bus->ub_hcpriv;
 	KASSERT(mutex_owned(&sc->sc_lock));
 	KASSERT(xfer->ux_pipe->up_intrxfer == xfer);
 	sc->sc_intrxfer = NULL;
 	xfer->ux_status = USBD_CANCELLED;
 	usb_transfer_complete(xfer);
+}
 /* Close the root pipe. */
 Static void
 ehci_root_intr_close(usbd_pipe_handle pipe)
+{
 	ehci_softc_t *sc = pipe->up_dev->ud_bus->ub_hcpriv;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	KASSERT(mutex_owned(&sc->sc_lock));
 	sc->sc_intrxfer = NULL;
+}
 Static void
 ehci_root_intr_done(usbd_xfer_handle xfer)
+{
 	xfer->ux_hcpriv = NULL;
+}
 /************************/
 Static ehci_soft_qh_t *
 ehci_alloc_sqh(ehci_softc_t *sc)
+{
 	ehci_soft_qh_t *sqh;
 	usbd_status err;
 	int i, offs;
 	usb_dma_t dma;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	if (sc->sc_freeqhs == NULL) {
 		USBHIST_LOG(ehcidebug, "allocating chunk", 0, 0, 0, 0);
 		err = usb_allocmem(&sc->sc_bus, EHCI_SQH_SIZE * EHCI_SQH_CHUNK,
 			  EHCI_PAGE_SIZE, &dma);
 #ifdef EHCI_DEBUG
 		if (err)
 			printf("ehci_alloc_sqh: usb_allocmem()=%d\n", err);
 #endif
 		if (err)
 			return NULL;
 		for (i = 0; i < EHCI_SQH_CHUNK; i++) {
 			offs = i * EHCI_SQH_SIZE;
 			sqh = KERNADDR(&dma, offs);
 			sqh->physaddr = DMAADDR(&dma, offs);
 			sqh->dma = dma;
 			sqh->offs = offs;
 			sqh->next = sc->sc_freeqhs;
 			sc->sc_freeqhs = sqh;
+		}
+	}
 	sqh = sc->sc_freeqhs;
 	sc->sc_freeqhs = sqh->next;
 	memset(&sqh->qh, 0, sizeof(ehci_qh_t));
 	sqh->next = NULL;
 	return sqh;
+}
 Static void
 ehci_free_sqh(ehci_softc_t *sc, ehci_soft_qh_t *sqh)
+{
 	sqh->next = sc->sc_freeqhs;
 	sc->sc_freeqhs = sqh;
+}
 Static ehci_soft_qtd_t *
 ehci_alloc_sqtd(ehci_softc_t *sc)
+{
 	ehci_soft_qtd_t *sqtd = NULL;
 	usbd_status err;
 	int i, offs;
 	usb_dma_t dma;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	if (sc->sc_freeqtds == NULL) {
 		USBHIST_LOG(ehcidebug, "allocating chunk", 0, 0, 0, 0);
 		err = usb_allocmem(&sc->sc_bus, EHCI_SQTD_SIZE*EHCI_SQTD_CHUNK,
 			  EHCI_PAGE_SIZE, &dma);
 #ifdef EHCI_DEBUG
 		if (err)
 			printf("ehci_alloc_sqtd: usb_allocmem()=%d\n", err);
 #endif
 		if (err)
 			goto done;
 		for (i = 0; i < EHCI_SQTD_CHUNK; i++) {
 			offs = i * EHCI_SQTD_SIZE;
 			sqtd = KERNADDR(&dma, offs);
 			sqtd->physaddr = DMAADDR(&dma, offs);
 			sqtd->dma = dma;
 			sqtd->offs = offs;
 			sqtd->nextqtd = sc->sc_freeqtds;
 			sc->sc_freeqtds = sqtd;
+		}
+	}
 	sqtd = sc->sc_freeqtds;
 	sc->sc_freeqtds = sqtd->nextqtd;
 	memset(&sqtd->qtd, 0, sizeof(ehci_qtd_t));
 	sqtd->nextqtd = NULL;
 	sqtd->xfer = NULL;
 done:
 	return sqtd;
+}
 Static void
 ehci_free_sqtd(ehci_softc_t *sc, ehci_soft_qtd_t *sqtd)
+{
 	KASSERT(sc->sc_bus.ub_usepolling || mutex_owned(&sc->sc_lock));
 	sqtd->nextqtd = sc->sc_freeqtds;
 	sc->sc_freeqtds = sqtd;
+}
 Static usbd_status
 ehci_alloc_sqtd_chain(struct ehci_pipe *epipe, ehci_softc_t *sc,
 		     int alen, int rd, usbd_xfer_handle xfer,
 		     ehci_soft_qtd_t **sp, ehci_soft_qtd_t **ep)
+{
 	ehci_soft_qtd_t *next, *cur;
 	ehci_physaddr_t nextphys;
 	uint32_t qtdstatus;
 	int len, curlen, mps;
 	int i, tog;
 	int pages, pageoffs;
 	bus_size_t curoffs;
 	vaddr_t va, va_offs;
 	usb_dma_t *dma = &xfer->ux_dmabuf;
 	uint16_t flags = xfer->ux_flags;
 	paddr_t a;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	USBHIST_LOG(ehcidebug, "start len=%d", alen, 0, 0, 0);
 	len = alen;
 	qtdstatus = EHCI_QTD_ACTIVE |
 	    EHCI_QTD_SET_PID(rd ? EHCI_QTD_PID_IN : EHCI_QTD_PID_OUT) |
 	    EHCI_QTD_SET_CERR(3)
 	    /* IOC set below */
 	    /* BYTES set below */
+	    ;
 	mps = UGETW(epipe->pipe.up_endpoint->ue_edesc->wMaxPacketSize);
 	tog = epipe->nexttoggle;
 	qtdstatus |= EHCI_QTD_SET_TOGGLE(tog);
 	cur = ehci_alloc_sqtd(sc);
 	*sp = cur;
 	if (cur == NULL)
 		goto nomem;
 	usb_syncmem(dma, 0, alen,
 	    rd ? BUS_DMASYNC_PREREAD : BUS_DMASYNC_PREWRITE);
 	curoffs = 0;
 	for (;;) {
 		/* The EHCI hardware can handle at most 5 pages. */
 		va_offs = (vaddr_t)KERNADDR(dma, curoffs);
 		va_offs = EHCI_PAGE_OFFSET(va_offs);
 		if (len-curoffs < EHCI_QTD_MAXTRANSFER - va_offs) {
 			/* we can handle it in this QTD */
 			curlen = len - curoffs;
 		} else {
 			/* must use multiple TDs, fill as much as possible. */
 			curlen = EHCI_QTD_MAXTRANSFER - va_offs;
 			/* the length must be a multiple of the max size */
 			curlen -= curlen % mps;
 			USBHIST_LOG(ehcidebug, "multiple QTDs, "
 				    "curlen=%d", curlen, 0, 0, 0);
 			KASSERT(curlen != 0);
+		}
 		USBHIST_LOG(ehcidebug, "len=%d curlen=%d curoffs=%zu",
 			len, curlen, (size_t)curoffs, 0);
 		/*
 		 * Allocate another transfer if there's more data left,
 		 * or if force last short transfer flag is set and we're
 		 * allocating a multiple of the max packet size.
 		 */
 		if (curoffs + curlen != len ||
 		    ((curlen % mps) == 0 && !rd && curlen != 0 &&
 		     (flags & USBD_FORCE_SHORT_XFER))) {
 			next = ehci_alloc_sqtd(sc);
 			if (next == NULL)
 				goto nomem;
 			nextphys = htole32(next->physaddr);
 		} else {
 			next = NULL;
 			nextphys = EHCI_NULL;
+		}
 		/* Find number of pages we'll be using, insert dma addresses */
 		pages = EHCI_PAGE(curlen + EHCI_PAGE_SIZE - 1) >> 12;
 		KASSERT(pages <= EHCI_QTD_NBUFFERS);
 		pageoffs = EHCI_PAGE(curoffs);
 		for (i = 0; i < pages; i++) {
 			a = DMAADDR(dma, pageoffs + i * EHCI_PAGE_SIZE);
 			cur->qtd.qtd_buffer[i] = htole32(a & 0xFFFFF000);
 			/* Cast up to avoid compiler warnings */
 			cur->qtd.qtd_buffer_hi[i] = htole32((uint64_t)a >> 32);
+		}
 		/* First buffer pointer requires a page offset to start at */
 		va = (vaddr_t)KERNADDR(dma, curoffs);
 		cur->qtd.qtd_buffer[0] |= htole32(EHCI_PAGE_OFFSET(va));
 		cur->nextqtd = next;
 		cur->qtd.qtd_next = cur->qtd.qtd_altnext = nextphys;
 		cur->qtd.qtd_status =
 		    htole32(qtdstatus | EHCI_QTD_SET_BYTES(curlen));
 		cur->xfer = xfer;
 		cur->len = curlen;
 		USBHIST_LOG(ehcidebug, "cbp=0x%08zx end=0x%08zx",
 		    (size_t)curoffs, (size_t)(curoffs + curlen), 0, 0);
 		/*
 		 * adjust the toggle based on the number of packets in this
 		 * qtd
 		 */
 		if (((curlen + mps - 1) / mps) & 1) {
 			tog ^= 1;
 			qtdstatus ^= EHCI_QTD_TOGGLE_MASK;
+		}
 		if (next == NULL)
 			break;
 		usb_syncmem(&cur->dma, cur->offs, sizeof(cur->qtd),
 		    BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
 		USBHIST_LOG(ehcidebug, "extend chain", 0, 0, 0, 0);
 		if (len)
 			curoffs += curlen;
 		cur = next;
+	}
 	cur->qtd.qtd_status |= htole32(EHCI_QTD_IOC);
 	usb_syncmem(&cur->dma, cur->offs, sizeof(cur->qtd),
 	    BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
 	*ep = cur;
 	epipe->nexttoggle = tog;
 	USBHIST_LOG(ehcidebug, "return sqtd=%p sqtdend=%p",
 	    *sp, *ep, 0, 0);
 	return USBD_NORMAL_COMPLETION;
  nomem:
 	/* XXX free chain */
 	USBHIST_LOG(ehcidebug, "no memory", 0, 0, 0, 0);
 	return USBD_NOMEM;
+}
 Static void
 ehci_free_sqtd_chain(ehci_softc_t *sc, ehci_soft_qtd_t *sqtd,
 		    ehci_soft_qtd_t *sqtdend)
+{
 	ehci_soft_qtd_t *p;
 	int i;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	USBHIST_LOG(ehcidebug, "sqtd=%p sqtdend=%p",
 	    sqtd, sqtdend, 0, 0);
 	for (i = 0; sqtd != sqtdend; sqtd = p, i++) {
 		p = sqtd->nextqtd;
 		ehci_free_sqtd(sc, sqtd);
+	}
+}
 Static ehci_soft_itd_t *
 ehci_alloc_itd(ehci_softc_t *sc)
+{
 	struct ehci_soft_itd *itd, *freeitd;
 	usbd_status err;
 	int i, offs, frindex, previndex;
 	usb_dma_t dma;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	mutex_enter(&sc->sc_lock);
 	/*
 	 * Find an itd that wasn't freed this frame or last frame. This can
 	 * discard itds that were freed before frindex wrapped around
 	 * XXX - can this lead to thrashing? Could fix by enabling wrap-around
 	 *       interrupt and fiddling with list when that happens
 	 */
 	frindex = (EOREAD4(sc, EHCI_FRINDEX) + 1) >> 3;
 	previndex = (frindex != 0) ? frindex - 1 : sc->sc_flsize;
 	freeitd = NULL;
 	LIST_FOREACH(itd, &sc->sc_freeitds, u.free_list) {
 		if (itd == NULL)
 			break;
 		if (itd->slot != frindex && itd->slot != previndex) {
 			freeitd = itd;
 			break;
+		}
+	}
 	if (freeitd == NULL) {
 		USBHIST_LOG(ehcidebug, "allocating chunk", 0, 0, 0, 0);
 		err = usb_allocmem(&sc->sc_bus, EHCI_ITD_SIZE * EHCI_ITD_CHUNK,
 				EHCI_PAGE_SIZE, &dma);
 		if (err) {
 			USBHIST_LOG(ehcidebug,
 			    "alloc returned %d", err, 0, 0, 0);
 			mutex_exit(&sc->sc_lock);
 			return NULL;
+		}
 		for (i = 0; i < EHCI_ITD_CHUNK; i++) {
 			offs = i * EHCI_ITD_SIZE;
 			itd = KERNADDR(&dma, offs);
 			itd->physaddr = DMAADDR(&dma, offs);
 	 		itd->dma = dma;
 			itd->offs = offs;
 			LIST_INSERT_HEAD(&sc->sc_freeitds, itd, u.free_list);
+		}
 		freeitd = LIST_FIRST(&sc->sc_freeitds);
+	}
 	itd = freeitd;
 	LIST_REMOVE(itd, u.free_list);
 	memset(&itd->itd, 0, sizeof(ehci_itd_t));
 	usb_syncmem(&itd->dma, itd->offs + offsetof(ehci_itd_t, itd_next),
 	    sizeof(itd->itd.itd_next),
 	    BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
 	itd->u.frame_list.next = NULL;
 	itd->u.frame_list.prev = NULL;
 	itd->xfer_next = NULL;
 	itd->slot = 0;
 	mutex_exit(&sc->sc_lock);
 	return itd;
+}
 Static ehci_soft_sitd_t *
 ehci_alloc_sitd(ehci_softc_t *sc)
+{
 	struct ehci_soft_sitd *sitd, *freesitd;
 	usbd_status err;
 	int i, offs, frindex, previndex;
 	usb_dma_t dma;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	mutex_enter(&sc->sc_lock);
 	/*
 	 * Find an sitd that wasn't freed this frame or last frame. This can
 	 * discard sitds that were freed before frindex wrapped around
 	 * XXX - can this lead to thrashing? Could fix by enabling wrap-around
 	 *       interrupt and fiddling with list when that happens
 	 */
 	frindex = (EOREAD4(sc, EHCI_FRINDEX) + 1) >> 3;
 	previndex = (frindex != 0) ? frindex - 1 : sc->sc_flsize;
 	freesitd = NULL;
 	LIST_FOREACH(sitd, &sc->sc_freesitds, u.free_list) {
 		if (sitd == NULL)
 			break;
 		if (sitd->slot != frindex && sitd->slot != previndex) {
 			freesitd = sitd;
 			break;
+		}
+	}
 	if (freesitd == NULL) {
 		USBHIST_LOG(ehcidebug, "allocating chunk", 0, 0, 0, 0);
 		err = usb_allocmem(&sc->sc_bus, EHCI_SITD_SIZE * EHCI_SITD_CHUNK,
 				EHCI_PAGE_SIZE, &dma);
 		if (err) {
 			USBHIST_LOG(ehcidebug,
 			    "alloc returned %d", err, 0, 0, 0);
 			mutex_exit(&sc->sc_lock);
 			return NULL;
+		}
 		for (i = 0; i < EHCI_SITD_CHUNK; i++) {
 			offs = i * EHCI_SITD_SIZE;
 			sitd = KERNADDR(&dma, offs);
 			sitd->physaddr = DMAADDR(&dma, offs);
 	 		sitd->dma = dma;
 			sitd->offs = offs;
 			LIST_INSERT_HEAD(&sc->sc_freesitds, sitd, u.free_list);
+		}
 		freesitd = LIST_FIRST(&sc->sc_freesitds);
+	}
 	sitd = freesitd;
 	LIST_REMOVE(sitd, u.free_list);
 	memset(&sitd->sitd, 0, sizeof(ehci_sitd_t));
 	usb_syncmem(&sitd->dma, sitd->offs + offsetof(ehci_sitd_t, sitd_next),
 		    sizeof(sitd->sitd.sitd_next), BUS_DMASYNC_PREWRITE |
 		    BUS_DMASYNC_PREREAD);
 	sitd->u.frame_list.next = NULL;
 	sitd->u.frame_list.prev = NULL;
 	sitd->xfer_next = NULL;
 	sitd->slot = 0;
 	mutex_exit(&sc->sc_lock);
 	return sitd;
+}
 Static void
 ehci_free_itd(ehci_softc_t *sc, ehci_soft_itd_t *itd)
+{
 	KASSERT(mutex_owned(&sc->sc_lock));
 	LIST_INSERT_HEAD(&sc->sc_freeitds, itd, u.free_list);
+}
 Static void
 ehci_free_sitd(ehci_softc_t *sc, ehci_soft_sitd_t *sitd)
+{
 	KASSERT(mutex_owned(&sc->sc_lock));
 	LIST_INSERT_HEAD(&sc->sc_freesitds, sitd, u.free_list);
+}
 /****************/
 /*
  * Close a reqular pipe.
  * Assumes that there are no pending transactions.
  */
 Static void
 ehci_close_pipe(usbd_pipe_handle pipe, ehci_soft_qh_t *head)
+{
 	struct ehci_pipe *epipe = (struct ehci_pipe *)pipe;
 	ehci_softc_t *sc = pipe->up_dev->ud_bus->ub_hcpriv;
 	ehci_soft_qh_t *sqh = epipe->sqh;
 	KASSERT(mutex_owned(&sc->sc_lock));
 	ehci_rem_qh(sc, sqh, head);
 	ehci_free_sqh(sc, epipe->sqh);
+}
 /*
  * Abort a device request.
  * If this routine is called at splusb() it guarantees that the request
  * will be removed from the hardware scheduling and that the callback
  * for it will be called with USBD_CANCELLED status.
  * It's impossible to guarantee that the requested transfer will not
  * have happened since the hardware runs concurrently.
  * If the transaction has already happened we rely on the ordinary
  * interrupt processing to process it.
  * XXX This is most probably wrong.
  * XXXMRG this doesn't make sense anymore.
  */
 Static void
 ehci_abort_xfer(usbd_xfer_handle xfer, usbd_status status)
+{
 #define exfer EXFER(xfer)
 	struct ehci_pipe *epipe = (struct ehci_pipe *)xfer->ux_pipe;
 	ehci_softc_t *sc = epipe->pipe.up_dev->ud_bus->ub_hcpriv;
 	ehci_soft_qh_t *sqh = epipe->sqh;
 	ehci_soft_qtd_t *sqtd;
 	ehci_physaddr_t cur;
 	uint32_t qhstatus;
 	int hit;
 	int wake;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	USBHIST_LOG(ehcidebug, "xfer=%p pipe=%p", xfer, epipe, 0, 0);
 	KASSERT(mutex_owned(&sc->sc_lock));
 	ASSERT_SLEEPABLE();
 	if (sc->sc_dying) {
 		/* If we're dying, just do the software part. */
 		xfer->ux_status = status;	/* make software ignore it */
 		callout_stop(&xfer->ux_callout);
 		usb_transfer_complete(xfer);
 		return;
+	}
 	/*
 	 * If an abort is already in progress then just wait for it to
 	 * complete and return.
 	 */
 	if (xfer->ux_hcflags & UXFER_ABORTING) {
 		USBHIST_LOG(ehcidebug, "already aborting", 0, 0, 0, 0);
 #ifdef DIAGNOSTIC
 		if (status == USBD_TIMEOUT)
 			printf("ehci_abort_xfer: TIMEOUT while aborting\n");
 #endif
 		/* Override the status which might be USBD_TIMEOUT. */
 		xfer->ux_status = status;
 		USBHIST_LOG(ehcidebug, "waiting for abort to finish",
 , 0, 0, 0);
 		xfer->ux_hcflags |= UXFER_ABORTWAIT;
 		while (xfer->ux_hcflags & UXFER_ABORTING)
 			cv_wait(&xfer->ux_hccv, &sc->sc_lock);
 		return;
+	}
 	xfer->ux_hcflags |= UXFER_ABORTING;
 	/*
 	 * Step 1: Make interrupt routine and hardware ignore xfer.
 	 */
 	xfer->ux_status = status;	/* make software ignore it */
 	callout_stop(&xfer->ux_callout);
 	usb_syncmem(&sqh->dma,
 	    sqh->offs + offsetof(ehci_qh_t, qh_qtd.qtd_status),
 	    sizeof(sqh->qh.qh_qtd.qtd_status),
 	    BUS_DMASYNC_POSTWRITE | BUS_DMASYNC_POSTREAD);
 	qhstatus = sqh->qh.qh_qtd.qtd_status;
 	sqh->qh.qh_qtd.qtd_status = qhstatus | htole32(EHCI_QTD_HALTED);
 	usb_syncmem(&sqh->dma,
 	    sqh->offs + offsetof(ehci_qh_t, qh_qtd.qtd_status),
 	    sizeof(sqh->qh.qh_qtd.qtd_status),
 	    BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
 	for (sqtd = exfer->ex_sqtdstart; ; sqtd = sqtd->nextqtd) {
 		usb_syncmem(&sqtd->dma,
 		    sqtd->offs + offsetof(ehci_qtd_t, qtd_status),
 		    sizeof(sqtd->qtd.qtd_status),
 		    BUS_DMASYNC_POSTWRITE | BUS_DMASYNC_POSTREAD);
 		sqtd->qtd.qtd_status |= htole32(EHCI_QTD_HALTED);
 		usb_syncmem(&sqtd->dma,
 		    sqtd->offs + offsetof(ehci_qtd_t, qtd_status),
 		    sizeof(sqtd->qtd.qtd_status),
 		    BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
 		if (sqtd == exfer->ex_sqtdend)
 			break;
+	}
 	/*
 	 * Step 2: Wait until we know hardware has finished any possible
 	 * use of the xfer.  Also make sure the soft interrupt routine
 	 * has run.
 	 */
 	ehci_sync_hc(sc);
 	sc->sc_softwake = 1;
 	usb_schedsoftintr(&sc->sc_bus);
 	cv_wait(&sc->sc_softwake_cv, &sc->sc_lock);
 	/*
 	 * Step 3: Remove any vestiges of the xfer from the hardware.
 	 * The complication here is that the hardware may have executed
 	 * beyond the xfer we're trying to abort.  So as we're scanning
 	 * the TDs of this xfer we check if the hardware points to
 	 * any of them.
 	 */
 	usb_syncmem(&sqh->dma,
 	    sqh->offs + offsetof(ehci_qh_t, qh_curqtd),
 	    sizeof(sqh->qh.qh_curqtd),
 	    BUS_DMASYNC_POSTWRITE | BUS_DMASYNC_POSTREAD);
 	cur = EHCI_LINK_ADDR(le32toh(sqh->qh.qh_curqtd));
 	hit = 0;
 	for (sqtd = exfer->ex_sqtdstart; ; sqtd = sqtd->nextqtd) {
 		hit |= cur == sqtd->physaddr;
 		if (sqtd == exfer->ex_sqtdend)
 			break;
+	}
 	sqtd = sqtd->nextqtd;
 	/* Zap curqtd register if hardware pointed inside the xfer. */
 	if (hit && sqtd != NULL) {
 		USBHIST_LOG(ehcidebug, "cur=0x%08x", sqtd->physaddr, 0, 0, 0);
 		sqh->qh.qh_curqtd = htole32(sqtd->physaddr); /* unlink qTDs */
 		usb_syncmem(&sqh->dma,
 		    sqh->offs + offsetof(ehci_qh_t, qh_curqtd),
 		    sizeof(sqh->qh.qh_curqtd),
 		    BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
 		sqh->qh.qh_qtd.qtd_status = qhstatus;
 		usb_syncmem(&sqh->dma,
 		    sqh->offs + offsetof(ehci_qh_t, qh_qtd.qtd_status),
 		    sizeof(sqh->qh.qh_qtd.qtd_status),
 		    BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
 	} else {
 		USBHIST_LOG(ehcidebug, "no hit", 0, 0, 0, 0);
+	}
 	/*
 	 * Step 4: Execute callback.
 	 */
 #ifdef DIAGNOSTIC
-	exfer->ex_isdone = 1;
+	exfer->ex_isdone = true;
 #endif
 	wake = xfer->ux_hcflags & UXFER_ABORTWAIT;
 	xfer->ux_hcflags &= ~(UXFER_ABORTING | UXFER_ABORTWAIT);
 	usb_transfer_complete(xfer);
 	if (wake) {
 		cv_broadcast(&xfer->ux_hccv);
+	}
 	KASSERT(mutex_owned(&sc->sc_lock));
 #undef exfer
+}
 Static void
 ehci_abort_isoc_xfer(usbd_xfer_handle xfer, usbd_status status)
+{
 	ehci_isoc_trans_t trans_status;
 	struct ehci_pipe *epipe;
 	struct ehci_xfer *exfer;
 	ehci_softc_t *sc;
 	struct ehci_soft_itd *itd;
 	struct ehci_soft_sitd *sitd;
 	int i, wake;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	epipe = (struct ehci_pipe *) xfer->ux_pipe;
 	exfer = EXFER(xfer);
 	sc = epipe->pipe.up_dev->ud_bus->ub_hcpriv;
 	USBHIST_LOG(ehcidebug, "xfer %p pipe %p", xfer, epipe, 0, 0);
 	KASSERT(mutex_owned(&sc->sc_lock));
 	if (sc->sc_dying) {
 		xfer->ux_status = status;
 		callout_stop(&xfer->ux_callout);
 		usb_transfer_complete(xfer);
 		return;
+	}
 	if (xfer->ux_hcflags & UXFER_ABORTING) {
 		USBHIST_LOG(ehcidebug, "already aborting", 0, 0, 0, 0);
 #ifdef DIAGNOSTIC
 		if (status == USBD_TIMEOUT)
 			printf("ehci_abort_isoc_xfer: TIMEOUT while aborting\n");
 #endif
 		xfer->ux_status = status;
 		USBHIST_LOG(ehcidebug,
 		    "waiting for abort to finish", 0, 0, 0, 0);
 		xfer->ux_hcflags |= UXFER_ABORTWAIT;
 		while (xfer->ux_hcflags & UXFER_ABORTING)
 			cv_wait(&xfer->ux_hccv, &sc->sc_lock);
 		goto done;
+	}
 	xfer->ux_hcflags |= UXFER_ABORTING;
 	xfer->ux_status = status;
 	callout_stop(&xfer->ux_callout);
 	if (xfer->ux_pipe->up_dev->ud_speed == USB_SPEED_HIGH) {
 		for (itd = exfer->ex_itdstart; itd != NULL;
 		     itd = itd->xfer_next) {
 			usb_syncmem(&itd->dma,
 			    itd->offs + offsetof(ehci_itd_t, itd_ctl),
 			    sizeof(itd->itd.itd_ctl),
 			    BUS_DMASYNC_POSTWRITE | BUS_DMASYNC_POSTREAD);
 			for (i = 0; i < 8; i++) {
 				trans_status = le32toh(itd->itd.itd_ctl[i]);
 				trans_status &= ~EHCI_ITD_ACTIVE;
 				itd->itd.itd_ctl[i] = htole32(trans_status);
+			}
 			usb_syncmem(&itd->dma,
 			    itd->offs + offsetof(ehci_itd_t, itd_ctl),
 			    sizeof(itd->itd.itd_ctl),
 			    BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
+		}
 	} else {
 		for (sitd = exfer->ex_sitdstart; sitd != NULL;
 		     sitd = sitd->xfer_next) {
 			usb_syncmem(&sitd->dma,
 			    sitd->offs + offsetof(ehci_sitd_t, sitd_buffer),
 			    sizeof(sitd->sitd.sitd_buffer),
 			    BUS_DMASYNC_POSTWRITE | BUS_DMASYNC_POSTREAD);
 			trans_status = le32toh(sitd->sitd.sitd_trans);
 			trans_status &= ~EHCI_SITD_ACTIVE;
 			sitd->sitd.sitd_trans = htole32(trans_status);
 			usb_syncmem(&sitd->dma,
 			    sitd->offs + offsetof(ehci_sitd_t, sitd_buffer),
 			    sizeof(sitd->sitd.sitd_buffer),
 			    BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
+		}
+	}
 	sc->sc_softwake = 1;
 	usb_schedsoftintr(&sc->sc_bus);
 	cv_wait(&sc->sc_softwake_cv, &sc->sc_lock);
 #ifdef DIAGNOSTIC
-	exfer->ex_isdone = 1;
+	exfer->ex_isdone = true;
 #endif
 	wake = xfer->ux_hcflags & UXFER_ABORTWAIT;
 	xfer->ux_hcflags &= ~(UXFER_ABORTING | UXFER_ABORTWAIT);
 	usb_transfer_complete(xfer);
 	if (wake) {
 		cv_broadcast(&xfer->ux_hccv);
+	}
 done:
 	KASSERT(mutex_owned(&sc->sc_lock));
 	return;
+}
 Static void
 ehci_timeout(void *addr)
+{
 	struct ehci_xfer *exfer = addr;
 	struct ehci_pipe *epipe = (struct ehci_pipe *)exfer->ex_xfer.ux_pipe;
 	ehci_softc_t *sc = epipe->pipe.up_dev->ud_bus->ub_hcpriv;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	USBHIST_LOG(ehcidebug, "exfer %p", exfer, 0, 0, 0);
 #ifdef EHCI_DEBUG
 	if (ehcidebug > 1)
 		usbd_dump_pipe(exfer->ex_xfer.ux_pipe);
 #endif
 	if (sc->sc_dying) {
 		mutex_enter(&sc->sc_lock);
 		ehci_abort_xfer(&exfer->ex_xfer, USBD_TIMEOUT);
 		mutex_exit(&sc->sc_lock);
 		return;
+	}
 	/* Execute the abort in a process context. */
 	usb_init_task(&exfer->ex_aborttask, ehci_timeout_task, addr,
 	    USB_TASKQ_MPSAFE);
 	usb_add_task(exfer->ex_xfer.ux_pipe->up_dev, &exfer->ex_aborttask,
 	    USB_TASKQ_HC);
+}
 Static void
 ehci_timeout_task(void *addr)
+{
 	usbd_xfer_handle xfer = addr;
 	ehci_softc_t *sc = xfer->ux_pipe->up_dev->ud_bus->ub_hcpriv;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	USBHIST_LOG(ehcidebug, "xfer=%p", xfer, 0, 0, 0);
 	mutex_enter(&sc->sc_lock);
 	ehci_abort_xfer(xfer, USBD_TIMEOUT);
 	mutex_exit(&sc->sc_lock);
+}
 /************************/
 Static usbd_status
 ehci_device_ctrl_transfer(usbd_xfer_handle xfer)
+{
 	ehci_softc_t *sc = xfer->ux_pipe->up_dev->ud_bus->ub_hcpriv;
 	usbd_status err;
 	/* Insert last in queue. */
 	mutex_enter(&sc->sc_lock);
 	err = usb_insert_transfer(xfer);
 	mutex_exit(&sc->sc_lock);
 	if (err)
 		return err;
 	/* Pipe isn't running, start first */
 	return ehci_device_ctrl_start(SIMPLEQ_FIRST(&xfer->ux_pipe->up_queue));
+}
 Static usbd_status
 ehci_device_ctrl_start(usbd_xfer_handle xfer)
+{
 	ehci_softc_t *sc = xfer->ux_pipe->up_dev->ud_bus->ub_hcpriv;
 	usbd_status err;
 	if (sc->sc_dying)
 		return USBD_IOERROR;
 	KASSERT(xfer->ux_rqflags & URQ_REQUEST);
 	err = ehci_device_request(xfer);
 	if (err) {
 		return err;
+	}
 	if (sc->sc_bus.ub_usepolling)
 		ehci_waitintr(sc, xfer);
 	return USBD_IN_PROGRESS;
+}
 Static void
 ehci_device_ctrl_done(usbd_xfer_handle xfer)
+{
 	struct ehci_xfer *ex = EXFER(xfer);
 	ehci_softc_t *sc = xfer->ux_pipe->up_dev->ud_bus->ub_hcpriv;
 	struct ehci_pipe *epipe = (struct ehci_pipe *)xfer->ux_pipe;
 	usb_device_request_t *req = &xfer->ux_request;
 	int len = UGETW(req->wLength);
 	int rd = req->bmRequestType & UT_READ;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	USBHIST_LOG(ehcidebug, "xfer=%p", xfer, 0, 0, 0);
 	KASSERT(sc->sc_bus.ub_usepolling || mutex_owned(&sc->sc_lock));
 	KASSERT(xfer->ux_rqflags & URQ_REQUEST);
 	if (xfer->ux_status != USBD_NOMEM && ehci_active_intr_list(ex)) {
 		ehci_del_intr_list(sc, ex);	/* remove from active list */
 		ehci_free_sqtd_chain(sc, ex->ex_sqtdstart, NULL);
 		usb_syncmem(&epipe->u.ctl.reqdma, 0, sizeof(*req),
 		    BUS_DMASYNC_POSTWRITE);
 		if (len)
 			usb_syncmem(&xfer->ux_dmabuf, 0, len,
 			    rd ? BUS_DMASYNC_POSTREAD : BUS_DMASYNC_POSTWRITE);
+	}
 	USBHIST_LOG(ehcidebug, "length=%d", xfer->ux_actlen, 0, 0, 0);
+}
 /* Abort a device control request. */
 Static void
 ehci_device_ctrl_abort(usbd_xfer_handle xfer)
+{
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	USBHIST_LOG(ehcidebug, "xfer=%p", xfer, 0, 0, 0);
 	ehci_abort_xfer(xfer, USBD_CANCELLED);
+}
 /* Close a device control pipe. */
 Static void
 ehci_device_ctrl_close(usbd_pipe_handle pipe)
+{
 	ehci_softc_t *sc = pipe->up_dev->ud_bus->ub_hcpriv;
 	/*struct ehci_pipe *epipe = (struct ehci_pipe *)pipe;*/
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	KASSERT(mutex_owned(&sc->sc_lock));
 	USBHIST_LOG(ehcidebug, "pipe=%p", pipe, 0, 0, 0);
 	ehci_close_pipe(pipe, sc->sc_async_head);
+}
 Static usbd_status
 ehci_device_request(usbd_xfer_handle xfer)
+{
 #define exfer EXFER(xfer)
 	struct ehci_pipe *epipe = (struct ehci_pipe *)xfer->ux_pipe;
 	usb_device_request_t *req = &xfer->ux_request;
 	usbd_device_handle dev = epipe->pipe.up_dev;
 	ehci_softc_t *sc = dev->ud_bus->ub_hcpriv;
 	ehci_soft_qtd_t *setup, *stat, *next;
 	ehci_soft_qh_t *sqh;
 	int isread;
 	int len;
 	usbd_status err;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	isread = req->bmRequestType & UT_READ;
 	len = UGETW(req->wLength);
 	USBHIST_LOG(ehcidebug, "type=0x%02x, request=0x%02x, "
 	    "wValue=0x%04x, wIndex=0x%04x",
 	    req->bmRequestType, req->bRequest, UGETW(req->wValue),
 	    UGETW(req->wIndex));
 	USBHIST_LOG(ehcidebug, "len=%d, addr=%d, endpt=%d",
 	    len, dev->ud_addr,
 	    epipe->pipe.up_endpoint->ue_edesc->bEndpointAddress, 0);
 	setup = ehci_alloc_sqtd(sc);
 	if (setup == NULL) {
 		err = USBD_NOMEM;
 		goto bad1;
+	}
 	stat = ehci_alloc_sqtd(sc);
 	if (stat == NULL) {
 		err = USBD_NOMEM;
 		goto bad2;
+	}
 	mutex_enter(&sc->sc_lock);
 	sqh = epipe->sqh;
 	KASSERTMSG(EHCI_QH_GET_ADDR(le32toh(sqh->qh.qh_endp)) == dev->ud_addr,
 	    "address QH %d pipe %d\n",
 	    EHCI_QH_GET_ADDR(le32toh(sqh->qh.qh_endp)), dev->ud_addr);
 	KASSERTMSG(EHCI_QH_GET_MPL(le32toh(sqh->qh.qh_endp)) ==
 	    UGETW(epipe->pipe.up_endpoint->ue_edesc->wMaxPacketSize),
 	    "MPS QH %d pipe %d\n",
 	    EHCI_QH_GET_MPL(le32toh(sqh->qh.qh_endp)),
 	    UGETW(epipe->pipe.up_endpoint->ue_edesc->wMaxPacketSize));
 	/* Set up data transaction */
 	if (len != 0) {
 		ehci_soft_qtd_t *end;
 		/* Start toggle at 1. */
 		epipe->nexttoggle = 1;
 		err = ehci_alloc_sqtd_chain(epipe, sc, len, isread, xfer,
 			  &next, &end);
 		if (err)
 			goto bad3;
 		end->qtd.qtd_status &= htole32(~EHCI_QTD_IOC);
 		end->nextqtd = stat;
 		end->qtd.qtd_next = end->qtd.qtd_altnext =
 		    htole32(stat->physaddr);
 		usb_syncmem(&end->dma, end->offs, sizeof(end->qtd),
 		   BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
 	} else {
 		next = stat;
+	}
 	memcpy(KERNADDR(&epipe->u.ctl.reqdma, 0), req, sizeof(*req));
 	usb_syncmem(&epipe->u.ctl.reqdma, 0, sizeof(*req), BUS_DMASYNC_PREWRITE);
 	/* Clear toggle */
 	setup->qtd.qtd_status = htole32(
 	    EHCI_QTD_ACTIVE |
 	    EHCI_QTD_SET_PID(EHCI_QTD_PID_SETUP) |
 	    EHCI_QTD_SET_CERR(3) |
 	    EHCI_QTD_SET_TOGGLE(0) |
 	    EHCI_QTD_SET_BYTES(sizeof(*req))
 	    );
 	setup->qtd.qtd_buffer[0] = htole32(DMAADDR(&epipe->u.ctl.reqdma, 0));
 	setup->qtd.qtd_buffer_hi[0] = 0;
 	setup->nextqtd = next;
 	setup->qtd.qtd_next = setup->qtd.qtd_altnext = htole32(next->physaddr);
 	setup->xfer = xfer;
 	setup->len = sizeof(*req);
 	usb_syncmem(&setup->dma, setup->offs, sizeof(setup->qtd),
 	    BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
 	stat->qtd.qtd_status = htole32(
 	    EHCI_QTD_ACTIVE |
 	    EHCI_QTD_SET_PID(isread ? EHCI_QTD_PID_OUT : EHCI_QTD_PID_IN) |
 	    EHCI_QTD_SET_CERR(3) |
 	    EHCI_QTD_SET_TOGGLE(1) |
 	    EHCI_QTD_IOC
 	    );
 	stat->qtd.qtd_buffer[0] = 0; /* XXX not needed? */
 	stat->qtd.qtd_buffer_hi[0] = 0; /* XXX not needed? */
 	stat->nextqtd = NULL;
 	stat->qtd.qtd_next = stat->qtd.qtd_altnext = EHCI_NULL;
 	stat->xfer = xfer;
 	stat->len = 0;
 	usb_syncmem(&stat->dma, stat->offs, sizeof(stat->qtd),
 	    BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
 #ifdef EHCI_DEBUG
 	USBHIST_LOGN(ehcidebug, 5, "dump:", 0, 0, 0, 0);
 	ehci_dump_sqh(sqh);
 	ehci_dump_sqtds(setup);
 #endif
 	exfer->ex_sqtdstart = setup;
 	exfer->ex_sqtdend = stat;
 	KASSERT(exfer->ex_isdone);
 #ifdef DIAGNOSTIC
-	if (!exfer->ex_isdone) {
+	exfer->ex_isdone = false;
 		printf("ehci_device_request: not done, exfer=%p\n", exfer);
 	exfer->ex_isdone = 0;
 #endif
 	/* Insert qTD in QH list. */
 	ehci_set_qh_qtd(sqh, setup); /* also does usb_syncmem(sqh) */
 	if (xfer->ux_timeout && !sc->sc_bus.ub_usepolling) {
 		callout_reset(&xfer->ux_callout, mstohz(xfer->ux_timeout),
 		    ehci_timeout, xfer);
+	}
 	ehci_add_intr_list(sc, exfer);
 	xfer->ux_status = USBD_IN_PROGRESS;
 	mutex_exit(&sc->sc_lock);
 #ifdef EHCI_DEBUG
 	USBHIST_LOGN(ehcidebug, 10, "status=%x, dump:",
 	    EOREAD4(sc, EHCI_USBSTS), 0, 0, 0);
 //	delay(10000);
 	ehci_dump_regs(sc);
 	ehci_dump_sqh(sc->sc_async_head);
 	ehci_dump_sqh(sqh);
 	ehci_dump_sqtds(setup);
 #endif
 	return USBD_NORMAL_COMPLETION;
  bad3:
 	mutex_exit(&sc->sc_lock);
 	ehci_free_sqtd(sc, stat);
  bad2:
 	ehci_free_sqtd(sc, setup);
  bad1:
 	USBHIST_LOG(ehcidebug, "no memory", 0, 0, 0, 0);
 	mutex_enter(&sc->sc_lock);
 	xfer->ux_status = err;
 	usb_transfer_complete(xfer);
 	mutex_exit(&sc->sc_lock);
 	return err;
 #undef exfer
+}
 /*
  * Some EHCI chips from VIA seem to trigger interrupts before writing back the
  * qTD status, or miss signalling occasionally under heavy load.  If the host
  * machine is too fast, we we can miss transaction completion - when we scan
  * the active list the transaction still seems to be active.  This generally
  * exhibits itself as a umass stall that never recovers.
+ *
  * We work around this behaviour by setting up this callback after any softintr
  * that completes with transactions still pending, giving us another chance to
  * check for completion after the writeback has taken place.
  */
 Static void
 ehci_intrlist_timeout(void *arg)
+{
 	ehci_softc_t *sc = arg;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	usb_schedsoftintr(&sc->sc_bus);
+}
 /************************/
 Static usbd_status
 ehci_device_bulk_transfer(usbd_xfer_handle xfer)
+{
 	ehci_softc_t *sc = xfer->ux_pipe->up_dev->ud_bus->ub_hcpriv;
 	usbd_status err;
 	/* Insert last in queue. */
 	mutex_enter(&sc->sc_lock);
 	err = usb_insert_transfer(xfer);
 	mutex_exit(&sc->sc_lock);
 	if (err)
 		return err;
 	/* Pipe isn't running, start first */
 	return ehci_device_bulk_start(SIMPLEQ_FIRST(&xfer->ux_pipe->up_queue));
+}
 Static usbd_status
 ehci_device_bulk_start(usbd_xfer_handle xfer)
+{
 #define exfer EXFER(xfer)
 	struct ehci_pipe *epipe = (struct ehci_pipe *)xfer->ux_pipe;
 	usbd_device_handle dev = epipe->pipe.up_dev;
 	ehci_softc_t *sc = dev->ud_bus->ub_hcpriv;
 	ehci_soft_qtd_t *data, *dataend;
 	ehci_soft_qh_t *sqh;
 	usbd_status err;
 	int len, isread, endpt;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	USBHIST_LOG(ehcidebug, "xfer=%p len=%d flags=%d",
 	    xfer, xfer->ux_length, xfer->ux_flags, 0);
 	if (sc->sc_dying)
 		return USBD_IOERROR;
 	KASSERT(!(xfer->ux_rqflags & URQ_REQUEST));
 	mutex_enter(&sc->sc_lock);
 	len = xfer->ux_length;
 	endpt = epipe->pipe.up_endpoint->ue_edesc->bEndpointAddress;
 	isread = UE_GET_DIR(endpt) == UE_DIR_IN;
 	sqh = epipe->sqh;
 	epipe->u.bulk.length = len;
 	err = ehci_alloc_sqtd_chain(epipe, sc, len, isread, xfer, &data,
 				   &dataend);
 	if (err) {
 		USBHIST_LOG(ehcidebug, "no memory", 0, 0, 0, 0);
 		xfer->ux_status = err;
 		usb_transfer_complete(xfer);
 		mutex_exit(&sc->sc_lock);
 		return err;
+	}
 #ifdef EHCI_DEBUG
 	USBHIST_LOGN(ehcidebug, 5, "--- dump start ---", 0, 0, 0, 0);
 	ehci_dump_sqh(sqh);
 	ehci_dump_sqtds(data);
 	USBHIST_LOGN(ehcidebug, 5, "--- dump end ---", 0, 0, 0, 0);
 #endif
 	/* Set up interrupt info. */
 	exfer->ex_sqtdstart = data;
 	exfer->ex_sqtdend = dataend;
 	KASSERT(exfer->ex_isdone);
 #ifdef DIAGNOSTIC
-	if (!exfer->ex_isdone) {
+	exfer->ex_isdone = false;
 		printf("ehci_device_bulk_start: not done, ex=%p\n", exfer);
 	exfer->ex_isdone = 0;
 #endif
 	ehci_set_qh_qtd(sqh, data); /* also does usb_syncmem(sqh) */
 	if (xfer->ux_timeout && !sc->sc_bus.ub_usepolling) {
 		callout_reset(&xfer->ux_callout, mstohz(xfer->ux_timeout),
 		    ehci_timeout, xfer);
+	}
 	ehci_add_intr_list(sc, exfer);
 	xfer->ux_status = USBD_IN_PROGRESS;
 	mutex_exit(&sc->sc_lock);
 #ifdef EHCI_DEBUG
 	USBHIST_LOGN(ehcidebug, 5, "data(2)", 0, 0, 0, 0);
 //	delay(10000);
 	USBHIST_LOGN(ehcidebug, 5, "data(3)", 0, 0, 0, 0);
 	ehci_dump_regs(sc);
 #if 0
 	printf("async_head:\n");
 	ehci_dump_sqh(sc->sc_async_head);
 #endif
 	USBHIST_LOG(ehcidebug, "sqh:", 0, 0, 0, 0);
 	ehci_dump_sqh(sqh);
 	ehci_dump_sqtds(data);
 #endif
 	if (sc->sc_bus.ub_usepolling)
 		ehci_waitintr(sc, xfer);
 	return USBD_IN_PROGRESS;
 #undef exfer
+}
 Static void
 ehci_device_bulk_abort(usbd_xfer_handle xfer)
+{
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	USBHIST_LOG(ehcidebug, "xfer %p", xfer, 0, 0, 0);
 	ehci_abort_xfer(xfer, USBD_CANCELLED);
+}
 /*
  * Close a device bulk pipe.
  */
 Static void
 ehci_device_bulk_close(usbd_pipe_handle pipe)
+{
 	ehci_softc_t *sc = pipe->up_dev->ud_bus->ub_hcpriv;
 	struct ehci_pipe *epipe = (struct ehci_pipe *)pipe;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	KASSERT(mutex_owned(&sc->sc_lock));
 	USBHIST_LOG(ehcidebug, "pipe=%p", pipe, 0, 0, 0);
 	pipe->up_endpoint->ue_toggle = epipe->nexttoggle;
 	ehci_close_pipe(pipe, sc->sc_async_head);
+}
 Static void
 ehci_device_bulk_done(usbd_xfer_handle xfer)
+{
 	struct ehci_xfer *ex = EXFER(xfer);
 	ehci_softc_t *sc = xfer->ux_pipe->up_dev->ud_bus->ub_hcpriv;
 	struct ehci_pipe *epipe = (struct ehci_pipe *)xfer->ux_pipe;
 	int endpt = epipe->pipe.up_endpoint->ue_edesc->bEndpointAddress;
 	int rd = UE_GET_DIR(endpt) == UE_DIR_IN;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	USBHIST_LOG(ehcidebug, "xfer=%p, actlen=%d",
 	    xfer, xfer->ux_actlen, 0, 0);
 	KASSERT(mutex_owned(&sc->sc_lock));
 	if (xfer->ux_status != USBD_NOMEM && ehci_active_intr_list(ex)) {
 		ehci_del_intr_list(sc, ex);	/* remove from active list */
 		ehci_free_sqtd_chain(sc, ex->ex_sqtdstart, NULL);
 		usb_syncmem(&xfer->ux_dmabuf, 0, xfer->ux_length,
 		    rd ? BUS_DMASYNC_POSTREAD : BUS_DMASYNC_POSTWRITE);
+	}
 	USBHIST_LOG(ehcidebug, "length=%d", xfer->ux_actlen, 0, 0, 0);
+}
 /************************/
 Static usbd_status
 ehci_device_setintr(ehci_softc_t *sc, ehci_soft_qh_t *sqh, int ival)
+{
 	struct ehci_soft_islot *isp;
 	int islot, lev;
 	/* Find a poll rate that is large enough. */
 	for (lev = EHCI_IPOLLRATES - 1; lev > 0; lev--)
 		if (EHCI_ILEV_IVAL(lev) <= ival)
 			break;
 	/* Pick an interrupt slot at the right level. */
 	/* XXX could do better than picking at random */
 	sc->sc_rand = (sc->sc_rand + 191) % sc->sc_flsize;
 	islot = EHCI_IQHIDX(lev, sc->sc_rand);
 	sqh->islot = islot;
 	isp = &sc->sc_islots[islot];
 	mutex_enter(&sc->sc_lock);
 	ehci_add_qh(sc, sqh, isp->sqh);
 	mutex_exit(&sc->sc_lock);
 	return USBD_NORMAL_COMPLETION;
+}
 Static usbd_status
 ehci_device_intr_transfer(usbd_xfer_handle xfer)
+{
 	ehci_softc_t *sc = xfer->ux_pipe->up_dev->ud_bus->ub_hcpriv;
 	usbd_status err;
 	/* Insert last in queue. */
 	mutex_enter(&sc->sc_lock);
 	err = usb_insert_transfer(xfer);
 	mutex_exit(&sc->sc_lock);
 	if (err)
 		return err;
 	/*
 	 * Pipe isn't running (otherwise err would be USBD_INPROG),
 	 * so start it first.
 	 */
 	return ehci_device_intr_start(SIMPLEQ_FIRST(&xfer->ux_pipe->up_queue));
+}
 Static usbd_status
 ehci_device_intr_start(usbd_xfer_handle xfer)
+{
 #define exfer EXFER(xfer)
 	struct ehci_pipe *epipe = (struct ehci_pipe *)xfer->ux_pipe;
 	usbd_device_handle dev = xfer->ux_pipe->up_dev;
 	ehci_softc_t *sc = dev->ud_bus->ub_hcpriv;
 	ehci_soft_qtd_t *data, *dataend;
 	ehci_soft_qh_t *sqh;
 	usbd_status err;
 	int len, isread, endpt;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	USBHIST_LOG(ehcidebug, "xfer=%p len=%d flags=%d",
 	    xfer, xfer->ux_length, xfer->ux_flags, 0);
 	if (sc->sc_dying)
 		return USBD_IOERROR;
 	KASSERT(!(xfer->ux_rqflags & URQ_REQUEST));
 	mutex_enter(&sc->sc_lock);
 	len = xfer->ux_length;
 	endpt = epipe->pipe.up_endpoint->ue_edesc->bEndpointAddress;
 	isread = UE_GET_DIR(endpt) == UE_DIR_IN;
 	sqh = epipe->sqh;
 	epipe->u.intr.length = len;
 	err = ehci_alloc_sqtd_chain(epipe, sc, len, isread, xfer, &data,
 	    &dataend);
 	if (err) {
 		USBHIST_LOG(ehcidebug, "no memory", 0, 0, 0, 0);
 		xfer->ux_status = err;
 		usb_transfer_complete(xfer);
 		mutex_exit(&sc->sc_lock);
 		return err;
+	}
 #ifdef EHCI_DEBUG
 	USBHIST_LOGN(ehcidebug, 5, "--- dump start ---", 0, 0, 0, 0);
 	ehci_dump_sqh(sqh);
 	ehci_dump_sqtds(data);
 	USBHIST_LOGN(ehcidebug, 5, "--- dump end ---", 0, 0, 0, 0);
 #endif
 	/* Set up interrupt info. */
 	exfer->ex_sqtdstart = data;
 	exfer->ex_sqtdend = dataend;
 	KASSERT(exfer->ex_isdone);
 #ifdef DIAGNOSTIC
-	if (!exfer->ex_isdone) {
+	exfer->ex_isdone = false;
 		printf("ehci_device_intr_start: not done, ex=%p\n", exfer);
 	exfer->ex_isdone = 0;
 #endif
 	ehci_set_qh_qtd(sqh, data); /* also does usb_syncmem(sqh) */
 	if (xfer->ux_timeout && !sc->sc_bus.ub_usepolling) {
 		callout_reset(&xfer->ux_callout, mstohz(xfer->ux_timeout),
 		    ehci_timeout, xfer);
+	}
 	ehci_add_intr_list(sc, exfer);
 	xfer->ux_status = USBD_IN_PROGRESS;
 	mutex_exit(&sc->sc_lock);
 #ifdef EHCI_DEBUG
 	USBHIST_LOGN(ehcidebug, 5, "data(2)", 0, 0, 0, 0);
 //	delay(10000);
 	USBHIST_LOGN(ehcidebug, 5, "data(3)", 0, 0, 0, 0);
 	ehci_dump_regs(sc);
 	USBHIST_LOGN(ehcidebug, 5, "sqh:", 0, 0, 0, 0);
 	ehci_dump_sqh(sqh);
 	ehci_dump_sqtds(data);
 #endif
 	if (sc->sc_bus.ub_usepolling)
 		ehci_waitintr(sc, xfer);
 	return USBD_IN_PROGRESS;
 #undef exfer
+}
 Static void
 ehci_device_intr_abort(usbd_xfer_handle xfer)
+{
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	USBHIST_LOG(ehcidebug, "xfer=%p", xfer, 0, 0, 0);
 	KASSERT(xfer->ux_pipe->up_intrxfer == xfer);
 	/*
 	 * XXX - abort_xfer uses ehci_sync_hc, which syncs via the advance
 	 *       async doorbell. That's dependent on the async list, wheras
 	 *       intr xfers are periodic, should not use this?
 	 */
 	ehci_abort_xfer(xfer, USBD_CANCELLED);
+}
 Static void
 ehci_device_intr_close(usbd_pipe_handle pipe)
+{
 	ehci_softc_t *sc = pipe->up_dev->ud_bus->ub_hcpriv;
 	struct ehci_pipe *epipe = (struct ehci_pipe *)pipe;
 	struct ehci_soft_islot *isp;
 	KASSERT(mutex_owned(&sc->sc_lock));
 	isp = &sc->sc_islots[epipe->sqh->islot];
 	ehci_close_pipe(pipe, isp->sqh);
+}
 Static void
 ehci_device_intr_done(usbd_xfer_handle xfer)
+{
 #define exfer EXFER(xfer)
 	struct ehci_xfer *ex = EXFER(xfer);
 	ehci_softc_t *sc = xfer->ux_pipe->up_dev->ud_bus->ub_hcpriv;
 	struct ehci_pipe *epipe = (struct ehci_pipe *)xfer->ux_pipe;
 	ehci_soft_qtd_t *data, *dataend;
 	ehci_soft_qh_t *sqh;
 	usbd_status err;
 	int len, isread, endpt;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	USBHIST_LOG(ehcidebug, "xfer=%p, actlen=%d",
 	    xfer, xfer->ux_actlen, 0, 0);
 	KASSERT(sc->sc_bus.ub_usepolling || mutex_owned(&sc->sc_lock));
 	if (xfer->ux_pipe->up_repeat) {
 		ehci_free_sqtd_chain(sc, ex->ex_sqtdstart, NULL);
 		len = epipe->u.intr.length;
 		xfer->ux_length = len;
 		endpt = epipe->pipe.up_endpoint->ue_edesc->bEndpointAddress;
 		isread = UE_GET_DIR(endpt) == UE_DIR_IN;
 		usb_syncmem(&xfer->ux_dmabuf, 0, len,
 		    isread ? BUS_DMASYNC_POSTREAD : BUS_DMASYNC_POSTWRITE);
 		sqh = epipe->sqh;
 		err = ehci_alloc_sqtd_chain(epipe, sc, len, isread, xfer,
 		    &data, &dataend);
 		if (err) {
 			USBHIST_LOG(ehcidebug, "no memory", 0, 0, 0, 0);
 			xfer->ux_status = err;
 			return;
+		}
 		/* Set up interrupt info. */
 		exfer->ex_sqtdstart = data;
 		exfer->ex_sqtdend = dataend;
 		KASSERT(exfer->ex_isdone);
 #ifdef DIAGNOSTIC
-		if (!exfer->ex_isdone) {
+		exfer->ex_isdone = false;
 			USBHIST_LOG(ehcidebug, "marked not done, ex = %p",
 				exfer, 0, 0, 0);
 			printf("ehci_device_intr_done: not done, ex=%p\n",
 			    exfer);
 		exfer->ex_isdone = 0;
 #endif
 		ehci_set_qh_qtd(sqh, data); /* also does usb_syncmem(sqh) */
 		if (xfer->ux_timeout && !sc->sc_bus.ub_usepolling) {
 			callout_reset(&xfer->ux_callout,
 			    mstohz(xfer->ux_timeout), ehci_timeout, xfer);
+		}
 		xfer->ux_status = USBD_IN_PROGRESS;
 	} else if (xfer->ux_status != USBD_NOMEM && ehci_active_intr_list(ex)) {
 		ehci_del_intr_list(sc, ex); /* remove from active list */
 		ehci_free_sqtd_chain(sc, ex->ex_sqtdstart, NULL);
 		endpt = epipe->pipe.up_endpoint->ue_edesc->bEndpointAddress;
 		isread = UE_GET_DIR(endpt) == UE_DIR_IN;
 		usb_syncmem(&xfer->ux_dmabuf, 0, xfer->ux_length,
 		    isread ? BUS_DMASYNC_POSTREAD : BUS_DMASYNC_POSTWRITE);
+	}
 #undef exfer
+}
 /************************/
 Static usbd_status
 ehci_device_fs_isoc_transfer(usbd_xfer_handle xfer)
+{
 	usbd_status err;
 	err = usb_insert_transfer(xfer);
 	if (err && err != USBD_IN_PROGRESS)
 		return err;
 	return ehci_device_fs_isoc_start(xfer);
+}
 Static usbd_status
 ehci_device_fs_isoc_start(usbd_xfer_handle xfer)
+{
 	struct ehci_pipe *epipe;
 	usbd_device_handle dev;
 	ehci_softc_t *sc;
 	struct ehci_xfer *exfer;
 	ehci_soft_sitd_t *sitd, *prev, *start, *stop;
 	usb_dma_t *dma_buf;
 	int i, j, k, frames;
 	int offs, total_length;
 	int frindex;
 	u_int huba, dir;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	start = NULL;
 	prev = NULL;
 	sitd = NULL;
 	total_length = 0;
 	exfer = (struct ehci_xfer *) xfer;
 	sc = xfer->ux_pipe->up_dev->ud_bus->ub_hcpriv;
 	dev = xfer->ux_pipe->up_dev;
 	epipe = (struct ehci_pipe *)xfer->ux_pipe;
 	/*
 	 * To allow continuous transfers, above we start all transfers
 	 * immediately. However, we're still going to get usbd_start_next call
 	 * this when another xfer completes. So, check if this is already
 	 * in progress or not
 	 */
 	if (exfer->ex_sitdstart != NULL)
 		return USBD_IN_PROGRESS;
 	USBHIST_LOG(ehcidebug, "xfer %p len %d flags %d",
 	    xfer, xfer->ux_length, xfer->ux_flags, 0);
 	if (sc->sc_dying)
 		return USBD_IOERROR;
 	/*
 	 * To avoid complication, don't allow a request right now that'll span
 	 * the entire frame table. To within 4 frames, to allow some leeway
 	 * on either side of where the hc currently is.
 	 */
 	if (epipe->pipe.up_endpoint->ue_edesc->bInterval *
 			xfer->ux_nframes >= sc->sc_flsize - 4) {
 		printf("ehci: isoc descriptor requested that spans the entire"
 		    "frametable, too many frames\n");
 		return USBD_INVAL;
+	}
 	KASSERT(!(xfer->ux_rqflags & URQ_REQUEST));
 	KASSERT(exfer->ex_isdone);
 #ifdef DIAGNOSTIC
-	if (!exfer->ex_isdone)
+	exfer->ex_isdone = false;
 		printf("ehci_device_fs_isoc_start: not done, ex = %p\n", exfer);
 	exfer->ex_isdone = 0;
 #endif
 	/*
 	 * Step 1: Allocate and initialize sitds.
 	 */
 	i = epipe->pipe.up_endpoint->ue_edesc->bInterval;
 	if (i > 16 || i == 0) {
 		/* Spec page 271 says intervals > 16 are invalid */
 		USBHIST_LOG(ehcidebug, "bInterval %d invalid", 0, 0, 0, 0);
 		return USBD_INVAL;
+	}
 	frames = xfer->ux_nframes;
 	if (frames == 0) {
 		USBHIST_LOG(ehcidebug, "frames == 0", 0, 0, 0, 0);
 		return USBD_INVAL;
+	}
 	dma_buf = &xfer->ux_dmabuf;
 	offs = 0;
 	for (i = 0; i < frames; i++) {
 		sitd = ehci_alloc_sitd(sc);
 		if (prev)
 			prev->xfer_next = sitd;
 		else
 			start = sitd;
 #ifdef DIAGNOSTIC
 		if (xfer->ux_frlengths[i] > 0x3ff) {
 			printf("ehci: invalid frame length\n");
 			xfer->ux_frlengths[i] = 0x3ff;
+		}
 #endif
 		sitd->sitd.sitd_trans = htole32(EHCI_SITD_ACTIVE |
 		    EHCI_SITD_SET_LEN(xfer->ux_frlengths[i]));
 		/* Set page0 index and offset. */
 		sitd->sitd.sitd_buffer[0] = htole32(DMAADDR(dma_buf, offs));
 		total_length += xfer->ux_frlengths[i];
 		offs += xfer->ux_frlengths[i];
 		sitd->sitd.sitd_buffer[1] =
 		    htole32(EHCI_SITD_SET_BPTR(DMAADDR(dma_buf, offs - 1)));
 		huba = dev->ud_myhsport->up_parent->ud_addr;
 /*		if (sc->sc_flags & EHCIF_FREESCALE) {
 			// Set hub address to 0 if embedded TT is used.
 			if (huba == sc->sc_addr)
 				huba = 0;
+		}
 */
 		k = epipe->pipe.up_endpoint->ue_edesc->bEndpointAddress;
 		dir = UE_GET_DIR(k) ? 1 : 0;
 		sitd->sitd.sitd_endp =
 		    htole32(EHCI_SITD_SET_ENDPT(UE_GET_ADDR(k)) |
 		    EHCI_SITD_SET_DADDR(dev->ud_addr) |
 		    EHCI_SITD_SET_PORT(dev->ud_myhsport->up_portno) |
 		    EHCI_SITD_SET_HUBA(huba) |
 		    EHCI_SITD_SET_DIR(dir));
 		sitd->sitd.sitd_back = htole32(EHCI_LINK_TERMINATE);
 		/* XXX */
 		u_char sa, sb;
 		u_int temp, tlen;
 		sa = 0;
 		if (dir == 0) {	/* OUT */
 			temp = 0;
 			tlen = xfer->ux_frlengths[i];
 			if (tlen <= 188) {
 				temp |= 1;	/* T-count = 1, TP = ALL */
 				tlen = 1;
 			} else {
 				tlen += 187;
 				tlen /= 188;
 				temp |= tlen;	/* T-count = [1..6] */
 				temp |= 8;	/* TP = Begin */
+			}
 			sitd->sitd.sitd_buffer[1] |= htole32(temp);
 			tlen += sa;
 			if (tlen >= 8) {
 				sb = 0;
 			} else {
 				sb = (1 << tlen);
+			}
 			sa = (1 << sa);
 			sa = (sb - sa) & 0x3F;
 			sb = 0;
 		} else {
 			sb = (-(4 << sa)) & 0xFE;
 			sa = (1 << sa) & 0x3F;
 			sa = 0x01;
 			sb = 0xfc;
+		}
 		sitd->sitd.sitd_sched = htole32(EHCI_SITD_SET_SMASK(sa) |
 		    EHCI_SITD_SET_CMASK(sb));
 		prev = sitd;
 	} /* End of frame */
 	sitd->sitd.sitd_trans |= htole32(EHCI_SITD_IOC);
 	stop = sitd;
 	stop->xfer_next = NULL;
 	usb_syncmem(&exfer->ex_xfer.ux_dmabuf, 0, total_length,
 		BUS_DMASYNC_PREREAD | BUS_DMASYNC_PREWRITE);
 	/*
 	 * Part 2: Transfer descriptors have now been set up, now they must
 	 * be scheduled into the periodic frame list. Erk. Not wanting to
 	 * complicate matters, transfer is denied if the transfer spans
 	 * more than the period frame list.
 	 */
 	mutex_enter(&sc->sc_lock);
 	/* Start inserting frames */
 	if (epipe->u.isoc.cur_xfers > 0) {
 		frindex = epipe->u.isoc.next_frame;
 	} else {
 		frindex = EOREAD4(sc, EHCI_FRINDEX);
 		frindex = frindex >> 3; /* Erase microframe index */
 		frindex += 2;
+	}
 	if (frindex >= sc->sc_flsize)
 		frindex &= (sc->sc_flsize - 1);
 	/* Whats the frame interval? */
 	i = epipe->pipe.up_endpoint->ue_edesc->bInterval;
 	sitd = start;
 	for (j = 0; j < frames; j++) {
 		if (sitd == NULL)
 			panic("ehci: unexpectedly ran out of isoc sitds\n");
 		sitd->sitd.sitd_next = sc->sc_flist[frindex];
 		if (sitd->sitd.sitd_next == 0)
 			/* FIXME: frindex table gets initialized to NULL
 			 * or EHCI_NULL? */
 			sitd->sitd.sitd_next = EHCI_NULL;
 		usb_syncmem(&sitd->dma,
 		    sitd->offs + offsetof(ehci_sitd_t, sitd_next),
 		    sizeof(ehci_sitd_t),
 		    BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
 		sc->sc_flist[frindex] =
 		    htole32(EHCI_LINK_SITD | sitd->physaddr);
 		usb_syncmem(&sc->sc_fldma,
 		    sizeof(ehci_link_t) * frindex,
 		    sizeof(ehci_link_t),
 		    BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
 		sitd->u.frame_list.next = sc->sc_softsitds[frindex];
 		sc->sc_softsitds[frindex] = sitd;
 		if (sitd->u.frame_list.next != NULL)
 			sitd->u.frame_list.next->u.frame_list.prev = sitd;
 		sitd->slot = frindex;
 		sitd->u.frame_list.prev = NULL;
 		frindex += i;
 		if (frindex >= sc->sc_flsize)
 			frindex -= sc->sc_flsize;
 		sitd = sitd->xfer_next;
+	}
 	epipe->u.isoc.cur_xfers++;
 	epipe->u.isoc.next_frame = frindex;
 	exfer->ex_sitdstart = start;
 	exfer->ex_sitdend = stop;
 	ehci_add_intr_list(sc, exfer);
 	xfer->ux_status = USBD_IN_PROGRESS;
 	xfer->ux_done = 0;
 	mutex_exit(&sc->sc_lock);
 	if (sc->sc_bus.ub_usepolling) {
 		printf("Starting ehci isoc xfer with polling. Bad idea?\n");
 		ehci_waitintr(sc, xfer);
+	}
 	return USBD_IN_PROGRESS;
+}
 Static void
 ehci_device_fs_isoc_abort(usbd_xfer_handle xfer)
+{
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	USBHIST_LOG(ehcidebug, "xfer = %p", xfer, 0, 0, 0);
 	ehci_abort_isoc_xfer(xfer, USBD_CANCELLED);
+}
 Static void
 ehci_device_fs_isoc_close(usbd_pipe_handle pipe)
+{
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	USBHIST_LOG(ehcidebug, "nothing in the pipe to free?", 0, 0, 0, 0);
+}
 Static void
 ehci_device_fs_isoc_done(usbd_xfer_handle xfer)
+{
 	struct ehci_xfer *exfer;
 	ehci_softc_t *sc;
 	struct ehci_pipe *epipe;
 	exfer = EXFER(xfer);
 	sc = xfer->ux_pipe->up_dev->ud_bus->ub_hcpriv;
 	epipe = (struct ehci_pipe *) xfer->ux_pipe;
 	KASSERT(mutex_owned(&sc->sc_lock));
 	epipe->u.isoc.cur_xfers--;
 	if (xfer->ux_status != USBD_NOMEM && ehci_active_intr_list(exfer)) {
 		ehci_del_intr_list(sc, exfer);
 		ehci_rem_free_sitd_chain(sc, exfer);
+	}
 	usb_syncmem(&xfer->ux_dmabuf, 0, xfer->ux_length, BUS_DMASYNC_POSTWRITE |
 		    BUS_DMASYNC_POSTREAD);
+}
 Static usbd_status
 ehci_device_isoc_transfer(usbd_xfer_handle xfer)
+{
 	ehci_softc_t *sc = xfer->ux_pipe->up_dev->ud_bus->ub_hcpriv;
 	usbd_status err;
 	mutex_enter(&sc->sc_lock);
 	err = usb_insert_transfer(xfer);
 	mutex_exit(&sc->sc_lock);
 	if (err && err != USBD_IN_PROGRESS)
 		return err;
 	return ehci_device_isoc_start(xfer);
+}
 Static usbd_status
 ehci_device_isoc_start(usbd_xfer_handle xfer)
+{
 	struct ehci_pipe *epipe;
 	ehci_softc_t *sc;
 	struct ehci_xfer *exfer;
 	ehci_soft_itd_t *itd, *prev, *start, *stop;
 	usb_dma_t *dma_buf;
 	int i, j, k, frames, uframes, ufrperframe;
 	int trans_count, offs, total_length;
 	int frindex;
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	start = NULL;
 	prev = NULL;
 	itd = NULL;
 	trans_count = 0;
 	total_length = 0;
 	exfer = (struct ehci_xfer *) xfer;
 	sc = xfer->ux_pipe->up_dev->ud_bus->ub_hcpriv;
 	epipe = (struct ehci_pipe *)xfer->ux_pipe;
 	/*
 	 * To allow continuous transfers, above we start all transfers
 	 * immediately. However, we're still going to get usbd_start_next call
 	 * this when another xfer completes. So, check if this is already
 	 * in progress or not
 	 */
 	if (exfer->ex_itdstart != NULL)
 		return USBD_IN_PROGRESS;
 	USBHIST_LOG(ehcidebug, "xfer %p len %d flags %d",
 	    xfer, xfer->ux_length, xfer->ux_flags, 0);
 	if (sc->sc_dying)
 		return USBD_IOERROR;
 	/*
 	 * To avoid complication, don't allow a request right now that'll span
 	 * the entire frame table. To within 4 frames, to allow some leeway
 	 * on either side of where the hc currently is.
 	 */
 	if ((1 << (epipe->pipe.up_endpoint->ue_edesc->bInterval)) *
 			xfer->ux_nframes >= (sc->sc_flsize - 4) * 8) {
 		USBHIST_LOG(ehcidebug,
 		    "isoc descriptor spans entire frametable", 0, 0, 0, 0);
 		printf("ehci: isoc descriptor requested that spans the entire frametable, too many frames\n");
 		return USBD_INVAL;
+	}
 	KASSERT(!(xfer->ux_rqflags & URQ_REQUEST));
 	KASSERT(exfer->ex_isdone);
 #ifdef DIAGNOSTIC
-	if (!exfer->ex_isdone) {
+	exfer->ex_isdone = false;
 		USBHIST_LOG(ehcidebug, "marked not done, ex = %p", exfer,
 , 0, 0);
 		printf("ehci_device_isoc_start: not done, ex = %p\n", exfer);
 	exfer->ex_isdone = 0;
 #endif
 	/*
 	 * Step 1: Allocate and initialize itds, how many do we need?
 	 * One per transfer if interval >= 8 microframes, fewer if we use
 	 * multiple microframes per frame.
 	 */
 	i = epipe->pipe.up_endpoint->ue_edesc->bInterval;
 	if (i > 16 || i == 0) {
 		/* Spec page 271 says intervals > 16 are invalid */
 		USBHIST_LOG(ehcidebug, "bInterval %d invalid", i, 0, 0, 0);
 		return USBD_INVAL;
+	}
 	ufrperframe = max(1, USB_UFRAMES_PER_FRAME / (1 << (i - 1)));
 	frames = (xfer->ux_nframes + (ufrperframe - 1)) / ufrperframe;
 	uframes = USB_UFRAMES_PER_FRAME / ufrperframe;
 	if (frames == 0) {
 		USBHIST_LOG(ehcidebug, "frames == 0", 0, 0, 0, 0);
 		return USBD_INVAL;
+	}
 	dma_buf = &xfer->ux_dmabuf;
 	offs = 0;
 	for (i = 0; i < frames; i++) {
 		int froffs = offs;
 		itd = ehci_alloc_itd(sc);
 		if (prev != NULL) {
 			prev->itd.itd_next =
 			    htole32(itd->physaddr | EHCI_LINK_ITD);
 			usb_syncmem(&itd->dma,
 			    itd->offs + offsetof(ehci_itd_t, itd_next),
 			    sizeof(itd->itd.itd_next), BUS_DMASYNC_POSTWRITE);
 			prev->xfer_next = itd;
 	    	} else {
 			start = itd;
+		}
 		/*
 		 * Step 1.5, initialize uframes
 		 */
 		for (j = 0; j < EHCI_ITD_NUFRAMES; j += uframes) {
 			/* Calculate which page in the list this starts in */
 			int addr = DMAADDR(dma_buf, froffs);
 			addr = EHCI_PAGE_OFFSET(addr);
 			addr += (offs - froffs);
 			addr = EHCI_PAGE(addr);
 			addr /= EHCI_PAGE_SIZE;
 			/*
 			 * This gets the initial offset into the first page,
 			 * looks how far further along the current uframe
 			 * offset is. Works out how many pages that is.
 			 */
 			itd->itd.itd_ctl[j] = htole32 ( EHCI_ITD_ACTIVE |
 			    EHCI_ITD_SET_LEN(xfer->ux_frlengths[trans_count]) |
 			    EHCI_ITD_SET_PG(addr) |
 			    EHCI_ITD_SET_OFFS(EHCI_PAGE_OFFSET(DMAADDR(dma_buf,offs))));
 			total_length += xfer->ux_frlengths[trans_count];
 			offs += xfer->ux_frlengths[trans_count];
 			trans_count++;
 			if (trans_count >= xfer->ux_nframes) { /*Set IOC*/
 				itd->itd.itd_ctl[j] |= htole32(EHCI_ITD_IOC);
 				break;
+			}
+		}
 		/*
 		 * Step 1.75, set buffer pointers. To simplify matters, all
 		 * pointers are filled out for the next 7 hardware pages in
 		 * the dma block, so no need to worry what pages to cover
 		 * and what to not.
 		 */
 		for (j = 0; j < EHCI_ITD_NBUFFERS; j++) {
 			/*
 			 * Don't try to lookup a page that's past the end
 			 * of buffer
 			 */
 			int page_offs = EHCI_PAGE(froffs + (EHCI_PAGE_SIZE * j));
 			if (page_offs >= dma_buf->udma_block->size)
 				break;
 			unsigned long long page = DMAADDR(dma_buf, page_offs);
 			page = EHCI_PAGE(page);
 			itd->itd.itd_bufr[j] =
 			    htole32(EHCI_ITD_SET_BPTR(page));
 			itd->itd.itd_bufr_hi[j] =
 			    htole32(page >> 32);
+		}
 		/*
 		 * Other special values
 		 */
 		k = epipe->pipe.up_endpoint->ue_edesc->bEndpointAddress;
 		itd->itd.itd_bufr[0] |= htole32(EHCI_ITD_SET_EP(UE_GET_ADDR(k)) |
 		    EHCI_ITD_SET_DADDR(epipe->pipe.up_dev->ud_addr));
 		k = (UE_GET_DIR(epipe->pipe.up_endpoint->ue_edesc->bEndpointAddress))
 		    ? 1 : 0;
 		j = UGETW(epipe->pipe.up_endpoint->ue_edesc->wMaxPacketSize);
 		itd->itd.itd_bufr[1] |= htole32(EHCI_ITD_SET_DIR(k) |
 		    EHCI_ITD_SET_MAXPKT(UE_GET_SIZE(j)));
 		/* FIXME: handle invalid trans */
 		itd->itd.itd_bufr[2] |=
 		    htole32(EHCI_ITD_SET_MULTI(UE_GET_TRANS(j)+1));
 		usb_syncmem(&itd->dma,
 		    itd->offs + offsetof(ehci_itd_t, itd_next),
 		    sizeof(ehci_itd_t),
 		    BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
 		prev = itd;
 	} /* End of frame */
 	stop = itd;
 	stop->xfer_next = NULL;
 	usb_syncmem(&exfer->ex_xfer.ux_dmabuf, 0, total_length,
 		BUS_DMASYNC_PREREAD | BUS_DMASYNC_PREWRITE);
 	/*
 	 * Part 2: Transfer descriptors have now been set up, now they must
 	 * be scheduled into the period frame list. Erk. Not wanting to
 	 * complicate matters, transfer is denied if the transfer spans
 	 * more than the period frame list.
 	 */
 	mutex_enter(&sc->sc_lock);
 	/* Start inserting frames */
 	if (epipe->u.isoc.cur_xfers > 0) {
 		frindex = epipe->u.isoc.next_frame;
 	} else {
 		frindex = EOREAD4(sc, EHCI_FRINDEX);
 		frindex = frindex >> 3; /* Erase microframe index */
 		frindex += 2;
+	}
 	if (frindex >= sc->sc_flsize)
 		frindex &= (sc->sc_flsize - 1);
 	/* What's the frame interval? */
 	i = (1 << (epipe->pipe.up_endpoint->ue_edesc->bInterval - 1));
 	if (i / USB_UFRAMES_PER_FRAME == 0)
 		i = 1;
 	else
 		i /= USB_UFRAMES_PER_FRAME;
 	itd = start;
 	for (j = 0; j < frames; j++) {
 		if (itd == NULL)
 			panic("ehci: unexpectedly ran out of isoc itds, isoc_start\n");
 		itd->itd.itd_next = sc->sc_flist[frindex];
 		if (itd->itd.itd_next == 0)
 			/* FIXME: frindex table gets initialized to NULL
 			 * or EHCI_NULL? */
 			itd->itd.itd_next = EHCI_NULL;
 		usb_syncmem(&itd->dma,
 		    itd->offs + offsetof(ehci_itd_t, itd_next),
 		    sizeof(itd->itd.itd_next),
 		    BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
 		sc->sc_flist[frindex] = htole32(EHCI_LINK_ITD | itd->physaddr);
 		usb_syncmem(&sc->sc_fldma,
 		    sizeof(ehci_link_t) * frindex,
 		    sizeof(ehci_link_t),
 		    BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
 		itd->u.frame_list.next = sc->sc_softitds[frindex];
 		sc->sc_softitds[frindex] = itd;
 		if (itd->u.frame_list.next != NULL)
 			itd->u.frame_list.next->u.frame_list.prev = itd;
 		itd->slot = frindex;
 		itd->u.frame_list.prev = NULL;
 		frindex += i;
 		if (frindex >= sc->sc_flsize)
 			frindex -= sc->sc_flsize;
 		itd = itd->xfer_next;
+	}
 	epipe->u.isoc.cur_xfers++;
 	epipe->u.isoc.next_frame = frindex;
 	exfer->ex_itdstart = start;
 	exfer->ex_itdend = stop;
 	ehci_add_intr_list(sc, exfer);
 	xfer->ux_status = USBD_IN_PROGRESS;
 	xfer->ux_done = 0;
 	mutex_exit(&sc->sc_lock);
 	if (sc->sc_bus.ub_usepolling) {
 		printf("Starting ehci isoc xfer with polling. Bad idea?\n");
 		ehci_waitintr(sc, xfer);
+	}
 	return USBD_IN_PROGRESS;
+}
 Static void
 ehci_device_isoc_abort(usbd_xfer_handle xfer)
+{
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	USBHIST_LOG(ehcidebug, "xfer = %p", xfer, 0, 0, 0);
 	ehci_abort_isoc_xfer(xfer, USBD_CANCELLED);
+}
 Static void
 ehci_device_isoc_close(usbd_pipe_handle pipe)
+{
 	USBHIST_FUNC(); USBHIST_CALLED(ehcidebug);
 	USBHIST_LOG(ehcidebug, "nothing in the pipe to free?", 0, 0, 0, 0);
+}
 Static void
 ehci_device_isoc_done(usbd_xfer_handle xfer)
+{
 	struct ehci_xfer *exfer;
 	ehci_softc_t *sc;
 	struct ehci_pipe *epipe;
 	exfer = EXFER(xfer);
 	sc = xfer->ux_pipe->up_dev->ud_bus->ub_hcpriv;
 	epipe = (struct ehci_pipe *) xfer->ux_pipe;
 	KASSERT(mutex_owned(&sc->sc_lock));
 	epipe->u.isoc.cur_xfers--;
 	if (xfer->ux_status != USBD_NOMEM && ehci_active_intr_list(exfer)) {
 		ehci_del_intr_list(sc, exfer);
 		ehci_rem_free_itd_chain(sc, exfer);
+	}
 	usb_syncmem(&xfer->ux_dmabuf, 0, xfer->ux_length, BUS_DMASYNC_POSTWRITE |
 	    BUS_DMASYNC_POSTREAD);
+}