build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44699 )
Change subject: soc/mediatek/mt8192: Add dram control register define and bits define ......................................................................
Patch Set 8:
(2 comments)
https://review.coreboot.org/c/coreboot/+/44699/8/src/soc/mediatek/mt8192/inc... File src/soc/mediatek/mt8192/include/soc/dramc_pi_api.h:
https://review.coreboot.org/c/coreboot/+/44699/8/src/soc/mediatek/mt8192/inc... PS8, Line 160: /* frequency set point: 0 means lower,un-terminated freq, 1 means higer,terminated freq */ 'higer' may be misspelled - perhaps 'higher'?
https://review.coreboot.org/c/coreboot/+/44699/8/src/soc/mediatek/mt8192/inc... PS8, Line 160: /* frequency set point: 0 means lower,un-terminated freq, 1 means higer,terminated freq */ line over 96 characters