GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

soc/mediatek/mt8195: fix apusys coding defects 

Use size_t for count variables.
Reduce 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
Reviewed-on: https://review.coreboot.org/c/coreboot/+/59038
Reviewed-by: Yu-Ping Wu <yupingso@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
Flora Fu 2021-11-09 16:42:05 +08:00 committed by Felix Held
parent 67b91b9344
commit 66f2cbb195
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/soc/mediatek/mt8195/apusys.c
View File

@ -10,10 +10,10 @@ DEFINE_BITFIELD(NO_MPU, 16, 16)
static void dump_apusys_reg(void) static void dump_apusys_reg(void)
{ {
int i; size_t i;
for (i = 0; i < ARRAY_SIZE(mt8195_apu_mbox); 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, (void *)&mt8195_apu_mbox[i]->mbox_func_cfg,
read32(&mt8195_apu_mbox[i]->mbox_func_cfg)); read32(&mt8195_apu_mbox[i]->mbox_func_cfg));
} }
@ -21,9 +21,9 @@ static void dump_apusys_reg(void)
void apusys_init(void) 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++) for (i = 0; i < ARRAY_SIZE(mt8195_apu_mbox); i++)
SET32_BITFIELDS(&mt8195_apu_mbox[i]->mbox_func_cfg, NO_MPU, 1, LOCK, 1); SET32_BITFIELDS(&mt8195_apu_mbox[i]->mbox_func_cfg, NO_MPU, 1, LOCK, 1);