summaryrefslogtreecommitdiff
path: root/src/vendorcode/google
diff options
context:
space:
mode:
authorPaul Kocialkowski <contact@paulk.fr>2016-05-14 15:30:52 +0200
committerMartin Roth <martinroth@google.com>2016-05-16 22:37:50 +0200
commit1811768c649ac0d0a8af30570ecaa1b1f3b44c6b (patch)
treedec43cc35510c1377be97962dd331d5687c13f07 /src/vendorcode/google
parentabe2de8854fe1d1ed2b836da3069470dd6b04d6c (diff)
downloadcoreboot-1811768c649ac0d0a8af30570ecaa1b1f3b44c6b.tar.xz
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)
Diffstat (limited to 'src/vendorcode/google')
-rw-r--r--src/vendorcode/google/chromeos/vboot2/vboot_loader.c6
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()) {