[coreboot-gerrit] Change in coreboot[master]: soc/intel/{icl.tgl,jsl}: Remove SMRAM register programming

[Subrata Banik (Code Review)]

Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44060 )

Change subject: soc/intel/{icl.tgl,jsl}: Remove SMRAM register programming
......................................................................

Patch Set 1:
> Patch Set 1:
>
> So if I understand correctly, right now SMRAM on these platforms remains unlocked after coreboot loads the payload?

This are compatible SMRAM region locking not the actual DRAM based SMRAM locking stuff. This register controls below 1 MB VGA region (0xA_0000 till 0xB_FFFF)

Here is the register range that this register protects:

C_BASE_SEG: This field indicates the location of SMM space. Only SMM space between A_0000h and
B_FFFFh is supported, so this field is hardwired to 010b