diff options
| author | Ronald G. Minnich <rminnich@gmail.com> | 2012-11-27 11:32:38 -0800 |
|---|---|---|
| committer | Ronald G. Minnich <rminnich@gmail.com> | 2012-11-28 07:57:17 +0100 |
| commit | 5079a0d32f2824445cf593dfbcb65598eaa97dee (patch) | |
| tree | 2e6edd336c00f1f3e4937410a9be9537553702be /src/northbridge/amd/agesa | |
| parent | acf443191bd035c26ee89c3ca56f065a5111901b (diff) | |
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/northbridge/amd/agesa')
5 files changed, 5 insertions, 5 deletions
diff --git a/src/northbridge/amd/agesa/family10/northbridge.c b/src/northbridge/amd/agesa/family10/northbridge.c index f1ed610e42..4bdc7b4fed 100644 --- a/src/northbridge/amd/agesa/family10/northbridge.c +++ b/src/northbridge/amd/agesa/family10/northbridge.c @@ -26,7 +26,7 @@ #include <device/hypertransport.h> #include <stdlib.h> #include <string.h> -#include <bitops.h> +#include <lib.h> #include <cpu/cpu.h> #include <cpu/x86/lapic.h> #include <cbmem.h> diff --git a/src/northbridge/amd/agesa/family12/northbridge.c b/src/northbridge/amd/agesa/family12/northbridge.c index ac3f3082e7..c4b43c5ed9 100644 --- a/src/northbridge/amd/agesa/family12/northbridge.c +++ b/src/northbridge/amd/agesa/family12/northbridge.c @@ -26,7 +26,7 @@ #include <device/hypertransport.h> #include <stdlib.h> #include <string.h> -#include <bitops.h> +#include <lib.h> #include <cpu/cpu.h> #include <cbmem.h> diff --git a/src/northbridge/amd/agesa/family14/northbridge.c b/src/northbridge/amd/agesa/family14/northbridge.c index 74473c065e..5e8d89b46d 100644 --- a/src/northbridge/amd/agesa/family14/northbridge.c +++ b/src/northbridge/amd/agesa/family14/northbridge.c @@ -26,7 +26,7 @@ #include <device/hypertransport.h> #include <stdlib.h> #include <string.h> -#include <bitops.h> +#include <lib.h> #include <cpu/cpu.h> #include <cbmem.h> diff --git a/src/northbridge/amd/agesa/family15/northbridge.c b/src/northbridge/amd/agesa/family15/northbridge.c index 78986e4a0c..abd53b2787 100644 --- a/src/northbridge/amd/agesa/family15/northbridge.c +++ b/src/northbridge/amd/agesa/family15/northbridge.c @@ -26,7 +26,7 @@ #include <device/hypertransport.h> #include <stdlib.h> #include <string.h> -#include <bitops.h> +#include <lib.h> #include <cpu/cpu.h> #include <cbmem.h> diff --git a/src/northbridge/amd/agesa/family15tn/northbridge.c b/src/northbridge/amd/agesa/family15tn/northbridge.c index ac605b0304..5659ac6539 100644 --- a/src/northbridge/amd/agesa/family15tn/northbridge.c +++ b/src/northbridge/amd/agesa/family15tn/northbridge.c @@ -26,7 +26,7 @@ #include <device/hypertransport.h> #include <stdlib.h> #include <string.h> -#include <bitops.h> +#include <lib.h> #include <cpu/cpu.h> #include <cbmem.h> |