summaryrefslogtreecommitdiff
path: root/src/lib
diff options
context:
space:
mode:
authorAndrey Petrov <andrey.petrov@intel.com>2016-03-10 22:14:41 -0800
committerAaron Durbin <adurbin@chromium.org>2016-03-11 09:52:46 +0100
commit447d9489a21aad3dda400aa519eaeaafed25573f (patch)
tree877304d2d792997abceee1c87ad09767d6c93bb9 /src/lib
parentc7a1a3e994135bb405662e2f52d0b22efa899c3d (diff)
downloadcoreboot-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')
-rw-r--r--src/lib/imd_cbmem.c6
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