On Tue, May 30, 2023 at 6:17 PM Biju Das <biju.das.jz@xxxxxxxxxxxxxx> wrote: > Currently {modprobe, bind} after {rmmod, unbind} results in probe failure. > > genirq: Flags mismatch irq 22. 00000004 (85070400.usb3drd) vs. 00000004 (85070400.usb3drd) > renesas_usb3: probe of 85070000.usb3peri failed with error -16 > > The reason is, it is trying to register an interrupt handler for the same > IRQ twice. The devm_request_irq() was called with the parent device. > So the interrupt handler won't be unregistered when the usb3-peri device > is unbound. > > Fix this issue by replacing "parent dev"->"dev" as the irq resource > is managed by this driver. > > Fixes: 9cad72dfc556 ("usb: gadget: Add support for RZ/V2M USB3DRD driver") > Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx> > --- > v1->v2: > * Added missing ')' at the end of the line for fixes tag. > * Updated commit description. > * Replaced "ddata->dev"->"&pdev->dev" for the devname parameter. Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds