summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorJulius Werner <jwerner@chromium.org>2020-04-22 22:31:36 +0000
committerJulius Werner <jwerner@chromium.org>2020-04-30 19:30:19 +0000
commit00961676fd399252e6ef1178dcda8391085694e2 (patch)
treeeaff56af83491ee09d103517b5d0c31dc775ea61 /src
parent4e06c6eeb1179cf553c99eff6f2ae9888376fd67 (diff)
downloadcoreboot-00961676fd399252e6ef1178dcda8391085694e2.tar.xz
Revert "soc/mediatek/mt8183: Force retraining memory if requested"
This reverts commit 285975dbba8c7f3bbb9f9950e79a30bb983d5123. Reason for revert: VB2_RECOVERY_TRAIN_AND_REBOOT was never meant to have any special effect on memory training behavior. It was just supposed to be a "reboot automatically after reaching kernel verification" recovery reason. On x86 devices this was used to prime the separate recovery MRC cache in the factory (make sure it is initialized before shipping). This isn't used on Kukui anyway, but in order to make sure nobody copies this code and keep the behavior consistent between platforms, let's remove it. Change-Id: I5df5e00526e90cb573131de3c8bac9f85f4e3a5f Signed-off-by: Julius Werner <jwerner@chromium.org> Reviewed-on: https://review.coreboot.org/c/coreboot/+/40623 Reviewed-by: Furquan Shaikh <furquan@google.com> Reviewed-by: Hung-Te Lin <hungte@chromium.org> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src')
-rw-r--r--src/soc/mediatek/mt8183/memory.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/src/soc/mediatek/mt8183/memory.c b/src/soc/mediatek/mt8183/memory.c
index 53763fd557..aa8f7d201d 100644
--- a/src/soc/mediatek/mt8183/memory.c
+++ b/src/soc/mediatek/mt8183/memory.c
@@ -163,8 +163,7 @@ static void mt_mem_init_run(struct dramc_param_ops *dparam_ops)
/* Load calibration params from flash and run fast calibration */
if (recovery_mode) {
printk(BIOS_WARNING, "Skip loading cached calibration data\n");
- if (get_recovery_mode_retrain_switch() ||
- vboot_check_recovery_request() == VB2_RECOVERY_TRAIN_AND_REBOOT) {
+ if (get_recovery_mode_retrain_switch()) {
printk(BIOS_WARNING, "Retrain memory in next boot\n");
/* Use 0xFF as erased flash data. */
memset(dparam, 0xff, sizeof(*dparam));