summaryrefslogtreecommitdiff
path: root/src/mainboard/google
diff options
context:
space:
mode:
authorAaron Durbin <adurbin@chromium.org>2016-07-21 10:00:39 -0500
committerAaron Durbin <adurbin@chromium.org>2016-07-22 18:51:08 +0200
commitdd8ed42ed6bf94259aa6bc81138e894e43ef4008 (patch)
tree612093c542b35da0df7614a9d4794b132c9a8d4e /src/mainboard/google
parent17dbec159363b4c778df9cc86d37049137e456d7 (diff)
downloadcoreboot-dd8ed42ed6bf94259aa6bc81138e894e43ef4008.tar.xz
mainboard/google/reef: indicate dual rank LPDDR4 skus
The 16Gb devices use two ranks per channel within the DRAM module. However, the density settings are really on a per rank basis so indicate dual rank with a device density of 8Gb. BUG=chrome-os-partner:55446 Change-Id: Ib5dba6f9ed248750d68b726996c71def9b75961e Signed-off-by: Aaron Durbin <adurbin@chromium.org> Reviewed-on: https://review.coreboot.org/15772 Reviewed-by: Duncan Laurie <dlaurie@chromium.org> Tested-by: build bot (Jenkins) Reviewed-by: Furquan Shaikh <furquan@google.com>
Diffstat (limited to 'src/mainboard/google')
-rw-r--r--src/mainboard/google/reef/romstage.c26
1 files changed, 19 insertions, 7 deletions
diff --git a/src/mainboard/google/reef/romstage.c b/src/mainboard/google/reef/romstage.c
index 8b7c7a3f58..74ead51bea 100644
--- a/src/mainboard/google/reef/romstage.c
+++ b/src/mainboard/google/reef/romstage.c
@@ -71,11 +71,17 @@ static const struct lpddr4_swizzle_cfg board_swizzle = {
#define PROTO_SKU 15
static const struct lpddr4_sku skus[] = {
- /* K4F6E304HB-MGCJ - both logical channels */
+ /*
+ * K4F6E304HB-MGCJ - both logical channels While the parts
+ * are listed at 16Gb there are 2 ranks per channel so indicate
+ * the deneisty as 8Gb per rank.
+ */
[0] = {
.speed = LP4_SPEED_2400,
- .ch0_density = LP4_16Gb_DENSITY,
- .ch1_density = LP4_16Gb_DENSITY,
+ .ch0_density = LP4_8Gb_DENSITY,
+ .ch1_density = LP4_8Gb_DENSITY,
+ .ch0_dual_rank = 1,
+ .ch1_dual_rank = 1,
},
/* K4F8E304HB-MGCJ - both logical channels */
[1] = {
@@ -83,13 +89,19 @@ static const struct lpddr4_sku skus[] = {
.ch0_density = LP4_8Gb_DENSITY,
.ch1_density = LP4_8Gb_DENSITY,
},
- /* MT53B512M32D2NP-062WT:C - both logical channels */
+ /*
+ * MT53B512M32D2NP-062WT:C - both logical channels. While the parts
+ * are listed at 16Gb there are 2 ranks per channel so indicate
+ * the deneisty as 8Gb per rank.
+ */
[2] = {
.speed = LP4_SPEED_2400,
- .ch0_density = LP4_16Gb_DENSITY,
- .ch1_density = LP4_16Gb_DENSITY,
- /* MT53B256M32D1NP-062 WT:C - both logical channels */
+ .ch0_density = LP4_8Gb_DENSITY,
+ .ch1_density = LP4_8Gb_DENSITY,
+ .ch0_dual_rank = 1,
+ .ch1_dual_rank = 1,
},
+ /* MT53B256M32D1NP-062 WT:C - both logical channels */
[3] = {
.speed = LP4_SPEED_2400,
.ch0_density = LP4_8Gb_DENSITY,