* Santosh Shilimkar <santosh.shilimkar@xxxxxx> [110328 22:47]: > > From: Tony Lindgren [mailto:tony@xxxxxxxxxxx] > > > > Do you really need to initialize all of this that early? > > > Yes. It's a interrupt controller extension and needs to work > together with GIC. Hmm to me it seems that the gic_arch_extn functions should be all NULL during runtime operation, and the wake-up generator should be programmed only before entering an idle state. Then the wake-up generator can be cleared after delivering the wake-up event(s). Or am I missing something on why this should be enabled all the time? BTW, looks like you got few typos in the comments in this patch, noticed these while reading: Hardware requirements is GIC rgeisters gic architecture specific fucntioms I know I'm making tons of typos too ;) Tony -- 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