diff options
| author | Colin Ian King <colin.king@canonical.com> | 2017-02-28 11:14:39 +0000 |
|---|---|---|
| committer | Kalle Valo <kvalo@qca.qualcomm.com> | 2017-03-16 10:56:46 +0200 |
| commit | 51f60b8b6979da361c97602d6b6df3c3f72c7c4c (patch) | |
| tree | 38aec006d541e00c9f6eff96c1d9adaea3ff7d00 /drivers/net/wireless/ath/ath10k/pci.c | |
| parent | 45c3d550b0684343b9d4b25aeec9b49c6effc1f9 (diff) | |
ath10k: remove redundant error check
The check on ret for an error is redundant because it is already been
checked for non-zero earlier on and ret is never non-zero at this point.
Fix this by removing the redundant check and error message.
Detected by CoverityScan, CID#1357170 ("Logically Dead Code")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
Diffstat (limited to 'drivers/net/wireless/ath/ath10k/pci.c')
| -rw-r--r-- | drivers/net/wireless/ath/ath10k/pci.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/drivers/net/wireless/ath/ath10k/pci.c b/drivers/net/wireless/ath/ath10k/pci.c index 6094372307aa..52896c20ca4e 100644 --- a/drivers/net/wireless/ath/ath10k/pci.c +++ b/drivers/net/wireless/ath/ath10k/pci.c @@ -970,12 +970,6 @@ static int ath10k_pci_diag_read_mem(struct ath10k *ar, u32 address, void *data, } remaining_bytes -= nbytes; - - if (ret) { - ath10k_warn(ar, "failed to read diag value at 0x%x: %d\n", - address, ret); - break; - } memcpy(data, data_buf, nbytes); address += nbytes; |