[coreboot-gerrit] Change in coreboot[master]: soc/mediatek/mt8183: Add get dram type API at ATF

[Julius Werner (Code Review)]

Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39097 )

Change subject: soc/mediatek/mt8183: Add get dram type API at ATF
......................................................................

Patch Set 4: Code-Review-1

Please don't subvert parameter types for something they weren't meant to be. You can create a new vendor-specific parameter type in TF if you need to.

(Do you really need to pass a parameter for this? Is there no way for TF to detect this information from the EMC registers that were already configured for the right type by coreboot?)