diff options
| author | Paul Kocialkowski <contact@paulk.fr> | 2016-05-14 15:30:52 +0200 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2016-05-16 22:37:50 +0200 |
| commit | 1811768c649ac0d0a8af30570ecaa1b1f3b44c6b (patch) | |
| tree | dec43cc35510c1377be97962dd331d5687c13f07 | |
| parent | abe2de8854fe1d1ed2b836da3069470dd6b04d6c (diff) | |
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
Signed-off-by: Paul Kocialkowski <contact@paulk.fr>
Reviewed-on: https://review.coreboot.org/14817
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Tested-by: build bot (Jenkins)
| -rw-r--r-- | src/vendorcode/google/chromeos/vboot2/vboot_loader.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/src/vendorcode/google/chromeos/vboot2/vboot_loader.c b/src/vendorcode/google/chromeos/vboot2/vboot_loader.c index 590061331d..f6efe0f2c7 100644 --- a/src/vendorcode/google/chromeos/vboot2/vboot_loader.c +++ b/src/vendorcode/google/chromeos/vboot2/vboot_loader.c @@ -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()) { |