diff options
author | Richard Spiegel <richard.spiegel@amd.corp-partner.google.com> | 2018-08-10 16:13:00 -0700 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2018-08-13 12:24:34 +0000 |
commit | 72b2022b31e6ee8d6d81a3761b90df2df88ba8a8 (patch) | |
tree | 1d5f23163372610a871f86dc26d416e7efe76106 /src/arch/arm64 | |
parent | 9fb8e22ffa30b88c440536a6bb3caf83d2a1c6d4 (diff) | |
download | coreboot-72b2022b31e6ee8d6d81a3761b90df2df88ba8a8.tar.xz |
cpu/x86/smm/smm_module_loader.c: Avoid static analysis error "unused value"
In procedure smm_load_module, variables fxsave_size and fxsave_area are set
to 0 and NULL, but if CONFIG_SSE is enabled, they are overwritten. Change
the code setting the value to an "else" of the "if" testing CONFIG_SSE, thus
avoiding static analysis error.
BUG=b:112253891
TEST=Build and boot grunt.
Change-Id: I3042b037017a082378b53ee419c769c535632038
Signed-off-by: Richard Spiegel <richard.spiegel@silverbackltd.com>
Reviewed-on: https://review.coreboot.org/28023
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Martin Roth <martinroth@google.com>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Diffstat (limited to 'src/arch/arm64')
0 files changed, 0 insertions, 0 deletions