uefi-ref/edk2
1
0
Fork 0
mirror of https://github.com/tianocore/edk2 synced 2026-08-27 00:23:19 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

OvmfPkg/Gop: clear the screen to black in SetMode()

Browse source 
Today's implementation forgot to clear the screen to black in
SetMode(). It causes SCT SetMode() test fails.

The patch adds the clear screen operation in SetMode() to fix
the SCT failure.

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com>
Cc: Jordan Justen <jordan.l.justen@intel.com>
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
This commit is contained in:
Ruiyu Ni 2018-03-13 18:05:16 +08:00
parent 9384e1c011
commit 81feb6d305
1 changed files with 20 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

24
OvmfPkg/QemuVideoDxe/Gop.c
View file

@ -1,7 +1,7 @@
/** @file
Graphics Output Protocol functions for the QEMU video controller.
Copyright (c) 2007 - 2017, Intel Corporation. All rights reserved.<BR>
Copyright (c) 2007 - 2018, Intel Corporation. All rights reserved.<BR>
This program and the accompanying materials
are licensed and made available under the terms and conditions of the BSD License
@ -196,9 +196,10 @@ Routine Description:
--*/
{
QEMU_VIDEO_PRIVATE_DATA *Private;
QEMU_VIDEO_MODE_DATA *ModeData;
RETURN_STATUS Status;
QEMU_VIDEO_PRIVATE_DATA *Private;
QEMU_VIDEO_MODE_DATA *ModeData;
RETURN_STATUS Status;
EFI_GRAPHICS_OUTPUT_BLT_PIXEL Black;
Private = QEMU_VIDEO_PRIVATE_DATA_FROM_GRAPHICS_OUTPUT_THIS (This);
@ -271,6 +272,21 @@ Routine Description:
}
ASSERT (Status == RETURN_SUCCESS);
//
// Per UEFI Spec, need to clear the visible portions of the output display to black.
//
ZeroMem (&Black, sizeof (Black));
Status = FrameBufferBlt (
Private->FrameBufferBltConfigure,
&Black,
EfiBltVideoFill,
0, 0,
0, 0,
This->Mode->Info->HorizontalResolution, This->Mode->Info->VerticalResolution,
0
);
ASSERT_RETURN_ERROR (Status);
return EFI_SUCCESS;
}