[coreboot-gerrit] Change in coreboot[master]: drivers/intel/fsp2_0/hand_off_block: Remove dead decrement

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/41697 )

Change subject: drivers/intel/fsp2_0/hand_off_block: Remove dead decrement
......................................................................

drivers/intel/fsp2_0/hand_off_block: Remove dead decrement

Value stored to 'size' is never read.
Also drop unused parameter.

Change-Id: If3e96ac90f06966ee408964e0748730bc237ec19
Signed-off-by: Elyes HAOUAS 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/41697
Tested-by: build bot (Jenkins) 
Reviewed-by: Angel Pons 
---
M src/drivers/intel/fsp2_0/hand_off_block.c
1 file changed, 2 insertions(+), 3 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Angel Pons: Looks good to me, approved

diff --git a/src/drivers/intel/fsp2_0/hand_off_block.c b/src/drivers/intel/fsp2_0/hand_off_block.c
index 44c91a2..60ab7cb 100644
--- a/src/drivers/intel/fsp2_0/hand_off_block.c
+++ b/src/drivers/intel/fsp2_0/hand_off_block.c
@@ -207,7 +207,7 @@
    return NULL;
 }

-static void display_fsp_version_info_hob(const void *hob, size_t size)
+static void display_fsp_version_info_hob(const void *hob)
 {
 #if CONFIG(DISPLAY_FSP_VERSION_INFO)
    const FIRMWARE_VERSION_INFO *fvi;
@@ -219,7 +219,6 @@
    fvi = (void *)&fvih[1];
    str_ptr = (char *)((uintptr_t)fvi +
    		 (fvih->Count * sizeof(FIRMWARE_VERSION_INFO)));
-	size -= sizeof(SMBIOS_STRUCTURE);

for (index = 0; index < fvih->Count; index++) {
    	cnt = strlen(str_ptr);
@@ -279,7 +278,7 @@

if (fsp_guid_compare(hob_uuid, uuid_fv_info)) {
    		size = hob->length - (HOB_HEADER_LEN + 16);
-			display_fsp_version_info_hob(hob, size);
+			display_fsp_version_info_hob(hob);
    	}
    }
 }