[PATCH] can: ctucanfd: ctucan_err_interrupt(): do not emit info messages on Arbitration Lost or CAN Bus Error IRQs
Marc Kleine-Budde <[email protected]>
| Newsgroups | org.kernel.vger.linux-can,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <20260806-ctucanfd-silence-can-bus-error-messages-v1-1-70eff67debfd@pengutronix.de> |
Since commit e74bae899529 ("can: ctucanfd: handle bus error interrupts")
CAN bus error interrupts are properly handled.
With activated CAN Bus Error reporting, Arbitration Lost (ALI) and Bus Error
(BEI) interrupts are enabled. These can occur at a very high rate (~10kHz).
To avoid flooding the system, don't emit a netdev_info() for these.
Fixes: e74bae899529 ("can: ctucanfd: handle bus error interrupts")
Signed-off-by: Marc Kleine-Budde <[email protected]>
---
drivers/net/can/ctucanfd/ctucanfd_base.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/net/can/ctucanfd/ctucanfd_base.c b/drivers/net/can/ctucanfd/ctucanfd_base.c
index 07d4aa43c700..9aec18681fc5 100644
--- a/drivers/net/can/ctucanfd/ctucanfd_base.c
+++ b/drivers/net/can/ctucanfd/ctucanfd_base.c
@@ -884,8 +884,6 @@ static void ctucan_err_interrupt(struct net_device *ndev, u32 isr)
/* Check for Arbitration Lost interrupt */
if (FIELD_GET(REG_INT_STAT_ALI, isr)) {
- if (dologerr)
- netdev_info(ndev, "arbitration lost\n");
priv->can.can_stats.arbitration_lost++;
if (skb) {
cf->can_id |= CAN_ERR_LOSTARB;
@@ -895,7 +893,6 @@ static void ctucan_err_interrupt(struct net_device *ndev, u32 isr)
/* Check for Bus Error interrupt */
if (FIELD_GET(REG_INT_STAT_BEI, isr)) {
- netdev_info(ndev, "bus error\n");
priv->can.can_stats.bus_error++;
stats->rx_errors++;
if (skb) {
---
base-commit: cd0784790d921ff4f7438d3b7ccda86068c8b766
change-id: 20260805-ctucanfd-silence-can-bus-error-messages-5c5bcd6860f8
Best regards,
--
Marc Kleine-Budde <[email protected]>