[coreboot-gerrit] Change in coreboot[master]: cpu/x86/16bit/entry16.inc: Stop PBET timer on Boot Guard platforms

[Michał Żygowski (Code Review)]

Michał Żygowski has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43395 )

Change subject: cpu/x86/16bit/entry16.inc: Stop PBET timer on Boot Guard platforms
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/c/coreboot/+/43395/1/src/cpu/x86/16bit/entry16.i... 
File src/cpu/x86/16bit/entry16.inc:

https://review.coreboot.org/c/coreboot/+/43395/1/src/cpu/x86/16bit/entry16.i... 
PS1, Line 129: 	wrmsr
> I wonder if you really need to do this in assembly.

FSP-M is supposed to have an option to stop PBET and the minimum expiration time is 60 seconds according the spec/ On the second hand whet PBET value is set to 0, the timer whouls not run, but still BIOS is responsible to write to this MSR. Due to this confusion I think it is better to stop it here.