[coreboot-gerrit] Change in coreboot[master]: [WIP] security/vboot: Remove selected_region from struct vboot_workin...

[Joel Kitching (Code Review)]

Joel Kitching has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36847 )

Change subject: [WIP] security/vboot: Remove selected_region from struct vboot_working_data
......................................................................

Patch Set 1:

I think your logic makes sense.  We're basically removing vboot_is_slot_selected() and directly reading recovery_reason instead, since we know that a slot will be selected iff a recovery reason is set.

I agree with Julius that we may want to handle all of the vboot_{set,get}_selected_region and vboot_is_slot_selected refactoring in one single CL.