[coreboot-gerrit] Change in coreboot[master]: soc/mediatek: dsi: fine tune the delta time for EoTp

[Yu-Ping Wu (Code Review)]

Attention is currently required from: jitao shi.
Yu-Ping Wu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/51432 )

Change subject: soc/mediatek: dsi: fine tune the delta time for EoTp
......................................................................

Patch Set 6:

(1 comment)

File src/soc/mediatek/common/dsi.c:

https://review.coreboot.org/c/coreboot/+/51432/comment/907e6a44_a093afd8 
PS2, Line 207: 10
> According to https://patchwork.kernel.org/project/linux-mediatek/patch/20210201033603.126.... […]

I'm also super confused by this. The MIPI_DSI_MODE_EOT_PACKET flag has been modified multiple times.

CB:41161 added the flag for anx7625.

+ if (CONFIG(DRIVER_ANALOGIX_ANX7625))                                     
 +     mipi_dsi_flags |= MIPI_DSI_MODE_EOT_PACKET;

CB:48868 further added

+ if (mode_flags & MIPI_DSI_MODE_EOT_PACKET)
 +     delta += 2;

However, CL:2604806 in kernel removed MIPI_DSI_MODE_EOT_PACKET for anx7625. Should we also *enable* EOTP (by removing the flag) in coreboot too?