Dear Marcin Wojtas, On Wed, 3 Feb 2016 22:41:15 +0100, Marcin Wojtas wrote: > Hi Gregory, > > > + port->irq = irq->start; > > + port->irqflags = 0; > > Same request as for v1, which you must have overlooked: > port->irqflags = IRQF_SHARED; Gregory didn't overlook this comment you made on v1, he even replied to it. Please check Gregory's reply to your review. Best regards, Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html