[coreboot-gerrit] Change in coreboot[master]: src/mediatek/mt8192: pass regulator function pointer to dram blob

[Xi Chen (Code Review)]

Xi Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48261 )

Change subject: src/mediatek/mt8192: pass regulator function pointer to dram blob
......................................................................

Patch Set 2:

(2 comments)

https://review.coreboot.org/c/coreboot/+/48261/1/src/soc/mediatek/mt8192/inc... 
File src/soc/mediatek/mt8192/include/soc/dramc_param.h:

https://review.coreboot.org/c/coreboot/+/48261/1/src/soc/mediatek/mt8192/inc... 
PS1, Line 57: ptr
> how about _cb  for callback ?

Ack

https://review.coreboot.org/c/coreboot/+/48261/1/src/soc/mediatek/mt8192/inc... 
PS1, Line 148: dram
> how about remove dram_ prefix to shorten the naming ?

Ack