diff options
| author | Arthur Heymans <arthur@aheymans.xyz> | 2022-02-12 17:29:59 +0100 |
|---|---|---|
| committer | Arthur Heymans <arthur@aheymans.xyz> | 2022-05-11 05:53:55 +0000 |
| commit | c2434f4b87047689eb6bba5476a9a65b03fc7dbe (patch) | |
| tree | bd9e664ce1ff43991ca88eb6daec355bc10eb418 /src | |
| parent | ca9d4feca6b68dce6bf473a672a317cc25dba3fc (diff) | |
drivers/intel/gma/opregion.c: Fix uninitialised variable use
Change-Id: I87cff1e0360e23e37201381ed8a6920ee36b2747
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/61892
Reviewed-by: Matt DeVillier <matt.devillier@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src')
| -rw-r--r-- | src/drivers/intel/gma/opregion.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/drivers/intel/gma/opregion.c b/src/drivers/intel/gma/opregion.c index ff77f653b0..dc99fd49e4 100644 --- a/src/drivers/intel/gma/opregion.c +++ b/src/drivers/intel/gma/opregion.c @@ -332,8 +332,8 @@ enum cb_err intel_gma_init_igd_opregion(void) 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) { |