UPSTREAM: vboot: Call verification_should_run directly in the if statement

Using a dedicated variable is slightly less readable and makes the code
less consistent, given that other test functions are called directly in
the if statements.

Change-Id: If52b2a4268acb1e2187574d15cc73a0c1d5fe9bb
Original-Signed-off-by: Paul Kocialkowski <contact@paulk.fr>
Original-Reviewed-on: https://review.coreboot.org/14817
Original-Reviewed-by: Aaron Durbin <adurbin@chromium.org>
(cherry-picked from commit 1811768c64)
Signed-off-by: Martin Roth <martinroth@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/346519
Reviewed-by: Stefan Reinauer <reinauer@google.com>
Commit-Queue: Stefan Reinauer <reinauer@google.com>
Tested-by: Stefan Reinauer <reinauer@google.com>
This commit is contained in:
Paul Kocialkowski 2016-05-14 15:30:52 +02:00 committed by ChromeOS Commit Bot
parent 1d4da5d984
commit 9305f44240

View file

@ -88,11 +88,7 @@ static int vboot_logic_executed(void)
static void vboot_prepare(void)
{
int run_verification;
run_verification = verification_should_run();
if (run_verification) {
if (verification_should_run()) {
verstage_main();
car_set_var(vboot_executed, 1);
} else if (verstage_should_load()) {