>> >> - ret = wiz_clock_init(wiz, node); >> + ret = wiz_clock_init(wiz, node, true); > > You are calling it one once? So what am I missing In patch 6/14, wiz_clock_init is called in resume_noirq callback: ret = wiz_clock_init(wiz, node, false); Regards -- Thomas Richard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com