summaryrefslogtreecommitdiff
path: root/src/lib
diff options
context:
space:
mode:
authorRichard Spiegel <richard.spiegel@amd.corp-partner.google.com>2018-08-06 15:26:54 -0700
committerMartin Roth <martinroth@google.com>2018-08-08 15:28:47 +0000
commitd3131e416417b537a0e93f72d8f0c6cd3d27e092 (patch)
tree47af0632f0fca0ebee392a7ea5975ea0bae30de9 /src/lib
parent2b68cb08a88117c2cc452a5a4f75f5c8fc401f21 (diff)
downloadcoreboot-d3131e416417b537a0e93f72d8f0c6cd3d27e092.tar.xz
lib/bootmem.c: Remove unused setting of variable "begin"
The variable "begin" is extracted from the structure, but 4 lines below it's overwritten with "end - size". This causes a static build scan error that should be fixed. Remove the initial assignment of variable "begin". BUG=b:112253891 TEST=Build and boot grunt. Change-Id: I0a265747e61289f045c5cac09e40478bd31e16fc Signed-off-by: Richard Spiegel <richard.spiegel@silverbackltd.com> Reviewed-on: https://review.coreboot.org/27886 Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com> Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net> Reviewed-by: Martin Roth <martinroth@google.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/lib')
-rw-r--r--src/lib/bootmem.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/src/lib/bootmem.c b/src/lib/bootmem.c
index 66d459207d..38fdb1b937 100644
--- a/src/lib/bootmem.c
+++ b/src/lib/bootmem.c
@@ -278,7 +278,6 @@ void *bootmem_allocate_buffer(size_t size)
return NULL;
/* region now points to the highest usable region for the given size. */
- begin = range_entry_base(region);
end = range_entry_end(region);
if (end > max_addr)
end = max_addr;