[M] Change in coreboot[master]: vboot: Introduce handy vboot reboot functions

[Jakub Czapiga (Code Review)]

Attention is currently required from: Angel Pons, Eric Lai.

Jakub Czapiga has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69208 )

Change subject: vboot: Introduce handy vboot reboot functions
......................................................................

Patch Set 5:

(2 comments)

File src/security/vboot/vboot_logic.c:

https://review.coreboot.org/c/coreboot/+/69208/comment/ad02578e_65c708cb 
PS4, Line 375: rv
> This was 0 before

Done

File src/soc/intel/common/block/cse/cse.c:

https://review.coreboot.org/c/coreboot/+/69208/comment/0ff53fd5_3dae68ad 
PS4, Line 935: vboot_get_context()
> Apparently this can return NULL. It would be best to keep the NULL check.

According to Yu-Ping comment in CB:66909 :
 This is actually impossible to happen by the logic of vboot_get_context .  
 vboot_recovery_mode_enabled also doesn't check the return value.

`vboot_get_context()` should never return incorrect pointer. It should die on assert instead (assuming fatal asserts are enabled)