[coreboot-gerrit] Change in coreboot[master]: soc/mediatek/mt8183: Add DSI driver

[Hung-Te Lin (Code Review)]

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

Change subject: soc/mediatek/mt8183: Add DSI driver
......................................................................

Patch Set 48:

(2 comments)

https://review.coreboot.org/c/coreboot/+/31591/47/src/soc/mediatek/common/in... 
File src/soc/mediatek/common/include/soc/dsi_common.h:

https://review.coreboot.org/c/coreboot/+/31591/47/src/soc/mediatek/common/in... 
PS47, Line 69: 	u32 dsi_lfr_con;
> nit: add comment which registers are only available since 8183?

Done

https://review.coreboot.org/c/coreboot/+/31591/47/src/soc/mediatek/mt8183/ds... 
File src/soc/mediatek/mt8183/dsi.c:

https://review.coreboot.org/c/coreboot/+/31591/47/src/soc/mediatek/mt8183/ds... 
PS47, Line 59: 26
> Is this related to CLK26M_HZ? If so, please use that constant for clarity.

Done