[coreboot-gerrit] Change in coreboot[master]: soc/mediatek/mt8173: Refactor display driver to share common parts

[Hung-Te Lin (Code Review)]

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

Change subject: soc/mediatek/mt8173: Refactor display driver to share common parts
......................................................................

Patch Set 15:

(1 comment)

https://review.coreboot.org/c/coreboot/+/34515/14/src/soc/mediatek/common/in... 
File src/soc/mediatek/common/include/soc/ddp_common.h:

https://review.coreboot.org/c/coreboot/+/34515/14/src/soc/mediatek/common/in... 
PS14, Line 151: void ovl_set_roi(u32 idx, u32 width, u32 height, u32 color);
> nit: Please namespace these better now that they're no longer static (e.g. mtk_ddp_ovl_set_roi()).

let's do that later since there's almost no one except SOC ddp module will call them. and this has been taking so long to where we are...