[v2] can: m_can: fix struct net_device_ops::{open,stop} callbacks under high bus load

[PATCH can v2 0/2] can: m_can: fix struct net_device_ops::{open,stop} callbacks under high bus load

Marc Kleine-Budde posted 2 patches 2 months, 3 weeks ago

Diff against v3
Download series mbox

There is a newer version of this series

drivers/net/can/m_can/m_can.c | 18 +++++++++++-------
1 file changed, 11 insertions(+), 7 deletions(-)

Expand all Fold all

[PATCH can v2 0/2] can: m_can: fix struct net_device_ops::{open,stop} callbacks under high bus load

Posted by Marc Kleine-Budde 2 months, 3 weeks ago

Under high CAN-bus load the struct net_device_ops::{open,stop}
callbacks (m_can_open(), m_can_close()) don't properly start and
shutdown the device.

Fix the problems by re-arranging the order of functions in
m_can_open() and m_can_close().

Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
---
Hello,

I've taken Jake's patch and split the timestamp wraparound changes
into separate patch. For now, I've added a Not-Signed-off-by, until
Jake gives the S-o-b.

regards,
Marc

Changes in v2:
- 1/1 split timestamp wraparound changes into separate patch
- 1/1 added "Not-Signed-off-by: Hamby, Jake (US) <Jake.Hamby@Teledyne.com>"
- 2/2 new patch: fix order of clock shutdown
- Link to v1: https://patch.msgid.link/DM8PR14MB5221D9DD3A7F2130EF161AF7EF9E2@DM8PR14MB5221.namprd14.prod.outlook.com

---
Hamby, Jake (US) (1):
      can: m_can: enable NAPI before enabling interrupts

Marc Kleine-Budde (1):
      can: m_can: m_can_close(): stop clocks after device has been shut down

 drivers/net/can/m_can/m_can.c | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)
---
base-commit: d759ee240d3c0c4a19f4d984eb21c36da76bc6ce
change-id: 20240909-can-m_can-fix-ifup-770306ca4aca

Best regards,
-- 
Marc Kleine-Budde <mkl@pengutronix.de>