MIPS: Fix flood of warnings about comparsion being always true.

ralfbaechle · ralfbaechle · commit 95486e4979e5 · 2015-11-16T12:07:10.000+01:00
./arch/mips/include/asm/page.h:204:13: warning: comparison of unsigned expression &amp;gt;= 0 is always true [-Wtype-limits]

The default value of ARCH_PFN_OFFSET is 0 thus triggering this warning
for all platforms using the default value.

Signed-off-by: Ralf Baechle &lt;ralf@linux-mips.org&gt;
diff --git a/arch/mips/include/asm/page.h b/arch/mips/include/asm/page.h
@@ -200,8 +200,9 @@ static inline int pfn_valid(unsigned long pfn)
 {
 	/* avoid <linux/mm.h> include hell */
 	extern unsigned long max_mapnr;
+	unsigned long pfn_offset = ARCH_PFN_OFFSET;
 
-	return pfn >= ARCH_PFN_OFFSET && pfn < max_mapnr;
+	return pfn >= pfn_offset && pfn < max_mapnr;
 }
 
 #elif defined(CONFIG_SPARSEMEM)

Original file line number	Diff line number	Diff line change
`@@ -200,8 +200,9 @@ static inline int pfn_valid(unsigned long pfn)`
`200`	`200`	`{`
`201`	`201`	`/* avoid <linux/mm.h> include hell */`
`202`	`202`	`extern unsigned long max_mapnr;`
	`203`	`+ unsigned long pfn_offset = ARCH_PFN_OFFSET;`
`203`	`204`
`204`		`- return pfn >= ARCH_PFN_OFFSET && pfn < max_mapnr;`
	`205`	`+ return pfn >= pfn_offset && pfn < max_mapnr;`
`205`	`206`	`}`
`206`	`207`
`207`	`208`	`#elif defined(CONFIG_SPARSEMEM)`