[XS] Change in coreboot[main]: security/vboot: Introduce vbnv_platform_init_cmos()

[Yu-Ping Wu (Code Review)]

Yu-Ping Wu has posted comments on this change by Yu-Ping Wu. ( https://review.coreboot.org/c/coreboot/+/83494?usp=email )

Change subject: security/vboot: Introduce vbnv_platform_init_cmos()
......................................................................

Patch Set 4:

(1 comment)

Patchset:

PS4:
> Is this really a vboot thing? Doesn't that bank selection affect all coreboot use of the CMOS, not j […]

Maybe I didn't explain this clear enough in the commit message.

Yes, the bank selection should be applied prior to any CMOS access on AMD. `__cmos_init()` already selects bank 0. The problem is, that function is not called until ramstage, but CMOS RAM is accessed in verstage. Therefore we need a hook in vboot code.