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

(3 comments)

https://review.coreboot.org/c/coreboot/+/34778/13/src/mainboard/google/kukui... 
File src/mainboard/google/kukui/mainboard.c:

https://review.coreboot.org/c/coreboot/+/34778/13/src/mainboard/google/kukui... 
PS13, Line 132: 			 (const struct lcm_init_command *)panel->init) < 0) {
> Just change the function to take a (u8 *) instead, you need casts in there anyway.

Done

https://review.coreboot.org/c/coreboot/+/34778/13/src/mainboard/google/kukui... 
File src/mainboard/google/kukui/panel.h:

https://review.coreboot.org/c/coreboot/+/34778/13/src/mainboard/google/kukui... 
PS13, Line 27: char
> u8

Done

https://review.coreboot.org/c/coreboot/+/34778/13/src/mainboard/google/kukui... 
PS13, Line 36: 	__VA_ARGS__
> Woah... this is one of those "just crazy enough it could work" things. […]

But Jenkins is not happy about it and I'm not sure how to solve that...