[coreboot-gerrit] Change in coreboot[master]: mediatek/mt8183: add dsi driver for mt8183

[Hung-Te Lin (Code Review)]

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

Change subject: mediatek/mt8183: add dsi driver for mt8183
......................................................................

Patch Set 16:

(3 comments)

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

https://review.coreboot.org/c/coreboot/+/31591/16/src/soc/mediatek/mt8183/ds... 
PS16, Line 399: count
we may remove count?

https://review.coreboot.org/c/coreboot/+/31591/16/src/soc/mediatek/mt8183/ds... 
PS16, Line 413: break
return;

https://review.coreboot.org/c/coreboot/+/31591/16/src/soc/mediatek/mt8183/ds... 
PS16, Line 459: u32 count
> 8173 init uses the END_OF_TABLE so there's need to count. […]

oh, sorry, 8173 didn't really do table init. but since you've declared END_OF_TABLE, why not use it?