[coreboot-gerrit] Change in ...coreboot[master]: google/kukui: boot up sspm

[Erin Lo (Code Review)]

Erin Lo has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31516 )

Change subject: google/kukui: boot up sspm
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/#/c/31516/1/src/mainboard/google/kukui/mainboard... 
File src/mainboard/google/kukui/mainboard.c:

https://review.coreboot.org/#/c/31516/1/src/mainboard/google/kukui/mainboard... 
PS1, Line 44: unsigned char buf[BUF_SIZE];
> Yeah, that's definitely the right thing to do, don't copy 64KB around more than necessary.

SSPM_SRAM_BASE is accessed through APB box. It can't support byte access only supports word access. Does it touch any limitation in cbfs_boot_load_file()? I found "lzma: Decoding error = 1" when directly load the file into SSPM_SRAM_BASE.