build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47359 )
Change subject: HACK: Random SPM debugging code ......................................................................
Patch Set 1:
(4 comments)
https://review.coreboot.org/c/coreboot/+/47359/1/src/soc/mediatek/mt8192/spm... File src/soc/mediatek/mt8192/spm.c:
https://review.coreboot.org/c/coreboot/+/47359/1/src/soc/mediatek/mt8192/spm... PS1, Line 534: printk(BIOS_INFO, "spm md32pcm_dma0_rlct @%08lx\n", (long)&mtk_spm->md32pcm_dma0_rlct); line over 96 characters
https://review.coreboot.org/c/coreboot/+/47359/1/src/soc/mediatek/mt8192/spm... PS1, Line 548: printk(BIOS_INFO, "SPM con0 0x%08x/0x%08x\n", con0, con0 | SPM_REGWR_CFG_KEY | PCM_CK_EN_LSB); line over 96 characters
https://review.coreboot.org/c/coreboot/+/47359/1/src/soc/mediatek/mt8192/spm... PS1, Line 817: u32* p = (u32 *)mtk_spm; "foo* bar" should be "foo *bar"
https://review.coreboot.org/c/coreboot/+/47359/1/src/soc/mediatek/mt8192/spm... PS1, Line 824: int spm_init(void) { open brace '{' following function definitions go on the next line