On 06/05/2013 03:19 PM, Uwe � wrote:
Hello,
On Wed, Jun 05, 2013 at 09:13:01AM +0800, Chen Gang wrote:
For arm and m68k, they customize find_*_bit(), but the API is different
with 'generic'.
avr32, s390, and unicore32 also customize find_*_bit(), but the API is
the same with 'generic', and the left architectures all use 'generic'.
So need change arm and m68k related API to match the 'generic', then
all another modules can face same public API for various architectures.
Also beautify code and comments to pass "./scripts/checkpatch.pl"
Signed-off-by: Chen Gang <gang.chen@xxxxxxxxxxx>
---
arch/arm/include/asm/bitops.h | 26 ++++++++++++++++++--------
arch/arm/lib/findbit.S | 14 ++++++++++----
arch/m68k/include/asm/bitops.h | 25 ++++++++++++++-----------
3 files changed, 42 insertions(+), 23 deletions(-)
diff --git a/arch/arm/include/asm/bitops.h b/arch/arm/include/asm/bitops.h
index e691ec9..10566c8 100644
--- a/arch/arm/include/asm/bitops.h
+++ b/arch/arm/include/asm/bitops.h
@@ -161,18 +161,28 @@ extern int _test_and_change_bit(int nr, volatile unsigned long * p);
/*
* Little endian assembly bitops. nr = 0 -> byte 0 bit 0.
*/
-extern int _find_first_zero_bit_le(const void * p, unsigned size);
-extern int _find_next_zero_bit_le(const void * p, int size, int offset);
-extern int _find_first_bit_le(const unsigned long *p, unsigned size);
-extern int _find_next_bit_le(const unsigned long *p, int size, int offset);
+extern unsigned long _find_first_zero_bit_le(const void *p,
doesn't checkpatch critizise the double space in the line above?
Thank you very much for your carefully checking (could you tell me how
do you find it ?) :-)
I need wait for a day at least, if no another suggestions or
completions, I should send patch v3.
+ unsigned long size);
+extern unsigned long _find_next_zero_bit_le(const void *p, unsigned long size,
+ unsigned long offset);
+extern unsigned long _find_first_bit_le(const unsigned long *p,
+ unsigned long size);
+extern unsigned long _find_next_bit_le(const unsigned long *p,
+ unsigned long size,
+ unsigned long offset);
/*
* Big endian assembly bitops. nr = 0 -> byte 3 bit 0.
*/
-extern int _find_first_zero_bit_be(const void * p, unsigned size);
-extern int _find_next_zero_bit_be(const void * p, int size, int offset);
-extern int _find_first_bit_be(const unsigned long *p, unsigned size);
-extern int _find_next_bit_be(const unsigned long *p, int size, int offset);
+extern unsigned long _find_first_zero_bit_be(const void *p,
+ unsigned long size);
+extern unsigned long _find_next_zero_bit_be(const void *p, unsigned long size,
+ unsigned long offset);
+extern unsigned long _find_first_bit_be(const unsigned long *p,
+ unsigned long size);
+extern unsigned long _find_next_bit_be(const unsigned long *p,
+ unsigned long size,
+ unsigned long offset);
#ifndef CONFIG_SMP
/*
diff --git a/arch/arm/lib/findbit.S b/arch/arm/lib/findbit.S
index 64f6bc1..9e3263a 100644
--- a/arch/arm/lib/findbit.S
+++ b/arch/arm/lib/findbit.S
@@ -19,7 +19,8 @@
/*
* Purpose : Find a 'zero' bit
- * Prototype: int find_first_zero_bit(void *addr, unsigned int maxbit);
+ * Prototype: unsigned long find_first_zero_bit(const void *p,
here is another one.
Thanks.
--
Chen Gang
Asianux Corporation
--
To unsubscribe from this list: send the line "unsubscribe linux-m68k" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
