Attention is currently required from: Paul Menzel, Yu-Ping Wu.
Julius Werner has posted comments on this change by Yu-Ping Wu. ( https://review.coreboot.org/c/coreboot/+/83652?usp=email )
Change subject: arch/arm64/armv8/mmu: Improve log format ......................................................................
Patch Set 2:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/83652/comment/07d13698_86b02702?usp... : PS2, Line 9: When using format string with "%p", "(nil)" will be printed for address I'm curious where you see this? coreboot's `vtxprintf()` doesn't do this as far as I can tell.
File src/arch/arm64/armv8/mmu.c:
https://review.coreboot.org/c/coreboot/+/83652/comment/68287cb9_d95e392e?usp... : PS2, Line 60: /* Func : table_level_name : * Desc : Get the descriptions table level name from the given size. : */
Unrelated: Not one of the recommended multi-line comment styles.
The code in this file precedes those style decisions, and until someone has time to update the whole file we should probably keep things consistent inside it.