Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35993 )
Change subject: cpu/x86: Add a prog_run hook to set up caching of XIP stages ......................................................................
Patch Set 18:
(2 comments)
https://review.coreboot.org/c/coreboot/+/35993/9/src/cpu/x86/mtrr/xip_cache.... File src/cpu/x86/mtrr/xip_cache.c:
https://review.coreboot.org/c/coreboot/+/35993/9/src/cpu/x86/mtrr/xip_cache.... PS9, Line 21: total amount of cache
fwiw, the topology and microarchitecture behavior matters. […]
Done
https://review.coreboot.org/c/coreboot/+/35993/9/src/cpu/x86/mtrr/xip_cache.... PS9, Line 26: uint32_t
why is this fixed width? size_t should be sufficient. […]
Done