Attention is currently required from: Furquan Shaikh, Maulik V Vaghela, Tim Wawrzynczak, Patrick Rudolph. Hello build bot (Jenkins), Nico Huber, Furquan Shaikh, Angel Pons, Tim Wawrzynczak, Patrick Rudolph,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/55341
to look at the new patch set (#4).
Change subject: drivers/intel/gma: Move extended VBT just below opregion ......................................................................
drivers/intel/gma: Move extended VBT just below opregion
Currently, we are allocating memory for opregion structure first and then we check if we need support for extended VBT (which is mostly needed with newer platforms). We allocate second chunk of memory for extended VBT and then pass physical address into RVDA.
With this changes, we're first checking if opregion requires extended VBT supports and if required, we allocate entire chunk of memory together. This will ensure that we'll align with opregion spec and extended VBT region will sit just below opregion structure. This will also help in optimizing memory allocation by allocation only single chunk of memory.
This is required for the case of virtualization where guest users don't have access to physical address and thus it needs relative address of VBT compared to absolute address.
BUG=None BRANCH=None TEST=check the address of extended VBT region and address is coming correctly.
Change-Id: Ic0e255df63145409096b0b9312c6c51c05f49931 Signed-off-by: Maulik V Vaghela maulik.v.vaghela@intel.com --- M src/drivers/intel/gma/opregion.c 1 file changed, 29 insertions(+), 15 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/41/55341/4