build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44701 )
Change subject: soc/mediatek/mt8192: Do the dramc pinmux selection ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/c/coreboot/+/44701/1/src/soc/mediatek/mt8192/dra... File src/soc/mediatek/mt8192/dramc_pi_basic_api.c:
https://review.coreboot.org/c/coreboot/+/44701/1/src/soc/mediatek/mt8192/dra... PS1, Line 17: [CHANNEL_A] = {1, 0, 3, 2, 4, 7, 6, 5,8, 9, 10, 14, 11, 15, 13, 12}, space required after that ',' (ctx:VxV)
https://review.coreboot.org/c/coreboot/+/44701/1/src/soc/mediatek/mt8192/dra... PS1, Line 18: [CHANNEL_B] = {0, 1, 4, 7, 3, 5, 6, 2,9, 8, 10, 12, 11, 14, 13, 15} space required after that ',' (ctx:VxV)