Tero, On Tuesday 23 July 2013 12:49 PM, Tero Kristo wrote: > + dd->control_reg = of_iomap(node, 0); > + dd->idlest_reg = of_iomap(node, 1); > + dd->autoidle_reg = of_iomap(node, 2); > + dd->mult_div1_reg = of_iomap(node, 3); > + []... > + reg = of_iomap(node, 0); Doing an of_iomap() for every single clock register seems like an overkill and might have performance penalties at boot. regards, Rajendra -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html