summaryrefslogtreecommitdiff
path: root/util/spd_tools
diff options
context:
space:
mode:
authorNick Vaccaro <nvaccaro@google.com>2020-08-20 16:19:44 -0700
committerAaron Durbin <adurbin@chromium.org>2020-08-28 04:35:56 +0000
commit48fc1640a83ce55aa1d7cc94298d3778116587e6 (patch)
tree124bbb85ebd0de8aeaa083fb29c252853e4e8d2b /util/spd_tools
parentc19140b49f1972cb3b0be5467a4c8a735226bb90 (diff)
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 <nvaccaro@google.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/44648 Reviewed-by: Furquan Shaikh <furquan@google.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'util/spd_tools')
-rw-r--r--util/spd_tools/ddr4/README.md4
-rw-r--r--util/spd_tools/ddr4/gen_part_id.go3
-rw-r--r--util/spd_tools/ddr4/gen_spd.go2
-rw-r--r--util/spd_tools/lp4x/gen_part_id.go2
-rw-r--r--util/spd_tools/lp4x/gen_spd.go4
5 files changed, 7 insertions, 8 deletions
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)
}