Attention is currently required from: Ryan Chuang. Hello Ryan Chuang,
I'd like you to do a code review. Please visit
https://review.coreboot.org/c/coreboot/+/56850
to review the following change.
Change subject: vc/mediatek/mt8195: Optimize DRAM init time by reducing I2C I/O ......................................................................
vc/mediatek/mt8195: Optimize DRAM init time by reducing I2C I/O
Disable reading of vdram/vddq/vmddr to reduce access of i2c to reduce DRAM init time by about 30ms. The values were only needed by HQA report and not needed on production units.
BUG=b:195274787
Signed-off-by: Ryan Chuang ryan.chuang@mediatek.corp-partner.google.com Change-Id: I32cd68fb8b52cec6e145d6772475fde0130ca6ac --- M src/vendorcode/mediatek/mt8195/dramc/dramc_pi_main.c 1 file changed, 2 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/50/56850/1
diff --git a/src/vendorcode/mediatek/mt8195/dramc/dramc_pi_main.c b/src/vendorcode/mediatek/mt8195/dramc/dramc_pi_main.c index b5b9ae1..f39023b 100644 --- a/src/vendorcode/mediatek/mt8195/dramc/dramc_pi_main.c +++ b/src/vendorcode/mediatek/mt8195/dramc/dramc_pi_main.c @@ -273,10 +273,12 @@ #ifndef DDR_INIT_TIME_PROFILING print("Read voltage for %d, %d\n", p->frequency, vGet_Current_SRAMIdx(p)); print("Vcore = %d\n", dramc_get_vcore_voltage()); +#ifdef FOR_HQA_REPORT_USED print("Vdram = %d\n", dramc_get_vmdd_voltage(p->dram_type)); print("Vddq = %d\n", dramc_get_vmddq_voltage(p->dram_type)); print("Vmddr = %d\n", dramc_get_vmddr_voltage()); #endif +#endif
#endif #endif