Hi Abhishek, > I think this code would be simplified by using a delayed_work struct > instead of a timer. And I pointed this out before that all the timers should be moved to a delayed_work. In addition such a complex support for hardware should move towards its own driver solely base on serdev. I am currently getting a bit fed up if I point these things out and the answer, please merge this now and we fix it later. Maybe I need to stop merging things and wait for a proper separate driver for this hardware. Regards Marcel