Change in coreboot[master]: soc/amd/psp_verstage: Get vb2_context early

[Kangheui Won (Code Review)]

Attention is currently required from: Jason Glenesk, Marshall Dawson, Rob Barnes, Felix Held.
Kangheui Won has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59017 )

Change subject: soc/amd/psp_verstage: Get vb2_context early
......................................................................

Patch Set 2:

(1 comment)

File src/soc/amd/common/psp_verstage/psp_verstage.c:

https://review.coreboot.org/c/coreboot/+/59017/comment/03c69e38_10d32ff1 
PS1, Line 224:
> Turns out, vbnv_init(ctx->nvdata) fails here because the cmos isn't ready until after verstage_soc_e […]

If map_fch_devices completely fails I don't think there's anything to do with our recovery process, similar to hardware failure cases. If i2c mapping has been failed vboot won't be able to communicate with the TPM.

Since we have very little control over how the PSP works and this is unlikely to fail, I think one option would be to use assert() here rather than vboot recovery.