Commit a4881d1c authored by Andy Shevchenko's avatar Andy Shevchenko Committed by Linus Torvalds

lib/test_bitmap: name EXP_BYTES properly

EXP_BYTES has been wrongly named.  It's a size of the exp array in bits.

While here, go ahead and rename to EXP1_IN_BITS to avoid double renaming
when exp will be renamed to exp1 in the next patch

Link: http://lkml.kernel.org/r/20191022172922.61232-4-andriy.shevchenko@linux.intel.comSigned-off-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Acked-by: default avatarLinus Walleij <linus.walleij@linaro.org>
Cc: Bartosz Golaszewski <bgolaszewski@baylibre.com>
Cc: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Marek Vasut <marek.vasut+renesas@gmail.com>
Cc: Rasmus Villemoes <linux@rasmusvillemoes.dk>
Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Cc: William Breathitt Gray <vilhelm.gray@gmail.com>
Cc: Yury Norov <yury.norov@gmail.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 54224044
...@@ -374,17 +374,17 @@ static void __init test_bitmap_parselist_user(void) ...@@ -374,17 +374,17 @@ static void __init test_bitmap_parselist_user(void)
__test_bitmap_parselist(1); __test_bitmap_parselist(1);
} }
#define EXP_BYTES (sizeof(exp) * 8) #define EXP1_IN_BITS (sizeof(exp) * 8)
static void __init test_bitmap_arr32(void) static void __init test_bitmap_arr32(void)
{ {
unsigned int nbits, next_bit; unsigned int nbits, next_bit;
u32 arr[sizeof(exp) / 4]; u32 arr[EXP1_IN_BITS / 32];
DECLARE_BITMAP(bmap2, EXP_BYTES); DECLARE_BITMAP(bmap2, EXP1_IN_BITS);
memset(arr, 0xa5, sizeof(arr)); memset(arr, 0xa5, sizeof(arr));
for (nbits = 0; nbits < EXP_BYTES; ++nbits) { for (nbits = 0; nbits < EXP1_IN_BITS; ++nbits) {
bitmap_to_arr32(arr, exp, nbits); bitmap_to_arr32(arr, exp, nbits);
bitmap_from_arr32(bmap2, arr, nbits); bitmap_from_arr32(bmap2, arr, nbits);
expect_eq_bitmap(bmap2, exp, nbits); expect_eq_bitmap(bmap2, exp, nbits);
...@@ -396,7 +396,7 @@ static void __init test_bitmap_arr32(void) ...@@ -396,7 +396,7 @@ static void __init test_bitmap_arr32(void)
" tail is not safely cleared: %d\n", " tail is not safely cleared: %d\n",
nbits, next_bit); nbits, next_bit);
if (nbits < EXP_BYTES - 32) if (nbits < EXP1_IN_BITS - 32)
expect_eq_uint(arr[DIV_ROUND_UP(nbits, 32)], expect_eq_uint(arr[DIV_ROUND_UP(nbits, 32)],
0xa5a5a5a5); 0xa5a5a5a5);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment