Change in coreboot[master]: soc/intel/common: Calculate and configure SF Mask 1

[Subrata Banik (Code Review)]

Attention is currently required from: Furquan Shaikh, Tim Wawrzynczak, Nick Vaccaro, EricR Lai.
Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/51374 )

Change subject: soc/intel/common: Calculate and configure SF Mask 1
......................................................................

Patch Set 18:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/51374/comment/9ed93ba5_6df3c002 
PS18, Line 555: 	cmp	$0x01, %eax /* Skip Step 2 if SFWayCnt = 1 */
              : 	jz	program_sf1
> This should be an effectively impossible situation though, shouldn't it? 1 SF way would effectively  […]

yes, i agree, and i just put this to have a discussion. 😊

Do we agree, we don't need to worry about this case, isn't it ?