Weiyi Lu would like Weiyi Lu to review this change.

View Change

mediatek/mt8183: postpone dcxo low power mode setting

Consider the association between MD and DCXO, this patch is a fix for
eb5e47d("mediatek/mt8183: update dcxo output buffer setting") [1]
We should not disable XO_CEL and block the bblpm request when MD is still ON.

[1] https://review.coreboot.org/c/coreboot/+/32323

BRANCH=none
TEST=Boots correctly on Krane.

Change-Id: I047ebed615e874977ca211aafd52b5551c71b764
Signed-off-by: Weiyi Lu <weiyi.lu@mediatek.com>
---
M src/soc/mediatek/mt8183/include/soc/rtc.h
M src/soc/mediatek/mt8183/rtc.c
M src/soc/mediatek/mt8183/soc.c
3 files changed, 14 insertions(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/77/34777/1

diff --git a/src/soc/mediatek/mt8183/include/soc/rtc.h b/src/soc/mediatek/mt8183/include/soc/rtc.h
index 5a61208..aa4cc43 100644
--- a/src/soc/mediatek/mt8183/include/soc/rtc.h
+++ b/src/soc/mediatek/mt8183/include/soc/rtc.h
@@ -147,6 +147,7 @@
 /* PMIC DCXO Register Definition */
 enum {
 	PMIC_RG_DCXO_CW00 = 0x0788,
+	PMIC_RG_DCXO_CW00_CLR = 0x078C,
 	PMIC_RG_DCXO_CW02 = 0x0790,
 	PMIC_RG_DCXO_CW07 = 0x079A,
 	PMIC_RG_DCXO_CW09 = 0x079E,
@@ -218,5 +219,6 @@
 void rtc_osc_init(void);
 int rtc_init(u8 recover);
 void rtc_boot(void);
+void dcxo_disable_unused(void);
 
 #endif /* SOC_MEDIATEK_MT8183_RTC_H */
diff --git a/src/soc/mediatek/mt8183/rtc.c b/src/soc/mediatek/mt8183/rtc.c
index af30d1f..464fff3 100644
--- a/src/soc/mediatek/mt8183/rtc.c
+++ b/src/soc/mediatek/mt8183/rtc.c
@@ -411,10 +411,9 @@
 	rtc_write(PMIC_RG_DCXO_CW16, 0x9855);
 
 	/* 26M enable control */
-	/* Enable clock buffer XO_SOC */
-	rtc_write(PMIC_RG_DCXO_CW00, 0x4005);
+	/* Enable clock buffer XO_SOC, XO_CEL */
+	rtc_write(PMIC_RG_DCXO_CW00, 0x4805);
 	rtc_write(PMIC_RG_DCXO_CW11, 0x8000);
-	rtc_write(PMIC_RG_DCXO_CW23, 0x0053);
 
 	/* Load thermal coefficient */
 	rtc_write(PMIC_RG_TOP_TMA_KEY, 0x9CA7);
@@ -432,6 +431,14 @@
 	mdelay(5);
 }
 
+void dcxo_disable_unused(void)
+{
+	/* Disable clock buffer XO_CEL */
+	rtc_write(PMIC_RG_DCXO_CW00_CLR, 0x0800);
+	/* Mask bblpm */
+	rtc_write(PMIC_RG_DCXO_CW23, 0x0053);
+}
+
 /* the rtc boot flow entry */
 void rtc_boot(void)
 {
diff --git a/src/soc/mediatek/mt8183/soc.c b/src/soc/mediatek/mt8183/soc.c
index c9c2147..f190480 100644
--- a/src/soc/mediatek/mt8183/soc.c
+++ b/src/soc/mediatek/mt8183/soc.c
@@ -17,6 +17,7 @@
 #include <soc/emi.h>
 #include <soc/md_ctrl.h>
 #include <soc/mmu_operations.h>
+#include <soc/rtc.h>
 #include <soc/sspm.h>
 #include <symbols.h>
 
@@ -29,6 +30,7 @@
 {
 	mtk_mmu_disable_l2c_sram();
 	mtk_md_early_init();
+	dcxo_disable_unused();
 	sspm_init();
 }
 

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