Paul Fagerburg has submitted this change. ( https://review.coreboot.org/c/coreboot/+/73650 )
Change subject: soc/amd/phoenix/mca.c: Remove excess MCA bank names ......................................................................
soc/amd/phoenix/mca.c: Remove excess MCA bank names
Documentation and hardware differ in the number of MCA bank names, so remove the excess ones to prevent a "CPU has an unexpected number of MCA banks!" warning message.
Signed-off-by: Fred Reitberger reitbergerfred@gmail.com Change-Id: I75a2348561833f3f19181b4f30a6971ecb317899 Reviewed-on: https://review.coreboot.org/c/coreboot/+/73650 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Felix Held felix-coreboot@felixheld.de --- M src/soc/amd/phoenix/mca.c 1 file changed, 17 insertions(+), 32 deletions(-)
Approvals: build bot (Jenkins): Verified Felix Held: Looks good to me, approved
diff --git a/src/soc/amd/phoenix/mca.c b/src/soc/amd/phoenix/mca.c index 63b040b..ccbb360 100644 --- a/src/soc/amd/phoenix/mca.c +++ b/src/soc/amd/phoenix/mca.c @@ -37,38 +37,6 @@ [29] = "", [30] = "", [31] = "", - [32] = "", - [33] = "", - [34] = "", - [35] = "", - [36] = "", - [37] = "", - [38] = "", - [39] = "", - [40] = "", - [41] = "", - [42] = "", - [43] = "", - [44] = "", - [45] = "", - [46] = "", - [47] = "", - [48] = "", - [49] = "", - [50] = "", - [51] = "", - [52] = "", - [53] = "", - [54] = "", - [55] = "", - [56] = "", - [57] = "", - [58] = "", - [59] = "", - [60] = "", - [61] = "", - [62] = "", - [63] = "", };
bool mca_has_expected_bank_count(void)