[coreboot-gerrit] Change in ...coreboot[master]: mediatek/mt8183: Add display driver

[yongqiang niu (Code Review)]

yongqiang niu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31478 )

Change subject: mediatek/mt8183: Add display driver
......................................................................

Patch Set 14:
> Patch Set 13:
>
> A lot of this seems duplicated from MT8173 code. Can you factor out the common parts?

there are 18 functions in 8183 ddp.c, but only 1(color_start) of them is same with 8173, the duplicate rate is 5.5%

there are 17 struct or enum define in 8183 ddp.h, but only 2 (disp_color_regs, OVL_INPUT_FORMAT) of them is same with 8173, the duplicate rate is 11.7%

i suppose the duplicate rate is too low, and no common them, please double confirm