commit 9f10950426e830752be1f49f1d50922ad6499420
author Ryan Chuang <ryan.chuang@mediatek.corp-partner.google.com> Thu Nov 25 16:15:27 2021 +0800
committer Hung-Te Lin <hungte@chromium.org> Wed Dec 01 09:48:17 2021 +0000
tree eed140f824525b5576f46fd24f4c9c7349ec7f19
parent 5fb0e5564d87785bf38999dbb664f1d2868fcd2f

vc/mediatek/mt8195: Fix rank1 CKE setting for single-rank DRAM

Fix the issue that power consumption of single rank DRAM is greater
than dual rank DRAM due to incorrect settings of rank1 CKE.
Set rank1 CKE to the correct state to fix this issue.

BUG=b:196867407
TEST=DUT can boot to OS.

Signed-off-by: Ryan Chuang <ryan.chuang@mediatek.corp-partner.google.com>
Change-Id: If336197aea4770dda1332b6e83da8ec9a4f9d77b
Reviewed-on: https://review.coreboot.org/c/coreboot/+/59715
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Yu-Ping Wu <yupingso@google.com>

src/vendorcode/mediatek/mt8195/dramc/dramc_pi_main.c

diff --git a/src/vendorcode/mediatek/mt8195/dramc/dramc_pi_main.c b/src/vendorcode/mediatek/mt8195/dramc/dramc_pi_main.c
index e7e8f89..e3baf7b 100644
--- a/src/vendorcode/mediatek/mt8195/dramc/dramc_pi_main.c
+++ b/src/vendorcode/mediatek/mt8195/dramc/dramc_pi_main.c
@@ -1946,6 +1946,11 @@
 #endif
     }
 
+    if (p->support_rank_num == RANK_SINGLE){
+        CKEFixOnOff(p, RANK_1, CKE_DYNAMIC, TO_ALL_CHANNEL);
+        mcSHOW_DBG_MSG(("Set RANK1 CKE to DYNAMIC\n"));
+    }
+
     #if (FOR_DV_SIMULATION_USED == 0 && SW_CHANGE_FOR_SIMULATION == 0)
     U32 backup_broadcast;
     backup_broadcast = GetDramcBroadcast();