Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38400 )
Change subject: soc/mediatek/mt8183: reduce the hbp and hfp for phy timing ......................................................................
Patch Set 1:
(5 comments)
https://review.coreboot.org/c/coreboot/+/38400/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/38400/1//COMMIT_MSG@9 PS1, Line 9: transfer tranfers
https://review.coreboot.org/c/coreboot/+/38400/1//COMMIT_MSG@12 PS1, Line 12: calc calculated
https://review.coreboot.org/c/coreboot/+/38400/1//COMMIT_MSG@14 PS1, Line 14: So dsi driver reduces the hbp and hfp to keep the line time. Please give exact time values.
https://review.coreboot.org/c/coreboot/+/38400/1/src/soc/mediatek/common/dsi... File src/soc/mediatek/common/dsi.c:
https://review.coreboot.org/c/coreboot/+/38400/1/src/soc/mediatek/common/dsi... PS1, Line 74: int data_rate That argument doesn’t seem to be used anymore.
https://review.coreboot.org/c/coreboot/+/38400/1/src/soc/mediatek/common/dsi... PS1, Line 201: * bytes_per_pixel - 10; Formatting shouldn’t be changed.