Commit 3c76c58fca03c1162ab8592f71c996e933af3a9e
1 parent
a719e0a81f
Exists in
master
and in
4 other branches
isdn: hfcsusb: Fix set-but-unused variables.
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>
Showing 1 changed file with 1 additions and 4 deletions Side-by-side Diff
drivers/isdn/hardware/mISDN/hfcsusb.c
... | ... | @@ -118,14 +118,12 @@ |
118 | 118 | ctrl_complete(struct urb *urb) |
119 | 119 | { |
120 | 120 | struct hfcsusb *hw = (struct hfcsusb *) urb->context; |
121 | - struct ctrl_buf *buf; | |
122 | 121 | |
123 | 122 | if (debug & DBG_HFC_CALL_TRACE) |
124 | 123 | printk(KERN_DEBUG "%s: %s\n", hw->name, __func__); |
125 | 124 | |
126 | 125 | urb->dev = hw->dev; |
127 | 126 | if (hw->ctrl_cnt) { |
128 | - buf = &hw->ctrl_buff[hw->ctrl_out_idx]; | |
129 | 127 | hw->ctrl_cnt--; /* decrement actual count */ |
130 | 128 | if (++hw->ctrl_out_idx >= HFC_CTRL_BUFSIZE) |
131 | 129 | hw->ctrl_out_idx = 0; /* pointer wrap */ |
... | ... | @@ -1726,7 +1724,6 @@ |
1726 | 1724 | static int |
1727 | 1725 | setup_hfcsusb(struct hfcsusb *hw) |
1728 | 1726 | { |
1729 | - int err; | |
1730 | 1727 | u_char b; |
1731 | 1728 | |
1732 | 1729 | if (debug & DBG_HFC_CALL_TRACE) |
... | ... | @@ -1745,7 +1742,7 @@ |
1745 | 1742 | } |
1746 | 1743 | |
1747 | 1744 | /* first set the needed config, interface and alternate */ |
1748 | - err = usb_set_interface(hw->dev, hw->if_used, hw->alt_used); | |
1745 | + (void) usb_set_interface(hw->dev, hw->if_used, hw->alt_used); | |
1749 | 1746 | |
1750 | 1747 | hw->led_state = 0; |
1751 | 1748 |