From 8fac0b3e904138f0642d7ab29c5867002aebf686 Mon Sep 17 00:00:00 2001 From: Edward O'Callaghan Date: Tue, 17 Jun 2014 18:53:36 +1000 Subject: src/lib/clog2.c: Fix style and clarity, remove some cruft Change-Id: I6b37cf945db12d2cf8096c9f49fff9e0bec139d6 Signed-off-by: Edward O'Callaghan Reviewed-on: http://review.coreboot.org/6058 Tested-by: build bot (Jenkins) Reviewed-by: Idwer Vollering --- src/lib/clog2.c | 29 +++++++++++------------------ 1 file changed, 11 insertions(+), 18 deletions(-) (limited to 'src/lib') diff --git a/src/lib/clog2.c b/src/lib/clog2.c index b908762917..5e0d591de4 100644 --- a/src/lib/clog2.c +++ b/src/lib/clog2.c @@ -1,9 +1,4 @@ -#undef DEBUG_LOG2 - -#ifdef DEBUG_LOG2 #include -#endif - #include /* Assume 8 bits per byte */ @@ -11,28 +6,26 @@ unsigned long log2(unsigned long x) { - // assume 8 bits per byte. - unsigned long i = 1ULL << (sizeof(x)* CHAR_BIT - 1ULL); - unsigned long pow = sizeof(x) * CHAR_BIT - 1ULL; + /* assume 8 bits per byte. */ + unsigned long pow = sizeof(x) * CHAR_BIT - 1ULL; + unsigned long i = 1ULL << pow; - if (! x) { -#ifdef DEBUG_LOG2 - printk(BIOS_WARNING, "%s called with invalid parameter of 0\n", + if (!x) { + printk(BIOS_WARNING, "%s called with invalid parameter of 0\n", __func__); -#endif - return -1; - } - for(; i > x; i >>= 1, pow--) - ; + return -1; + } - return pow; + for (; i > x; i >>= 1, pow--); + + return pow; } unsigned long log2_ceil(unsigned long x) { unsigned long pow; - if (! x) + if (!x) return -1; pow = log2(x); -- cgit v1.2.3