OvmfPkg: QemuVideoDxe: eliminate useless Private->HardwareNeedsStarting
Currently, QemuVideoGraphicsOutputQueryMode() reports EFI_NOT_STARTED when this boolean field is set. However, QemuVideoGraphicsOutputQueryMode() is only available to callers after the GOP interface has been installed. That in turn implies that the following partial call tree has succeeded without errors: QemuVideoControllerDriverStart() QemuVideoGraphicsOutputConstructor() QemuVideoGraphicsOutputSetMode(... 0 ...) HardwareNeedsStarting = FALSE InstallMultipleProtocolInterfaces(... GOP ...) That is, when QemuVideoGraphicsOutputQueryMode() is reached, HardwareNeedsStarting is always FALSE. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Laszlo Ersek <lersek@redhat.com> Reviewed-by: Jordan Justen <jordan.l.justen@intel.com> git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@15286 6f19259b-4bc3-4df7-8a09-765794883524
This commit is contained in:
@@ -114,10 +114,6 @@ Routine Description:
|
||||
|
||||
Private = QEMU_VIDEO_PRIVATE_DATA_FROM_GRAPHICS_OUTPUT_THIS (This);
|
||||
|
||||
if (Private->HardwareNeedsStarting) {
|
||||
return EFI_NOT_STARTED;
|
||||
}
|
||||
|
||||
if (Info == NULL || SizeOfInfo == NULL || ModeNumber >= This->Mode->MaxMode) {
|
||||
return EFI_INVALID_PARAMETER;
|
||||
}
|
||||
@@ -209,8 +205,6 @@ Routine Description:
|
||||
This->Mode->Info
|
||||
);
|
||||
|
||||
Private->HardwareNeedsStarting = FALSE;
|
||||
|
||||
return EFI_SUCCESS;
|
||||
}
|
||||
|
||||
@@ -331,7 +325,6 @@ QemuVideoGraphicsOutputConstructor (
|
||||
}
|
||||
Private->GraphicsOutput.Mode->MaxMode = (UINT32) Private->MaxMode;
|
||||
Private->GraphicsOutput.Mode->Mode = GRAPHICS_OUTPUT_INVALIDE_MODE_NUMBER;
|
||||
Private->HardwareNeedsStarting = TRUE;
|
||||
Private->LineBuffer = NULL;
|
||||
|
||||
//
|
||||
|
@@ -111,7 +111,6 @@ typedef struct {
|
||||
UINTN MaxMode;
|
||||
QEMU_VIDEO_MODE_DATA *ModeData;
|
||||
UINT8 *LineBuffer;
|
||||
BOOLEAN HardwareNeedsStarting;
|
||||
QEMU_VIDEO_VARIANT Variant;
|
||||
} QEMU_VIDEO_PRIVATE_DATA;
|
||||
|
||||
|
Reference in New Issue
Block a user