[coreboot-gerrit] Change in coreboot[master]: mediatek/mt8183: Allow modifying vdram2 voltage

[Hung-Te Lin (Code Review)]

Hung-Te Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35147 )

Change subject: mediatek/mt8183: Allow modifying vdram2 voltage
......................................................................

Patch Set 10:

(1 comment)

https://review.coreboot.org/c/coreboot/+/35147/4/src/soc/mediatek/mt8183/mt6... 
File src/soc/mediatek/mt8183/mt6358.c:

https://review.coreboot.org/c/coreboot/+/35147/4/src/soc/mediatek/mt8183/mt6... 
PS4, Line 910: if (vdram2_uv > 680000) {
             : 		assert(0);
             : 		return;
             : 	} else if (vdram2_uv < 530000) {
             : 		assert(0);
             : 		return;
             : 	}
> Why not just […]

Are we going to fix this?