Module Name:    src
Committed By:   dyoung
Date:           Thu Sep  3 20:54:03 UTC 2009

Modified Files:
        src/sys/dev/usb: uhub.c usb_subr.c

Log Message:
Expand some of the portability macros from sys/dev/usb/usb_port.h.
There is no change in the generated assembly.


To generate a diff of this commit:
cvs rdiff -u -r1.104 -r1.105 src/sys/dev/usb/uhub.c
cvs rdiff -u -r1.163 -r1.164 src/sys/dev/usb/usb_subr.c

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/uhub.c
diff -u src/sys/dev/usb/uhub.c:1.104 src/sys/dev/usb/uhub.c:1.105
--- src/sys/dev/usb/uhub.c:1.104	Tue Apr  7 18:15:45 2009
+++ src/sys/dev/usb/uhub.c	Thu Sep  3 20:54:03 2009
@@ -1,4 +1,4 @@
-/*	$NetBSD: uhub.c,v 1.104 2009/04/07 18:15:45 dyoung Exp $	*/
+/*	$NetBSD: uhub.c,v 1.105 2009/09/03 20:54:03 dyoung Exp $	*/
 /*	$FreeBSD: src/sys/dev/usb/uhub.c,v 1.18 1999/11/17 22:33:43 n_hibma Exp $	*/
 
 /*
@@ -36,7 +36,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: uhub.c,v 1.104 2009/04/07 18:15:45 dyoung Exp $");
+__KERNEL_RCSID(0, "$NetBSD: uhub.c,v 1.105 2009/09/03 20:54:03 dyoung Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -106,7 +106,8 @@
 CFATTACH_DECL2_NEW(uroothub, sizeof(struct uhub_softc), uhub_match,
     uhub_attach, uhub_detach, uhub_activate, uhub_rescan, uhub_childdet);
 
-USB_MATCH(uhub)
+int
+uhub_match(device_t parent, cfdata_t match, void *aux)
 {
 	USB_MATCH_START(uhub, uaa);
 
@@ -120,7 +121,8 @@
 	return (UMATCH_NONE);
 }
 
-USB_ATTACH(uhub)
+void
+uhub_attach(device_t parent, device_t self, void *aux)
 {
 	USB_ATTACH_START(uhub, sc, uaa);
 	usbd_device_handle dev = uaa->device;
@@ -155,7 +157,7 @@
 	err = usbd_set_config_index(dev, 0, 1);
 	if (err) {
 		DPRINTF(("%s: configuration failed, error=%s\n",
-			 USBDEVNAME(sc->sc_dev), usbd_errstr(err)));
+		    device_xname(sc->sc_dev), usbd_errstr(err)));
 		USB_ATTACH_ERROR_RETURN;
 	}
 
@@ -181,7 +183,7 @@
 	}
 	if (err) {
 		DPRINTF(("%s: getting hub descriptor failed, error=%s\n",
-			 USBDEVNAME(sc->sc_dev), usbd_errstr(err)));
+		    device_xname(sc->sc_dev), usbd_errstr(err)));
 		USB_ATTACH_ERROR_RETURN;
 	}
 
@@ -254,7 +256,7 @@
 	/* Wait with power off for a while. */
 	usbd_delay_ms(dev, USB_POWER_DOWN_TIME);
 
-	usbd_add_drv_event(USB_EVENT_DRIVER_ATTACH, dev, USBDEV(sc->sc_dev));
+	usbd_add_drv_event(USB_EVENT_DRIVER_ATTACH, dev, sc->sc_dev);
 
 	/*
 	 * To have the best chance of success we do things in the exact same
@@ -467,7 +469,7 @@
 			/* Disconnected */
 			DPRINTF(("uhub_explore: device addr=%d disappeared "
 				 "on port %d\n", up->device->address, port));
-			usb_disconnect_port(up, USBDEV(sc->sc_dev));
+			usb_disconnect_port(up, sc->sc_dev);
 			usbd_clear_port_feature(dev, port,
 						UHF_C_PORT_CONNECTION);
 		}
@@ -519,7 +521,7 @@
 		else
 			speed = USB_SPEED_FULL;
 		/* Get device info and set its address. */
-		err = usbd_new_device(USBDEV(sc->sc_dev), dev->bus,
+		err = usbd_new_device(sc->sc_dev, dev->bus,
 			  dev->depth + 1, speed, port, up);
 		/* XXX retry a few times? */
 		if (err) {
@@ -584,7 +586,8 @@
  * Called from process context when the hub is gone.
  * Detach all devices on active ports.
  */
-USB_DETACH(uhub)
+int
+uhub_detach(device_t self, int flags)
 {
 	USB_DETACH_START(uhub, sc);
 	struct usbd_hub *hub = sc->sc_hub->hub;
@@ -607,8 +610,7 @@
 			usb_disconnect_port(rup, self);
 	}
 
-	usbd_add_drv_event(USB_EVENT_DRIVER_DETACH, sc->sc_hub,
-			   USBDEV(sc->sc_dev));
+	usbd_add_drv_event(USB_EVENT_DRIVER_DETACH, sc->sc_hub, sc->sc_dev);
 
 #if 0
 	if (hub->ports[0].tt)
@@ -636,8 +638,7 @@
 		dev = hub->ports[port].device;
 		if (dev == NULL)
 			continue;
-		err = usbd_reattach_device(USBDEV(sc->sc_dev), dev,
-					   port, locators);
+		err = usbd_reattach_device(sc->sc_dev, dev, port, locators);
 	}
 	return 0;
 }
@@ -679,8 +680,7 @@
  * to be explored again.
  */
 void
-uhub_intr(usbd_xfer_handle xfer, usbd_private_handle addr,
-    usbd_status status)
+uhub_intr(usbd_xfer_handle xfer, usbd_private_handle addr, usbd_status status)
 {
 	struct uhub_softc *sc = addr;
 

Index: src/sys/dev/usb/usb_subr.c
diff -u src/sys/dev/usb/usb_subr.c:1.163 src/sys/dev/usb/usb_subr.c:1.164
--- src/sys/dev/usb/usb_subr.c:1.163	Sun Aug 16 13:06:32 2009
+++ src/sys/dev/usb/usb_subr.c	Thu Sep  3 20:54:03 2009
@@ -1,4 +1,4 @@
-/*	$NetBSD: usb_subr.c,v 1.163 2009/08/16 13:06:32 martin Exp $	*/
+/*	$NetBSD: usb_subr.c,v 1.164 2009/09/03 20:54:03 dyoung Exp $	*/
 /*	$FreeBSD: src/sys/dev/usb/usb_subr.c,v 1.18 1999/11/17 22:33:47 n_hibma Exp $	*/
 
 /*
@@ -32,7 +32,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: usb_subr.c,v 1.163 2009/08/16 13:06:32 martin Exp $");
+__KERNEL_RCSID(0, "$NetBSD: usb_subr.c,v 1.164 2009/09/03 20:54:03 dyoung Exp $");
 
 #include "opt_compat_netbsd.h"
 #include "opt_usbverbose.h"
@@ -78,8 +78,8 @@
 Static void usbd_free_iface_data(usbd_device_handle dev, int ifcno);
 Static void usbd_kill_pipe(usbd_pipe_handle);
 usbd_status usbd_attach_roothub(device_t, usbd_device_handle);
-Static usbd_status usbd_probe_and_attach(device_ptr_t parent,
-				 usbd_device_handle dev, int port, int addr);
+Static usbd_status usbd_probe_and_attach(device_t parent,
+                                 usbd_device_handle dev, int port, int addr);
 
 Static u_int32_t usb_cookie_no = 0;
 
@@ -640,7 +640,7 @@
 		if (msg)
 			printf("%s: device addr %d (config %d): "
 				 "can't set self powered configuration\n",
-			       USBDEVNAME(dev->bus->bdev), dev->address,
+			       device_xname(dev->bus->bdev), dev->address,
 			       cdp->bConfigurationValue);
 		err = USBD_NO_POWER;
 		goto bad;
@@ -904,8 +904,8 @@
 }
 
 usbd_status
-usbd_probe_and_attach(device_ptr_t parent, usbd_device_handle dev,
-		      int port, int addr)
+usbd_probe_and_attach(device_t parent, usbd_device_handle dev,
+                      int port, int addr)
 {
 	usb_device_descriptor_t *dd = &dev->ddesc;
 	int confi, nifaces;
@@ -927,11 +927,11 @@
 		if (err) {
 #ifdef USB_DEBUG
 			DPRINTF(("%s: port %d, set config at addr %d failed, "
-				 "error=%s\n", USBDEVPTRNAME(parent), port,
-				 addr, usbd_errstr(err)));
+			    "error=%s\n", device_xname(parent), port,
+			    addr, usbd_errstr(err)));
 #else
 			printf("%s: port %d, set config at addr %d failed\n",
-			       USBDEVPTRNAME(parent), port, addr);
+			    device_xname(parent), port, addr);
 #endif
 			return (err);
 		}
@@ -976,8 +976,8 @@
  * called before calling this.
  */
 usbd_status
-usbd_reattach_device(device_ptr_t parent, usbd_device_handle dev,
-		     int port, const int *locators)
+usbd_reattach_device(device_t parent, usbd_device_handle dev,
+                     int port, const int *locators)
 {
 	int i, loc;
 
@@ -1024,8 +1024,8 @@
  * and attach a driver.
  */
 usbd_status
-usbd_new_device(device_ptr_t parent, usbd_bus_handle bus, int depth,
-		int speed, int port, struct usbd_port *up)
+usbd_new_device(device_t parent, usbd_bus_handle bus, int depth,
+                int speed, int port, struct usbd_port *up)
 {
 	usbd_device_handle dev, adev;
 	struct usbd_device *hub;
@@ -1211,9 +1211,9 @@
 	if (err) {
 		usbd_remove_device(dev, up);
 		return (err);
-  	}
+	}
 
-  	return (USBD_NORMAL_COMPLETION);
+	return (USBD_NORMAL_COMPLETION);
 }
 
 usbd_status
@@ -1337,16 +1337,16 @@
 			     j < USB_MAX_DEVNAMES; i++) {
 			if (!dev->subdevs[i])
 				continue;
-			strncpy(di->udi_devnames[j], USBDEVPTRNAME(dev->subdevs[i]),
-				USB_MAX_DEVNAMELEN);
+			strncpy(di->udi_devnames[j],
+			    device_xname(dev->subdevs[i]), USB_MAX_DEVNAMELEN);
 			di->udi_devnames[j][USB_MAX_DEVNAMELEN-1] = '\0';
 			j++;
-                }
-        } else {
-                j = 0;
-        }
-        for (/* j is set */; j < USB_MAX_DEVNAMES; j++)
-                di->udi_devnames[j][0] = 0;                 /* empty */
+		}
+	} else {
+		j = 0;
+	}
+	for (/* j is set */; j < USB_MAX_DEVNAMES; j++)
+		di->udi_devnames[j][0] = 0;                 /* empty */
 
 	if (dev->hub) {
 		for (i = 0;
@@ -1403,8 +1403,8 @@
 			     j < USB_MAX_DEVNAMES; i++) {
 			if (!dev->subdevs[i])
 				continue;
-			strncpy(di->udi_devnames[j], USBDEVPTRNAME(dev->subdevs[i]),
-				USB_MAX_DEVNAMELEN);
+			strncpy(di->udi_devnames[j],
+			    device_xname(dev->subdevs[i]), USB_MAX_DEVNAMELEN);
 			di->udi_devnames[j][USB_MAX_DEVNAMELEN-1] = '\0';
 			j++;
 		}
@@ -1482,10 +1482,10 @@
  * been disconnected.
  */
 void
-usb_disconnect_port(struct usbd_port *up, device_ptr_t parent)
+usb_disconnect_port(struct usbd_port *up, device_t parent)
 {
 	usbd_device_handle dev = up->device;
-	const char *hubname = USBDEVPTRNAME(parent);
+	const char *hubname = device_xname(parent);
 	int i;
 
 	DPRINTFN(3,("uhub_disconnect: up=%p dev=%p port=%d\n",
@@ -1503,7 +1503,7 @@
 		for (i = 0; i < dev->subdevlen; i++) {
 			if (!dev->subdevs[i])
 				continue;
-			printf("%s: at %s", USBDEVPTRNAME(dev->subdevs[i]),
+			printf("%s: at %s", device_xname(dev->subdevs[i]),
 			       hubname);
 			if (up->portno != 0)
 				printf(" port %d", up->portno);

Reply via email to