]> git.karo-electronics.de Git - linux-beck.git/commitdiff
staging: android: ashmem: clean up range inline functions
authorGuillaume Tucker <guillaume.tucker@gmail.com>
Mon, 5 Dec 2016 18:55:38 +0000 (18:55 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 6 Dec 2016 09:18:07 +0000 (10:18 +0100)
Clean up the code in inline functions that deal with page and
range addresses.  Use bool instead of int for boolean return
types and remove superfluous brackets.

Signed-off-by: Guillaume Tucker <guillaume.tucker@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/android/ashmem.c

index facc8a247707ed492bce3d116a8265ba4b28094f..7cbad0d45b9c8053dc77b7ec696966ed6318324d 100644 (file)
@@ -110,33 +110,33 @@ static inline bool range_on_lru(struct ashmem_range *range)
        return range->purged == ASHMEM_NOT_PURGED;
 }
 
-static inline int page_range_subsumes_range(struct ashmem_range *range,
-                                           size_t start, size_t end)
+static inline bool page_range_subsumes_range(struct ashmem_range *range,
+                                            size_t start, size_t end)
 {
-       return (((range)->pgstart >= (start)) && ((range)->pgend <= (end)));
+       return (range->pgstart >= start) && (range->pgend <= end);
 }
 
-static inline int page_range_subsumed_by_range(struct ashmem_range *range,
-                                              size_t start, size_t end)
+static inline bool page_range_subsumed_by_range(struct ashmem_range *range,
+                                               size_t start, size_t end)
 {
-       return (((range)->pgstart <= (start)) && ((range)->pgend >= (end)));
+       return (range->pgstart <= start) && (range->pgend >= end);
 }
 
-static inline int page_in_range(struct ashmem_range *range, size_t page)
+static inline bool page_in_range(struct ashmem_range *range, size_t page)
 {
-       return (((range)->pgstart <= (page)) && ((range)->pgend >= (page)));
+       return (range->pgstart <= page) && (range->pgend >= page);
 }
 
-static inline int page_range_in_range(struct ashmem_range *range,
-                                     size_t start, size_t end)
+static inline bool page_range_in_range(struct ashmem_range *range,
+                                      size_t start, size_t end)
 {
-       return (page_in_range(range, start) || page_in_range(range, end) ||
-               page_range_subsumes_range(range, start, end));
+       return page_in_range(range, start) || page_in_range(range, end) ||
+               page_range_subsumes_range(range, start, end);
 }
 
-static inline int range_before_page(struct ashmem_range *range, size_t page)
+static inline bool range_before_page(struct ashmem_range *range, size_t page)
 {
-       return ((range)->pgend < (page));
+       return range->pgend < page;
 }
 
 #define PROT_MASK              (PROT_EXEC | PROT_READ | PROT_WRITE)