MIPS: Make virt_addr_valid() return bool
authorPaul Burton <paul.burton@mips.com>
Tue, 28 May 2019 17:05:04 +0000 (17:05 +0000)
committerPaul Burton <paul.burton@mips.com>
Wed, 29 May 2019 19:04:44 +0000 (12:04 -0700)
virt_addr_valid() really returns a boolean value, but currently uses an
integer to represent it. Switch to the bool type to make it clearer that
we really are returning a true or false value.

Signed-off-by: Paul Burton <paul.burton@mips.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Tested-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Cc: linux-mips@vger.kernel.org
arch/mips/include/asm/page.h
arch/mips/mm/mmap.c

index 6b31c93b5eaad3bf4423779b188bc996f519071b..a25643d258cb55be5830d262590f014f1d064d07 100644 (file)
@@ -249,7 +249,7 @@ static inline int pfn_valid(unsigned long pfn)
 #define virt_to_pfn(kaddr)     PFN_DOWN(virt_to_phys((void *)(kaddr)))
 #define virt_to_page(kaddr)    pfn_to_page(virt_to_pfn(kaddr))
 
-extern int __virt_addr_valid(const volatile void *kaddr);
+extern bool __virt_addr_valid(const volatile void *kaddr);
 #define virt_addr_valid(kaddr)                                         \
        __virt_addr_valid((const volatile void *) (kaddr))
 
index 7755a1fad05adbd1e1a322154a75f3441210c04f..50ee7213b432812ebf8563f8bd786da7ed68c139 100644 (file)
@@ -201,12 +201,12 @@ unsigned long arch_randomize_brk(struct mm_struct *mm)
        return ret;
 }
 
-int __virt_addr_valid(const volatile void *kaddr)
+bool __virt_addr_valid(const volatile void *kaddr)
 {
        unsigned long vaddr = (unsigned long)vaddr;
 
        if ((vaddr < PAGE_OFFSET) || (vaddr >= MAP_BASE))
-               return 0;
+               return false;
 
        return pfn_valid(PFN_DOWN(virt_to_phys(kaddr)));
 }