Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48259 )
Change subject: libpayload: Unified the type of _start, _end in ARM64 mmu.h ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/48259/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/48259/2//COMMIT_MSG@9 PS2, Line 9: The type of _start and _end symbols in depthcharge is "uint8_t". Can we fix depthcharge instead? I don't care about uint8_t vs char, but generally it is better to define these as arrays so that they do the right thing with or without an address-of operator in front of them (coreboot is doing it the same way).