diff options
author | Paul Menzel <paulepanter@users.sourceforge.net> | 2017-10-11 15:41:43 +0200 |
---|---|---|
committer | Martin Roth <martinroth@google.com> | 2017-11-03 15:20:15 +0000 |
commit | 01873808755406d2b72ea47761554ec3025c81c7 (patch) | |
tree | 26ebdb9e0e9a80af5c6957ea4a7c10616ed62079 /src/include/cbfs.h | |
parent | d540d740b6e76ece7a3fbd5d8803f652a176b50c (diff) |
arch/x86/gdt: Move variable assignment down
The Clang Static Analyzer warns about an unused assignment, when
building the image for the Lenovo X60.
```
src/arch/x86/gdt.c:39:6: warning: Value stored to 'num_gdt_bytes' \
during its initialization is never read
u16 num_gdt_bytes = (uintptr_t)&gdt_end - (uintptr_t)&gdt;
^~~~~~~~~~~~~ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
If `CONFIG_RELOCATABLE_RAMSTAGE` is selected, the function returns
before the assignment is used. So, move the assignment below the if
statement.
Change-Id: Ibcb8bce743d8cb3625647804816fb97f937dc429
Found-by: clang version 4.0.1-6 (tags/RELEASE_401/final), Debian Sid/unstable
Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-on: https://review.coreboot.org/21957
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Sumeet R Pawnikar <sumeet.r.pawnikar@intel.com>
Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'src/include/cbfs.h')
0 files changed, 0 insertions, 0 deletions