[coreboot-gerrit] Change in coreboot[master]: soc/mediatek: Change DSI init commands to take flexible length array

[Hung-Te Lin (Code Review)]

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

Change subject: soc/mediatek: Change DSI init commands to take flexible length array
......................................................................

Patch Set 24:

(2 comments)

https://review.coreboot.org/c/coreboot/+/34778/12/src/soc/mediatek/common/ds... 
File src/soc/mediatek/common/dsi.c:

https://review.coreboot.org/c/coreboot/+/34778/12/src/soc/mediatek/common/ds... 
PS12, Line 333: 	 */
> Please use one of the allowed comment styles from the coding style.

Done

https://review.coreboot.org/c/coreboot/+/34778/13/src/soc/mediatek/common/ds... 
File src/soc/mediatek/common/dsi.c:

https://review.coreboot.org/c/coreboot/+/34778/13/src/soc/mediatek/common/ds... 
PS13, Line 347: 		buf += sizeof(*init);
> nit: why not do the whole […]

because if command is delay, len must be ignored.