Attention is currently required from: Hung-Te Lin. flora.fu@mediatek.com has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/59038 )
Change subject: soc/mediatek/mt8195: fix apusys coding defects ......................................................................
soc/mediatek/mt8195: fix apusys coding defects
Use size_t for count variables. Reducing debug log level and fix typo. Fix commit: https://review.coreboot.org/c/coreboot/+/58794
BUG=b:203145462 BRANCH=cherry TEST=boot cherry correctly
Signed-off-by: Flora Fu flora.fu@mediatek.com Change-Id: Ic03f71b7a9038edb5877ebd9b6aed5e9bd63c918 --- M src/soc/mediatek/mt8195/apusys.c 1 file changed, 4 insertions(+), 4 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/38/59038/1
diff --git a/src/soc/mediatek/mt8195/apusys.c b/src/soc/mediatek/mt8195/apusys.c index bc3c4ab..fc95cab 100644 --- a/src/soc/mediatek/mt8195/apusys.c +++ b/src/soc/mediatek/mt8195/apusys.c @@ -10,10 +10,10 @@
static void dump_apusys_reg(void) { - int i; + size_t i;
for (i = 0; i < ARRAY_SIZE(mt8195_apu_mbox); i++) { - printk(BIOS_INFO, "APU_MBOX %p = %#x\n", + printk(BIOS_DEBUG, "APU_MBOX %p = %#x\n", (void *)&mt8195_apu_mbox[i]->mbox_func_cfg, read32(&mt8195_apu_mbox[i]->mbox_func_cfg)); } @@ -21,9 +21,9 @@
void apusys_init(void) { - int i; + size_t i;
- /* Setup MBOX MPU for non secure access */ + /* Set up MBOX MPU for non secure access */ for (i = 0; i < ARRAY_SIZE(mt8195_apu_mbox); i++) SET32_BITFIELDS(&mt8195_apu_mbox[i]->mbox_func_cfg, NO_MPU, 1, LOCK, 1);