Hello Patrick Rudolph, Balaji Manigandan, Lance Zhao, build bot (Jenkins), Martin Roth, Furquan Shaikh,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/32903
to look at the new patch set (#3).
Change subject: soc/intel/apollolake: Fix value stored to gnvs is never read ......................................................................
soc/intel/apollolake: Fix value stored to gnvs is never read
Clang Static Analyzer found value stored to gnvs is never read. Common code southbridge_inject_dsdt already takes of allocating ACPI NVS in CBMEM cbmem_add(CBMEM_ID_ACPI_GNVS, sizeof(*gnvs)). Remove the redundant ACPI GNVS addition to CBMEM.
TEST=Built and boot up to kernel.
Signed-off-by: John Zhao john.zhao@intel.com Change-Id: Ie9421af4a556d1d88183aa938ee2a124a10ab727 --- M src/soc/intel/apollolake/chip.c 1 file changed, 0 insertions(+), 6 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/03/32903/3