mb/lenovo/x1_carbon_gen1: add support for hynix memory
All different memory configuration should be supported by now.
Thanks to Igor Lee.
Change-Id: Ib93c0e3cbdc29cbf6cff26292df4fbbb8208082f
Signed-off-by: Alexander Couzens <lynxis@fe80.eu>
Tested-by: Igor Lee <getrun@gmail.com>
Reviewed-on: https://review.coreboot.org/27781
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
diff --git a/src/mainboard/lenovo/x1_carbon_gen1/romstage.c b/src/mainboard/lenovo/x1_carbon_gen1/romstage.c
index c12c1fc..a34d1db 100644
--- a/src/mainboard/lenovo/x1_carbon_gen1/romstage.c
+++ b/src/mainboard/lenovo/x1_carbon_gen1/romstage.c
@@ -95,7 +95,13 @@
/* 4gb model = 0, 8gb model = 1 */
/* int extended_memory_version = get_gpio(44); */
- /* TODO: how do they differ? Guess only one slot is connected */
+
+ /*
+ * So far there is no need to parse gpio 44, as the 4GiB use
+ * the hynix or elpida memory and 8 GiB versions use samsung.
+ * All version use both channels.
+ * But we might miss some versions.
+ */
/*
* GPIO45 GPIO25
@@ -105,10 +111,8 @@
* 1 1 reserved
*/
- /* We only support elpida and samsung.
- Because the spd data is missing. */
- if (spd_index != 0 && spd_index != 2)
- die("Unsupported Memory. Please add your SPD dump to coreboot.");
+ if (spd_index == 3)
+ die("Unsupported Memory. (detected 'reserved' memory configuration).");
memory = get_spd_data(spd_index);
memcpy(&spd[0], memory, 256);