Commit 87e9af6cc67d842cd92b52b81f3f14e665e7ab05
Committed by
David S. Miller
1 parent
75bd0cbdc2
Exists in
master
and in
40 other branches
can: fix SJA1000 dlc for RTR packets
RTR frames do have a valid data length code on CAN. The driver for SJA1000 did not handle that situation properly. Signed-off-by: Kurt Van Dijck <kurt.van.dijck@eia.be> Acked-by: Marc Kleine-Budde <mkl@pengutronix.de> Signed-off-by: David S. Miller <davem@davemloft.net>
Showing 1 changed file with 1 additions and 1 deletions Side-by-side Diff
drivers/net/can/sja1000/sja1000.c
... | ... | @@ -346,10 +346,10 @@ |
346 | 346 | | (priv->read_reg(priv, REG_ID2) >> 5); |
347 | 347 | } |
348 | 348 | |
349 | + cf->can_dlc = get_can_dlc(fi & 0x0F); | |
349 | 350 | if (fi & FI_RTR) { |
350 | 351 | id |= CAN_RTR_FLAG; |
351 | 352 | } else { |
352 | - cf->can_dlc = get_can_dlc(fi & 0x0F); | |
353 | 353 | for (i = 0; i < cf->can_dlc; i++) |
354 | 354 | cf->data[i] = priv->read_reg(priv, dreg++); |
355 | 355 | } |