| Message ID | 20230301185209.274134-2-jjhiblot@traphandler.com (mailing list archive) |
|---|---|
| State | Under Review |
| Delegated to: | Geert Uytterhoeven |
| Headers | show |
| Series | of: irq: Fixes refcount issues with of_irq_parse_one()/of_irq_parse_raw() | expand |
diff --git a/arch/powerpc/platforms/fsl_uli1575.c b/arch/powerpc/platforms/fsl_uli1575.c index 84afae7a25617..ba104f6474bc8 100644 --- a/arch/powerpc/platforms/fsl_uli1575.c +++ b/arch/powerpc/platforms/fsl_uli1575.c @@ -334,6 +334,7 @@ static void hpcd_final_uli5288(struct pci_dev *dev) laddr[1] = laddr[2] = 0; of_irq_parse_raw(laddr, &oirq); dev->irq = irq_create_of_mapping(&oirq); + of_node_put(oirq.np); } DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_AL, 0x1575, hpcd_quirk_uli1575); diff --git a/drivers/bcma/main.c b/drivers/bcma/main.c index 0a8469e0b13ad..11219dd79d327 100644 --- a/drivers/bcma/main.c +++ b/drivers/bcma/main.c @@ -193,7 +193,10 @@ static unsigned int bcma_of_get_irq(struct device *parent, return 0; } - return irq_create_of_mapping(&out_irq); + ret = irq_create_of_mapping(&out_irq); + of_node_put(out_irq.np); + + return ret; } static void bcma_of_fill_device(struct device *parent,
of_irq_parse_raw() does a get() on the device node returned in out_irq->np. Callers of of_irq_parse_raw() must do a put() when they are done with it. Signed-off-by: Jean-Jacques Hiblot <jjhiblot@traphandler.com> --- arch/powerpc/platforms/fsl_uli1575.c | 1 + drivers/bcma/main.c | 5 ++++- 2 files changed, 5 insertions(+), 1 deletion(-)