[coreboot-gerrit] Change in ...coreboot[master]: google/kukui: Boot up SSPM

[You-Cheng Syu (Code Review)]

You-Cheng Syu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31516 )

Change subject: google/kukui: Boot up SSPM
......................................................................

Patch Set 7:

(1 comment)

https://review.coreboot.org/#/c/31516/7/src/soc/mediatek/mt8183/sspm.c 
File src/soc/mediatek/mt8183/sspm.c:

https://review.coreboot.org/#/c/31516/7/src/soc/mediatek/mt8183/sspm.c@24 
PS7, Line 24: static uint8_t sspm_bin[BUF_SIZE];
> Maybe I need to implement memory copy by myself since destination sram need word aligned access? […]

You need to make sure your sspm_bin[] is aligned so that memcpy will do word aligned access.
See: https://github.com/coreboot/coreboot/blob/master/payloads/libpayload/libc/me...