On Wed, Apr 13, 2016 at 04:18:58PM +0200, Raphael Poggi wrote: > Signed-off-by: Raphael Poggi <poggi.raph@xxxxxxxxx> > --- > 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 | _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox
