[coreboot-gerrit] Change in coreboot[master]: mediatek/mt8183: Init SPM driver

[DAWEI CHIEN (Code Review)]

DAWEI CHIEN has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34545 )

Change subject: mediatek/mt8183: Init SPM driver
......................................................................

Patch Set 12:

(1 comment)

https://review.coreboot.org/c/coreboot/+/34545/12/src/mainboard/google/kukui... 
File src/mainboard/google/kukui/mainboard.c:

https://review.coreboot.org/c/coreboot/+/34545/12/src/mainboard/google/kukui... 
PS12, Line 59: spm_init(get_dram_type());
> Please try not to call get_sdram_config() from ramstage, it will have to load the whole blob from SP […]

I just talk with MTK dram driver owner, how about if we move get_dram_type function to emi.c, and spm driver include emi.h to call get_dram_type.

int get_dram_type(void)
{
    return params.dram_type;
}