[PATCH v2 2/2] powerpc: cell: iommu: Hold reference returned by of_find_node_by_name()

Liang He windhl at 126.com
Sat Jul 2 00:49:49 AEST 2022


In cell_iommu_init_disabled(), we should hold the reference returned
by of_find_node_by_name() and use it to call of_node_put() for reference
balance.

Signed-off-by: Liang He <windhl at 126.com>
---
 changelog:

 v2: (1) split v1's two files in to two commits
     (2) using 'check-then-put' coding style
 v1: hold the reference returned by of_find_xxx OF APIs

 v1-link: https://lore.kernel.org/all/20220621075333.4081413-1-windhl@126.com/

 arch/powerpc/platforms/cell/iommu.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/arch/powerpc/platforms/cell/iommu.c b/arch/powerpc/platforms/cell/iommu.c
index 0ca3efeef293..8c7133039566 100644
--- a/arch/powerpc/platforms/cell/iommu.c
+++ b/arch/powerpc/platforms/cell/iommu.c
@@ -720,8 +720,10 @@ static int __init cell_iommu_init_disabled(void)
 	cell_disable_iommus();
 
 	/* If we have no Axon, we set up the spider DMA magic offset */
-	if (of_find_node_by_name(NULL, "axon") == NULL)
+	np = of_find_node_by_name(NULL, "axon");
+	if (!np)
 		cell_dma_nommu_offset = SPIDER_DMA_OFFSET;
+	of_node_put(np);
 
 	/* Now we need to check to see where the memory is mapped
 	 * in PCI space. We assume that all busses use the same dma
-- 
2.25.1



More information about the Linuxppc-dev mailing list