ath9k: Remove unnecessary print function dev_err()
authorYang Li <yang.lee@linux.alibaba.com>
Fri, 8 Apr 2022 00:01:13 +0000 (08:01 +0800)
committerKalle Valo <quic_kvalo@quicinc.com>
Tue, 12 Apr 2022 13:13:43 +0000 (16:13 +0300)
The print function dev_err() is redundant because
platform_get_irq_byname() already prints an error.

Eliminate the follow coccicheck warning:
./drivers/net/wireless/ath/ath9k/ahb.c:103:2-9: line 103 is redundant
because platform_get_irq() already prints an error

Reported-by: Abaci Robot <abaci@linux.alibaba.com>
Signed-off-by: Yang Li <yang.lee@linux.alibaba.com>
Acked-by: Toke Høiland-Jørgensen <toke@toke.dk>
Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
Link: https://lore.kernel.org/r/20220408000113.129906-2-yang.lee@linux.alibaba.com
drivers/net/wireless/ath/ath9k/ahb.c

index c9b853af41d1dada39a2d0882001eef0ca510555..9cd12b20b18d809b4734f3b873e20cd73b7d0366 100644 (file)
@@ -99,10 +99,8 @@ static int ath_ahb_probe(struct platform_device *pdev)
        }
 
        irq = platform_get_irq(pdev, 0);
-       if (irq < 0) {
-               dev_err(&pdev->dev, "no IRQ resource found\n");
+       if (irq < 0)
                return irq;
-       }
 
        ath9k_fill_chanctx_ops();
        hw = ieee80211_alloc_hw(sizeof(struct ath_softc), &ath9k_ops);