[coreboot-gerrit] Change in coreboot[master]: vc/amd/agesa/f14: Fix array length

[Angel Pons (Code Review)]

Angel Pons has submitted this change. ( https://review.coreboot.org/c/coreboot/+/39014 )

Change subject: vc/amd/agesa/f14: Fix array length
......................................................................

vc/amd/agesa/f14: Fix array length

This array is declared to have length MAX_FF_TYPES (aka 6) in several
other places, so update it here so the length matches. This fixes a
-Wlto-type-mismatch compiler error when using LTO. Extending the length
is harmless, since the only code that uses this array will stop once it
reaches the NULL pointer.

Change-Id: Ie00e969fa8cda88a934bf416c8775f7ae0b2747e
Signed-off-by: Jacob Garber 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/39014
Reviewed-by: Angel Pons 
Tested-by: build bot (Jenkins) 
---
M src/vendorcode/amd/agesa/f14/Config/OptionMemoryInstall.h
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  build bot (Jenkins): Verified
  Angel Pons: Looks good to me, approved

diff --git a/src/vendorcode/amd/agesa/f14/Config/OptionMemoryInstall.h b/src/vendorcode/amd/agesa/f14/Config/OptionMemoryInstall.h
index 231b263..600ae9b 100644
--- a/src/vendorcode/amd/agesa/f14/Config/OptionMemoryInstall.h
+++ b/src/vendorcode/amd/agesa/f14/Config/OptionMemoryInstall.h
@@ -569,7 +569,7 @@
     #define PLAT_SP_ON_FF_SDIMM3    MemPConstructPsUDef,
     #define PLAT_SP_ON_FF_UDIMM3    MemPConstructPsUDef,
   #endif
-  MEM_PLAT_SPEC_CFG* memPlatSpecFFInstalledON[] = {
+  MEM_PLAT_SPEC_CFG* memPlatSpecFFInstalledON[MAX_FF_TYPES] = {
     PLAT_SP_ON_FF_SDIMM3
     PLAT_SP_ON_FF_UDIMM3
     NULL