build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46409 )
Change subject: soc/mediatek/mt8192: eint: unmask eint event mask register ......................................................................
Patch Set 2:
(2 comments)
https://review.coreboot.org/c/coreboot/+/46409/2/src/soc/mediatek/mt8192/ein... File src/soc/mediatek/mt8192/eint_event.c:
https://review.coreboot.org/c/coreboot/+/46409/2/src/soc/mediatek/mt8192/ein... PS2, Line 7: for (i = 0; i < 7; i++) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/c/coreboot/+/46409/2/src/soc/mediatek/mt8192/ein... PS2, Line 12: } void function return statements are not generally useful