diff options
| author | Elyes HAOUAS <ehaouas@noos.fr> | 2019-06-23 06:55:41 +0200 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2019-10-28 11:57:23 +0000 |
| commit | 845652b8bbaeca8373bcf574e12e7d4e406b72e9 (patch) | |
| tree | e4d3c9cd1cb12ae13aef9a07d5bb86d4b0468b83 | |
| parent | 400f9ca2613297b3034d96286eeeb009a1eb2ac0 (diff) | |
| download | coreboot-845652b8bbaeca8373bcf574e12e7d4e406b72e9.tar.xz | |
src/cpu: Use 'include <stdlib.h>' when appropriate
Also:
add some missing includes spotted by Jenkins.
Including <types.h>, is supposed to provide stdint and stddef.
Change-Id: I7bd999b59d1c0bdfa5999bf5805576f94c9a2390
Signed-off-by: Elyes HAOUAS <ehaouas@noos.fr>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/33680
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Martin Roth <martinroth@google.com>
| -rw-r--r-- | src/cpu/amd/family_10h-family_15h/fidvid.c | 1 | ||||
| -rw-r--r-- | src/cpu/amd/family_10h-family_15h/processor_name.c | 2 | ||||
| -rw-r--r-- | src/cpu/x86/lapic/lapic_cpu_init.c | 1 | ||||
| -rw-r--r-- | src/cpu/x86/mirror_payload.c | 3 |
4 files changed, 5 insertions, 2 deletions
diff --git a/src/cpu/amd/family_10h-family_15h/fidvid.c b/src/cpu/amd/family_10h-family_15h/fidvid.c index 917cde307f..57207a431a 100644 --- a/src/cpu/amd/family_10h-family_15h/fidvid.c +++ b/src/cpu/amd/family_10h-family_15h/fidvid.c @@ -91,6 +91,7 @@ b.- prep_fid_change(...) #include <device/pci_ops.h> #include <stdint.h> #include <northbridge/amd/amdht/AsPsDefs.h> +#include <stdlib.h> static inline void print_debug_fv(const char *str, u32 val) { diff --git a/src/cpu/amd/family_10h-family_15h/processor_name.c b/src/cpu/amd/family_10h-family_15h/processor_name.c index fbd2bb5a33..c5e31fd59d 100644 --- a/src/cpu/amd/family_10h-family_15h/processor_name.c +++ b/src/cpu/amd/family_10h-family_15h/processor_name.c @@ -27,6 +27,8 @@ #include <device/pci.h> #include <device/pnp.h> #include <device/pci_ops.h> +#include <stdlib.h> +#include <types.h> /* The maximum length of CPU names is 48 bytes, including the final NULL byte. * If you change these names your BIOS will _NOT_ pass the AMD validation and diff --git a/src/cpu/x86/lapic/lapic_cpu_init.c b/src/cpu/x86/lapic/lapic_cpu_init.c index 668988a70e..e7dfc5798d 100644 --- a/src/cpu/x86/lapic/lapic_cpu_init.c +++ b/src/cpu/x86/lapic/lapic_cpu_init.c @@ -28,6 +28,7 @@ #include <smp/spinlock.h> #include <cpu/cpu.h> #include <cpu/intel/speedstep.h> +#include <stdlib.h> #include <thread.h> /* This is a lot more paranoid now, since Linux can NOT handle diff --git a/src/cpu/x86/mirror_payload.c b/src/cpu/x86/mirror_payload.c index 9dec8dbc68..9987347f33 100644 --- a/src/cpu/x86/mirror_payload.c +++ b/src/cpu/x86/mirror_payload.c @@ -11,13 +11,12 @@ * GNU General Public License for more details. */ -#include <stdint.h> -#include <stdlib.h> #include <string.h> #include <commonlib/helpers.h> #include <console/console.h> #include <bootmem.h> #include <program_loading.h> +#include <types.h> void mirror_payload(struct prog *payload) { |