Hi Marc, On 03/05/2017 14:16, Marc Kleine-Budde wrote: > On 05/03/2017 02:11 PM, Quentin Schulz wrote: >> This adds Power Management deep Suspend/Resume support for Bosch M_CAN >> chip. >> >> When the chip resumes from deep sleep, the RAM needs to be initialized >> as it is done when the driver probes. The net interface also needs to be >> closed and reopened to be fully functional. > > Are you sure it's the closing and opening of the net interface. Maybe > it's the m_can_start() and/or the subsequent m_can_chip_config()? >
You're right. The chip needs to be completely reinitialized (clocks, chip itself) and it also closes and reopens the interface if I'm correct with close_candev and open_candev. I'm sending a v2 right away to fix the commit log. Thanks, Quentin -- Quentin Schulz, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com
signature.asc
Description: OpenPGP digital signature
