[coreboot-gerrit] Change in ...coreboot[master]: mediatek/mt8183: add SAMSUNG and MICRON EMCP LPDDR4X support

[huayang duan (Code Review)]

huayang duan has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32243 )

Change subject: mediatek/mt8183: add SAMSUNG and MICRON EMCP LPDDR4X support
......................................................................

Patch Set 3: Code-Review+1

(1 comment)

https://review.coreboot.org/#/c/32243/3/src/soc/mediatek/mt8183/dramc_pi_cal... 
File src/soc/mediatek/mt8183/dramc_pi_calibration_api.c:

https://review.coreboot.org/#/c/32243/3/src/soc/mediatek/mt8183/dramc_pi_cal... 
PS3, Line 35: -16
> why is this changed, and will this be a problem for other existing DRAM modules? […]

From the calibration log of EMCP-SAMSUNG DDR, we found the begin pass range of RX window early than other DDR. Because the pass window will be affected by SOC chip manufacturing process, DDR chip manufacturing process, current volatge, temperature and some other environment. If we not increase the begin scan range, the pass window maybe get wrong result at some board.