LoongArch: mm: Remove the unneeded result variable

ye xingchen · chenhuacai · commit ac9284db6b7b · 2022-09-03T18:01:27.000+08:00
Return the value pa_to_nid() directly instead of storing it in another
redundant variable.

Reported-by: Zeal Robot &lt;zealci@zte.com.cn&gt;
Signed-off-by: ye xingchen &lt;ye.xingchen@zte.com.cn&gt;
Signed-off-by: Huacai Chen &lt;chenhuacai@loongson.cn&gt;
diff --git a/arch/loongarch/mm/init.c b/arch/loongarch/mm/init.c
@@ -146,10 +146,7 @@ void arch_remove_memory(u64 start, u64 size, struct vmem_altmap *altmap)
 #ifdef CONFIG_NUMA
 int memory_add_physaddr_to_nid(u64 start)
 {
-	int nid;
-
-	nid = pa_to_nid(start);
-	return nid;
+	return pa_to_nid(start);
 }
 EXPORT_SYMBOL_GPL(memory_add_physaddr_to_nid);
 #endif

Original file line number	Diff line number	Diff line change
`@@ -146,10 +146,7 @@ void arch_remove_memory(u64 start, u64 size, struct vmem_altmap *altmap)`
`146`	`146`	`#ifdef CONFIG_NUMA`
`147`	`147`	`int memory_add_physaddr_to_nid(u64 start)`
`148`	`148`	`{`
`149`		`- int nid;`
`150`		`-`
`151`		`- nid = pa_to_nid(start);`
`152`		`- return nid;`
	`149`	`+ return pa_to_nid(start);`
`153`	`150`	`}`
`154`	`151`	`EXPORT_SYMBOL_GPL(memory_add_physaddr_to_nid);`
`155`	`152`	`#endif`