[S] Change in coreboot[master]: soc/amd/common/psp_verstage: Report previous boot status

[Karthik Ramasubramanian (Code Review)]

Attention is currently required from: Jason Glenesk, Matt DeVillier, Julius Werner, Fred Reitberger, Yu-Ping Wu, Felix Held.

Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/70382 )

Change subject: soc/amd/common/psp_verstage: Report previous boot status
......................................................................

Patch Set 8:

(2 comments)

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

https://review.coreboot.org/c/coreboot/+/70382/comment/db5c0f5c_4b17a334 
PS7, Line 143: 	svc_get_prev_boot_status(&boot_status);
> I'm not familiar with SVC commands, but do we need to check the return value here?

I am not expecting the SVC call to fail. Still I handled it as if the previous boot failed. Ignoring the SVC call failure might cause the system to boot loop.

https://review.coreboot.org/c/coreboot/+/70382/comment/4d03dc1d_fde73d4d 
PS7, Line 148: Error in previous boot
> This seems a bit too general. […]

Done