[PATCH] bitops: fls64: add missing include

[Sascha Hauer]

On Wed, Apr 13, 2016 at 04:18:58PM +0200, Raphael Poggi wrote:
> Signed-off-by: Raphael Poggi <poggi.raph at gmail.com>
> ---
>  include/asm-generic/bitops/fls64.h | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/include/asm-generic/bitops/fls64.h b/include/asm-generic/bitops/fls64.h
> index 86d403f..e8eff76 100644
> --- a/include/asm-generic/bitops/fls64.h
> +++ b/include/asm-generic/bitops/fls64.h
> @@ -2,6 +2,7 @@
>  #define _ASM_GENERIC_BITOPS_FLS64_H_
>  
>  #include <asm/types.h>
> +#include <asm-generic/bitops/__fls.h>

This is wrong. This include must not be here to give the architecture
the chance to provide an optimized version of __fls(), but the generic
version of fls64(). This may not be clear in barebox context since all
users use the generic version, but in the kernel there are some
architectures implementing their own version of __fls() while others use
the generic variant:

arch/mips/include/asm/bitops.h:480:static inline unsigned long __fls(unsigned long word)
...
#include <asm-generic/bitops/fls64.h>

arch/metag/include/asm/bitops.h:118:#include <asm-generic/bitops/__fls.h>
...
#include <asm-generic/bitops/fls64.h>

Sascha



-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |