Hi Juan, On Tue, May 8, 2012 at 1:09 AM, Gutierrez, Juan <jgutierrez@xxxxxx> wrote: > Startup it is now only used to enable pm runtime of the whole mbox module. Btw I really doubt we need to do that in a machine-specific handler, or whether we really need this startup/shutdown abstraction at all. But that's not related to your patch of course... > I think, if request_irq is called from the generic layer, it is kind > of logical to enable_irq next to it. I somewhat agree, though I was wondering about the implications for OMAP1, which didn't seem to enable_irq at startup at all. Not sure there's much to do here (unless you have some OMAP1 hardware ? :). Thanks, Ohad. -- 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