On 03/09/2016 12:13 PM, Jakub Kicinski wrote: >> I not entirely sure what i have to do... >> - Reenable the ONE_SHOT >> - Disable irq when running in the loop, and enable when returning? >> - Implement threaded irq? >> >> /Sean > > Do not reenable ONE_SHOT. Disable interrupt (disable_irq_nosync()) > in sc16is7xx_irq() and reenable at the end of sc16is7xx_ist(). > I think reenabling at the end of sc16is7xx_ist() is fine since it's a > level triggered IRQ. Josh's patch [0] which drops the ONE_SHOT flag has not yet been taken by Greg. It sits only in the -RT tree at the moment. It might have something to do with Greg not being in TO: or CC: *I* don't care how you fix this in the end but please do something before the merge window opens. An ack by the maintainer of the driver and re-send to Greg (the usual way) would be one way. [0] http://lkml.iu.edu/hypermail/linux/kernel/1602.2/02637.html Sebastian -- To unsubscribe from this list: send the line "unsubscribe linux-serial" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
