Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44673 )
Change subject: cpu/intel/car/non-evict/cache_as_ram.S: Add support for longmode ......................................................................
Patch Set 3: Code-Review+1
(2 comments)
https://review.coreboot.org/c/coreboot/+/44673/3/src/cpu/intel/car/non-evict... File src/cpu/intel/car/non-evict/cache_as_ram.S:
https://review.coreboot.org/c/coreboot/+/44673/3/src/cpu/intel/car/non-evict... PS3, Line 233: * BUG: On some x86 platforms acessing MMX register in long mode
'acessing' may be misspelled - perhaps 'accessing'?
Would be good to fix
https://review.coreboot.org/c/coreboot/+/44673/3/src/cpu/intel/car/non-evict... PS3, Line 236: A single space should be enough