Xi Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44699 )
Change subject: soc/mediatek/mt8192: Define DRAM registers and APIs ......................................................................
Patch Set 44:
(6 comments)
https://review.coreboot.org/c/coreboot/+/44699/41//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/44699/41//COMMIT_MSG@7 PS41, Line 7: Add dram control register define and bits define
Define DRAM registers and APIs
Ack
https://review.coreboot.org/c/coreboot/+/44699/41/src/soc/mediatek/mt8192/in... File src/soc/mediatek/mt8192/include/soc/dramc_pi_api.h:
https://review.coreboot.org/c/coreboot/+/44699/41/src/soc/mediatek/mt8192/in... PS41, Line 111: DutyScan_K_DQ
Uppercase for enums
Ack
https://review.coreboot.org/c/coreboot/+/44699/41/src/soc/mediatek/mt8192/in... PS41, Line 160: /* frequency set point: : * 0 means lower,un-terminated freq; : * 1 means higher,terminated freq : */
Format: […]
Ack
https://review.coreboot.org/c/coreboot/+/44699/41/src/soc/mediatek/mt8192/in... PS41, Line 276: u8 dqs_final_delay[2][2]
Either […]
Ack
https://review.coreboot.org/c/coreboot/+/44699/41/src/soc/mediatek/mt8192/in... PS41, Line 284: calType
lowercase
Ack
https://review.coreboot.org/c/coreboot/+/44699/41/src/soc/mediatek/mt8192/in... PS41, Line 285: u1VrefScanEnable
lowercase
Ack