From 05208b50c591ed29e1eece4fe2c3ec0476d1019f Mon Sep 17 00:00:00 2001 From: Jon Murphy Date: Fri, 5 Aug 2022 13:59:44 -0600 Subject: [PATCH] util/spd_tools: Rename Sabrina to Mendocino 'Mendocino' was an embargoed name and could previously not be used. Update amdfwtool for consistency with the correct naming convention. BUG=b:239072117 TEST=Builds Signed-off-by: Jon Murphy Change-Id: I404fcf59e89b75cd2488bcb51981aee2eb4ff0df Reviewed-on: https://review.coreboot.org/c/coreboot/+/66468 Reviewed-by: Raul Rangel Tested-by: build bot (Jenkins) --- util/spd_tools/src/spd_gen/lp5.go | 12 ++++++------ util/spd_tools/src/spd_gen/spd_gen.go | 4 ++-- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/util/spd_tools/src/spd_gen/lp5.go b/util/spd_tools/src/spd_gen/lp5.go index 09387c740b..2adaafd74f 100644 --- a/util/spd_tools/src/spd_gen/lp5.go +++ b/util/spd_tools/src/spd_gen/lp5.go @@ -122,7 +122,7 @@ const ( */ LP5SPDValueRevision1_0 = 0x10 /* - * Revision 1.1. Expected by Sabrina + * Revision 1.1. Expected by Mendocino */ LP5SPDValueRevision1_1 = 0x11 @@ -173,7 +173,7 @@ const ( var LP5PlatformSetMap = map[int][]int{ 0: {PlatformMTL, PlatformADL}, - 1: {PlatformSBR}, + 1: {PlatformMDN}, } var LP5SetInfo = map[int]LP5Set{ @@ -214,20 +214,20 @@ var LP5SetInfo = map[int]LP5Set{ SPDRevision: LP5SPDValueRevision1_1, getBankArch: LP5GetBankArchSet1, /* - * For Sabrina (as per advisory b/211510456): + * For Mendocino (as per advisory b/211510456): * 5:4 (Maximum Activate Window) = 01 (4096 * tREFI) * 3:0 (Maximum Activate Count) = 1000 (Unlimited MAC) * Set to 0x18. */ optionalFeatures: 0x18, /* - * For Sabrina (as per advisory b/211510456): + * For Mendocino (as per advisory b/211510456): * 7:6 (PPR) = 1 (Post Package Repair is supported) * Set to 0x40. */ otherOptionalFeatures: 0x40, /* - * For Sabrina (as per advisory b/211510456): + * For Mendocino (as per advisory b/211510456): * 7:5 (Number of system channels) = 000 (1 channel always) * 4:3 (Bus width extension) = 00 (no ECC) * 2:0 (Bus width) = 010 (x32 always) @@ -432,7 +432,7 @@ func LP5GetBankArchSet0(memAttribs *LP5MemAttributes) int { func LP5GetBankArchSet1(memAttribs *LP5MemAttributes) int { /* - * Sabrina does not support 8B. It uses 16B Bank Architecture for speed <= 3200 Mbps. + * Mendocino does not support 8B. It uses 16B Bank Architecture for speed <= 3200 Mbps. * It uses BG Bank Architecture for speed > 3200 Mbps. */ if memAttribs.SpeedMbps <= 3200 { diff --git a/util/spd_tools/src/spd_gen/spd_gen.go b/util/spd_tools/src/spd_gen/spd_gen.go index f8eb52896b..d25490504a 100644 --- a/util/spd_tools/src/spd_gen/spd_gen.go +++ b/util/spd_tools/src/spd_gen/spd_gen.go @@ -72,7 +72,7 @@ const ( PlatformJSL PlatformPCO PlatformCZN - PlatformSBR + PlatformMDN PlatformMTL PlatformMax ) @@ -92,7 +92,7 @@ var platformNames = map[int]string{ PlatformJSL: "JSL", PlatformPCO: "PCO", PlatformCZN: "CZN", - PlatformSBR: "SBR", + PlatformMDN: "MDN", PlatformMTL: "MTL", }