summaryrefslogtreecommitdiff
path: root/src/superio/smsc/mec1308/superio.c
diff options
context:
space:
mode:
authorRonald G. Minnich <rminnich@gmail.com>2012-11-27 11:32:38 -0800
committerRonald G. Minnich <rminnich@gmail.com>2012-11-28 07:57:17 +0100
commit5079a0d32f2824445cf593dfbcb65598eaa97dee (patch)
tree2e6edd336c00f1f3e4937410a9be9537553702be /src/superio/smsc/mec1308/superio.c
parentacf443191bd035c26ee89c3ca56f065a5111901b (diff)
downloadcoreboot-5079a0d32f2824445cf593dfbcb65598eaa97dee.tar.xz
Remove assembly coded log2 function
As we move to supporting other systems we need to get rid of assembly where we can. The log2 function in src/lib is identical to the assembly one (tested for all 32-bit signed integers :-) and takes about 10 ns to run as opposed to 5ns for the non-portable assembly version. While speed is good, I think we can spare the 15 ns or so we add to boot time by using the C version only. Change-Id: Icafa565eae282c85fa5fc01b3bd1f110cd9aaa91 Signed-off-by: Ronald G. Minnich <rminnich@gmail.com> Reviewed-on: http://review.coreboot.org/1928 Tested-by: build bot (Jenkins)
Diffstat (limited to 'src/superio/smsc/mec1308/superio.c')
-rw-r--r--src/superio/smsc/mec1308/superio.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/src/superio/smsc/mec1308/superio.c b/src/superio/smsc/mec1308/superio.c
index 070871d106..0de06f8f32 100644
--- a/src/superio/smsc/mec1308/superio.c
+++ b/src/superio/smsc/mec1308/superio.c
@@ -25,7 +25,6 @@
#include <console/console.h>
#include <device/smbus.h>
#include <string.h>
-#include <bitops.h>
#include <pc80/keyboard.h>
#include <stdlib.h>
#include "chip.h"