Hi Hauke, read through the series a couple of times now and the only thing i can find is a superfluous Travolta bracket On 28/10/2015 23:37, Hauke Mehrtens wrote: > + if (deactivate) { > + /* Disable it during the initialitin. Module should enable when used */ > + pmu_disable(clk); > + } > clkdev_add(&clk->cl); > } i think that bracket is not needed apart from that, for the whole series Acked-by: John Crispin <blogic@xxxxxxxxxxx> Thanks!