]> git.proxmox.com Git - mirror_edk2.git/commit
OvmfPkg: add PcdVideoResolutionSource
authorGerd Hoffmann <kraxel@redhat.com>
Sat, 29 Jan 2022 16:26:14 +0000 (17:26 +0100)
committermergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Sat, 29 Jan 2022 17:04:21 +0000 (17:04 +0000)
commit929804b1721b73452a977f31af2d7de83193301b
tree1a355da75e09af253138b2a3fb1effc1209889f3
parente95b44c90eede07c832cada9ce8e6d4035f5e8b7
OvmfPkg: add PcdVideoResolutionSource

It's a UINT8 (enum) PCD telling where the PcdVideoHorizontalResolution
and PcdVideoVerticalResolution values are coming from.  It can be:

 0 (unset aka default from dsc file), or
 1 (from PlatformConfig), or
 2 (set by Video Driver).

It will be used by video drivers to avoid overriding PlatformConfig
values, or override each others values in case multiple display devices
are present.

The underlying problem this tries to solve is that the GOP protocol has
no way to indicate the preferred video mode.  On physical hardware this
isn't much of a problem because using the highest resolution available
works just fine as that is typically the native display resolution

But in a virtual machine you don't want come up with a huge 4k window by
default just because the virtual vga is able to handle that.  Cutting
down the video mode list isn't a great solution either as that would
also remove the modes from the platform configuration so the user
wouldn't be able to pick a resolution higher than the default any more.

So with patch drivers can use use PcdVideoHorizontalResolution and
PcdVideoVerticalResolution to indicate what the preferred display
resolution is, without overwriting the user preferences from
PlatformConfig if present.

A possible alternative approach would be to extend the GOP protocol, but
I'm not sure this is a good plan given this is mostly a problem for
virtual machines and using PCDs allows to keep this local to OvmfPkg.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Acked-by: Ard Biesheuvel <ardb@kernel.org>
OvmfPkg/AmdSev/AmdSevX64.dsc
OvmfPkg/Microvm/MicrovmX64.dsc
OvmfPkg/OvmfPkg.dec
OvmfPkg/OvmfPkgIa32.dsc
OvmfPkg/OvmfPkgIa32X64.dsc
OvmfPkg/OvmfPkgX64.dsc
OvmfPkg/OvmfXen.dsc
OvmfPkg/PlatformDxe/Platform.c
OvmfPkg/PlatformDxe/Platform.inf