Attention is currently required from: Angel Pons.
Nico Huber has posted comments on this change by Nico Huber. ( https://review.coreboot.org/c/coreboot/+/82766?usp=email )
Change subject: cpu/via: Implement cache as RAM ......................................................................
Patch Set 2:
(1 comment)
File src/cpu/via/car/cache_as_ram.S:
https://review.coreboot.org/c/coreboot/+/82766/comment/ddb8b756_12f6b3e8?usp... : PS1, Line 111: /* TODO: Or also enable fixed MTRRs? Bug in the code? */
Does this need to be investigated?
Probably not. Took me a moment to understand that this is a comment to the comment (I just copied most of the code from the past).
Given that it used to work, works, and enabling empty, fixed MTRRs shouldn't change a thing, I'll just drop it.