[coreboot-gerrit] Change in coreboot[master]: mb/google/kukui: Initialize display

[Julius Werner (Code Review)]

Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32511 )

Change subject: mb/google/kukui: Initialize display
......................................................................

Patch Set 38:

(2 comments)

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

https://review.coreboot.org/c/coreboot/+/32511/38/src/mainboard/google/kukui... 
PS38, Line 32: _INIT_DCS_CMD
Let's not start the thing that we actually want to use in the panel file with an underscore (underscores are for intermediary macros hidden away in some header). I'd either just call this something else (e.g. INIT_DCS() or PANEL_INIT_DCS or something) or rename the integer constant (e.g. call the macro INIT_DCS_CMD() and the constant INIT_DCS_CMD_OPCODE or something).

https://review.coreboot.org/c/coreboot/+/32511/38/src/mainboard/google/kukui... 
PS38, Line 34: 	.len = sizeof((u8[]){__VA_ARGS__}), \
Nice, that's way more clever than what I said!