Felix Singer has uploaded this change for review.

View Change

mb/up/squared: Do RAM config based on SKU ID

Change-Id: Ic121652213d5b1f65cff2f3096e919a3cf88db72
Signed-off-by: Felix Singer <felix.singer@9elements.com>
---
M src/mainboard/up/squared/romstage.c
A src/mainboard/up/squared/romstage.h
2 files changed, 109 insertions(+), 33 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/38/34838/1
diff --git a/src/mainboard/up/squared/romstage.c b/src/mainboard/up/squared/romstage.c
index 2cfaa00..c1c52ad 100644
--- a/src/mainboard/up/squared/romstage.c
+++ b/src/mainboard/up/squared/romstage.c
@@ -15,10 +15,20 @@

#include <string.h>
#include <soc/romstage.h>
+#include <soc/gpio_apl.h>
#include <fsp/api.h>
#include <FspmUpd.h>
#include <console/console.h>
+#include <gpio.h>
#include "gpio.h"
+#include "romstage.h"
+
+/*
+ * Offsets:
+ * - GPIO_27: 0xd8
+ * - GPIO_28: 0xe0
+ */
+const uint8_t memory_skuid_pads[] = { GPIO_27, GPIO_28 };

static const uint8_t ch0_bit_swizzling[] = {
0x0D, 0x0A, 0x08, 0x0B, 0x0C, 0x0F, 0x0E, 0x09,
@@ -48,49 +58,101 @@
0x19, 0x1F, 0x1D, 0x1B, 0x1E, 0x18, 0x1C, 0x1A
};

-
void mainboard_memory_init_params(FSPM_UPD *memupd)
{
printk(BIOS_DEBUG, "MAINBOARD: %s/%s called\n", __FILE__, __func__);

+ FSP_M_CONFIG *config = &memupd->FspmConfig;
+
+ uint8_t memory_skuid = get_memory_skuid();
+
+ switch (memory_skuid) {
+ case 0: /* 2GB */
+ config->DualRankSupportEnable = 0;
+ config->Ch0_RankEnable = 1;
+ config->Ch0_DramDensity = 2;
+ config->Ch1_RankEnable = 1;
+ config->Ch1_DramDensity = 2;
+ config->Ch2_RankEnable = 0;
+ config->Ch3_RankEnable = 0;
+ break;
+ case 1: /* 4GB */
+ config->DualRankSupportEnable = 1;
+ config->Ch0_RankEnable = 1;
+ config->Ch0_DramDensity = 2;
+ config->Ch1_RankEnable = 1;
+ config->Ch1_DramDensity = 2;
+ config->Ch2_RankEnable = 1;
+ config->Ch2_DramDensity = 2;
+ config->Ch3_RankEnable = 1;
+ config->Ch3_DramDensity = 2;
+ break;
+ case 2: /* 8GB */
+ config->DualRankSupportEnable = 1;
+ config->Ch0_RankEnable = 3;
+ config->Ch0_DramDensity = 2;
+ config->Ch1_RankEnable = 3;
+ config->Ch1_DramDensity = 2;
+ config->Ch2_RankEnable = 3;
+ config->Ch2_DramDensity = 2;
+ config->Ch3_RankEnable = 3;
+ config->Ch3_DramDensity = 2;
+ break;
+ default:
+ break;
+ }
+
gpio_configure_pads(gpio_table, ARRAY_SIZE(gpio_table));

- memupd->FspmConfig.Package = 0x1; // 0x0
- memupd->FspmConfig.Profile = 0xB; // 0x19
- memupd->FspmConfig.MemoryDown = 0x1; // 0x0
- memupd->FspmConfig.DDR3LPageSize = 0x0; // 0x1
- memupd->FspmConfig.DIMM0SPDAddress = 0x0; // 0xa0
- memupd->FspmConfig.DIMM1SPDAddress = 0x0; // 0xa4
- memupd->FspmConfig.RmtCheckRun = 0x3; // 0x0
- memupd->FspmConfig.RmtMarginCheckScaleHighThreshold = 0xC8; // 0x0
- memupd->FspmConfig.EnhancePort8xhDecoding = 0x0; // 0x1
- memupd->FspmConfig.NpkEn = 0x0; // 0x3
- memupd->FspmConfig.PrimaryVideoAdaptor = 0x2; // 0x0
+ config->Package = 0x1; // 0x0
+ config->Profile = 0xB; // 0x19
+ config->MemoryDown = 0x1; // 0x0
+ config->DDR3LPageSize = 0x0; // 0x1
+ config->DIMM0SPDAddress = 0x0; // 0xa0
+ config->DIMM1SPDAddress = 0x0; // 0xa4
+ config->RmtCheckRun = 0x3; // 0x0
+ config->RmtMarginCheckScaleHighThreshold = 0xC8; // 0x0
+ config->EnhancePort8xhDecoding = 0x0; // 0x1
+ config->NpkEn = 0x0; // 0x3
+ config->PrimaryVideoAdaptor = 0x2; // 0x0

- memupd->FspmConfig.Ch0_RankEnable = 0x1; // 0x0
- memupd->FspmConfig.Ch0_DeviceWidth = 0x1; // 0x0
- memupd->FspmConfig.Ch0_DramDensity = 0x2; // 0x0
- memupd->FspmConfig.Ch0_Option = 0x3; // 0x0
- memupd->FspmConfig.Ch1_RankEnable = 0x1; // 0x0
- memupd->FspmConfig.Ch1_DeviceWidth = 0x1; // 0x0
- memupd->FspmConfig.Ch1_DramDensity = 0x2; // 0x0
- memupd->FspmConfig.Ch1_Option = 0x3; // 0x0
- memupd->FspmConfig.Ch2_RankEnable = 0x1; // 0x0
- memupd->FspmConfig.Ch2_DeviceWidth = 0x1; // 0x0
- memupd->FspmConfig.Ch2_DramDensity = 0x2; // 0x0
- memupd->FspmConfig.Ch2_Option = 0x3; // 0x0
- memupd->FspmConfig.Ch3_RankEnable = 0x1; // 0x0
- memupd->FspmConfig.Ch3_DeviceWidth = 0x1; // 0x0
- memupd->FspmConfig.Ch3_DramDensity = 0x2; // 0x0
- memupd->FspmConfig.Ch3_Option = 0x3; // 0x0
- memupd->FspmConfig.StartTimerTickerOfPfetAssert = 0x4E20; // 0x0
+ config->Ch0_RankEnable = 0x1; // 0x0
+ config->Ch0_DeviceWidth = 0x1; // 0x0
+ config->Ch0_DramDensity = 0x2; // 0x0
+ config->Ch0_Option = 0x3; // 0x0
+ config->Ch1_RankEnable = 0x1; // 0x0
+ config->Ch1_DeviceWidth = 0x1; // 0x0
+ config->Ch1_DramDensity = 0x2; // 0x0
+ config->Ch1_Option = 0x3; // 0x0
+ config->Ch2_RankEnable = 0x1; // 0x0
+ config->Ch2_DeviceWidth = 0x1; // 0x0
+ config->Ch2_DramDensity = 0x2; // 0x0
+ config->Ch2_Option = 0x3; // 0x0
+ config->Ch3_RankEnable = 0x1; // 0x0
+ config->Ch3_DeviceWidth = 0x1; // 0x0
+ config->Ch3_DramDensity = 0x2; // 0x0
+ config->Ch3_Option = 0x3; // 0x0
+ config->StartTimerTickerOfPfetAssert = 0x4E20; // 0x0

- memcpy(memupd->FspmConfig.Ch0_Bit_swizzling, &ch0_bit_swizzling,
+ memcpy(config->Ch0_Bit_swizzling, &ch0_bit_swizzling,
sizeof(ch0_bit_swizzling));
- memcpy(memupd->FspmConfig.Ch1_Bit_swizzling, &ch1_bit_swizzling,
+ memcpy(config->Ch1_Bit_swizzling, &ch1_bit_swizzling,
sizeof(ch1_bit_swizzling));
- memcpy(memupd->FspmConfig.Ch2_Bit_swizzling, &ch2_bit_swizzling,
+ memcpy(config->Ch2_Bit_swizzling, &ch2_bit_swizzling,
sizeof(ch2_bit_swizzling));
- memcpy(memupd->FspmConfig.Ch3_Bit_swizzling, &ch3_bit_swizzling,
+ memcpy(config->Ch3_Bit_swizzling, &ch3_bit_swizzling,
sizeof(ch3_bit_swizzling));
}
+
+uint8_t get_memory_skuid(void)
+{
+ uint8_t memory_skuid = 0;
+
+ for (uint8_t i = 0; i < ARRAY_SIZE(memory_skuid_pads); i++) {
+ uint32_t gpio_value = gpio_get(memory_skuid_pads[i]);
+ uint8_t rx_state = gpio_value & 0x2;
+ memory_skuid |= rx_state << i;
+ }
+ memory_skuid = memory_skuid & 0x3;
+ return memory_skuid;
+}
diff --git a/src/mainboard/up/squared/romstage.h b/src/mainboard/up/squared/romstage.h
new file mode 100644
index 0000000..5c0f4ba
--- /dev/null
+++ b/src/mainboard/up/squared/romstage.h
@@ -0,0 +1,14 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 2 of the License.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ */
+
+uint8_t get_memory_skuid(void);

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

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ic121652213d5b1f65cff2f3096e919a3cf88db72
Gerrit-Change-Number: 34838
Gerrit-PatchSet: 1
Gerrit-Owner: Felix Singer <felixsinger@posteo.net>
Gerrit-MessageType: newchange