From d99092baab86a62d98b315c58c2860eaba106291 Mon Sep 17 00:00:00 2001 From: Felix Fietkau Date: Sat, 2 Dec 2017 10:43:35 +0100 Subject: [PATCH] mt7603: remove WARN_ON_ONCE for workaround checks It can happen during normal operation and it confuses users more than it actually helps. workaround trigger counts are still available via debugfs Signed-off-by: Felix Fietkau --- mt7603_mac.c | 30 +++++++++++++++--------------- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/mt7603_mac.c b/mt7603_mac.c index e53ab114..8637c7d2 100644 --- a/mt7603_mac.c +++ b/mt7603_mac.c @@ -1274,21 +1274,21 @@ void mt7603_mac_work(struct work_struct *work) mutex_lock(&dev->mutex); - if (WARN_ON_ONCE(mt7603_watchdog_check(dev, &dev->tx_dma_check, - RESET_CAUSE_TX_BUSY, - mt7603_tx_dma_busy)) || - WARN_ON_ONCE(mt7603_watchdog_check(dev, &dev->rx_dma_check, - RESET_CAUSE_RX_BUSY, - mt7603_rx_dma_busy)) || - WARN_ON_ONCE(mt7603_watchdog_check(dev, &dev->tx_hang_check, - RESET_CAUSE_TX_HANG, - mt7603_tx_hang)) || - WARN_ON_ONCE(mt7603_watchdog_check(dev, &dev->rx_pse_check, - RESET_CAUSE_RX_PSE_BUSY, - mt7603_rx_pse_busy)) || - WARN_ON_ONCE(mt7603_watchdog_check(dev, &dev->beacon_check, - RESET_CAUSE_BEACON_STUCK, - NULL))) { + if (mt7603_watchdog_check(dev, &dev->tx_dma_check, + RESET_CAUSE_TX_BUSY, + mt7603_tx_dma_busy) || + mt7603_watchdog_check(dev, &dev->rx_dma_check, + RESET_CAUSE_RX_BUSY, + mt7603_rx_dma_busy) || + mt7603_watchdog_check(dev, &dev->tx_hang_check, + RESET_CAUSE_TX_HANG, + mt7603_tx_hang) || + mt7603_watchdog_check(dev, &dev->rx_pse_check, + RESET_CAUSE_RX_PSE_BUSY, + mt7603_rx_pse_busy) || + mt7603_watchdog_check(dev, &dev->beacon_check, + RESET_CAUSE_BEACON_STUCK, + NULL)) { dev->beacon_check = 0; dev->tx_dma_check = 0; dev->tx_hang_check = 0;