can: c_can: fix "BUG! echo_skb is occupied!" during transmit
authorAnilKumar Ch <anilkumar@ti.com>
Wed, 23 May 2012 12:15:09 +0000 (17:45 +0530)
committerMarc Kleine-Budde <mkl@pengutronix.de>
Sun, 3 Jun 2012 16:59:18 +0000 (18:59 +0200)
commit617caccebe451716df21c069b079d5936ed7b0f3
tree48e896e5da53154401eaaf406229e2f42f1c7519
parent9ca3cc6f3026946ba655e863ca2096339e667639
can: c_can: fix "BUG! echo_skb is occupied!" during transmit

This patch fixes an issue with transmit routine, which causes
"can_put_echo_skb: BUG! echo_skb is occupied!" message when
using "cansequence -p" on D_CAN controller.

In c_can driver, while transmitting packets tx_echo flag holds
the no of can frames put for transmission into the hardware.

As the comment above c_can_do_tx() indicates, if we find any packet
which is not transmitted then we should stop looking for more.
In the current implementation this is not taken care of causing the
said message.

Also, fix the condition used to find if the packet is transmitted
or not. Current code skips the first tx message object and ends up
checking one extra invalid object.

While at it, fix the comment on top of c_can_do_tx() to use the
terminology "packet" instead of "package" since it is more
standard.

Cc: stable@kernel.org # 2.6.39+
Signed-off-by: AnilKumar Ch <anilkumar@ti.com>
Acked-by: Wolfgang Grandegger <wg@grandegger.com>
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
drivers/net/can/c_can/c_can.c