Kill devinfo handling in drivers, set device description in one place -
authorHasso Tepper <hasso@dragonflybsd.org>
Tue, 6 Nov 2007 07:37:02 +0000 (07:37 +0000)
committerHasso Tepper <hasso@dragonflybsd.org>
Tue, 6 Nov 2007 07:37:02 +0000 (07:37 +0000)
usbd_probe_and_attach() and let generic device code to output descriptions.

Obtained-from: FreeBSD with modifications

29 files changed:
sys/bus/usb/uhub.c
sys/bus/usb/usb_subr.c
sys/dev/netif/aue/if_aue.c
sys/dev/netif/axe/if_axe.c
sys/dev/netif/cue/if_cue.c
sys/dev/netif/kue/if_kue.c
sys/dev/netif/rue/if_rue.c
sys/dev/netif/rum/if_rum.c
sys/dev/netif/ural/if_ural.c
sys/dev/usbmisc/uark/uark.c
sys/dev/usbmisc/ubsa/ubsa.c
sys/dev/usbmisc/udbp/udbp.c
sys/dev/usbmisc/ufm/ufm.c
sys/dev/usbmisc/uftdi/uftdi.c
sys/dev/usbmisc/ugen/ugen.c
sys/dev/usbmisc/uhid/uhid.c
sys/dev/usbmisc/ukbd/ukbd.c
sys/dev/usbmisc/ulpt/ulpt.c
sys/dev/usbmisc/umass/umass.c
sys/dev/usbmisc/umct/umct.c
sys/dev/usbmisc/umodem/umodem.c
sys/dev/usbmisc/ums/ums.c
sys/dev/usbmisc/umsm/umsm.c
sys/dev/usbmisc/uplcom/uplcom.c
sys/dev/usbmisc/urio/urio.c
sys/dev/usbmisc/uscanner/uscanner.c
sys/dev/usbmisc/uslcom/uslcom.c
sys/dev/usbmisc/uvisor/uvisor.c
sys/dev/usbmisc/uvscom/uvscom.c

index 9c2a30b..7d90b29 100644 (file)
@@ -1,6 +1,6 @@
 /*     $NetBSD: uhub.c,v 1.68 2004/06/29 06:30:05 mycroft Exp $        */
 /*     $FreeBSD: src/sys/dev/usb/uhub.c,v 1.69.2.1 2005/12/18 15:51:31 iedowse Exp $   */
-/*     $DragonFly: src/sys/bus/usb/uhub.c,v 1.18 2007/07/02 23:52:04 hasso Exp $       */
+/*     $DragonFly: src/sys/bus/usb/uhub.c,v 1.19 2007/11/06 07:37:00 hasso Exp $       */
 
 /*
  * Copyright (c) 1998 The NetBSD Foundation, Inc.
@@ -170,7 +170,6 @@ uhub_attach(device_t self)
        struct uhub_softc *sc = device_get_softc(self);
        struct usb_attach_arg *uaa = device_get_ivars(self);
        usbd_device_handle dev = uaa->device;
-       char *devinfo;
        usbd_status err;
        struct usbd_hub *hub = NULL;
        usb_device_request_t req;
@@ -180,12 +179,9 @@ uhub_attach(device_t self)
        usb_endpoint_descriptor_t *ed;
        struct usbd_tt *tts = NULL;
 
-       devinfo = kmalloc(1024, M_TEMP, M_WAITOK);
        DPRINTFN(1,("uhub_attach\n"));
        sc->sc_hub = dev;
-       usbd_devinfo(dev, 1, devinfo);
        sc->sc_dev = self;
-       device_set_desc_copy(self, devinfo);
 
        if (dev->depth > 0 && UHUB_IS_HIGH_SPEED(sc)) {
                kprintf("%s: %s transaction translator%s\n",
@@ -197,14 +193,12 @@ uhub_attach(device_t self)
        if (err) {
                DPRINTF(("%s: configuration failed, error=%s\n",
                         device_get_nameunit(sc->sc_dev), usbd_errstr(err)));
-               kfree(devinfo, M_TEMP);
                return ENXIO;
        }
 
        if (dev->depth > USB_HUB_MAX_DEPTH) {
                kprintf("%s: hub depth (%d) exceeded, hub ignored\n",
                       device_get_nameunit(sc->sc_dev), USB_HUB_MAX_DEPTH);
-               kfree(devinfo, M_TEMP);
                return ENXIO;
        }
 
@@ -224,7 +218,6 @@ uhub_attach(device_t self)
        if (err) {
                DPRINTF(("%s: getting hub descriptor failed, error=%s\n",
                         device_get_nameunit(sc->sc_dev), usbd_errstr(err)));
-               kfree(devinfo, M_TEMP);
                return ENXIO;
        }
 
@@ -365,7 +358,6 @@ uhub_attach(device_t self)
  bad:
        if (hub)
                kfree(hub, M_USBDEV);
-       kfree(devinfo, M_TEMP);
        dev->hub = NULL;
        return ENXIO;
 }
index 575d36b..291eace 100644 (file)
@@ -1,6 +1,6 @@
 /*     $NetBSD: usb_subr.c,v 1.99 2002/07/11 21:14:34 augustss Exp $   */
 /*     $FreeBSD: src/sys/dev/usb/usb_subr.c,v 1.76.2.3 2006/03/01 01:59:05 iedowse Exp $       */
-/*     $DragonFly: src/sys/bus/usb/usb_subr.c,v 1.25 2007/11/05 19:09:42 hasso Exp $   */
+/*     $DragonFly: src/sys/bus/usb/usb_subr.c,v 1.26 2007/11/06 07:37:00 hasso Exp $   */
 
 /* Also already have from NetBSD:
  *     $NetBSD: usb_subr.c,v 1.102 2003/01/01 16:21:50 augustss Exp $
@@ -748,17 +748,24 @@ usbd_probe_and_attach(device_t parent, usbd_device_handle dev,
        usbd_status err;
        device_t *tmpdv;
        usbd_interface_handle ifaces[256]; /* 256 is the absolute max */
+       char *devinfo;
 
        /* XXX FreeBSD may leak resources on failure cases -- fixme */
        device_t bdev;
        struct usb_attach_arg *uaap;
 
+       devinfo = kmalloc(1024, M_USB, M_NOWAIT);
+       if (devinfo == NULL) {
+               device_printf(parent,
+                             "Can't allocate memory for probe string\n");
+               return (USBD_NOMEM);
+       }
        bdev = device_add_child(parent, NULL, -1);
        if (!bdev) {
+               kfree(devinfo, M_USB);
                device_printf(parent, "Device creation failed\n");
                return (USBD_INVAL);
        }
-       device_quiet(bdev);
        uaap = kmalloc(sizeof(uaa), M_USB, M_WAITOK);
        device_set_ivars(bdev, uaap);
        uaa.device = dev;
@@ -781,7 +788,10 @@ usbd_probe_and_attach(device_t parent, usbd_device_handle dev,
        dev->subdevs[0] = bdev;
        dev->subdevs[1] = 0;
        *uaap = uaa;
+       usbd_devinfo(dev, 1, devinfo);
+       device_set_desc_copy(bdev, devinfo);
        if (device_probe_and_attach(bdev) == 0) {
+               kfree(devinfo, M_USB);
                return (USBD_NORMAL_COMPLETION);
        }
 
@@ -810,6 +820,7 @@ usbd_probe_and_attach(device_t parent, usbd_device_handle dev,
                        kprintf("%s: port %d, set config at addr %d failed\n",
                               device_get_nameunit(parent), port, addr);
 #endif
+                       kfree(devinfo, M_USB);
                        return (err);
                }
                nifaces = dev->cdesc->bNumInterface;
@@ -833,6 +844,8 @@ usbd_probe_and_attach(device_t parent, usbd_device_handle dev,
                        dev->subdevs[found + 1] = 0;
                        dev->ifacenums[found] = i;
                        *uaap = uaa;
+                       usbd_devinfo(dev, 1, devinfo);
+                       device_set_desc_copy(bdev, devinfo);
                        if (device_probe_and_attach(bdev) == 0) {
                                ifaces[i] = 0; /* consumed */
                                found++;
@@ -842,6 +855,7 @@ usbd_probe_and_attach(device_t parent, usbd_device_handle dev,
                                if (!bdev) {
                                        device_printf(parent,
                                            "Device add failed\n");
+                                       kfree(devinfo, M_USB);
                                        return (USBD_NORMAL_COMPLETION);
                                }
                                uaap = kmalloc(sizeof(uaa), M_USB, M_WAITOK);
@@ -853,6 +867,8 @@ usbd_probe_and_attach(device_t parent, usbd_device_handle dev,
                }
                if (found != 0) {
                        /* remove the last created child.  It is unused */
+                       kfree(uaap, M_USB);
+                       kfree(devinfo, M_USB);
                        device_delete_child(parent, bdev);
                        /* kfree(uaap, M_USB); */ /* May be needed? xxx */
                        return (USBD_NORMAL_COMPLETION);
@@ -879,6 +895,9 @@ usbd_probe_and_attach(device_t parent, usbd_device_handle dev,
        dev->subdevs[0] = bdev;
        dev->subdevs[1] = 0;
        *uaap = uaa;
+       usbd_devinfo(dev, 1, devinfo);
+       device_set_desc_copy(bdev, devinfo);
+       kfree(devinfo, M_USB);
        if (device_probe_and_attach(bdev) == 0)
                return (USBD_NORMAL_COMPLETION);
 
index 12ff152..bac1641 100644 (file)
@@ -30,7 +30,7 @@
  * THE POSSIBILITY OF SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/dev/usb/if_aue.c,v 1.78 2003/12/17 14:23:07 sanpei Exp $
- * $DragonFly: src/sys/dev/netif/aue/if_aue.c,v 1.35 2007/11/05 19:09:43 hasso Exp $
+ * $DragonFly: src/sys/dev/netif/aue/if_aue.c,v 1.36 2007/11/06 07:37:00 hasso Exp $
  */
 
 /*
@@ -619,7 +619,6 @@ aue_attach(device_t self)
 {
        struct aue_softc *sc = device_get_softc(self);
        struct usb_attach_arg *uaa = device_get_ivars(self);
-       char                    devinfo[1024];
        u_char                  eaddr[ETHER_ADDR_LEN];
        struct ifnet            *ifp;
        usbd_interface_handle   iface;
@@ -628,8 +627,6 @@ aue_attach(device_t self)
        usb_endpoint_descriptor_t       *ed;
        int                     i;
 
-       usbd_devinfo(uaa->device, 0, devinfo);
-
        sc->aue_udev = uaa->device;
        callout_init(&sc->aue_stat_timer);
 
@@ -653,10 +650,6 @@ aue_attach(device_t self)
 
        id = usbd_get_interface_descriptor(sc->aue_iface);
 
-       usbd_devinfo(uaa->device, 0, devinfo);
-       device_set_desc_copy(self, devinfo);
-       device_printf(self, "%s\n", devinfo);
-
        /* Find endpoints. */
        for (i = 0; i < id->bNumEndpoints; i++) {
                ed = usbd_interface2endpoint_descriptor(iface, i);
index a2133e6..5ef859a 100644 (file)
@@ -30,7 +30,7 @@
  * THE POSSIBILITY OF SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/dev/usb/if_axe.c,v 1.10 2003/12/08 07:54:14 obrien Exp $
- * $DragonFly: src/sys/dev/netif/axe/if_axe.c,v 1.25 2007/11/05 19:09:43 hasso Exp $
+ * $DragonFly: src/sys/dev/netif/axe/if_axe.c,v 1.26 2007/11/06 07:37:00 hasso Exp $
  */
 /*
  * ASIX Electronics AX88172 USB 2.0 ethernet driver. Used in the
@@ -387,7 +387,6 @@ axe_attach(device_t self)
 {
        struct axe_softc *sc = device_get_softc(self);
        struct usb_attach_arg *uaa = device_get_ivars(self);
-       char                    devinfo[1024];
        u_char                  eaddr[ETHER_ADDR_LEN];
        struct ifnet            *ifp;
        usb_interface_descriptor_t      *id;
@@ -411,10 +410,6 @@ axe_attach(device_t self)
 
        id = usbd_get_interface_descriptor(sc->axe_iface);
 
-       usbd_devinfo(uaa->device, 0, devinfo);
-       device_set_desc_copy(self, devinfo);
-       device_printf(self, "%s\n", devinfo);
-
        /* Find endpoints. */
        for (i = 0; i < id->bNumEndpoints; i++) {
                ed = usbd_interface2endpoint_descriptor(sc->axe_iface, i);
index b75ff8f..7c7a890 100644 (file)
@@ -30,7 +30,7 @@
  * THE POSSIBILITY OF SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/dev/usb/if_cue.c,v 1.45 2003/12/08 07:54:14 obrien Exp $
- * $DragonFly: src/sys/dev/netif/cue/if_cue.c,v 1.30 2007/11/05 19:09:43 hasso Exp $
+ * $DragonFly: src/sys/dev/netif/cue/if_cue.c,v 1.31 2007/11/06 07:37:00 hasso Exp $
  */
 
 /*
@@ -411,7 +411,6 @@ cue_attach(device_t self)
 {
        struct cue_softc *sc = device_get_softc(self);
        struct usb_attach_arg *uaa = device_get_ivars(self);
-       char                    devinfo[1024];
        u_char                  eaddr[ETHER_ADDR_LEN];
        struct ifnet            *ifp;
        usb_interface_descriptor_t      *id;
@@ -430,10 +429,6 @@ cue_attach(device_t self)
 
        id = usbd_get_interface_descriptor(uaa->iface);
 
-       usbd_devinfo(uaa->device, 0, devinfo);
-       device_set_desc_copy(self, devinfo);
-       device_printf(self, "%s\n", devinfo);
-
        /* Find endpoints. */
        for (i = 0; i < id->bNumEndpoints; i++) {
                ed = usbd_interface2endpoint_descriptor(uaa->iface, i);
index 7527226..e556eb3 100644 (file)
@@ -30,7 +30,7 @@
  * THE POSSIBILITY OF SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/dev/usb/if_kue.c,v 1.17.2.9 2003/04/13 02:39:25 murray Exp $
- * $DragonFly: src/sys/dev/netif/kue/if_kue.c,v 1.27 2007/11/05 19:09:43 hasso Exp $
+ * $DragonFly: src/sys/dev/netif/kue/if_kue.c,v 1.28 2007/11/06 07:37:00 hasso Exp $
  */
 
 /*
@@ -399,7 +399,6 @@ kue_attach(device_t self)
 {
        struct kue_softc *sc = device_get_softc(self);
        struct usb_attach_arg *uaa = device_get_ivars(self);
-       char                    devinfo[1024];
        struct ifnet            *ifp;
        usbd_status             err;
        usb_interface_descriptor_t      *id;
@@ -412,10 +411,6 @@ kue_attach(device_t self)
 
        id = usbd_get_interface_descriptor(uaa->iface);
 
-       usbd_devinfo(uaa->device, 0, devinfo);
-       device_set_desc_copy(self, devinfo);
-       kprintf("%s: %s\n", device_get_nameunit(self), devinfo);
-
        /* Find endpoints. */
        for (i = 0; i < id->bNumEndpoints; i++) {
                ed = usbd_interface2endpoint_descriptor(uaa->iface, i);
index 7b67c96..dec7bf0 100644 (file)
@@ -55,7 +55,7 @@
  * THE POSSIBILITY OF SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/dev/usb/if_rue.c,v 1.14 2004/06/09 14:34:03 naddy Exp $
- * $DragonFly: src/sys/dev/netif/rue/if_rue.c,v 1.11 2007/11/05 19:09:43 hasso Exp $
+ * $DragonFly: src/sys/dev/netif/rue/if_rue.c,v 1.12 2007/11/06 07:37:00 hasso Exp $
  */
 
 /*
@@ -528,7 +528,6 @@ rue_attach(device_t self)
 {
        struct rue_softc *sc = device_get_softc(self);
        struct usb_attach_arg *uaa = device_get_ivars(self);
-       char                            devinfo[1024];
        uint8_t                         eaddr[ETHER_ADDR_LEN];
        struct ifnet                    *ifp;
        usb_interface_descriptor_t      *id;
@@ -551,10 +550,6 @@ rue_attach(device_t self)
 
        id = usbd_get_interface_descriptor(sc->rue_iface);
 
-       usbd_devinfo(uaa->device, 0, devinfo);
-       device_set_desc_copy(self, devinfo);
-       device_printf(self, "%s\n", devinfo);
-
        /* Find endpoints */
        for (i = 0; i < id->bNumEndpoints; i++) {
                ed = usbd_interface2endpoint_descriptor(sc->rue_iface, i);
index 82a3aa7..5e47e37 100644 (file)
@@ -1,5 +1,5 @@
 /*     $OpenBSD: if_rum.c,v 1.40 2006/09/18 16:20:20 damien Exp $      */
-/*     $DragonFly: src/sys/dev/netif/rum/if_rum.c,v 1.21 2007/11/05 19:09:43 hasso Exp $       */
+/*     $DragonFly: src/sys/dev/netif/rum/if_rum.c,v 1.22 2007/11/06 07:37:00 hasso Exp $       */
 
 /*-
  * Copyright (c) 2005, 2006 Damien Bergamini <damien.bergamini@free.fr>
@@ -259,15 +259,11 @@ rum_attach(device_t self)
        usb_interface_descriptor_t *id;
        usb_endpoint_descriptor_t *ed;
        usbd_status error;
-       char devinfo[1024];
        int i, ntries;
        uint32_t tmp;
 
        sc->sc_udev = uaa->device;
-
-       usbd_devinfo(uaa->device, 0, devinfo);
        sc->sc_dev = self;
-       device_set_desc_copy(self, devinfo);
 
        if (usbd_set_config_no(sc->sc_udev, RT2573_CONFIG_NO, 0) != 0) {
                kprintf("%s: could not set configuration no\n",
index 127c511..486c14b 100644 (file)
@@ -1,5 +1,5 @@
 /*     $FreeBSD: src/sys/dev/usb/if_ural.c,v 1.10.2.8 2006/07/08 07:48:43 maxim Exp $  */
-/*     $DragonFly: src/sys/dev/netif/ural/if_ural.c,v 1.22 2007/11/05 19:09:43 hasso Exp $     */
+/*     $DragonFly: src/sys/dev/netif/ural/if_ural.c,v 1.23 2007/11/06 07:37:00 hasso Exp $     */
 
 /*-
  * Copyright (c) 2005, 2006
@@ -383,15 +383,12 @@ ural_attach(device_t self)
        usb_interface_descriptor_t *id;
        usb_endpoint_descriptor_t *ed;
        usbd_status error;
-       char devinfo[1024];
        int i;
 
        sc->sc_udev = uaa->device;
        sc->sc_tx_retries = 7;  /* TODO tunable/sysctl */
 
-       usbd_devinfo(sc->sc_udev, 0, devinfo);
        sc->sc_dev = self;
-       device_set_desc_copy(self, devinfo);
 
        if (usbd_set_config_no(sc->sc_udev, RAL_CONFIG_NO, 0) != 0) {
                kprintf("%s: could not set configuration no\n",
index cc942f6..bb746b6 100644 (file)
@@ -1,4 +1,4 @@
-/*     $DragonFly: src/sys/dev/usbmisc/uark/uark.c,v 1.5 2007/11/05 19:09:43 hasso Exp $       */
+/*     $DragonFly: src/sys/dev/usbmisc/uark/uark.c,v 1.6 2007/11/06 07:37:01 hasso Exp $       */
 /*     $OpenBSD: uark.c,v 1.9 2007/06/13 06:25:03 mbalmer Exp $        */
 
 /*
@@ -132,24 +132,15 @@ uark_attach(device_t self)
        usb_interface_descriptor_t *id;
        usb_endpoint_descriptor_t *ed;
        usbd_status error;
-       char *devinfo;
        int i;
 
-       devinfo = kmalloc(1024, M_USBDEV, M_INTWAIT);
        ucom = &sc->sc_ucom;
-
        bzero(sc, sizeof (struct uark_softc));
 
-       usbd_devinfo(uaa->device, 0, devinfo);
        ucom->sc_dev = self;
-       device_set_desc_copy(self, devinfo);
-
        ucom->sc_udev = uaa->device;
        ucom->sc_iface = uaa->iface;
 
-       device_printf(ucom->sc_dev, "%s\n", devinfo);
-       kfree(devinfo, M_USBDEV);
-
        if (usbd_set_config_index(ucom->sc_udev, UARK_CONFIG_NO, 1) != 0) {
                device_printf(ucom->sc_dev, "could not set configuration no\n");
                goto error;
index 103c4d4..81c3843 100644 (file)
@@ -60,7 +60,7 @@
  * POSSIBILITY OF SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/dev/usb/ubsa.c,v 1.11 2003/11/16 12:13:39 akiyama Exp $
- * $DragonFly: src/sys/dev/usbmisc/ubsa/ubsa.c,v 1.18 2007/11/05 19:09:43 hasso Exp $
+ * $DragonFly: src/sys/dev/usbmisc/ubsa/ubsa.c,v 1.19 2007/11/06 07:37:01 hasso Exp $
  */
 
 #include <sys/param.h>
@@ -258,14 +258,11 @@ ubsa_attach(device_t self)
        usb_config_descriptor_t *cdesc;
        usb_interface_descriptor_t *id;
        usb_endpoint_descriptor_t *ed;
-       char *devinfo;
        usbd_status err;
        int i;
 
        dev = uaa->device;
-       devinfo = kmalloc(1024, M_USBDEV, M_INTWAIT);
        ucom = &sc->sc_ucom;
-
        bzero(sc, sizeof (struct ubsa_softc));
 
        /*
@@ -275,15 +272,10 @@ ubsa_attach(device_t self)
        sc->sc_dtr = -1;
        sc->sc_rts = -1;
 
-       usbd_devinfo(dev, 0, devinfo);
        ucom->sc_dev = self;
-       device_set_desc_copy(self, devinfo);
-
        ucom->sc_udev = dev;
        ucom->sc_iface = uaa->iface;
 
-       device_printf(ucom->sc_dev, "%s\n", devinfo);
-
        DPRINTF(("ubsa attach: sc = %p\n", sc));
 
        /* initialize endpoints */
@@ -382,11 +374,9 @@ ubsa_attach(device_t self)
 
        ucom_attach(ucom);
 
-       kfree(devinfo, M_USBDEV);
        return 0;
 
 error:
-       kfree(devinfo, M_USBDEV);
        return ENXIO;
 }
 
index 0f61289..4667837 100644 (file)
@@ -27,7 +27,7 @@
  * POSSIBILITY OF SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/dev/usb/udbp.c,v 1.24 2003/08/24 17:55:55 obrien Exp $
- * $DragonFly: src/sys/dev/usbmisc/udbp/Attic/udbp.c,v 1.17 2007/11/05 19:09:43 hasso Exp $
+ * $DragonFly: src/sys/dev/usbmisc/udbp/Attic/udbp.c,v 1.18 2007/11/06 07:37:01 hasso Exp $
  */
 
 /* Driver for arbitrary double bulk pipe devices.
@@ -262,7 +262,6 @@ udbp_attach(device_t self)
        usb_interface_descriptor_t *id;
        usb_endpoint_descriptor_t *ed, *ed_bulkin = NULL, *ed_bulkout = NULL;
        usbd_status err;
-       char devinfo[1024];
        int i;
        static int ngudbp_done_init=0;
 
@@ -270,11 +269,7 @@ udbp_attach(device_t self)
        /* fetch the interface handle for the first interface */
        (void) usbd_device2interface_handle(uaa->device, 0, &iface);
        id = usbd_get_interface_descriptor(iface);
-       usbd_devinfo(uaa->device, 0, devinfo);
        sc->sc_dev = self;
-       device_set_desc_copy(self, devinfo);
-       kprintf("%s: %s, iclass %d/%d\n", device_get_nameunit(sc->sc_dev),
-              devinfo, id->bInterfaceClass, id->bInterfaceSubClass);
 
        /* Find the two first bulk endpoints */
        for (i = 0 ; i < id->bNumEndpoints; i++) {
index 36aa32d..0b5bb35 100644 (file)
@@ -30,7 +30,7 @@
 
 /*
  * $FreeBSD: src/sys/dev/usb/ufm.c,v 1.16 2003/10/04 21:41:01 joe Exp $
- * $DragonFly: src/sys/dev/usbmisc/ufm/ufm.c,v 1.22 2007/11/05 19:09:43 hasso Exp $
+ * $DragonFly: src/sys/dev/usbmisc/ufm/ufm.c,v 1.23 2007/11/06 07:37:01 hasso Exp $
  */
 
 #include <sys/param.h>
@@ -145,7 +145,6 @@ ufm_attach(device_t self)
 {
        struct ufm_softc *sc = device_get_softc(self);
        struct usb_attach_arg *uaa = device_get_ivars(self);
-       char devinfo[1024];
        usb_endpoint_descriptor_t *edesc;
        usbd_device_handle udev;
        usbd_interface_handle iface;
@@ -154,11 +153,7 @@ ufm_attach(device_t self)
        char * ermsg = "<none>";
 
        DPRINTFN(10,("ufm_attach: sc=%p\n", sc));
-       usbd_devinfo(uaa->device, 0, devinfo);
        sc->sc_dev = self;
-       device_set_desc_copy(self, devinfo);
-       kprintf("%s: %s\n", device_get_nameunit(sc->sc_dev), devinfo);
-
        sc->sc_udev = udev = uaa->device;
 
        if ((!uaa->device) || (!uaa->iface)) {
index ef2c373..666def3 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * $NetBSD: uftdi.c,v 1.13 2002/09/23 05:51:23 simonb Exp $
  * $FreeBSD: src/sys/dev/usb/uftdi.c,v 1.37 2007/06/22 05:53:05 imp Exp $
- * $DragonFly: src/sys/dev/usbmisc/uftdi/uftdi.c,v 1.22 2007/11/05 19:09:43 hasso Exp $
+ * $DragonFly: src/sys/dev/usbmisc/uftdi/uftdi.c,v 1.23 2007/11/06 07:37:01 hasso Exp $
  */
 
 /*-
@@ -536,19 +536,13 @@ uftdi_attach(device_t self)
        usbd_interface_handle iface;
        usb_interface_descriptor_t *id;
        usb_endpoint_descriptor_t *ed;
-       char *devinfo;
        int i;
        struct ucom_softc *ucom = &sc->sc_ucom;
        DPRINTFN(10,("\nuftdi_attach: sc=%p\n", sc));
-       devinfo = kmalloc(1024, M_USBDEV, M_INTWAIT);
 
        ucom->sc_dev = self;
        ucom->sc_udev = dev;
 
-       usbd_devinfo(dev, 0, devinfo);
-       device_printf(ucom->sc_dev, "%s\n", devinfo);
-       kfree(devinfo, M_USBDEV);
-
        iface = uaa->iface;
        id = usbd_get_interface_descriptor(iface);
        ucom->sc_iface = iface;
index 9423348..a6631fd 100644 (file)
@@ -2,7 +2,7 @@
  * $NetBSD: ugen.c,v 1.27 1999/10/28 12:08:38 augustss Exp $
  * $NetBSD: ugen.c,v 1.59 2002/07/11 21:14:28 augustss Exp $
  * $FreeBSD: src/sys/dev/usb/ugen.c,v 1.81 2003/11/09 09:17:22 tanimura Exp $
- * $DragonFly: src/sys/dev/usbmisc/ugen/ugen.c,v 1.32 2007/07/03 19:28:16 hasso Exp $
+ * $DragonFly: src/sys/dev/usbmisc/ugen/ugen.c,v 1.33 2007/11/06 07:37:01 hasso Exp $
  */
 
 /* 
@@ -217,15 +217,10 @@ ugen_attach(device_t self)
        struct ugen_softc *sc = device_get_softc(self);
        struct usb_attach_arg *uaa = device_get_ivars(self);
        usbd_device_handle udev;
-       char devinfo[1024];
        usbd_status err;
        int conf;
 
-       usbd_devinfo(uaa->device, 0, devinfo);
        sc->sc_dev = self;
-       device_set_desc_copy(self, devinfo);
-       kprintf("%s: %s\n", device_get_nameunit(sc->sc_dev), devinfo);
-
        sc->sc_udev = udev = uaa->device;
 
        memset(sc->sc_endpoints, 0, sizeof sc->sc_endpoints);
index 0ec5226..e8c3b22 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * $NetBSD: uhid.c,v 1.46 2001/11/13 06:24:55 lukem Exp $
  * $FreeBSD: src/sys/dev/usb/uhid.c,v 1.65 2003/11/09 09:17:22 tanimura Exp $
- * $DragonFly: src/sys/dev/usbmisc/uhid/uhid.c,v 1.31 2007/11/05 19:09:43 hasso Exp $
+ * $DragonFly: src/sys/dev/usbmisc/uhid/uhid.c,v 1.32 2007/11/06 07:37:01 hasso Exp $
  */
 
 /* Also already merged from NetBSD:
@@ -202,16 +202,11 @@ uhid_attach(device_t self)
        int size;
        void *desc;
        usbd_status err;
-       char devinfo[1024];
 
        sc->sc_udev = uaa->device;
        sc->sc_iface = iface;
        id = usbd_get_interface_descriptor(iface);
-       usbd_devinfo(uaa->device, 0, devinfo);
        sc->sc_dev = self;
-       device_set_desc_copy(self, devinfo);
-       kprintf("%s: %s, iclass %d/%d\n", device_get_nameunit(sc->sc_dev),
-              devinfo, id->bInterfaceClass, id->bInterfaceSubClass);
 
        ed = usbd_interface2endpoint_descriptor(iface, 0);
        if (ed == NULL) {
index ed660de..4cb6d2e 100644 (file)
@@ -1,6 +1,6 @@
 /*
  * $FreeBSD: src/sys/dev/usb/ukbd.c,v 1.45 2003/10/04 21:41:01 joe Exp $
- * $DragonFly: src/sys/dev/usbmisc/ukbd/ukbd.c,v 1.25 2007/11/05 19:09:43 hasso Exp $
+ * $DragonFly: src/sys/dev/usbmisc/ukbd/ukbd.c,v 1.26 2007/11/06 07:37:01 hasso Exp $
  */
 
 /*
@@ -174,7 +174,6 @@ ukbd_attach(device_t self)
        struct usb_attach_arg *uaa = device_get_ivars(self);
        usbd_interface_handle iface = uaa->iface;
        usb_interface_descriptor_t *id;
-       char devinfo[1024];
 
        keyboard_switch_t *sw;
        keyboard_t *kbd;
@@ -186,11 +185,7 @@ ukbd_attach(device_t self)
                return ENXIO;
 
        id = usbd_get_interface_descriptor(iface);
-       usbd_devinfo(uaa->device, 0, devinfo);
        sc->sc_dev = self;
-       device_set_desc_copy(self, devinfo);
-       kprintf("%s: %s, iclass %d/%d\n", device_get_nameunit(sc->sc_dev),
-              devinfo, id->bInterfaceClass, id->bInterfaceSubClass);
 
        arg[0] = (void *)uaa;
        arg[1] = (void *)ukbd_intr;
index e30b1cb..c8161d4 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * $NetBSD: ulpt.c,v 1.55 2002/10/23 09:14:01 jdolecek Exp $
  * $FreeBSD: src/sys/dev/usb/ulpt.c,v 1.59 2003/09/28 20:48:13 phk Exp $
- * $DragonFly: src/sys/dev/usbmisc/ulpt/ulpt.c,v 1.23 2007/11/05 19:09:43 hasso Exp $
+ * $DragonFly: src/sys/dev/usbmisc/ulpt/ulpt.c,v 1.24 2007/11/06 07:37:01 hasso Exp $
  */
 
 /*
@@ -200,17 +200,12 @@ ulpt_attach(device_t self)
        usb_interface_descriptor_t *id, *iend;
        usb_config_descriptor_t *cdesc;
        usbd_status err;
-       char devinfo[1024];
        usb_endpoint_descriptor_t *ed;
        u_int8_t epcount;
        int i, altno;
 
        DPRINTFN(10,("ulpt_attach: sc=%p\n", sc));
-       usbd_devinfo(dev, 0, devinfo);
        sc->sc_dev = self;
-       device_set_desc_copy(self, devinfo);
-       kprintf("%s: %s, iclass %d/%d\n", device_get_nameunit(sc->sc_dev),
-              devinfo, ifcd->bInterfaceClass, ifcd->bInterfaceSubClass);
 
        /* XXX
         * Stepping through the alternate settings needs to be abstracted out.
@@ -297,42 +292,6 @@ ulpt_attach(device_t self)
        sc->sc_ifaceno = id->bInterfaceNumber;
        sc->sc_udev = dev;
 
-#if 0
-/*
- * This code is disabled because for some mysterious reason it causes
- * printing not to work.  But only sometimes, and mostly with
- * UHCI and less often with OHCI.  *sigh*
- */
-       {
-       usb_config_descriptor_t *cd = usbd_get_config_descriptor(dev);
-       usb_device_request_t req;
-       int len, alen;
-
-       req.bmRequestType = UT_READ_CLASS_INTERFACE;
-       req.bRequest = UR_GET_DEVICE_ID;
-       USETW(req.wValue, cd->bConfigurationValue);
-       USETW2(req.wIndex, id->bInterfaceNumber, id->bAlternateSetting);
-       USETW(req.wLength, sizeof devinfo - 1);
-       err = usbd_do_request_flags(dev, &req, devinfo, USBD_SHORT_XFER_OK,
-                 &alen, USBD_DEFAULT_TIMEOUT);
-       if (err) {
-               kprintf("%s: cannot get device id\n", device_get_nameunit(sc->sc_dev));
-       } else if (alen <= 2) {
-               kprintf("%s: empty device id, no printer connected?\n",
-                      device_get_nameunit(sc->sc_dev));
-       } else {
-               /* devinfo now contains an IEEE-1284 device ID */
-               len = ((devinfo[0] & 0xff) << 8) | (devinfo[1] & 0xff);
-               if (len > sizeof devinfo - 3)
-                       len = sizeof devinfo - 3;
-               devinfo[len] = 0;
-               kprintf("%s: device id <", device_get_nameunit(sc->sc_dev));
-               ieee1284_print_id(devinfo+2);
-               kprintf(">\n");
-       }
-       }
-#endif
-
        dev_ops_add(&ulpt_ops, -1, device_get_unit(self));
        make_dev(&ulpt_ops, device_get_unit(self),
                UID_ROOT, GID_OPERATOR, 0644, "ulpt%d", device_get_unit(self));
index 13de9a4..3ebbd99 100644 (file)
@@ -26,7 +26,7 @@
  *
  * $NetBSD: umass.c,v 1.28 2000/04/02 23:46:53 augustss Exp $
  * $FreeBSD: src/sys/dev/usb/umass.c,v 1.96 2003/12/19 12:19:11 sanpei Exp $
- * $DragonFly: src/sys/dev/usbmisc/umass/umass.c,v 1.29 2007/11/05 19:09:43 hasso Exp $
+ * $DragonFly: src/sys/dev/usbmisc/umass/umass.c,v 1.30 2007/11/06 07:37:01 hasso Exp $
  */
 
 /*
@@ -853,7 +853,6 @@ umass_attach(device_t self)
        struct usb_attach_arg *uaa = device_get_ivars(self);
        usb_interface_descriptor_t *id;
        usb_endpoint_descriptor_t *ed;
-       char devinfo[1024];
        int i;
        int err;
 
@@ -862,9 +861,7 @@ umass_attach(device_t self)
         * call umass_detach without specifically initialising the struct.
         */
 
-       usbd_devinfo(uaa->device, 0, devinfo);
        sc->sc_dev = self;
-       device_set_desc_copy(self, devinfo);
 
        sc->iface = uaa->iface;
        sc->ifaceno = uaa->ifaceno;
@@ -873,7 +870,6 @@ umass_attach(device_t self)
        (void) umass_match_proto(sc, sc->iface, uaa->device);
 
        id = usbd_get_interface_descriptor(sc->iface);
-       kprintf("%s: %s\n", device_get_nameunit(sc->sc_dev), devinfo);
 #ifdef USB_DEBUG
        kprintf("%s: ", device_get_nameunit(sc->sc_dev));
        switch (sc->proto&UMASS_PROTO_COMMAND) {
index 0fdc541..ccee369 100644 (file)
@@ -24,7 +24,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/dev/usb/umct.c,v 1.12 2006/09/07 00:06:42 imp Exp $
- * $DragonFly: src/sys/dev/usbmisc/umct/umct.c,v 1.14 2007/11/05 19:09:44 hasso Exp $
+ * $DragonFly: src/sys/dev/usbmisc/umct/umct.c,v 1.15 2007/11/06 07:37:01 hasso Exp $
  */
 
 /*
@@ -158,22 +158,16 @@ umct_attach(device_t self)
        usb_config_descriptor_t *cdesc;
        usb_interface_descriptor_t *id;
        usb_endpoint_descriptor_t *ed;
-       char *devinfo;
        usbd_status err;
        int i;
 
        dev = uaa->device;
-       devinfo = kmalloc(1024, M_USBDEV, M_INTWAIT | M_ZERO);
        bzero(sc, sizeof(struct umct_softc));
        ucom = &sc->sc_ucom;
        ucom->sc_dev = self;
        ucom->sc_udev = dev;
        ucom->sc_iface = uaa->iface;
 
-       usbd_devinfo(dev, 0, devinfo);
-       device_set_desc_copy(self, devinfo);
-       device_printf(ucom->sc_dev, "%s\n", devinfo);
-
        ucom->sc_bulkout_no = -1;
        ucom->sc_bulkin_no = -1;
        sc->sc_intr_number = -1;
@@ -269,11 +263,9 @@ umct_attach(device_t self)
        TASK_INIT(&sc->sc_task, 0, umct_notify, sc);
        ucom_attach(ucom);
 
-       kfree(devinfo, M_USBDEV);
        return 0;
 
 error:
-       kfree(devinfo, M_USBDEV);
        return ENXIO;
 }
 
index 5c0134c..8812025 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * $NetBSD: umodem.c,v 1.45 2002/09/23 05:51:23 simonb Exp $
  * $FreeBSD: src/sys/dev/usb/umodem.c,v 1.48 2003/08/24 17:55:55 obrien Exp $
- * $DragonFly: src/sys/dev/usbmisc/umodem/umodem.c,v 1.21 2007/11/05 19:09:44 hasso Exp $
+ * $DragonFly: src/sys/dev/usbmisc/umodem/umodem.c,v 1.22 2007/11/06 07:37:01 hasso Exp $
  */
 
 /*-
@@ -241,19 +241,15 @@ umodem_attach(device_t self)
        usbd_device_handle dev = uaa->device;
        usb_interface_descriptor_t *id;
        usb_endpoint_descriptor_t *ed;
-       char *devinfo = NULL;
        const char *devname;
        usbd_status err;
        int data_ifcno;
        int i;
        struct ucom_softc *ucom;
 
-       devinfo = kmalloc(1024, M_USBDEV, M_INTWAIT);
-       usbd_devinfo(dev, 0, devinfo);
        ucom = &sc->sc_ucom;
        ucom->sc_dev = self;
        sc->sc_dev = self;
-       device_set_desc_copy(self, devinfo);
        ucom->sc_udev = dev;
        ucom->sc_iface = uaa->iface;
 
@@ -263,8 +259,6 @@ umodem_attach(device_t self)
        devname = device_get_nameunit(sc->sc_dev);
        /* XXX ? use something else ? XXX */
        id = usbd_get_interface_descriptor(sc->sc_ctl_iface);
-       kprintf("%s: %s, iclass %d/%d\n", devname, devinfo,
-         id->bInterfaceClass, id->bInterfaceSubClass);
        sc->sc_ctl_iface_no = id->bInterfaceNumber;
 
        sc->sc_data_iface_no = data_ifcno =
@@ -384,12 +378,10 @@ umodem_attach(device_t self)
 
        ucom_attach(&sc->sc_ucom);
 
-       kfree(devinfo, M_USBDEV);
        return 0;
 
  bad:
        ucom->sc_dying = 1;
-       kfree(devinfo, M_USBDEV);
        return ENXIO;
 }
 
index 17eadaa..165fbb7 100644 (file)
@@ -1,6 +1,6 @@
 /*
  * $FreeBSD: src/sys/dev/usb/ums.c,v 1.64 2003/11/09 09:17:22 tanimura Exp $
- * $DragonFly: src/sys/dev/usbmisc/ums/ums.c,v 1.29 2007/11/05 19:09:44 hasso Exp $
+ * $DragonFly: src/sys/dev/usbmisc/ums/ums.c,v 1.30 2007/11/06 07:37:01 hasso Exp $
  */
 
 /*
@@ -215,7 +215,6 @@ ums_attach(device_t self)
        int size;
        void *desc;
        usbd_status err;
-       char devinfo[1024];
        u_int32_t flags;
        int i;
        struct hid_location loc_btn;
@@ -223,11 +222,7 @@ ums_attach(device_t self)
        sc->sc_disconnected = 1;
        sc->sc_iface = iface;
        id = usbd_get_interface_descriptor(iface);
-       usbd_devinfo(uaa->device, 0, devinfo);
        sc->sc_dev = self;
-       device_set_desc_copy(self, devinfo);
-       kprintf("%s: %s, iclass %d/%d\n", device_get_nameunit(sc->sc_dev),
-              devinfo, id->bInterfaceClass, id->bInterfaceSubClass);
        ed = usbd_interface2endpoint_descriptor(iface, 0);
        if (!ed) {
                kprintf("%s: could not read endpoint descriptor\n",
index 52e6750..a4fcea3 100644 (file)
@@ -1,4 +1,4 @@
-/*     $DragonFly: src/sys/dev/usbmisc/umsm/Attic/umsm.c,v 1.4 2007/11/05 19:09:44 hasso Exp $ */
+/*     $DragonFly: src/sys/dev/usbmisc/umsm/Attic/umsm.c,v 1.5 2007/11/06 07:37:01 hasso Exp $ */
 /*     $OpenBSD: umsm.c,v 1.15 2007/06/14 10:11:16 mbalmer Exp $       */
 
 /*
@@ -129,25 +129,17 @@ umsm_attach(device_t self)
        usb_interface_descriptor_t *id;
        usb_endpoint_descriptor_t *ed;
        usbd_status error;
-       char *devinfo;
        const char *devname;
        int i;
 
-       devinfo = kmalloc(1024, M_USBDEV, M_INTWAIT);
        ucom = &sc->sc_ucom;
-
        bzero(sc, sizeof (struct umsm_softc));
 
-       usbd_devinfo(uaa->device, 0, devinfo);
        ucom->sc_dev = self;
-       device_set_desc_copy(self, devinfo);
-
        ucom->sc_udev = uaa->device;
        ucom->sc_iface = uaa->iface;
 
        devname = device_get_nameunit(ucom->sc_dev);
-       kprintf("%s: %s\n", devname, devinfo);
-       kfree(devinfo, M_USBDEV);
 
        if (usbd_set_config_index(ucom->sc_udev, UMSM_CONFIG_NO, 1) != 0) {
                device_printf(ucom->sc_dev, "could not set configuration no\n");
index d089a9a..a6313ed 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * $NetBSD: uplcom.c,v 1.21 2001/11/13 06:24:56 lukem Exp $
  * $FreeBSD: src/sys/dev/usb/uplcom.c,v 1.39 2006/09/07 00:06:42 imp Exp $
- * $DragonFly: src/sys/dev/usbmisc/uplcom/uplcom.c,v 1.24 2007/11/05 19:09:44 hasso Exp $
+ * $DragonFly: src/sys/dev/usbmisc/uplcom/uplcom.c,v 1.25 2007/11/06 07:37:01 hasso Exp $
  */
 
 /*-
@@ -324,24 +324,16 @@ uplcom_attach(device_t self)
        usb_config_descriptor_t *cdesc;
        usb_interface_descriptor_t *id;
        usb_endpoint_descriptor_t *ed;
-       char *devinfo;
        usbd_status err;
        int i;
 
-       devinfo = kmalloc(1024, M_USBDEV, M_INTWAIT);
        ucom = &sc->sc_ucom;
-
        bzero(sc, sizeof (struct uplcom_softc));
 
-       usbd_devinfo(dev, 0, devinfo);
        ucom->sc_dev = self;
-       device_set_desc_copy(self, devinfo);
-
        ucom->sc_udev = dev;
        ucom->sc_iface = uaa->iface;
 
-       device_printf(ucom->sc_dev, "%s\n", devinfo);
-
        DPRINTF(("uplcom attach: sc = %p\n", sc));
 
        dd = usbd_get_device_descriptor(uaa->device);
@@ -519,11 +511,9 @@ uplcom_attach(device_t self)
        TASK_INIT(&sc->sc_task, 0, uplcom_notify, sc);
        ucom_attach(&sc->sc_ucom);
 
-       kfree(devinfo, M_USBDEV);
        return 0;
 
 error:
-       kfree(devinfo, M_USBDEV);
        return ENXIO;
 }
 
index 10dfa3a..46de685 100644 (file)
@@ -30,7 +30,7 @@
 
 /*
  * $FreeBSD: src/sys/dev/usb/urio.c,v 1.28 2003/08/25 22:01:06 joe Exp $
- * $DragonFly: src/sys/dev/usbmisc/urio/urio.c,v 1.25 2007/11/05 19:09:44 hasso Exp $
+ * $DragonFly: src/sys/dev/usbmisc/urio/urio.c,v 1.26 2007/11/06 07:37:01 hasso Exp $
  */
 
 /*
@@ -173,7 +173,6 @@ urio_attach(device_t self)
 {
        struct urio_softc *sc = device_get_softc(self);
        struct usb_attach_arg *uaa = device_get_ivars(self);
-       char devinfo[1024];
        usbd_device_handle udev;
        usbd_interface_handle iface;
        u_int8_t epcount;
@@ -182,11 +181,7 @@ urio_attach(device_t self)
        int i;
 
        DPRINTFN(10,("urio_attach: sc=%p\n", sc));
-       usbd_devinfo(uaa->device, 0, devinfo);
        sc->sc_dev = self;
-       device_set_desc_copy(self, devinfo);
-       kprintf("%s: %s\n", device_get_nameunit(sc->sc_dev), devinfo);
-
        sc->sc_udev = udev = uaa->device;
 
        if ((!uaa->device) || (!uaa->iface)) {
index e4a33a4..8af5706 100644 (file)
@@ -1,7 +1,7 @@
 /* 
  * $NetBSD: uscanner.c,v 1.30 2002/07/11 21:14:36 augustss Exp $
  * $FreeBSD: src/sys/dev/usb/uscanner.c,v 1.48 2003/12/22 19:58:27 sanpei Exp $
- * $DragonFly: src/sys/dev/usbmisc/uscanner/uscanner.c,v 1.25 2007/11/05 19:09:44 hasso Exp $
+ * $DragonFly: src/sys/dev/usbmisc/uscanner/uscanner.c,v 1.26 2007/11/06 07:37:01 hasso Exp $
  */
 
 /* Also already merged from NetBSD:
@@ -292,14 +292,10 @@ uscanner_attach(device_t self)
        struct usb_attach_arg *uaa = device_get_ivars(self);
        usb_interface_descriptor_t *id = 0;
        usb_endpoint_descriptor_t *ed, *ed_bulkin = NULL, *ed_bulkout = NULL;
-       char devinfo[1024];
        int i;
        usbd_status err;
 
-       usbd_devinfo(uaa->device, 0, devinfo);
        sc->sc_dev = self;
-       device_set_desc_copy(self, devinfo);
-       kprintf("%s: %s\n", device_get_nameunit(sc->sc_dev), devinfo);
 
        sc->sc_dev_flags = uscanner_lookup(uaa->vendor, uaa->product)->flags;
 
index 83aa1fb..d441f69 100644 (file)
@@ -1,4 +1,4 @@
-/*     $DragonFly: src/sys/dev/usbmisc/uslcom/uslcom.c,v 1.7 2007/11/05 19:09:44 hasso Exp $   */
+/*     $DragonFly: src/sys/dev/usbmisc/uslcom/uslcom.c,v 1.8 2007/11/06 07:37:01 hasso Exp $   */
 /*     $OpenBSD: uslcom.c,v 1.12 2007/06/13 06:25:03 mbalmer Exp $     */
 
 /*
@@ -178,24 +178,16 @@ uslcom_attach(device_t self)
        usb_interface_descriptor_t *id;
        usb_endpoint_descriptor_t *ed;
        usbd_status error;
-       char *devinfo;
        int i;
 
-       devinfo = kmalloc(1024, M_USBDEV, M_INTWAIT);
        ucom = &sc->sc_ucom;
 
        bzero(sc, sizeof (struct uslcom_softc));
 
-       usbd_devinfo(uaa->device, 0, devinfo);
        ucom->sc_dev = self;
-       device_set_desc_copy(self, devinfo);
-
        ucom->sc_udev = uaa->device;
        ucom->sc_iface = uaa->iface;
 
-       device_printf(ucom->sc_dev, "%s\n", devinfo);
-       kfree(devinfo, M_USBDEV);
-
        if (usbd_set_config_index(ucom->sc_udev, USLCOM_CONFIG_NO, 1) != 0) {
                device_printf(ucom->sc_dev, "could not set configuration no\n");
                goto error;
index 1252d4a..8353970 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * $NetBSD: uvisor.c,v 1.9 2001/01/23 14:04:14 augustss Exp $
  * $FreeBSD: src/sys/dev/usb/uvisor.c,v 1.16 2003/11/08 11:23:07 joe Exp $
- * $DragonFly: src/sys/dev/usbmisc/uvisor/uvisor.c,v 1.15 2007/11/05 19:09:44 hasso Exp $
+ * $DragonFly: src/sys/dev/usbmisc/uvisor/uvisor.c,v 1.16 2007/11/06 07:37:01 hasso Exp $
  */
 
 /*
@@ -246,26 +246,20 @@ uvisor_attach(device_t self)
        usbd_interface_handle iface;
        usb_interface_descriptor_t *id;
        usb_endpoint_descriptor_t *ed;
-       char *devinfo;
        const char *devname;
        int i;
        usbd_status err;
        struct ucom_softc *ucom;
 
-       devinfo = kmalloc(1024, M_USBDEV, M_INTWAIT);
        ucom = &sc->sc_ucom;
 
        bzero(sc, sizeof (struct uvisor_softc));
-       usbd_devinfo(dev, 0, devinfo);
 
        ucom->sc_dev = self;
-       device_set_desc_copy(self, devinfo);
-
        ucom->sc_udev = dev;
        ucom->sc_iface = uaa->iface;
 
        devname = device_get_nameunit(ucom->sc_dev);
-       kprintf("%s: %s\n", devname, devinfo);
 
        DPRINTFN(10,("\nuvisor_attach: sc=%p\n", sc));
 
@@ -284,8 +278,6 @@ uvisor_attach(device_t self)
                goto bad;
        }
 
-       kprintf("%s: %s\n", devname, devinfo);
-
        sc->sc_flags = uvisor_lookup(uaa->vendor, uaa->product)->uv_flags;
 
        id = usbd_get_interface_descriptor(iface);
index cc03e98..c7283a5 100644 (file)
@@ -25,7 +25,7 @@
  *
  * $NetBSD: usb/uvscom.c,v 1.1 2002/03/19 15:08:42 augustss Exp $
  * $FreeBSD: src/sys/dev/usb/uvscom.c,v 1.19 2003/11/16 12:26:10 akiyama Exp $
- * $DragonFly: src/sys/dev/usbmisc/uvscom/uvscom.c,v 1.18 2007/11/05 19:09:44 hasso Exp $
+ * $DragonFly: src/sys/dev/usbmisc/uvscom/uvscom.c,v 1.19 2007/11/06 07:37:02 hasso Exp $
  */
 
 /*
@@ -248,25 +248,19 @@ uvscom_attach(device_t self)
        usb_config_descriptor_t *cdesc;
        usb_interface_descriptor_t *id;
        usb_endpoint_descriptor_t *ed;
-       char *devinfo;
        const char *devname;
        usbd_status err;
        int i;
 
-       devinfo = kmalloc(1024, M_USBDEV, M_INTWAIT);
        ucom = &sc->sc_ucom;
 
        bzero(sc, sizeof (struct uvscom_softc));
 
-       usbd_devinfo(dev, 0, devinfo);
        ucom->sc_dev = self;
-       device_set_desc_copy(self, devinfo);
-
        ucom->sc_udev = dev;
        ucom->sc_iface = uaa->iface;
 
        devname = device_get_nameunit(ucom->sc_dev);
-       kprintf("%s: %s\n", devname, devinfo);
 
        DPRINTF(("uvscom attach: sc = %p\n", sc));
 
@@ -367,12 +361,10 @@ uvscom_attach(device_t self)
 
        ucom_attach(&sc->sc_ucom);
 
-       kfree(devinfo, M_USBDEV);
        return 0;
 
 error:
        ucom->sc_dying = 1;
-       kfree(devinfo, M_USBDEV);
        return ENXIO;
 }