[coreboot-gerrit] Change in coreboot[master]: mb/google/kukui: Add panels for Krane

[Hung-Te Lin (Code Review)]

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

Change subject: mb/google/kukui: Add panels for Krane
......................................................................

Patch Set 16:

(1 comment)

https://review.coreboot.org/c/coreboot/+/33413/15/src/mainboard/google/kukui... 
File src/mainboard/google/kukui/panel_krane.c:

https://review.coreboot.org/c/coreboot/+/33413/15/src/mainboard/google/kukui... 
PS15, Line 357: static struct panel_description *krane_panels[] = {
> let's do that later when we find this really needed. […]

I did some experiment and found it not being very easy/clean at the moment.

The major reason is to load from CBFS we have to know the size first. SDRAM_config is usually in fixed size (or at least knowing the maximum). The table above may vary a lot and hard to define a size for it.