usb: dwc3: omap: remove IRQ_NOAUTOEN used with shared irq
authorVignesh R <vigneshr@ti.com>
Fri, 7 Jul 2017 06:22:52 +0000 (11:52 +0530)
committerFelipe Balbi <felipe.balbi@linux.intel.com>
Mon, 17 Jul 2017 07:59:31 +0000 (10:59 +0300)
commitee249b4554947de3be77be4e9e6077b20c0fe055
tree78b553f7f73f906afce6d80bf4fae29cef5bc84d
parent541768b08a400d9d292cfd9c898401b8178856ac
usb: dwc3: omap: remove IRQ_NOAUTOEN used with shared irq

IRQ_NOAUTOEN cannot be used with shared IRQs, since commit 04c848d39879
("genirq: Warn when IRQ_NOAUTOEN is used with shared interrupts") and
kernel now throws a warn dump. But OMAP DWC3 driver uses this flag. As
per commit 12a7f17fac5b ("usb: dwc3: omap: fix race of pm runtime with
irq handler in probe") that introduced this flag, PM runtime can race
with IRQ handler when deferred probing happens due to extcon,
therefore IRQ_NOAUTOEN needs to be set so that irq is not enabled until
extcon is registered.

Remove setting of IRQ_NOAUTOEN and move the registration of
shared irq to a point after dwc3_omap_extcon_register() and
of_platform_populate(). This avoids possibility of probe deferring and
above said race condition.

Reviewed-by: Grygorii Strashko <grygorii.strashko@ti.com>
Signed-off-by: Vignesh R <vigneshr@ti.com>
Signed-off-by: Felipe Balbi <felipe.balbi@linux.intel.com>
drivers/usb/dwc3/dwc3-omap.c