diff options
author | Paul Menzel <pmenzel@molgen.mpg.de> | 2020-06-25 09:11:56 +0200 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2020-07-26 21:26:04 +0000 |
commit | 6d412d738c48c5e65da4e2054109726ba3f558d7 (patch) | |
tree | 9eb64076b24ad4e8945a7279ea7f041347c2c8ec /src/include/smmstore.h | |
parent | 253b7d22fe9fea679ccbafd4d55142088e203127 (diff) |
drivers/mrc_cache: Avoid unused variable assignment
Fix the scan-build warning below:
CC romstage/drivers/mrc_cache/mrc_cache.o
src/drivers/mrc_cache/mrc_cache.c:450:26: warning: Value stored to 'flash' during its initialization is never read
const struct spi_flash *flash = boot_device_spi_flash();
^~~~~ ~~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.
The function can return early before the value is read. Fix this, by
getting rid of the variable, as the value is only read once.
Change-Id: I3c94b123f4994eed9d7568b63971fd5b1d94bc09
Found-by: scan-build (clang-tools-9 1:9.0.1-12)
Signed-off-by: Paul Menzel <pmenzel@molgen.mpg.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/42798
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Patrick Rudolph <siro@das-labor.org>
Diffstat (limited to 'src/include/smmstore.h')
0 files changed, 0 insertions, 0 deletions