diff options
author | Marc Kleine-Budde <mkl@pengutronix.de> | 2019-08-22 15:35:35 +0200 |
---|---|---|
committer | Marcel Ziswiler <marcel.ziswiler@toradex.com> | 2020-02-09 22:38:57 +0100 |
commit | b46373b96b114098eed63f450249ba234821c24e (patch) | |
tree | b36aca972f516a0cc7ecc6e55c90eabe27f83cd3 | |
parent | e90c3d81f35f4002add925193f41c1b2c3f534c8 (diff) |
can: mcp25xxfd: mcp25xxfd_can_tx_start_xmit() fix possible race condition
Put sbk into echo queue before triggering send.
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
(cherry picked from commit 2c1df7da765455d86f5630e050905aa3b599ed97)
-rw-r--r-- | drivers/net/can/spi/mcp25xxfd/mcp25xxfd_can_tx.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/drivers/net/can/spi/mcp25xxfd/mcp25xxfd_can_tx.c b/drivers/net/can/spi/mcp25xxfd/mcp25xxfd_can_tx.c index 6b02d8d2149b..07526a23c054 100644 --- a/drivers/net/can/spi/mcp25xxfd/mcp25xxfd_can_tx.c +++ b/drivers/net/can/spi/mcp25xxfd/mcp25xxfd_can_tx.c @@ -601,13 +601,14 @@ netdev_tx_t mcp25xxfd_can_tx_start_xmit(struct sk_buff *skb, ret = spi_async(spi, &smsg->fill_fifo.msg); if (ret) goto out_async_failed; - ret = spi_async(spi, &smsg->trigger_fifo.msg); - if (ret) - goto out_async_failed; /* keep it for reference until the message really got transmitted */ can_put_echo_skb(skb, net, smsg->fifo); + ret = spi_async(spi, &smsg->trigger_fifo.msg); + if (ret) + goto out_async_failed; + return NETDEV_TX_OK; out_async_failed: netdev_err(net, "spi_async submission of fifo %i failed - %i\n", |