Staging: wlan-ng: Remove redundant break/goto statements in hfa384x_usb.c
authorIlja Sidoroff <ilja.sidoroff@iki.fi>
Mon, 8 Sep 2014 14:37:53 +0000 (17:37 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 8 Sep 2014 21:02:48 +0000 (14:02 -0700)
This patch removes redundant goto or break statements in hfa384x_usb.c as found by checkpatch.pl

Signed-off-by: Ilja Sidoroff <ilja.sidoroff@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/wlan-ng/hfa384x_usb.c

index 2f63e0c..cac55c6 100644 (file)
@@ -3474,7 +3474,7 @@ static void hfa384x_usbin_rx(wlandevice_t *wlandev, struct sk_buff *skb)
                /* If exclude and we receive an unencrypted, drop it */
                if ((wlandev->hostwep & HOSTWEP_EXCLUDEUNENCRYPTED) &&
                    !WLAN_GET_FC_ISWEP(fc)) {
-                       goto done;
+                       break;
                }
 
                data_len = le16_to_cpu(usbin->rxfrm.desc.data_len);
@@ -3528,11 +3528,9 @@ static void hfa384x_usbin_rx(wlandevice_t *wlandev, struct sk_buff *skb)
                netdev_warn(hw->wlandev->netdev, "Received frame on unsupported port=%d\n",
                            HFA384x_RXSTATUS_MACPORT_GET(
                                    usbin->rxfrm.desc.status));
-               goto done;
                break;
        }
 
-done:
        return;
 }
 
@@ -4127,7 +4125,6 @@ static int hfa384x_isgood_pdrcode(u16 pdrcode)
        case HFA384x_PDR_HFA3861_MANF_TESTI:
                /* code is OK */
                return 1;
-               break;
        default:
                if (pdrcode < 0x1000) {
                        /* code is OK, but we don't know exactly what it is */
@@ -4140,7 +4137,6 @@ static int hfa384x_isgood_pdrcode(u16 pdrcode)
                                 pdrcode);
                        return 0;
                }
-               break;
        }
        return 0;               /* avoid compiler warnings */
 }