]> Pileus Git - ~andy/linux/commitdiff
isdn: hfcsusb: Fix set-but-unused variables.
authorDavid S. Miller <davem@davemloft.net>
Sun, 17 Apr 2011 23:35:27 +0000 (16:35 -0700)
committerDavid S. Miller <davem@davemloft.net>
Sun, 17 Apr 2011 23:35:27 +0000 (16:35 -0700)
The variable 'buf' is set but unused in
ctrl_complete().  Just kill it off.

Similarly for the variable 'err' in setup_hfcsusb().

Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/isdn/hardware/mISDN/hfcsusb.c

index 8700474747e8cd0251bc54f99f80cb2a7b8eb8e1..3ccbff13eaf238541beb0564542db9a647cbdd8b 100644 (file)
@@ -118,14 +118,12 @@ static void
 ctrl_complete(struct urb *urb)
 {
        struct hfcsusb *hw = (struct hfcsusb *) urb->context;
-       struct ctrl_buf *buf;
 
        if (debug & DBG_HFC_CALL_TRACE)
                printk(KERN_DEBUG "%s: %s\n", hw->name, __func__);
 
        urb->dev = hw->dev;
        if (hw->ctrl_cnt) {
-               buf = &hw->ctrl_buff[hw->ctrl_out_idx];
                hw->ctrl_cnt--; /* decrement actual count */
                if (++hw->ctrl_out_idx >= HFC_CTRL_BUFSIZE)
                        hw->ctrl_out_idx = 0;   /* pointer wrap */
@@ -1726,7 +1724,6 @@ hfcsusb_stop_endpoint(struct hfcsusb *hw, int channel)
 static int
 setup_hfcsusb(struct hfcsusb *hw)
 {
-       int err;
        u_char b;
 
        if (debug & DBG_HFC_CALL_TRACE)
@@ -1745,7 +1742,7 @@ setup_hfcsusb(struct hfcsusb *hw)
        }
 
        /* first set the needed config, interface and alternate */
-       err = usb_set_interface(hw->dev, hw->if_used, hw->alt_used);
+       (void) usb_set_interface(hw->dev, hw->if_used, hw->alt_used);
 
        hw->led_state = 0;