diff options
| author | Richard Spiegel <richard.spiegel@amd.corp-partner.google.com> | 2018-08-06 15:26:54 -0700 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2018-08-08 15:28:47 +0000 |
| commit | d3131e416417b537a0e93f72d8f0c6cd3d27e092 (patch) | |
| tree | 47af0632f0fca0ebee392a7ea5975ea0bae30de9 /src/lib | |
| parent | 2b68cb08a88117c2cc452a5a4f75f5c8fc401f21 (diff) | |
| download | coreboot-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.c | 1 |
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; |