[coreboot-gerrit] Change in coreboot[master]: google/kukui: Load and run dram init blob

[Tristan Hsieh (Code Review)]

Tristan Hsieh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34872 )

Change subject: google/kukui: Load and run dram init blob
......................................................................

Patch Set 5:
> Patch Set 5:
>
> ... on the other hand, it's probably also not a bad idea to put everything into the buffer, say
>
> struct dram_calibration_param {
>   u32 result;
>   u32 length;
>   u8 data[];
>  }
>
> This may be easier to prevent ABI issues

I think we should do it this way (with struct).
Since we use PROG_RUN(), it passes only one argument to entry.