Re: [PATCHv2] PCI: xilinx: use fwnode_irq_get() for INTx IRQ lookup

[Manivannan Sadhasivam]

On Wed, Jul 29, 2026 at 11:18:15AM -0700, Rosen Penev wrote:
> On Wed, Jul 29, 2026 at 9:00 AM Manivannan Sadhasivam <mani@xxxxxxxxxx> wrote:
> >
> > On Thu, Jul 16, 2026 at 01:58:11PM -0700, Rosen Penev wrote:
> > > Replace irq_of_parse_and_map() with fwnode_irq_get(dev_fwnode(dev), 0)
> > > in xilinx_pcie_parse_dt(). For an OF-backed device this is equivalent to
> > > the previous call, but uses the generic firmware-node API.
> > >
> >
> > What is the benefit of using fwnode API for OF only driver?
> I get conflicting information. Apparently of_irq_get needs a call to
> irq_dispose_mapping but fwnode_irq_get does not.

That's not true. fwnode_irq_get() becomes a wrapper to of_irq_get() if OF is
used. So both needs a call to irq_dispose_mapping() in the error path.

- Mani

-- 
மணிவண்ணன் சதாசிவம்