]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commit
can: cc770: Fix queue stall & dropped RTR reply
authorAndri Yngvason <andri.yngvason@marel.com>
Wed, 14 Mar 2018 11:52:57 +0000 (11:52 +0000)
committerMarc Kleine-Budde <mkl@pengutronix.de>
Wed, 14 Mar 2018 12:01:22 +0000 (13:01 +0100)
commit746201235b3f876792099079f4c6fea941d76183
tree7238fcd524a676531bfa61f7f7580bf61088b645
parentf4353daf4905c0099fd25fa742e2ffd4a4bab26a
can: cc770: Fix queue stall & dropped RTR reply

While waiting for the TX object to send an RTR, an external message with a
matching id can overwrite the TX data. In this case we must call the rx
routine and then try transmitting the message that was overwritten again.

The queue was being stalled because the RX event did not generate an
interrupt to wake up the queue again and the TX event did not happen
because the TXRQST flag is reset by the chip when new data is received.

According to the CC770 datasheet the id of a message object should not be
changed while the MSGVAL bit is set. This has been fixed by resetting the
MSGVAL bit before modifying the object in the transmit function and setting
it after. It is not enough to set & reset CPUUPD.

It is important to keep the MSGVAL bit reset while the message object is
being modified. Otherwise, during RTR transmission, a frame with matching
id could trigger an rx-interrupt, which would cause a race condition
between the interrupt routine and the transmit function.

Signed-off-by: Andri Yngvason <andri.yngvason@marel.com>
Tested-by: Richard Weinberger <richard@nod.at>
Cc: linux-stable <stable@vger.kernel.org>
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
drivers/net/can/cc770/cc770.c
drivers/net/can/cc770/cc770.h