Hung-Te Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46392 )
Change subject: soc/mediatek/mt8192: Load MCUPM firmware and boot up MCUPM. ......................................................................
Patch Set 4:
(2 comments)
https://review.coreboot.org/c/coreboot/+/46392/4/src/soc/mediatek/mt8192/mcu... File src/soc/mediatek/mt8192/mcupm.c:
https://review.coreboot.org/c/coreboot/+/46392/4/src/soc/mediatek/mt8192/mcu... PS4, Line 26: void *)(0x0C55FAA0) define a variable to explain what is 0x0c55faa0
https://review.coreboot.org/c/coreboot/+/46392/4/src/soc/mediatek/mt8192/mcu... PS4, Line 15: const char *file_name = "mcupm.bin"; : size_t fw_size = cbfs_boot_load_file(file_name, : mcupm_bin, : sizeof(mcupm_bin), : CBFS_TYPE_RAW); : : if (fw_size == 0) { : die("MCUPM file :mcupm.bin not found."); : } : : write32(&mt8192_mcupm->sw_rstn, 0x0); : write32((void *)(0x0C55FAA0), 0x0); : memcpy((void *)MCUPM_SRAM_BASE, mcupm_bin, fw_size); : /* Memory barrier to ensure that all fw code is loaded : before we release the reset pin. */ : mb(); : write32(&mt8192_mcupm->sw_rstn, 0x1); please indent properly.