On 10/20/20 8:45 PM, Joakim Zhang wrote: > With below sequence, we can see wakeup default is enabled after re-load > module, so need disable wakeup in flexcan_remove(). > > root@imx8qmevk:~# lsmod > Module Size Used by > crct10dif_ce 16384 1 > flexcan 32768 0 > can_dev 36864 1 flexcan > root@imx8qmevk:~# cat /sys/bus/platform/drivers/flexcan/5a8e0000.can/power/wakeup > disabled > root@imx8qmevk:~# echo enabled > /sys/bus/platform/drivers/flexcan/5a8e0000.can/power/wakeup > root@imx8qmevk:~# cat /sys/bus/platform/drivers/flexcan/5a8e0000.can/power/wakeup > enabled > root@imx8qmevk:~# rmmod flexcan > root@imx8qmevk:~# rmmod can_dev > root@imx8qmevk:~# modprobe flexcan > [ 147.877689] CAN device driver interface > root@imx8qmevk:~# cat /sys/bus/platform/drivers/flexcan/5a8e0000.can/power/wakeup > enabled Very good, that explain the problem! I've streamlined the commit message a bit and applied to linux-can/testing Thanks, Marc -- Pengutronix e.K. | Marc Kleine-Budde | Embedded Linux | https://www.pengutronix.de | Vertretung West/Dortmund | Phone: +49-231-2826-924 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
Attachment:
signature.asc
Description: OpenPGP digital signature
