[coreboot-gerrit] Change in coreboot[master]: soc/amd/picasso: Die if the workbuf is missing two boots in a row

[Martin Roth (Code Review)]

Martin Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45964 )

Change subject: soc/amd/picasso: Die if the workbuf is missing two boots in a row
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/45964/2/src/soc/amd/picasso/bootblo... 
File src/soc/amd/picasso/bootblock/bootblock.c:

https://review.coreboot.org/c/coreboot/+/45964/2/src/soc/amd/picasso/bootblo... 
PS2, Line 142: 		if (cmos_read(RTC_CLK_ALTCENTURY) != 0xff &&
> src/soc/amd/picasso/pmutil.c already has vbnv_cmos_failed() implementation. […]

I agree, but vbnv_cmos_failed is just this one cmos read, which I'm already using in this function, so I just pulled the cmos read in.