wl12xx: remove redundant null check on wl->scan.ssid
authorColin Ian King <colin.king@canonical.com>
Fri, 18 Mar 2016 16:20:48 +0000 (16:20 +0000)
committerKalle Valo <kvalo@codeaurora.org>
Wed, 6 Apr 2016 18:39:23 +0000 (21:39 +0300)
ssid is an array of u8, so it can never be null, so the null check on
wl->scan.ssid is redundant and can be removed.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/ti/wl12xx/scan.c

index ebed13a..a0dfc59 100644 (file)
@@ -149,7 +149,7 @@ static int wl1271_scan_send(struct wl1271 *wl, struct wl12xx_vif *wlvif,
        else
                cmd->params.band = WL1271_SCAN_BAND_5_GHZ;
 
-       if (wl->scan.ssid_len && wl->scan.ssid) {
+       if (wl->scan.ssid_len) {
                cmd->params.ssid_len = wl->scan.ssid_len;
                memcpy(cmd->params.ssid, wl->scan.ssid, wl->scan.ssid_len);
        }