Attention is currently required from: Hung-Te Lin, Jarried Lin, Xi Chen, Yu-Ping Wu.
Paul Menzel has posted comments on this change by Jarried Lin. ( https://review.coreboot.org/c/coreboot/+/83925?usp=email )
The change is no longer submittable: All-Comments-Resolved is unsatisfied now.
Change subject: soc/mediatek/mt8196: Enable mmu operation for L2C SRAM and DMA ......................................................................
Patch Set 8:
(3 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/83925/comment/9f2c8cea_7672a539?usp... : PS8, Line 7: mmu MMU
https://review.coreboot.org/c/coreboot/+/83925/comment/f109e360_39c8d5a0?usp... : PS8, Line 17: TEST=build pass Any way to test it on hardware yet?
File src/soc/mediatek/mt8196/l2c_ops.c:
https://review.coreboot.org/c/coreboot/+/83925/comment/9ea1573a_1b9e7fff?usp... : PS8, Line 36: } while (((v >> CLUST_DIS_SHIFT) & CLUST_DIS_VAL) != CLUST_DIS_VAL); Could this be an infinite loop, or is it guaranteed to finish?