Patrick Georgi submitted this change.

View Change

Approvals: build bot (Jenkins): Verified Hung-Te Lin: Looks good to me, approved
mb/google/kukui: Add discrete LPDDR4X DDR table support for burnet/esche

LPDDR4x DRAM table for burnet/esche:
[1] = "sdram-lpddr4x-H9HCNNNCPMALHR-4GB"
[2] = "sdram-lpddr4x-MT53E1G32D4NQ-4GB"
[3] = "sdram-lpddr4x-K4UBE3D4AA-MGCL-4GB"
[4] = "sdram-lpddr4x-H9HCNNNCPMMLXR-NEE-4GB"

BUG=b:161768221,b:159301679
BRANCH=master
TEST=emerge-jacuzzi coreboot
Signed-off-by: Kevin Chiu <kevin.chiu@quantatw.com>
Change-Id: Ida7ab877c3f7e10a67680b69a1d724ec734d2928
Reviewed-on: https://review.coreboot.org/c/coreboot/+/43752
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Hung-Te Lin <hungte@chromium.org>
---
M src/mainboard/google/kukui/sdram_configs.c
M src/mainboard/google/kukui/sdram_params/Makefile.inc
2 files changed, 23 insertions(+), 9 deletions(-)

diff --git a/src/mainboard/google/kukui/sdram_configs.c b/src/mainboard/google/kukui/sdram_configs.c
index c1e75ef6..7638853 100644
--- a/src/mainboard/google/kukui/sdram_configs.c
+++ b/src/mainboard/google/kukui/sdram_configs.c
@@ -6,6 +6,12 @@
#include <soc/emi.h>

static const char *const sdram_configs[] = {
+#if CONFIG(BOARD_GOOGLE_BURNET)
+ [1] = "sdram-lpddr4x-H9HCNNNCPMALHR-4GB",
+ [2] = "sdram-lpddr4x-MT53E1G32D4NQ-4GB",
+ [3] = "sdram-lpddr4x-K4UBE3D4AA-MGCL-4GB",
+ [4] = "sdram-lpddr4x-H9HCNNNCPMMLXR-NEE-4GB",
+#else /* The default unified table */
[1] = "sdram-lpddr4x-H9HCNNNCPMALHR-4GB",
[2] = "sdram-lpddr4x-MT53E1G32D4NQ-4GB",
[3] = "sdram-lpddr4x-KMDH6001DA-B422-4GB",
@@ -15,6 +21,7 @@
[7] = "sdram-lpddr4x-SDADA4CR-128G-4GB",
[8] = "sdram-lpddr4x-K4UBE3D4AA-MGCL-4GB",
[10] = "sdram-lpddr4x-H9HCNNNCPMMLXR-NEE-4GB",
+#endif
};

static struct sdram_params params;
diff --git a/src/mainboard/google/kukui/sdram_params/Makefile.inc b/src/mainboard/google/kukui/sdram_params/Makefile.inc
index af13aa1..2b43519 100644
--- a/src/mainboard/google/kukui/sdram_params/Makefile.inc
+++ b/src/mainboard/google/kukui/sdram_params/Makefile.inc
@@ -1,13 +1,20 @@
sdram-params :=
-sdram-params += sdram-lpddr4x-K4UBE3D4AA-MGCL-4GB
-sdram-params += sdram-lpddr4x-H9HCNNNCPMALHR-4GB
-sdram-params += sdram-lpddr4x-MT53E1G32D4NQ-4GB
-sdram-params += sdram-lpddr4x-KMDH6001DA-B422-4GB
-sdram-params += sdram-lpddr4x-KMDP6001DA-B425-4GB
-sdram-params += sdram-lpddr4x-MT29VZZZAD8DQKSL-4GB
-sdram-params += sdram-lpddr4x-KMDV6001DA-B620-4GB
-sdram-params += sdram-lpddr4x-SDADA4CR-128G-4GB
-sdram-params += sdram-lpddr4x-H9HCNNNCPMMLXR-NEE-4GB
+ifeq ($(CONFIG_BOARD_GOOGLE_BURNET),y)
+ sdram-params += sdram-lpddr4x-H9HCNNNCPMALHR-4GB
+ sdram-params += sdram-lpddr4x-MT53E1G32D4NQ-4GB
+ sdram-params += sdram-lpddr4x-K4UBE3D4AA-MGCL-4GB
+ sdram-params += sdram-lpddr4x-H9HCNNNCPMMLXR-NEE-4GB
+else
+ sdram-params += sdram-lpddr4x-K4UBE3D4AA-MGCL-4GB
+ sdram-params += sdram-lpddr4x-H9HCNNNCPMALHR-4GB
+ sdram-params += sdram-lpddr4x-MT53E1G32D4NQ-4GB
+ sdram-params += sdram-lpddr4x-KMDH6001DA-B422-4GB
+ sdram-params += sdram-lpddr4x-KMDP6001DA-B425-4GB
+ sdram-params += sdram-lpddr4x-MT29VZZZAD8DQKSL-4GB
+ sdram-params += sdram-lpddr4x-KMDV6001DA-B620-4GB
+ sdram-params += sdram-lpddr4x-SDADA4CR-128G-4GB
+ sdram-params += sdram-lpddr4x-H9HCNNNCPMMLXR-NEE-4GB
+endif

$(foreach params,$(sdram-params), \
$(eval cbfs-files-y += $(params)) \

To view, visit change 43752. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ida7ab877c3f7e10a67680b69a1d724ec734d2928
Gerrit-Change-Number: 43752
Gerrit-PatchSet: 4
Gerrit-Owner: Kevin Chiu <Kevin.Chiu@quantatw.com>
Gerrit-Reviewer: Hung-Te Lin <hungte@chromium.org>
Gerrit-Reviewer: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-MessageType: merged