Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update ovmf + armvirt display configuration for setup #10631

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

kraxel
Copy link
Member

@kraxel kraxel commented Jan 15, 2025

  • OvmfPkg: move display pcds to OvmfDisplayPcds.dsc.inc include file
  • OvmfPkg/OvmfXen.dsc: use OvmfDisplayPcds.dsc.inc
  • OvmfPkg/RiscVVirtQemu.dsc: use OvmfDisplayPcds.dsc.inc
  • ArmVirtPkg/ArmVirtQemu.dsc: use OvmfDisplayPcds.dsc.inc
  • OvmfPkg/QemuVideoDxe: set SetupVideoResolution too
  • OvmfPkg/VirtioGpuDxe: set SetupVideoResolution too
  • OvmfPkg/PlatformDxe: set SetupVideoResolution too
  • OvmfPkg/OvmfDisplayPcds.dsc.inc: set SetupVideoResolution too
  • OvmfPkg/OvmfDisplayPcds.dsc.inc: set SetupConOut too

This patch series makes sure the video PCDs for normal mode and for setup mode are identical. This gets rid of pointless video mode changes when entering/leaving the firmware setup application.

The patch series also moves all the PCDs to an include file to reduce duplication.

kraxel added 10 commits January 15, 2025 13:44
This way we have the display configuration in a single place and
need to change one file only to update all build variants.

Signed-off-by: Gerd Hoffmann <[email protected]>
Set both PcdVideo*Resolution and PcdSetupVideo*Resolution PCDs.

This avoids pointless video mode changes when entering and leaving
the firmware setup application.

Signed-off-by: Gerd Hoffmann <[email protected]>
Set both PcdVideo*Resolution and PcdSetupVideo*Resolution PCDs.

This avoids pointless video mode changes when entering and leaving
the firmware setup application.

Signed-off-by: Gerd Hoffmann <[email protected]>
Set both PcdVideo*Resolution and PcdSetupVideo*Resolution PCDs.

This avoids pointless video mode changes when entering and leaving
the firmware setup application.

Signed-off-by: Gerd Hoffmann <[email protected]>
Set both PcdVideo*Resolution and PcdSetupVideo*Resolution PCDs.

This avoids pointless video mode changes when entering and leaving
the firmware setup application.

Signed-off-by: Gerd Hoffmann <[email protected]>
Setting PcdSetupConOut* to zero turns on autodetection mode
for rows and cols, so the firmware setup application will use
the use complete available screen space.

Signed-off-by: Gerd Hoffmann <[email protected]>
@kraxel kraxel force-pushed the devel/setup-display branch from bda33a2 to 64b3fc7 Compare January 16, 2025 07:58
@kraxel kraxel changed the title ovmf display setup update ovmf + armvirt display configuration for setup Jan 16, 2025
@kraxel kraxel marked this pull request as ready for review January 16, 2025 09:30
@ardbiesheuvel
Copy link
Member

Can you squash the [fixup] please so I can merge this?Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants