Change in coreboot[master]: drivers/intel/gma/opregion.c: Fix uninitialised variable use

[Arthur Heymans (Code Review)]

Arthur Heymans has submitted this change. ( https://review.coreboot.org/c/coreboot/+/61892 )

Change subject: drivers/intel/gma/opregion.c: Fix uninitialised variable use
......................................................................

drivers/intel/gma/opregion.c: Fix uninitialised variable use

Change-Id: I87cff1e0360e23e37201381ed8a6920ee36b2747
Signed-off-by: Arthur Heymans 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/61892
Reviewed-by: Matt DeVillier 
Tested-by: build bot (Jenkins) 
---
M src/drivers/intel/gma/opregion.c
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Matt DeVillier: Looks good to me, approved

diff --git a/src/drivers/intel/gma/opregion.c b/src/drivers/intel/gma/opregion.c
index ff77f65..dc99fd4 100644
--- a/src/drivers/intel/gma/opregion.c
+++ b/src/drivers/intel/gma/opregion.c
@@ -332,8 +332,8 @@
    	return CB_ERR;
    }

-	if (is_ext_vbt_required(opregion, vbt))
-		opregion_size += vbt->hdr_vbt_size;
+	/* Add the space for the extended VBT header even if it's not used */
+	opregion_size += vbt->hdr_vbt_size;

opregion = cbmem_add(CBMEM_ID_IGD_OPREGION, opregion_size);
    if (!opregion) {