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

[Julius Werner (Code Review)]

Julius Werner 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 2:

(1 comment)

This change is ready for review.

https://review.coreboot.org/c/coreboot/+/34872/1/src/mainboard/google/kukui/... 
File src/mainboard/google/kukui/romstage.c:

https://review.coreboot.org/c/coreboot/+/34872/1/src/mainboard/google/kukui/... 
PS1, Line 44: memset(_dram_init_code + blob_size, 0, region_size - blob_size);
> To initial .bss to zero. […]

Guys, why are we inventing our own loading conventions? Please provide the DRAM blob as an ELF, not a raw .bin, and then treat it as a stage in CBFS. That way, .bss initialization will be handled automatically.