[coreboot-gerrit] Change in coreboot[master]: soc/mediatek: dsi: Refactor video timing calculation

[Hung-Te Lin (Code Review)]

Hung-Te Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34772 )

Change subject: soc/mediatek: dsi: Refactor video timing calculation
......................................................................

Patch Set 12:

(2 comments)

https://review.coreboot.org/c/coreboot/+/34772/11/src/soc/mediatek/common/ds... 
File src/soc/mediatek/common/dsi.c:

https://review.coreboot.org/c/coreboot/+/34772/11/src/soc/mediatek/common/ds... 
PS11, Line 209: 	u32 d_phy = data_phy_cycles * lanes + delta;
> nit: I hope you checked that this comes out to 12 for 8173, because I'm too lazy ^^

Done

https://review.coreboot.org/c/coreboot/+/34772/11/src/soc/mediatek/common/ds... 
PS11, Line 268: 	mtk_dsi_config_vdo_timing(mode_flags, format, lanes, edid, &phy_timing);
> Oh, this is why you passed it out. Okay, fair enough.

Ack