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

[Julius Werner (Code Review)]

Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34772 )

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

Patch Set 11: Code-Review+2

(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 ^^

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.