On Wed, 20 May 2015, Jiang Liu wrote: > /* Chained IRQ handler for IPU error interrupt */ > -static void ipu_irq_err(unsigned int irq, struct irq_desc *desc) > +static void ipu_irq_err(struct irq_desc *desc) > { > + unsigned int irq = irq_desc_to_irq(desc); Why initializing? It's overwritten in the loop. > /* Chained IRQ handler for IPU function interrupt */ > -static void ipu_irq_fn(unsigned int irq, struct irq_desc *desc) > +static void ipu_irq_fn(struct irq_desc *desc) > { > + unsigned int irq = irq_desc_to_irq(desc); Ditto. Thanks, tglx -- To unsubscribe from this list: send the line "unsubscribe dmaengine" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html