Commit ee6bf367 authored by Dario Binacchi's avatar Dario Binacchi Committed by Marc Kleine-Budde
Browse files

can: sun4i_can: sun4i_can_err(): call can_change_state() even if cf is NULL 



Call the function can_change_state() if the allocation of the skb
fails, as it handles the cf parameter when it is null.

Additionally, this ensures that the statistics related to state error
counters (i. e. warning, passive, and bus-off) are updated.

Fixes: 0738eff1 ("can: Allwinner A10/A20 CAN Controller support - Kernel module")
Signed-off-by: default avatarDario Binacchi <dario.binacchi@amarulasolutions.com>
Link: https://patch.msgid.link/20241122221650.633981-3-dario.binacchi@amarulasolutions.com


Signed-off-by: default avatarMarc Kleine-Budde <mkl@pengutronix.de>
parent 9e662425

drivers/net/can/sun4i_can.c

+4 −4
Original line number Diff line number Diff line
@@ -629,10 +629,10 @@ static int sun4i_can_err(struct net_device *dev, u8 isrc, u8 status) 
		tx_state = txerr >= rxerr ? state : 0;

		rx_state = txerr <= rxerr ? state : 0;



		if (likely(skb))

		/* The skb allocation might fail, but can_change_state()

		 * handles cf == NULL.

		 */

		can_change_state(dev, cf, tx_state, rx_state);

		else

			priv->can.state = state;

		if (state == CAN_STATE_BUS_OFF)

			can_bus_off(dev);

	}