Duan huayang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41949 )
Change subject: soc/mediatek/mt8183: Support 6GB, 8GB DDR bootup ......................................................................
Patch Set 3:
(2 comments)
https://review.coreboot.org/c/coreboot/+/41949/3/src/soc/mediatek/mt8183/dra... File src/soc/mediatek/mt8183/dramc_param.c:
https://review.coreboot.org/c/coreboot/+/41949/3/src/soc/mediatek/mt8183/dra... PS3, Line 34: static void set_ddr_size_type(struct dramc_param *param) : { : u32 ramcode = get_rom_code(); : u16 ddr_size_type = DDR_TYPE_2CH_2RK_4GB_2_2; : : switch (ramcode) { : case 1: : case 2: : case 3: : case 4: : case 5: : case 6: : case 7: : case 8: : ddr_size_type = DDR_TYPE_2CH_2RK_4GB_2_2; : break; : default: : break; : } : : param->header.ddr_size_type = ddr_size_type; : }
no, we should not do this. The ddr_size_type should be a data directly from sdram_config.
the sdram_config are uesless now, it only can be called at the partial k flow. current the full K and fast K do not use the sdram_config
https://review.coreboot.org/c/coreboot/+/41949/3/src/soc/mediatek/mt8183/dra... PS3, Line 64: set_ddr_size_type(param);
Please move this to be a param of initialize_dramc_param.
please more details.