powerpc/pasemi: Drop redundant res assignment

krzk · maddy-kerneldev · commit f583bd5f64d4 · 2026-05-06T07:49:19.000+05:30
Return value of pas_add_bridge() is not used, so code can be simplified to fix W=1 clang warnings: arch/powerpc/platforms/pasemi/pci.c:275:6: error: variable 'res' set but not used [-Werror,-Wunused-but-set-variable] Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com> Signed-off-by: Madhavan Srinivasan <maddy@linux.ibm.com> Link: https://patch.msgid.link/20260317130823.240279-4-krzysztof.kozlowski@oss.qualcomm.com
diff --git a/arch/powerpc/platforms/pasemi/pci.c b/arch/powerpc/platforms/pasemi/pci.c
@@ -272,13 +272,12 @@ void __init pas_pci_init(void)
 {
 	struct device_node *root = of_find_node_by_path("/");
 	struct device_node *np;
-	int res;
 
 	pci_set_flags(PCI_SCAN_ALL_PCIE_DEVS);
 
 	np = of_find_compatible_node(root, NULL, "pasemi,rootbus");
 	if (np) {
-		res = pas_add_bridge(np);
+		pas_add_bridge(np);
 		of_node_put(np);
 	}
 	of_node_put(root);

Original file line number	Diff line number	Diff line change
`@@ -272,13 +272,12 @@ void __init pas_pci_init(void)`
`272`	`272`	`{`
`273`	`273`	`struct device_node *root = of_find_node_by_path("/");`
`274`	`274`	`struct device_node *np;`
`275`		`- int res;`
`276`	`275`
`277`	`276`	`pci_set_flags(PCI_SCAN_ALL_PCIE_DEVS);`
`278`	`277`
`279`	`278`	`np = of_find_compatible_node(root, NULL, "pasemi,rootbus");`
`280`	`279`	`if (np) {`
`281`		`- res = pas_add_bridge(np);`
	`280`	`+ pas_add_bridge(np);`
`282`	`281`	`of_node_put(np);`
`283`	`282`	`}`
`284`	`283`	`of_node_put(root);`