> > > + ret = devm_request_irq(&pdev->dev, irq, mchp_corei2c_isr, IRQF_SHARED, > > > + pdev->name, idev); > > > > Really SHARED? > > Yeah, this driver supports both the hard peripherals & soft FPGA cores. > The hard peripherals do not have shared IRQs, but we don't really have > any control over what way the interrupts are wired for the soft cores. > Worth adding a comment to explain the reason for it being shared? Yes. Such information never hurst, I'd say. Thanks, have a nice weekend, Wolfram
Attachment:
signature.asc
Description: PGP signature
