New patch to review for coreboot: chromeos: vboot_common: Avoid code duplication when grabbing the handoff info

[Paul Kocialkowski]

Paul Kocialkowski ([e-mail]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/11498

-gerrit

commit dac52e6563de6926c32854c9a1de6daea7676849
Author: Paul Kocialkowski 
Date:   Thu Sep 3 11:44:56 2015 +0200

chromeos: vboot_common: Avoid code duplication when grabbing the handoff info

vboot_handoff_flag was duplicating the logic to grab the handoff info, that is
    already made available with vboot_get_handoff_info.
    This uses vboot_get_handoff_info in vboot_handoff_flag instead.

Change-Id: I28f1decce98f988f90c446a3a0dbe7409d714527
    Signed-off-by: Paul Kocialkowski 
---
 src/vendorcode/google/chromeos/vboot_common.c | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/vendorcode/google/chromeos/vboot_common.c b/src/vendorcode/google/chromeos/vboot_common.c
index 1c216d0..74b95cd 100644
--- a/src/vendorcode/google/chromeos/vboot_common.c
+++ b/src/vendorcode/google/chromeos/vboot_common.c
@@ -57,11 +57,13 @@ int vboot_get_handoff_info(void **addr, uint32_t *size)

static int vboot_handoff_flag(uint32_t flag)
 {
-	struct vboot_handoff *vbho;
+	struct vboot_handoff *vbho = NULL;
+	uint32_t size = 0;
+	int ret;

-	vbho = cbmem_find(CBMEM_ID_VBOOT_HANDOFF);
+	ret = vboot_get_handoff_info((void **)&vbho, &size);

-	if (vbho == NULL)
+	if (ret || vbho == NULL)
    	return 0;

return !!(vbho->init_params.out_flags & flag);