Commit ed72bc8b authored Sep 18, 2018 by Oleksij Rempel Committed by Marc Kleine-Budde Nov 09, 2018

can: flexcan: handle tx-complete CAN frames via rx-offload infrastructure



Current flexcan driver will put TX-ECHO in regular unsorted way, in
this case TX-ECHO can come after the response to the same TXed message.
In some cases, for example for J1939 stack, things will break.
This patch is using new rx-offload API to put the messages just in the
right place.

Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de>
Cc: linux-stable <stable@vger.kernel.org>
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>

parent 55059f2b

drivers/net/can/flexcan.c

+4 −1

Original line number	Diff line number	Diff line
		@@ -787,8 +787,11 @@ static irqreturn_t flexcan_irq(int irq, void *dev_id)

		/* transmission complete interrupt */
		if (reg_iflag2 & FLEXCAN_IFLAG_MB(FLEXCAN_TX_MB)) {
		u32 reg_ctrl = priv->read(&regs->mb[FLEXCAN_TX_MB].can_ctrl);

		handled = IRQ_HANDLED;
		stats->tx_bytes += can_get_echo_skb(dev, 0);
		stats->tx_bytes += can_rx_offload_get_echo_skb(&priv->offload,
		0, reg_ctrl << 16);
		stats->tx_packets++;
		can_led_event(dev, CAN_LED_EVENT_TX);

Admin message