[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 3:

(3 comments)

https://review.coreboot.org/#/c/31478/2//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/#/c/31478/2//COMMIT_MSG@8 
PS2, Line 8:
> What datasheet did you use?

does datasheet means hardware register map?

https://review.coreboot.org/#/c/31478/2//COMMIT_MSG@9 
PS2, Line 9: BUG=b:80501386
> Also add b:117254947

add this bug id in patch3

https://review.coreboot.org/#/c/31478/2/src/mainboard/google/kukui/mainboard... 
File src/mainboard/google/kukui/mainboard.c:

https://review.coreboot.org/#/c/31478/2/src/mainboard/google/kukui/mainboard... 
PS2, Line 66: start
> Starting

modified in patch3