[coreboot-gerrit] Change in coreboot[master]: src/soc/intel: Add support for CAR_HAS_SF_MASKS and select for TGL

[Tim Wawrzynczak (Code Review)]

Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48286 )

Change subject: src/soc/intel: Add support for CAR_HAS_SF_MASKS and select for TGL
......................................................................

Patch Set 18:

(1 comment)

https://review.coreboot.org/c/coreboot/+/48286/18/src/soc/intel/common/block... 
File src/soc/intel/common/block/cpu/car/exit_car.S:

https://review.coreboot.org/c/coreboot/+/48286/18/src/soc/intel/common/block... 
PS18, Line 94: 	/* Reset CLOS selector to 0 */
             : 	mov	$IA32_PQR_ASSOC, %ecx
             : 	rdmsr
             : 	and	$~IA32_PQR_ASSOC_MASK, %edx
             : 	wrmsr
Shouldn't we have the same logic here as in NEM programming, e.g.,
```
    mov	$IA32_PQR_ASSOC, %ecx
    rdmsr
#if CONFIG(COS_MAPPED_TO_MSB)
    and	$~IA32_PQR_ASSOC_MASK, %edx
#else
    and	$~IA32_PQR_ASSOC_MASK, %eax
#endif
    wrmsr
```