Re: [PATCH 2/4] powerpc: enable ARCH_USE_BUILTIN_BSWAP

2012-12-04 Thread David Woodhouse
On Tue, 2012-12-04 at 22:02 +1100, Stephen Rothwell wrote: > > +config ARCH_USE_BUILTIN_BSWAP > > + def_bool y > > + > > This should be defined as bool in arch/Kconfig (probably in the previous > patch) and then selected from appropriate architectures. Thanks. Updated series at git://git.

Re: [PATCH 2/4] powerpc: enable ARCH_USE_BUILTIN_BSWAP

2012-12-04 Thread Stephen Rothwell
Hi David, On Tue, 4 Dec 2012 10:15:28 + David Woodhouse wrote: > > diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig > index a902a5c..b4ea516 100644 > --- a/arch/powerpc/Kconfig > +++ b/arch/powerpc/Kconfig > @@ -78,6 +78,9 @@ config ARCH_HAS_ILOG2_U64 > bool > default y i

[PATCH 2/4] powerpc: enable ARCH_USE_BUILTIN_BSWAP

2012-12-04 Thread David Woodhouse
From: David Woodhouse By using the compiler intrinsics instead of hand-crafted opaque inline assembler for byte-swapping, we let the compiler see what's actually happening and it gets to use lwbrx/stwbrx instructions instead of a normal load/store coupled with a sequence of rlwimi instructions to