[coreboot-gerrit] Change in coreboot[master]: soc/intel/common/cache_as_ram.S: Add macro to find a free MTRR

[Angel Pons (Code Review)]

Attention is currently required from: Nico Huber, Arthur Heymans, Patrick Rudolph.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37190 )

Change subject: soc/intel/common/cache_as_ram.S: Add macro to find a free MTRR
......................................................................

Patch Set 9:

(1 comment)

File src/soc/intel/common/block/cpu/car/cache_as_ram.S:

https://review.coreboot.org/c/coreboot/+/37190/comment/312f05cf_7a8af489 
PS4, Line 123: 	jne	1f
> > Or even jz .halt_forever […]

https://stackoverflow.com/questions/14267081/difference-between-je-jne-and-j...

Yes, they're the same machine operation, but have different semantic meaning. One should use them depending on what one is doing. If one is testing for something being equal to zero, `jz` and `jnz` are more appropriate. `je` and `jne` are more appropriate immediately after a `cmp` instruction.