Hung-Te Lin has submitted this change. ( https://review.coreboot.org/c/coreboot/+/55154 )
Change subject: mb/google/cherry: Initialize SPM ......................................................................
mb/google/cherry: Initialize SPM
This patch adds support for SPM. This adds 43ms to the boot time.
TEST=program counter of SPM is correct value after booting up.
Signed-off-by: Dawei Chien dawei.chien@mediatek.com Signed-off-by: Edward-JW Yang edward-jw.yang@mediatek.corp-partner.google.com Signed-off-by: Rex-BC Chen rex-bc.chen@mediatek.com Change-Id: I5f17f6d51fc9ad2d23c71c3c5cd29fdc777dc071 Reviewed-on: https://review.coreboot.org/c/coreboot/+/55154 Reviewed-by: Yu-Ping Wu yupingso@google.com Reviewed-by: Rex-BC Chen rex-bc.chen@mediatek.corp-partner.google.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/mainboard/google/cherry/mainboard.c 1 file changed, 4 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Yu-Ping Wu: Looks good to me, approved Rex-BC Chen: Looks good to me, but someone else must approve
diff --git a/src/mainboard/google/cherry/mainboard.c b/src/mainboard/google/cherry/mainboard.c index 2607a2d..04f4293 100644 --- a/src/mainboard/google/cherry/mainboard.c +++ b/src/mainboard/google/cherry/mainboard.c @@ -11,6 +11,7 @@ #include <soc/i2c.h> #include <soc/mt6360.h> #include <soc/regulator.h> +#include <soc/spm.h> #include <soc/usb.h>
#include "gpio.h" @@ -124,6 +125,9 @@ configure_sdcard(); setup_usb_host();
+ if (spm_init()) + printk(BIOS_ERR, "spm init failed, system suspend may not work\n"); + register_reset_to_bl31(); }