Edward O'Callaghan has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41988 )
Change subject: chromeos/cr50_enable_update.c: Modify recovery flow for cr50 ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/c/coreboot/+/41988/1/src/vendorcode/google/chrom... File src/vendorcode/google/chromeos/cr50_enable_update.c:
https://review.coreboot.org/c/coreboot/+/41988/1/src/vendorcode/google/chrom... PS1, Line 76: vboot_recovery_mode_enabled
nit: This check is redundant because the other one already implies this.
Done
https://review.coreboot.org/c/coreboot/+/41988/1/src/vendorcode/google/chrom... PS1, Line 77: (vboot_get_context()->flags & VB2_CONTEXT_FORCE_RECOVERY_MODE))
Can you please add a comment here explaining the reason behind doing this?
Done