Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36126 )
Change subject: Documentation: Add cache as ram documentation ......................................................................
Patch Set 3:
(6 comments)
https://review.coreboot.org/c/coreboot/+/36126/3/Documentation/cpu/x86/car.m... File Documentation/cpu/x86/car.md:
https://review.coreboot.org/c/coreboot/+/36126/3/Documentation/cpu/x86/car.m... PS3, Line 14: using It can be misunderstood. It's not the compiler that only uses cpu registers.
https://review.coreboot.org/c/coreboot/+/36126/3/Documentation/cpu/x86/car.m... PS3, Line 18: X86 This is usually written as x86
https://review.coreboot.org/c/coreboot/+/36126/3/Documentation/cpu/x86/car.m... PS3, Line 22: dram DRAM
https://review.coreboot.org/c/coreboot/+/36126/3/Documentation/cpu/x86/car.m... PS3, Line 34: Cache-as-Ram cache as RAM. Maybe worth explaining that cache is backed by SRAM?
https://review.coreboot.org/c/coreboot/+/36126/3/Documentation/cpu/x86/car.m... PS3, Line 37: is And static heap?
https://review.coreboot.org/c/coreboot/+/36126/3/Documentation/cpu/x86/car.m... PS3, Line 84: romstage Romcc?