On 02/08/2018 03:46 PM, Marc Kleine-Budde wrote:

On 02/08/2018 07:47 AM, Heiko Schocher wrote:

quoted

the driver reads in the ISR first the IRQpending register,
and clears after that in a write *all* bits in it.

It could happen that the isr register raise bits between
this 2 register accesses, which leads in lost bits ...

In case it clears "TX message sent successfully", the driver
never sends any Tx data, and buffers to userspace run over.

Fixed this:
clear only the bits in the IRQpending register, the
driver had read.

Signed-off-by: Heiko Schocher <redacted>
Reviewed-by: Marek Vasut <marex@denx.de>

Applied both to linux-can.

Can you also apply them to stable, so they get into 4.9.x etc ?

Thanks!

-- 
Best regards,
Marek Vasut