[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());
> The point is that 'params' is not accessible in ramstage (or at least you shouldn't access it, for e […]

Many thank you for your information, now we got the point.
On next version, we would not run sdram_config.c in ram stage any more, and try to pass params.dram_type at ram stage if possible, thank you.

@Huayang, would you please comment for passing dram_type at ram stage, thank you.