On Sun, Jan 26, 2014 at 09:04:12AM +0100, Thomas Petazzoni wrote: > Dear Ezequiel Garcia, > > On Wed, 22 Jan 2014 20:05:04 -0300, Ezequiel Garcia wrote: > > DT-enabled where an irqchip driver for the brigde interrupt controller is > > available can handle the watchdog IRQ properly. Therefore, we request > > the interruption and add a dummy handler that merely calls panic(). > > I don't quiite understand the first sentence of this commit log, and > the commit title looks wrong. Maybe a bad copy/paste or something? > Hm... yes it doesn't look right. It should read: "DT-enabled platforms, where the irqchip driver for the brigde interrupt controller is available, can handle the watchdog IRQ properly. Therefore, we request the interrupt and add a dummy handler that merely calls panic()". I guess we can re-phrase it be a bit more readable. Why does th commit title looks wrong? By requesting the IRQ we are "handling it", no? > > > > This is done in order to have an initial 'ack' of the interruption, > > interruption -> interrupt > Right. -- Ezequiel García, Free Electrons Embedded Linux, Kernel and Android Engineering http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html