[coreboot-gerrit] [XS] Change in coreboot[main]: Cezanne: Add an option to enable A/B recovery scheme

[Marshall Dawson (Code Review)]

Attention is currently required from: Bao Zheng, Felix Held, Fred Reitberger, Jason Glenesk, Matt DeVillier, Zheng Bao.

Marshall Dawson has posted comments on this change by Bao Zheng. ( https://review.coreboot.org/c/coreboot/+/84195?usp=email )

Change subject: Cezanne: Add an option to enable A/B recovery scheme
......................................................................

Patch Set 2:

(3 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/84195/comment/c1e9027a_ffc5e36d?usp... :
PS1, Line 9: Non A/B recovery
> Done

Wow, thank you for the additional detail.  But please also mention the NDA spec #56995.

Commit Message:

https://review.coreboot.org/c/coreboot/+/84195/comment/f8055c1e_eda98b05?usp... :
PS2, Line 7: Cezanne: Add an option to enable A/B recovery scheme
           : 
One more nit, please begin this with "soc/amd/cezanne".

If needed for shortness, you can simplify the remaining part down to "add option for A/B recovery".

File src/soc/amd/cezanne/Kconfig:

https://review.coreboot.org/c/coreboot/+/84195/comment/c440c6a9_323ad592?usp... :
PS1, Line 415: Recovery A/B
> I dont know much about vboot's role in A/B recovery.

Done