On 02.03.2021 08:19:00, Marc wrote:

On 02.03.2021 10:40:21, Vincent MAILHOL wrote:

quoted

This mcp251x_hw_wake() function was introduced in below commit:
8ce8c0abcba3 ("can: mcp251x: only reset hardware as required") 
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/com
mit/?id=8ce8c0abcba314e1fe954a1840f6568bf5aef2ef

quoted

I am adding Timo to the loop. He is the author of the two commits above.
I think he will be of better support than me here :)

Of course I wanted to add Timo, thanks. I think I've picked your e-mail, as you've done the newest change on the driver after me :)

This CAN controller demands a software reset after power on which is done in mcp251x_can_probe -> mcp251x_hw_probe.
Since then resets are only performed when the power to the controller has been disabled. E.g. when mcp251x_can_suspend is called,
power is turned off and by setting AFTER_SUSPEND_POWER in priv->after_suspend it will be reenabled and reset in mcp251x_can_resume
and mcp251x_restart_work_handler.
Could it be that mcp251x_can_probe does not get called in any configuration? Or could it be that the controller gets power cycled during
operation with these Orange Pi boards?

Regards
Timo