ath11k: remove an unneeded NULL check
authorDan Carpenter <dan.carpenter@oracle.com>
Fri, 13 Dec 2019 10:52:19 +0000 (13:52 +0300)
committerKalle Valo <kvalo@codeaurora.org>
Wed, 18 Dec 2019 17:56:26 +0000 (19:56 +0200)
The list iterator is always non-NULL so it doesn't need to be checked.
I also removed the unnecessary initializer because the list iterator is
always initialized.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/ath/ath11k/dp_rx.c

index 1b3b65c..3a3dc76 100644 (file)
@@ -1225,12 +1225,12 @@ static
 struct htt_ppdu_stats_info *ath11k_dp_htt_get_ppdu_desc(struct ath11k *ar,
                                                        u32 ppdu_id)
 {
-       struct htt_ppdu_stats_info *ppdu_info = NULL;
+       struct htt_ppdu_stats_info *ppdu_info;
 
        spin_lock_bh(&ar->data_lock);
        if (!list_empty(&ar->ppdu_stats_info)) {
                list_for_each_entry(ppdu_info, &ar->ppdu_stats_info, list) {
-                       if (ppdu_info && ppdu_info->ppdu_id == ppdu_id) {
+                       if (ppdu_info->ppdu_id == ppdu_id) {
                                spin_unlock_bh(&ar->data_lock);
                                return ppdu_info;
                        }