> > > > > > The warning comes from calling generic_handle_irq() in usb tasklet > context. > > > This is not correct. > > > > > > Per my understanding, if there's chained irq, we could introduce > > > extra irqdomain. E.g > > > > > > GIC <--> another irqchip controller <--> HW device > > > > > Correct, IRQ domain is generally used in chained irq controllers. > > Yes, We need to check why irq domain is used in the current driver. > > > > It's introduced in the commit cc89c323a30e > Hi Jisheng, I had spent some time to look into the history of commit. The reason for having irq domain in driver is because of the unavailability of real hardware interrupt, here we have USB interrupt pipe which is not actual interrupt. So changes were proposed to have pseudo phy interrupt and handlers in phy lib. But later it was suggested to implement linux interrupt controller in driver itself. You can see the archive here https://patchwork.ozlabs.org/patch/564511/ I want to understand if there is any functionality impact with this warning? Because I'm afraid if the current changes are removed we might hit some other issues (or older ones). We have to go through rigorous testing before going ahead. Thanks.
