From 48fc1640a83ce55aa1d7cc94298d3778116587e6 Mon Sep 17 00:00:00 2001 From: Nick Vaccaro Date: Thu, 20 Aug 2020 16:19:44 -0700 Subject: util: volteer/dedede: move generic SPDs to common location Now that generic SPD files have the memory type prepended to the filename, they can be stored in the same location. This CL moves the generic SPDs to the new location. Change the ddr4 gen_part_id.go and gen_spd.go tools to use "ddr4_spd_manifest.generated" instead of "spd_manifest.generated". Change the lpddr4x gen_part_id.go and gen_spd.go tools to use "lp4x_spd_manifest.generated" instead of "spd_manifest.generated". Move TGL DDR4 and LPDDR4x generic SPDs into a common location. Move JSL DDR4 and LPDDR4x generic SPDs into a common location. Change the volteer/spd/Makefile.inc to use the new path for the spds. Change the dedede/spd/Makefile.inc to use the new path for the spds. BUG=b:165854055 TEST="emerge-volteer coreboot" and verify all variants build correctly. Change-Id: I83b088cb718d15ffd3012c84a12b5231ae84a3e4 Signed-off-by: Nick Vaccaro Reviewed-on: https://review.coreboot.org/c/coreboot/+/44648 Reviewed-by: Furquan Shaikh Tested-by: build bot (Jenkins) --- util/spd_tools/ddr4/README.md | 4 ++-- util/spd_tools/ddr4/gen_part_id.go | 3 +-- util/spd_tools/ddr4/gen_spd.go | 2 +- util/spd_tools/lp4x/gen_part_id.go | 2 +- util/spd_tools/lp4x/gen_spd.go | 4 ++-- 5 files changed, 7 insertions(+), 8 deletions(-) (limited to 'util/spd_tools') diff --git a/util/spd_tools/ddr4/README.md b/util/spd_tools/ddr4/README.md index 7921f48a5d..81ab97a80a 100644 --- a/util/spd_tools/ddr4/README.md +++ b/util/spd_tools/ddr4/README.md @@ -175,7 +175,7 @@ memory parts in JSON format as described above: and placed in the directory provided as an input to the tool. * CSV file representing which of the deduplicated SPD files is used by which memory part. This file is named as - `spd_manifest.generated.txt` and placed in the directory provided + `ddr4_spd_manifest.generated.txt` and placed in the directory provided as an input to the tool along with the generated SPD files. Example CSV file: ``` @@ -190,7 +190,7 @@ memory parts in JSON format as described above: This program takes as input: * Pointer to directory where the SPD files and the manifest file - `spd_manifest.generated.txt` (in CSV format) are placed by + `ddr4_spd_manifest.generated.txt` (in CSV format) are placed by gen_spd.go * File containing list of memory parts used by the board. Each line of the file is supposed to contain one memory part `name` as present in diff --git a/util/spd_tools/ddr4/gen_part_id.go b/util/spd_tools/ddr4/gen_part_id.go index f67b4a9434..e0adaaf994 100644 --- a/util/spd_tools/ddr4/gen_part_id.go +++ b/util/spd_tools/ddr4/gen_part_id.go @@ -25,7 +25,7 @@ import ( * is expected to have one memory part name. */ const ( - SPDManifestFileName = "spd_manifest.generated.txt" + SPDManifestFileName = "ddr4_spd_manifest.generated.txt" MakefileName = "Makefile.inc" DRAMIdFileName = "dram_id.generated.txt" ) @@ -174,7 +174,6 @@ func genMakefile(partIdList []partIds, makefileDirName string) error { var s string s += fmt.Sprintf("%s\n%s\n\n", generatedCodeLicense, autoGeneratedInfo) - s += fmt.Sprintf("MEMORY_TYPE = ddr4\n\n") s += fmt.Sprintf("SPD_SOURCES =\n") for i := 0; i < len(partIdList); i++ { diff --git a/util/spd_tools/ddr4/gen_spd.go b/util/spd_tools/ddr4/gen_spd.go index e99239acd2..99fae7d3b4 100644 --- a/util/spd_tools/ddr4/gen_spd.go +++ b/util/spd_tools/ddr4/gen_spd.go @@ -26,7 +26,7 @@ import ( * JSON file containing a list of memory parts with their attributes as per datasheet. */ const ( - SPDManifestFileName = "spd_manifest.generated.txt" + SPDManifestFileName = "ddr4_spd_manifest.generated.txt" PlatformTGL = 0 PlatformPCO = 1 diff --git a/util/spd_tools/lp4x/gen_part_id.go b/util/spd_tools/lp4x/gen_part_id.go index 7ed255c83b..6037981605 100644 --- a/util/spd_tools/lp4x/gen_part_id.go +++ b/util/spd_tools/lp4x/gen_part_id.go @@ -25,7 +25,7 @@ import ( * is expected to have one memory part name. */ const ( - SPDManifestFileName = "spd_manifest.generated.txt" + SPDManifestFileName = "lp4x_spd_manifest.generated.txt" MakefileName = "Makefile.inc" DRAMIdFileName = "dram_id.generated.txt" ) diff --git a/util/spd_tools/lp4x/gen_spd.go b/util/spd_tools/lp4x/gen_spd.go index 2465815e49..e63ca8df6f 100644 --- a/util/spd_tools/lp4x/gen_spd.go +++ b/util/spd_tools/lp4x/gen_spd.go @@ -25,7 +25,7 @@ import ( * JSON file containing a list of memory parts with their attributes as per datasheet. */ const ( - SPDManifestFileName = "spd_manifest.generated.txt" + SPDManifestFileName = "lp4x_spd_manifest.generated.txt" PlatformTGL = 0 PlatformJSL = 1 @@ -666,7 +666,7 @@ func dedupeMemoryPart(dedupedParts []*memPart, memPart *memPart) bool { func generateSPD(memPart *memPart, SPDId int, SPDDirName string) { s := createSPD(&memPart.Attribs) - memPart.SPDFileName = fmt.Sprintf("spd-%d.hex", SPDId) + memPart.SPDFileName = fmt.Sprintf("lp4x-spd-%d.hex", SPDId) ioutil.WriteFile(filepath.Join(SPDDirName, memPart.SPDFileName), []byte(s), 0644) } -- cgit v1.2.3