Re: [PATCH v4 1/3] lib: add bitrev8x4()

2016-12-01 Thread Anatolij Gustschin
Hi Joshua, On Thu, 1 Dec 2016 16:04:09 -0800 Joshua Clayton stillcompil...@gmail.com wrote: ... >>> +static __always_inline __attribute_const__ u32 __arch_bitrev8x4(u32 x) >>> +{ >>> + __asm__ ("rbit %0, %1; rev %0, %0" : "=r" (x) : "r" (x)); >> return x; >Oops thats a little embarrassi

Re: [PATCH v4 1/3] lib: add bitrev8x4()

2016-12-01 Thread Joshua Clayton
Hello Anatolij, Thanks for the review. On 12/01/2016 02:45 PM, Anatolij Gustschin wrote: > On Thu, 1 Dec 2016 09:04:50 -0800 > Joshua Clayton stillcompil...@gmail.com wrote: > ... >> diff --git a/arch/arm/include/asm/bitrev.h b/arch/arm/include/asm/bitrev.h >> index ec291c3..6d2e9ca 100644 >> --

Re: [PATCH v4 1/3] lib: add bitrev8x4()

2016-12-01 Thread Anatolij Gustschin
On Thu, 1 Dec 2016 09:04:50 -0800 Joshua Clayton stillcompil...@gmail.com wrote: ... >diff --git a/arch/arm/include/asm/bitrev.h b/arch/arm/include/asm/bitrev.h >index ec291c3..6d2e9ca 100644 >--- a/arch/arm/include/asm/bitrev.h >+++ b/arch/arm/include/asm/bitrev.h >@@ -17,4 +17,9 @@ static __alwa

[PATCH v4 1/3] lib: add bitrev8x4()

2016-12-01 Thread Joshua Clayton
Add a function to reverse bytes within a 32 bit word. This function is more efficient than using the 8 bit version when iterating over an array Signed-off-by: Joshua Clayton --- Looking for an ACK from Russell King on this patch (or at least the arm specific implementation) arch/arm/include/a