Hello Xi Chen,
I'd like you to do a code review. Please visit
https://review.coreboot.org/c/coreboot/+/46398
to review the following change.
Change subject: soc/mediatek/mt8192: Add multi freq calibration ......................................................................
soc/mediatek/mt8192: Add multi freq calibration
For calibration highest freq.
Signed-off-by: mtk11206 xixi.chen@mediatek.com Change-Id: Iddab4dd1a331c668b967207e7f1542ea29424b68 --- M src/soc/mediatek/mt8192/Kconfig M src/soc/mediatek/mt8192/dramc_pi_main.c M src/soc/mediatek/mt8192/include/soc/dramc_param.h 3 files changed, 3 insertions(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/98/46398/1
diff --git a/src/soc/mediatek/mt8192/Kconfig b/src/soc/mediatek/mt8192/Kconfig index ea82a8b..6723cc5 100644 --- a/src/soc/mediatek/mt8192/Kconfig +++ b/src/soc/mediatek/mt8192/Kconfig @@ -38,7 +38,7 @@
config MT8192_DRAM_DVFS_LIMIT_FREQ_CNT bool - default n + default y select MT8192_DRAM_DVFS help This options limit DRAM frequency calibration count from total 7 to 3, diff --git a/src/soc/mediatek/mt8192/dramc_pi_main.c b/src/soc/mediatek/mt8192/dramc_pi_main.c old mode 100644 new mode 100755 index e4772c2..e7c6ecb --- a/src/soc/mediatek/mt8192/dramc_pi_main.c +++ b/src/soc/mediatek/mt8192/dramc_pi_main.c @@ -480,7 +480,7 @@ after_calib(&cali); enable_dfs_hw_mode_clk();
- if (CONFIG(MT8192_DRAM_DVFS)) { + if (CONFIG(MT8192_DRAM_DVFS) || CONFIG(MT8192_DRAM_DVFS_LIMIT_FREQ_CNT)) { dram_cali_seq bootup_cali_seq = CALI_SEQ5; dram_dfs_shu bootup_shu = get_shu_save_by_k_shu(bootup_cali_seq);
diff --git a/src/soc/mediatek/mt8192/include/soc/dramc_param.h b/src/soc/mediatek/mt8192/include/soc/dramc_param.h old mode 100644 new mode 100755 index b4e982f..2d4f2ca --- a/src/soc/mediatek/mt8192/include/soc/dramc_param.h +++ b/src/soc/mediatek/mt8192/include/soc/dramc_param.h @@ -8,7 +8,7 @@ #include <soc/dramc_common_mt8192.h>
enum { - DRAMC_PARAM_HEADER_VERSION = 3, + DRAMC_PARAM_HEADER_VERSION = 4, };
enum DRAMC_PARAM_STATUS_CODES {