Remove some useless variables and assignments from USB code.
authorHasso Tepper <hasso@dragonflybsd.org>
Thu, 14 Aug 2008 20:55:54 +0000 (20:55 +0000)
committerHasso Tepper <hasso@dragonflybsd.org>
Thu, 14 Aug 2008 20:55:54 +0000 (20:55 +0000)
Found-by: LLVM/Clang Static Analyzer
sys/bus/usb/ehci.c
sys/bus/usb/ohci.c
sys/dev/usbmisc/ufm/ufm.c
sys/dev/usbmisc/uhid/uhid.c
sys/dev/usbmisc/ukbd/ukbd.c
sys/dev/usbmisc/ums/ums.c
sys/dev/usbmisc/urio/urio.c

index eb968f8..628d64c 100644 (file)
@@ -1,6 +1,6 @@
 /*     $NetBSD: ehci.c,v 1.91 2005/02/27 00:27:51 perry Exp $ */
 /*     $FreeBSD: src/sys/dev/usb/ehci.c,v 1.36.2.3 2006/09/24 13:39:04 iedowse Exp $   */
-/*     $DragonFly: src/sys/bus/usb/ehci.c,v 1.35 2008/06/10 10:04:05 hasso Exp $       */
+/*     $DragonFly: src/sys/bus/usb/ehci.c,v 1.36 2008/08/14 20:55:53 hasso Exp $       */
 
 /*
  * Copyright (c) 2004 The NetBSD Foundation, Inc.
@@ -587,7 +587,6 @@ ehci_intr1(ehci_softc_t *sc)
 void
 ehci_pcd(ehci_softc_t *sc, usbd_xfer_handle xfer)
 {
-       usbd_pipe_handle pipe;
        u_char *p;
        int i, m;
 
@@ -596,8 +595,6 @@ ehci_pcd(ehci_softc_t *sc, usbd_xfer_handle xfer)
                return;
        }
 
-       pipe = xfer->pipe;
-
        p = KERNADDR(&xfer->dmabuf, 0);
        m = min(sc->sc_noport, xfer->length * 8 - 1);
        memset(p, 0, xfer->length);
@@ -2446,7 +2443,6 @@ ehci_abort_xfer(usbd_xfer_handle xfer, usbd_status status)
                 * (if there is one). We only need to do this if
                 * it was previously pointing to us.
                 */
-               sqtd = exfer->sqtdstart;
                for (sqtd = exfer->sqtdstart; ; sqtd = sqtd->nextqtd) {
                        if (cur == sqtd->physaddr) {
                                hit++;
index 25b1143..67362a8 100644 (file)
@@ -1,6 +1,6 @@
 /*     $NetBSD: ohci.c,v 1.138 2003/02/08 03:32:50 ichiro Exp $        */
 /*     $FreeBSD: src/sys/dev/usb/ohci.c,v 1.154.2.4 2006/06/26 00:31:25 iedowse Exp $  */
-/*     $DragonFly: src/sys/bus/usb/ohci.c,v 1.27 2008/05/24 09:11:09 swildner Exp $    */
+/*     $DragonFly: src/sys/bus/usb/ohci.c,v 1.28 2008/08/14 20:55:53 hasso Exp $       */
 
 /* Also, already ported:
  *     $NetBSD: ohci.c,v 1.140 2003/05/13 04:42:00 gson Exp $
@@ -1537,7 +1537,6 @@ ohci_device_bulk_done(usbd_xfer_handle xfer)
 void
 ohci_rhsc(ohci_softc_t *sc, usbd_xfer_handle xfer)
 {
-       usbd_pipe_handle pipe;
        u_char *p;
        int i, m;
        int hstatus;
@@ -1551,8 +1550,6 @@ ohci_rhsc(ohci_softc_t *sc, usbd_xfer_handle xfer)
                return;
        }
 
-       pipe = xfer->pipe;
-
        p = KERNADDR(&xfer->dmabuf, 0);
        m = min(sc->sc_noport, xfer->length * 8 - 1);
        memset(p, 0, xfer->length);
index 0b5bb35..12e4c3d 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.23 2007/11/06 07:37:01 hasso Exp $
+ * $DragonFly: src/sys/dev/usbmisc/ufm/ufm.c,v 1.24 2008/08/14 20:55:53 hasso Exp $
  */
 
 #include <sys/param.h>
@@ -146,7 +146,6 @@ ufm_attach(device_t self)
        struct ufm_softc *sc = device_get_softc(self);
        struct usb_attach_arg *uaa = device_get_ivars(self);
        usb_endpoint_descriptor_t *edesc;
-       usbd_device_handle udev;
        usbd_interface_handle iface;
        u_int8_t epcount;
        usbd_status r;
@@ -154,7 +153,7 @@ ufm_attach(device_t self)
 
        DPRINTFN(10,("ufm_attach: sc=%p\n", sc));
        sc->sc_dev = self;
-       sc->sc_udev = udev = uaa->device;
+       sc->sc_udev = uaa->device;
 
        if ((!uaa->device) || (!uaa->iface)) {
                ermsg = "device or iface";
index e8c3b22..d9296f5 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.32 2007/11/06 07:37:01 hasso Exp $
+ * $DragonFly: src/sys/dev/usbmisc/uhid/uhid.c,v 1.33 2008/08/14 20:55:53 hasso Exp $
  */
 
 /* Also already merged from NetBSD:
@@ -197,7 +197,6 @@ uhid_attach(device_t self)
        struct uhid_softc *sc = device_get_softc(self);
        struct usb_attach_arg *uaa = device_get_ivars(self);
        usbd_interface_handle iface = uaa->iface;
-       usb_interface_descriptor_t *id;
        usb_endpoint_descriptor_t *ed;
        int size;
        void *desc;
@@ -205,7 +204,6 @@ uhid_attach(device_t self)
 
        sc->sc_udev = uaa->device;
        sc->sc_iface = iface;
-       id = usbd_get_interface_descriptor(iface);
        sc->sc_dev = self;
 
        ed = usbd_interface2endpoint_descriptor(iface, 0);
@@ -511,7 +509,6 @@ uhid_do_write(struct uhid_softc *sc, struct uio *uio, int flag)
                return (EIO);
 
        size = sc->sc_osize;
-       error = 0;
        if (uio->uio_resid != size)
                return (EINVAL);
        error = uiomove(sc->sc_obuf, size, uio);
index 4cb6d2e..82df85e 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.26 2007/11/06 07:37:01 hasso Exp $
+ * $DragonFly: src/sys/dev/usbmisc/ukbd/ukbd.c,v 1.27 2008/08/14 20:55:53 hasso Exp $
  */
 
 /*
@@ -172,8 +172,6 @@ ukbd_attach(device_t self)
 {
        struct ukbd_softc *sc = device_get_softc(self);
        struct usb_attach_arg *uaa = device_get_ivars(self);
-       usbd_interface_handle iface = uaa->iface;
-       usb_interface_descriptor_t *id;
 
        keyboard_switch_t *sw;
        keyboard_t *kbd;
@@ -184,7 +182,6 @@ ukbd_attach(device_t self)
        if (sw == NULL)
                return ENXIO;
 
-       id = usbd_get_interface_descriptor(iface);
        sc->sc_dev = self;
 
        arg[0] = (void *)uaa;
index 165fbb7..af01edd 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.30 2007/11/06 07:37:01 hasso Exp $
+ * $DragonFly: src/sys/dev/usbmisc/ums/ums.c,v 1.31 2008/08/14 20:55:54 hasso Exp $
  */
 
 /*
@@ -210,7 +210,6 @@ ums_attach(device_t self)
        struct ums_softc *sc = device_get_softc(self);
        struct usb_attach_arg *uaa = device_get_ivars(self);
        usbd_interface_handle iface = uaa->iface;
-       usb_interface_descriptor_t *id;
        usb_endpoint_descriptor_t *ed;
        int size;
        void *desc;
@@ -221,7 +220,6 @@ ums_attach(device_t self)
 
        sc->sc_disconnected = 1;
        sc->sc_iface = iface;
-       id = usbd_get_interface_descriptor(iface);
        sc->sc_dev = self;
        ed = usbd_interface2endpoint_descriptor(iface, 0);
        if (!ed) {
index 46de685..10fa331 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.26 2007/11/06 07:37:01 hasso Exp $
+ * $DragonFly: src/sys/dev/usbmisc/urio/urio.c,v 1.27 2008/08/14 20:55:54 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);
-       usbd_device_handle udev;
        usbd_interface_handle iface;
        u_int8_t epcount;
        usbd_status r;
@@ -182,7 +181,7 @@ urio_attach(device_t self)
 
        DPRINTFN(10,("urio_attach: sc=%p\n", sc));
        sc->sc_dev = self;
-       sc->sc_udev = udev = uaa->device;
+       sc->sc_udev = uaa->device;
 
        if ((!uaa->device) || (!uaa->iface)) {
                ermsg = "device or iface";