ath10k: remove redundant check of len with buf_len
authorColin Ian King <colin.king@canonical.com>
Wed, 15 Mar 2017 16:36:41 +0000 (16:36 +0000)
committerKalle Valo <kvalo@qca.qualcomm.com>
Mon, 20 Mar 2017 15:24:45 +0000 (17:24 +0200)
The check of len > buf_len is redundant as len is initialized
to 0 and buf_len to 4096, so this comparison is always false.
Remove it.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
drivers/net/wireless/ath/ath10k/debug.c

index ac9090b..00b424d 100644 (file)
@@ -249,9 +249,6 @@ static ssize_t ath10k_read_wmi_services(struct file *file,
 
        mutex_lock(&ar->conf_mutex);
 
-       if (len > buf_len)
-               len = buf_len;
-
        spin_lock_bh(&ar->data_lock);
        for (i = 0; i < WMI_SERVICE_MAX; i++) {
                enabled = test_bit(i, ar->wmi.svc_map);