diff options
| author | Andrey Petrov <andrey.petrov@intel.com> | 2016-03-10 22:14:41 -0800 |
|---|---|---|
| committer | Aaron Durbin <adurbin@chromium.org> | 2016-03-11 09:52:46 +0100 |
| commit | 447d9489a21aad3dda400aa519eaeaafed25573f (patch) | |
| tree | 877304d2d792997abceee1c87ad09767d6c93bb9 /src/lib/imd_cbmem.c | |
| parent | c7a1a3e994135bb405662e2f52d0b22efa899c3d (diff) | |
| download | coreboot-447d9489a21aad3dda400aa519eaeaafed25573f.tar.xz | |
cbmem: Fix cbmem_add_bootmem()
Change 13363 (555d6c2) introduced a bug where cbmem_add_bootmem() was
converted to use a new function. Unfortunately instead of passing a
pointer, NULL was passed due to type confusion. This change fixes that
problem by passing address of stack variable instead of NULL.
Change-Id: Ib8e1add3547cda01f71bf1dea14d3e58bdd99730
Signed-off-by: Andrey Petrov <andrey.petrov@intel.com>
Reviewed-on: https://review.coreboot.org/14033
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Tested-by: build bot (Jenkins)
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Tested-by: Raptor Engineering Automated Test Stand <noreply@raptorengineeringinc.com>
Diffstat (limited to 'src/lib/imd_cbmem.c')
| -rw-r--r-- | src/lib/imd_cbmem.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/lib/imd_cbmem.c b/src/lib/imd_cbmem.c index e75c538cea..a0780719fe 100644 --- a/src/lib/imd_cbmem.c +++ b/src/lib/imd_cbmem.c @@ -272,11 +272,11 @@ void cbmem_region_used(uintptr_t *base, size_t *size) void cbmem_add_bootmem(void) { - void *base = NULL; + uintptr_t base = 0; size_t size = 0; - cbmem_region_used(base, &size); - bootmem_add_range((uintptr_t)base, size, LB_MEM_TABLE); + cbmem_region_used(&base, &size); + bootmem_add_range(base, size, LB_MEM_TABLE); } #if ENV_RAMSTAGE |