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.

View Change

To view, visit change 36847. To unsubscribe, or for help writing mail filters, visit settings.