Hi Paul, A minor comment below. On Sun, Dec 09, 2012 at 23:23:01, Paul Walmsley wrote: [...] > + > + pr_debug("powerdomain: convert pwrst (%0x,%0x) to fpwrst %0x\n", > + pwrst, logic, *fpwrst); > + This function alone does not print the powerdomain name. Can you add that in the final version? Regards, Vaibhav -- 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