Module Name:    src
Committed By:   skrll
Date:           Tue Oct 27 15:28:22 UTC 2015

Modified Files:
        src/sys/dev/usb [nick-nhusb]: uhci.c uhcivar.h

Log Message:
Rename uhci_xfer struct members.

No funciotnal change.


To generate a diff of this commit:
cvs rdiff -u -r1.264.4.40 -r1.264.4.41 src/sys/dev/usb/uhci.c
cvs rdiff -u -r1.52.14.11 -r1.52.14.12 src/sys/dev/usb/uhcivar.h

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/dev/usb/uhci.c
diff -u src/sys/dev/usb/uhci.c:1.264.4.40 src/sys/dev/usb/uhci.c:1.264.4.41
--- src/sys/dev/usb/uhci.c:1.264.4.40	Tue Oct 27 14:22:38 2015
+++ src/sys/dev/usb/uhci.c	Tue Oct 27 15:28:22 2015
@@ -1,4 +1,4 @@
-/*	$NetBSD: uhci.c,v 1.264.4.40 2015/10/27 14:22:38 skrll Exp $	*/
+/*	$NetBSD: uhci.c,v 1.264.4.41 2015/10/27 15:28:22 skrll Exp $	*/
 
 /*
  * Copyright (c) 1998, 2004, 2011, 2012 The NetBSD Foundation, Inc.
@@ -42,7 +42,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: uhci.c,v 1.264.4.40 2015/10/27 14:22:38 skrll Exp $");
+__KERNEL_RCSID(0, "$NetBSD: uhci.c,v 1.264.4.41 2015/10/27 15:28:22 skrll Exp $");
 
 #include "opt_usb.h"
 
@@ -367,13 +367,13 @@ const struct usbd_pipe_methods uhci_devi
 };
 
 #define uhci_add_intr_info(sc, ux) \
-	TAILQ_INSERT_TAIL(&(sc)->sc_intrhead, (ux), list)
+	TAILQ_INSERT_TAIL(&(sc)->sc_intrhead, (ux), ux_list)
 #define uhci_del_intr_info(sc, ux) \
 	do { \
-		TAILQ_REMOVE(&(sc)->sc_intrhead, (ux), list); \
-		(ux)->list.tqe_prev = NULL; \
+		TAILQ_REMOVE(&(sc)->sc_intrhead, (ux), ux_list); \
+		(ux)->ux_list.tqe_prev = NULL; \
 	} while (0)
-#define uhci_active_intr_info(ux) ((ux)->list.tqe_prev != NULL)
+#define uhci_active_intr_info(ux) ((ux)->ux_list.tqe_prev != NULL)
 
 static inline uhci_soft_qh_t *
 uhci_find_prev_qh(uhci_soft_qh_t *pqh, uhci_soft_qh_t *sqh)
@@ -625,7 +625,7 @@ uhci_allocx(struct usbd_bus *bus, unsign
 
 #ifdef DIAGNOSTIC
 		struct uhci_xfer *uxfer = UHCI_XFER2UXFER(xfer);
-		uxfer->isdone = true;
+		uxfer->ux_isdone = true;
 		xfer->ux_state = XFER_BUSY;
 #endif
 	}
@@ -640,7 +640,7 @@ uhci_freex(struct usbd_bus *bus, struct 
 
 	KASSERTMSG(xfer->ux_state == XFER_BUSY, "xfer %p state %d\n", xfer,
 	    xfer->ux_state);
-	KASSERTMSG(uxfer->isdone, "xfer %p not done\n", xfer);
+	KASSERTMSG(uxfer->ux_isdone, "xfer %p not done\n", xfer);
 #ifdef DIAGNOSTIC
 	xfer->ux_state = XFER_FREE;
 #endif
@@ -905,25 +905,25 @@ uhci_dump_ii(struct uhci_xfer *ux)
 		printf("ux NULL\n");
 		return;
 	}
-	pipe = ux->xfer.ux_pipe;
+	pipe = ux->ux_xfer.ux_pipe;
 	if (pipe == NULL) {
-		printf("ux %p: done=%d pipe=NULL\n", ux, ux->isdone);
+		printf("ux %p: done=%d pipe=NULL\n", ux, ux->ux_isdone);
 		return;
 	}
 	if (pipe->up_endpoint == NULL) {
 		printf("ux %p: done=%d pipe=%p pipe->up_endpoint=NULL\n",
-		       ux, ux->isdone, pipe);
+		       ux, ux->ux_isdone, pipe);
 		return;
 	}
 	if (pipe->up_dev == NULL) {
 		printf("ux %p: done=%d pipe=%p pipe->up_dev=NULL\n",
-		       ux, ux->isdone, pipe);
+		       ux, ux->ux_isdone, pipe);
 		return;
 	}
 	ed = pipe->up_endpoint->ue_edesc;
 	dev = pipe->up_dev;
 	printf("ux %p: done=%d dev=%p vid=0x%04x pid=0x%04x addr=%d pipe=%p ep=0x%02x attr=0x%02x\n",
-	       ux, ux->isdone, dev,
+	       ux, ux->ux_isdone, dev,
 	       UGETW(dev->ud_ddesc.idVendor),
 	       UGETW(dev->ud_ddesc.idProduct),
 	       dev->ud_addr, pipe,
@@ -937,7 +937,7 @@ uhci_dump_iis(struct uhci_softc *sc)
 	struct uhci_xfer *ux;
 
 	printf("interrupt list:\n");
-	for (ux = TAILQ_FIRST(&sc->sc_intrhead); ux; ux = TAILQ_NEXT(ux, list))
+	for (ux = TAILQ_FIRST(&sc->sc_intrhead); ux; ux = TAILQ_NEXT(ux, ux_list))
 		uhci_dump_ii(ux);
 }
 
@@ -1370,7 +1370,7 @@ uhci_softintr(void *v)
 	 * completed.
 	 */
 	for (ux = TAILQ_FIRST(&sc->sc_intrhead); ux; ux = nextux) {
-		nextux = TAILQ_NEXT(ux, list);
+		nextux = TAILQ_NEXT(ux, ux_list);
 		uhci_check_intr(sc, ux);
 	}
 
@@ -1392,16 +1392,16 @@ uhci_check_intr(uhci_softc_t *sc, struct
 
 	KASSERT(ux != NULL);
 
-	struct usbd_xfer *xfer = &ux->xfer;
+	struct usbd_xfer *xfer = &ux->ux_xfer;
 	if (xfer->ux_status == USBD_CANCELLED ||
 	    xfer->ux_status == USBD_TIMEOUT) {
 		DPRINTF("aborted xfer %p", xfer, 0, 0, 0);
 		return;
 	}
 
-	if (ux->stdstart == NULL)
+	if (ux->ux_stdstart == NULL)
 		return;
-	lstd = ux->stdend;
+	lstd = ux->ux_stdend;
 
 	KASSERT(lstd != NULL);
 
@@ -1431,7 +1431,7 @@ uhci_check_intr(uhci_softc_t *sc, struct
 	 * short packet (SPD and not ACTIVE).
 	 */
 	DPRINTFN(12, "active ux=%p", ux, 0, 0, 0);
-	for (std = ux->stdstart; std != lstd; std = std->link.std) {
+	for (std = ux->ux_stdstart; std != lstd; std = std->link.std) {
 		usb_syncmem(&std->dma,
 		    std->offs + offsetof(uhci_td_t, td_status),
 		    sizeof(std->td.td_status),
@@ -1466,7 +1466,7 @@ uhci_check_intr(uhci_softc_t *sc, struct
 			uhci_soft_td_t *stat = upipe->ctrl.stat;
 
 			DPRINTFN(12, "ux=%p std=%p control status"
-			    "phase needs completion", ux, ux->stdstart, 0, 0);
+			    "phase needs completion", ux, ux->ux_stdstart, 0, 0);
 
 			sqh->qh.qh_elink =
 			    htole32(stat->physaddr | UHCI_PTR_TD);
@@ -1493,7 +1493,7 @@ uhci_check_intr(uhci_softc_t *sc, struct
 void
 uhci_idone(struct uhci_xfer *ux)
 {
-	struct usbd_xfer *xfer = &ux->xfer;
+	struct usbd_xfer *xfer = &ux->ux_xfer;
 	uhci_softc_t *sc __diagused = UHCI_XFER2SC(xfer);
 	struct uhci_pipe *upipe = (struct uhci_pipe *)xfer->ux_pipe;
 	uhci_soft_td_t *std;
@@ -1507,14 +1507,14 @@ uhci_idone(struct uhci_xfer *ux)
 
 #ifdef DIAGNOSTIC
 #ifdef UHCI_DEBUG
-	if (ux->isdone) {
+	if (ux->ux_isdone) {
 		DPRINTF("--- dump start ---", 0, 0, 0, 0);
 		uhci_dump_ii(ux);
 		DPRINTF("--- dump end ---", 0, 0, 0, 0);
 	}
 #endif
-	KASSERT(!ux->isdone);
-	ux->isdone = true;
+	KASSERT(!ux->ux_isdone);
+	ux->ux_isdone = true;
 #endif
 
 	if (xfer->ux_nframes != 0) {
@@ -1526,7 +1526,7 @@ uhci_idone(struct uhci_xfer *ux)
 
 		nframes = xfer->ux_nframes;
 		actlen = 0;
-		n = UHCI_XFER2UXFER(xfer)->curframe;
+		n = UHCI_XFER2UXFER(xfer)->ux_curframe;
 		for (i = 0; i < nframes; i++) {
 			std = stds[n];
 #ifdef UHCI_DEBUG
@@ -1556,12 +1556,12 @@ uhci_idone(struct uhci_xfer *ux)
 	DPRINTFN(10, "ux=%p, xfer=%p, pipe=%p ready",
 	    ux, xfer, upipe, 0);
 	if (uhcidebug > 10)
-		uhci_dump_tds(ux->stdstart);
+		uhci_dump_tds(ux->ux_stdstart);
 #endif
 
 	/* The transfer is done, compute actual length and status. */
 	actlen = 0;
-	for (std = ux->stdstart; std != NULL; std = std->link.std) {
+	for (std = ux->ux_stdstart; std != NULL; std = std->link.std) {
 		usb_syncmem(&std->dma, std->offs, sizeof(std->td),
 		    BUS_DMASYNC_POSTWRITE | BUS_DMASYNC_POSTREAD);
 		nstatus = le32toh(std->td.td_status);
@@ -1645,9 +1645,9 @@ uhci_timeout(void *addr)
 	}
 
 	/* Execute the abort in a process context. */
-	usb_init_task(&uxfer->abort_task, uhci_timeout_task, xfer,
+	usb_init_task(&uxfer->ux_aborttask, uhci_timeout_task, xfer,
 	    USB_TASKQ_MPSAFE);
-	usb_add_task(uxfer->xfer.ux_pipe->up_dev, &uxfer->abort_task,
+	usb_add_task(uxfer->ux_xfer.ux_pipe->up_dev, &uxfer->ux_aborttask,
 	    USB_TASKQ_HC);
 }
 
@@ -1700,8 +1700,8 @@ uhci_waitintr(uhci_softc_t *sc, struct u
 	/* Timeout */
 	DPRINTF("timeout", 0, 0, 0, 0);
 	for (ux = TAILQ_FIRST(&sc->sc_intrhead); ux != NULL;
-	    ux = TAILQ_NEXT(ux, list))
-		if (&ux->xfer == xfer)
+	    ux = TAILQ_NEXT(ux, ux_list))
+		if (&ux->ux_xfer == xfer)
 			break;
 
 	KASSERT(ux != NULL);
@@ -2078,12 +2078,12 @@ uhci_device_bulk_start(struct usbd_xfer 
 #endif
 
 	/* Set up interrupt info. */
-	ux->stdstart = data;
-	ux->stdend = dataend;
+	ux->ux_stdstart = data;
+	ux->ux_stdend = dataend;
 
-	KASSERT(ux->isdone);
+	KASSERT(ux->ux_isdone);
 #ifdef DIAGNOSTIC
-	ux->isdone = false;
+	ux->ux_isdone = false;
 #endif
 
 	sqh->elink = data;
@@ -2187,7 +2187,7 @@ uhci_abort_xfer(struct usbd_xfer *xfer, 
 	xfer->ux_status = status;	/* make software ignore it */
 	callout_stop(&xfer->ux_callout);
 	DPRINTF("stop ux=%p", ux, 0, 0, 0);
-	for (std = ux->stdstart; std != NULL; std = std->link.std) {
+	for (std = ux->ux_stdstart; std != NULL; std = std->link.std) {
 		usb_syncmem(&std->dma,
 		    std->offs + offsetof(uhci_td_t, td_status),
 		    sizeof(std->td.td_status),
@@ -2216,7 +2216,7 @@ uhci_abort_xfer(struct usbd_xfer *xfer, 
 	 */
 	DPRINTF("callback", 0, 0, 0, 0);
 #ifdef DIAGNOSTIC
-	ux->isdone = true;
+	ux->ux_isdone = true;
 #endif
 	wake = xfer->ux_hcflags & UXFER_ABORTWAIT;
 	xfer->ux_hcflags &= ~(UXFER_ABORTING | UXFER_ABORTWAIT);
@@ -2356,11 +2356,11 @@ uhci_device_intr_start(struct usbd_xfer 
 	DPRINTFN(10, "--- dump end ---", 0, 0, 0, 0);
 
 	/* Set up interrupt info. */
-	ux->stdstart = data;
-	ux->stdend = dataend;
-	KASSERT(ux->isdone);
+	ux->ux_stdstart = data;
+	ux->ux_stdend = dataend;
+	KASSERT(ux->ux_isdone);
 #ifdef DIAGNOSTIC
-	ux->isdone = false;
+	ux->ux_isdone = false;
 #endif
 
 	DPRINTFN(10, "qhs[0]=%p", upipe->intr.qhs[0], 0, 0, 0);
@@ -2537,11 +2537,11 @@ uhci_device_request(struct usbd_xfer *xf
 	DPRINTFN(10, "--- dump end ---", 0, 0, 0, 0);
 
 	/* Set up interrupt info. */
-	uxfer->stdstart = setup;
-	uxfer->stdend = stat;
-	KASSERT(uxfer->isdone);
+	uxfer->ux_stdstart = setup;
+	uxfer->ux_stdend = stat;
+	KASSERT(uxfer->ux_isdone);
 #ifdef DIAGNOSTIC
-	uxfer->isdone = false;
+	uxfer->ux_isdone = false;
 #endif
 
 	sqh->elink = setup;
@@ -2658,7 +2658,7 @@ uhci_device_isoc_enter(struct usbd_xfer 
 	}
 
 	xfer->ux_status = USBD_IN_PROGRESS;
-	UHCI_XFER2UXFER(xfer)->curframe = next;
+	UHCI_XFER2UXFER(xfer)->ux_curframe = next;
 
 	buf = DMAADDR(&xfer->ux_dmabuf, 0);
 	offs = 0;
@@ -2724,7 +2724,7 @@ uhci_device_isoc_start(struct usbd_xfer 
 #endif
 
 	/* Find the last TD */
-	i = UHCI_XFER2UXFER(xfer)->curframe + xfer->ux_nframes;
+	i = UHCI_XFER2UXFER(xfer)->ux_curframe + xfer->ux_nframes;
 	if (i >= UHCI_VFRAMELIST_COUNT)
 		i -= UHCI_VFRAMELIST_COUNT;
 	end = upipe->isoc.stds[i];
@@ -2732,12 +2732,12 @@ uhci_device_isoc_start(struct usbd_xfer 
 	KASSERT(end != NULL);
 
 	/* Set up interrupt info. */
-	ux->stdstart = end;
-	ux->stdend = end;
+	ux->ux_stdstart = end;
+	ux->ux_stdend = end;
 
-	KASSERT(ux->isdone);
+	KASSERT(ux->ux_isdone);
 #ifdef DIAGNOSTIC
-	ux->isdone = false;
+	ux->ux_isdone = false;
 #endif
 	uhci_add_intr_info(sc, ux);
 
@@ -2768,7 +2768,7 @@ uhci_device_isoc_abort(struct usbd_xfer 
 
 	/* make hardware ignore it, */
 	nframes = xfer->ux_nframes;
-	n = UHCI_XFER2UXFER(xfer)->curframe;
+	n = UHCI_XFER2UXFER(xfer)->ux_curframe;
 	maxlen = 0;
 	for (i = 0; i < nframes; i++) {
 		std = stds[n];
@@ -2796,7 +2796,7 @@ uhci_device_isoc_abort(struct usbd_xfer 
 	delay(maxlen);
 
 #ifdef DIAGNOSTIC
-	UHCI_XFER2UXFER(xfer)->isdone = true;
+	UHCI_XFER2UXFER(xfer)->ux_isdone = true;
 #endif
 	/* Run callback and remove from interrupt list. */
 	usb_transfer_complete(xfer);
@@ -2957,7 +2957,7 @@ uhci_device_isoc_done(struct usbd_xfer *
 		return;
 
 #ifdef DIAGNOSTIC
-	if (ux->stdend == NULL) {
+	if (ux->ux_stdend == NULL) {
 		printf("uhci_device_isoc_done: xfer=%p stdend==NULL\n", xfer);
 #ifdef UHCI_DEBUG
 		uhci_dump_ii(ux);
@@ -2967,14 +2967,14 @@ uhci_device_isoc_done(struct usbd_xfer *
 #endif
 
 	/* Turn off the interrupt since it is active even if the TD is not. */
-	usb_syncmem(&ux->stdend->dma,
-	    ux->stdend->offs + offsetof(uhci_td_t, td_status),
-	    sizeof(ux->stdend->td.td_status),
+	usb_syncmem(&ux->ux_stdend->dma,
+	    ux->ux_stdend->offs + offsetof(uhci_td_t, td_status),
+	    sizeof(ux->ux_stdend->td.td_status),
 	    BUS_DMASYNC_POSTWRITE | BUS_DMASYNC_POSTREAD);
-	ux->stdend->td.td_status &= htole32(~UHCI_TD_IOC);
-	usb_syncmem(&ux->stdend->dma,
-	    ux->stdend->offs + offsetof(uhci_td_t, td_status),
-	    sizeof(ux->stdend->td.td_status),
+	ux->ux_stdend->td.td_status &= htole32(~UHCI_TD_IOC);
+	usb_syncmem(&ux->ux_stdend->dma,
+	    ux->ux_stdend->offs + offsetof(uhci_td_t, td_status),
+	    sizeof(ux->ux_stdend->td.td_status),
 	    BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
 
 	uhci_del_intr_info(sc, ux);	/* remove from active list */
@@ -3011,7 +3011,7 @@ uhci_device_intr_done(struct usbd_xfer *
 		    sizeof(sqh->qh.qh_elink),
 		    BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
 	}
-	uhci_free_std_chain(sc, ux->stdstart, NULL);
+	uhci_free_std_chain(sc, ux->ux_stdstart, NULL);
 
 	isread = UE_GET_DIR(upipe->pipe.up_endpoint->ue_edesc->bEndpointAddress) == UE_DIR_IN;
 	usb_syncmem(&xfer->ux_dmabuf, 0, xfer->ux_length,
@@ -3042,11 +3042,11 @@ uhci_device_intr_done(struct usbd_xfer *
 #endif
 		DPRINTFN(10, "--- dump end ---", 0, 0, 0, 0);
 
-		ux->stdstart = data;
-		ux->stdend = dataend;
-		KASSERT(ux->isdone);
+		ux->ux_stdstart = data;
+		ux->ux_stdend = dataend;
+		KASSERT(ux->ux_isdone);
 #ifdef DIAGNOSTIC
-		ux->isdone = false;
+		ux->ux_isdone = false;
 #endif
 		for (i = 0; i < npoll; i++) {
 			sqh = upipe->intr.qhs[i];
@@ -3093,7 +3093,7 @@ uhci_device_ctrl_done(struct usbd_xfer *
 		uhci_remove_hs_ctrl(sc, upipe->ctrl.sqh);
 
 	if (upipe->ctrl.length != 0)
-		uhci_free_std_chain(sc, ux->stdstart->link.std, ux->stdend);
+		uhci_free_std_chain(sc, ux->ux_stdstart->link.std, ux->ux_stdend);
 
 	if (len) {
 		usb_syncmem(&xfer->ux_dmabuf, 0, len,
@@ -3126,7 +3126,7 @@ uhci_device_bulk_done(struct usbd_xfer *
 
 	uhci_remove_bulk(sc, upipe->bulk.sqh);
 
-	uhci_free_std_chain(sc, ux->stdstart, NULL);
+	uhci_free_std_chain(sc, ux->ux_stdstart, NULL);
 
 	DPRINTFN(5, "length=%d", xfer->ux_actlen, 0, 0, 0);
 }
@@ -3693,7 +3693,7 @@ uhci_root_intr_abort(struct usbd_xfer *x
 
 	xfer->ux_status = USBD_CANCELLED;
 #ifdef DIAGNOSTIC
-	UHCI_XFER2UXFER(xfer)->isdone = 1;
+	UHCI_XFER2UXFER(xfer)->ux_isdone = true;
 #endif
 	usb_transfer_complete(xfer);
 }

Index: src/sys/dev/usb/uhcivar.h
diff -u src/sys/dev/usb/uhcivar.h:1.52.14.11 src/sys/dev/usb/uhcivar.h:1.52.14.12
--- src/sys/dev/usb/uhcivar.h:1.52.14.11	Tue Oct 27 14:05:29 2015
+++ src/sys/dev/usb/uhcivar.h	Tue Oct 27 15:28:22 2015
@@ -1,4 +1,4 @@
-/*	$NetBSD: uhcivar.h,v 1.52.14.11 2015/10/27 14:05:29 skrll Exp $	*/
+/*	$NetBSD: uhcivar.h,v 1.52.14.12 2015/10/27 15:28:22 skrll Exp $	*/
 
 /*
  * Copyright (c) 1998 The NetBSD Foundation, Inc.
@@ -60,13 +60,13 @@ typedef union {
 } uhci_soft_td_qh_t;
 
 struct uhci_xfer {
-	struct usbd_xfer xfer;
-	struct usb_task	abort_task;
-	uhci_soft_td_t *stdstart;
-	uhci_soft_td_t *stdend;
-	TAILQ_ENTRY(uhci_xfer) list;
-	int curframe;
-	bool isdone;	/* used only when DIAGNOSTIC is defined */
+	struct usbd_xfer ux_xfer;
+	struct usb_task	ux_aborttask;
+	uhci_soft_td_t *ux_stdstart;
+	uhci_soft_td_t *ux_stdend;
+	TAILQ_ENTRY(uhci_xfer) ux_list;
+	int ux_curframe;
+	bool ux_isdone;	/* used only when DIAGNOSTIC is defined */
 };
 
 #define UHCI_BUS2SC(bus)	((bus)->ub_hcpriv)

Reply via email to