diff options
author | Gregory Fong <gregory.0xf0@gmail.com> | 2015-08-14 18:35:21 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2015-08-14 18:56:32 -0400 |
commit | f21838e056ebc499f5d6ca4cb734e82cf9c275a5 (patch) | |
tree | a5e3f2c700e7ca8639128de9c42dea5fcb76e2fa | |
parent | 4ce321f574a97f3453bca5a4117610b43dabd3ee (diff) |
mm: cma: mark cma_bitmap_maxno() inline in header
cma_bitmap_maxno() was marked as static and not static inline, which can
cause warnings about this function not being used if this file is included
in a file that does not call that function, and violates the conventions
used elsewhere. The two options are to move the function implementation
back to mm/cma.c or make it inline here, and it's simple enough for the
latter to make sense.
Signed-off-by: Gregory Fong <gregory.0xf0@gmail.com>
Cc: Joonsoo Kim <iamjoonsoo.kim@lge.com>
Cc: Sasha Levin <sasha.levin@oracle.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | mm/cma.h | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -16,7 +16,7 @@ struct cma { | |||
16 | extern struct cma cma_areas[MAX_CMA_AREAS]; | 16 | extern struct cma cma_areas[MAX_CMA_AREAS]; |
17 | extern unsigned cma_area_count; | 17 | extern unsigned cma_area_count; |
18 | 18 | ||
19 | static unsigned long cma_bitmap_maxno(struct cma *cma) | 19 | static inline unsigned long cma_bitmap_maxno(struct cma *cma) |
20 | { | 20 | { |
21 | return cma->count >> cma->order_per_bit; | 21 | return cma->count >> cma->order_per_bit; |
22 | } | 22 | } |