xtensa: fix __ffs result type
authorMax Filippov <jcmvbkbc@gmail.com>
Mon, 25 Apr 2016 01:06:37 +0000 (04:06 +0300)
committerMax Filippov <jcmvbkbc@gmail.com>
Sun, 24 Jul 2016 03:33:29 +0000 (06:33 +0300)
Make __ffs result type unsigned long to match generic asm
implementation. This fixes the following build warning:

  mm/nobootmem.c: In function '__free_pages_memory':
  include/linux/kernel.h:742:17: warning: comparison of distinct pointer
                                          types lacks a cast
    (void) (&_min1 == &_min2);  \
    ^
  mm/nobootmem.c:100:11: note: in expansion of macro 'min'
     order = min(MAX_ORDER - 1UL, __ffs(start));

Signed-off-by: Max Filippov <jcmvbkbc@gmail.com>
arch/xtensa/include/asm/bitops.h

index 3f44fa2a53e9fcde4266717bd245d15c0b530559..d3490189792b08a22962d9784c40472eaf5907b4 100644 (file)
@@ -48,7 +48,7 @@ static inline int ffz(unsigned long x)
  * __ffs: Find first bit set in word. Return 0 for bit 0
  */
 
-static inline int __ffs(unsigned long x)
+static inline unsigned long __ffs(unsigned long x)
 {
        return 31 - __cntlz(x & -x);
 }