awokd@danwin1210.me uploaded patch set #2 to this change.

View Change

vc/amd/agesa/[...]/Config: Avoid out-of-bounds warnings

The memNTrainFlowControl array is generating Coverity warnings
in multiple places in code where it attempts to write to index 1.
The array is defined as either 2 elements or 1 of NULL depending
on #if (AGESA_ENTRY_INIT_POST == TRUE). This is likely a false
alarm from Coverity (memory should not be training outside of a
POST), but adding a second NULL element for the
AGESA_ENTRY_INIT_POST == FALSE case.

Change-Id: Ib56fb6d98f5afa0a73004c29d05647683e7a63d9
Signed-off-by: Joe Moore <awokd@danwin1210.me>
Found-by: Coverity CID 1357451, 1357452, 1357453
---
M src/vendorcode/amd/agesa/f14/Config/OptionMemoryInstall.h
M src/vendorcode/amd/agesa/f14/Proc/Mem/NB/mn.c
M src/vendorcode/amd/agesa/f14/Proc/Mem/mn.h
M src/vendorcode/amd/agesa/f15tn/Config/OptionMemoryInstall.h
M src/vendorcode/amd/agesa/f15tn/Proc/Mem/NB/mn.c
M src/vendorcode/amd/agesa/f15tn/Proc/Mem/mn.h
M src/vendorcode/amd/agesa/f16kb/Config/OptionMemoryInstall.h
7 files changed, 14 insertions(+), 5 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/73/38173/2

To view, visit change 38173. To unsubscribe, or for help writing mail filters, visit settings.