12 comments:
Patch Set #4, Line 7: boot up sspm
Boot up SSPM
Done
Patch Set #4, Line 9: enables
enable
Done
Patch Set #4, Line 14: worng
wrong
Removed it
File src/mainboard/google/kukui/Kconfig:
Patch Set #4, Line 68: config SSPM_BIN_FILE
Still think this shouldn't be a Kconfig...
Removed it
Patch Set #4, Line 71: ---help---
That syntax seems new.
Removed it
Patch Set #4, Line 72: SSPM is Secure System Power Manager for power control in secure domain.
Please add a dot at the end.
Removed it
File src/mainboard/google/kukui/Makefile.inc:
Patch Set #4, Line 32: sspm-compression := $(CBFS_COMPRESS_FLAG)
Oh... yes, actually, that's the cause of your problem. […]
Done
File src/mainboard/google/kukui/mainboard.c:
Patch Set #4, Line 41: static void configure_sspm(void)
nit: You don't really need single line functions, can just put sspm_init() into mainboard_init().
Done
File src/soc/mediatek/mt8183/sspm.c:
Probably better to just die() if something goes wrong, assuming the board cannot really boot without […]
Done
Patch Set #4, Line 35: printk(BIOS_DEBUG, "no sspm\n");
No SSPM firmware(?) found.
Done
Patch Set #4, Line 35: BIOS_DEBUG
I'd say at least ERR, but probably a die()
Done
Patch Set #4, Line 38: "sspm[0]=%#x, [%zd]=%#x\n",
nit: I'm not sure this is useful enough to keep in here after you're done debugging?
Done
To view, visit change 31516. To unsubscribe, or for help writing mail filters, visit settings.