From nobody Mon Apr 29 16:54:41 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of groups.io designates 66.175.222.12 as permitted sender) client-ip=66.175.222.12; envelope-from=bounce+27952+40373+1787277+3901457@groups.io; helo=web01.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zoho.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+40373+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=intel.com ARC-Seal: i=1; a=rsa-sha256; t=1557423597; cv=none; d=zoho.com; s=zohoarc; b=QE/lwdFODAq9BXnL+q+yyhKPSIPTKg7UQBkcwc3dIj7xFXrHIjmzV1ZsAsXuyjM4WzBr/snJ4Qf0u+ZMqLGNrZaHgbwD7KDy6LXqCtXfhaqX1FE6d/ahlD3/QxRe5V9mAKvuyE7ji3soYs7blZLJQ6na8ecvzw9EcWoMfk9f3jk= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1557423597; h=Content-Transfer-Encoding:Cc:Date:From:List-Id:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:Sender:Subject:To:ARC-Authentication-Results; bh=pXtrlMBpy60fwAL5qF2Z9qIvBoetWAnNNGX9bFwumcc=; b=XaQEafmXNywTBl/kxQmwCxUfijo0iA5R4cBj9eGe8juLr7T86sxFbB1rQLpJ4gf3B3T64EdWuifrRTe6b0X4J1l3MQv4DFXAt0QSolipB7Ay5nOWcvjjdNmZJVNtl6aTQuWwA6iTX6AIRfhdHWfJGI51Y1p/V7qSLNsG1Y8zNoc= ARC-Authentication-Results: i=1; mx.zoho.com; dkim=pass; spf=pass (zoho.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+40373+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) header.from= Received: from web01.groups.io (web01.groups.io [66.175.222.12]) by mx.zohomail.com with SMTPS id 1557423597573956.8888451225569; Thu, 9 May 2019 10:39:57 -0700 (PDT) Return-Path: X-Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by groups.io with SMTP; Thu, 09 May 2019 10:39:55 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-Received: from fmsmga007.fm.intel.com ([10.253.24.52]) by orsmga102.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 09 May 2019 10:39:50 -0700 X-ExtLoop1: 1 X-Received: from unknown (HELO mdkinney-MOBL2.amr.corp.intel.com) ([10.241.98.74]) by fmsmga007.fm.intel.com with ESMTP; 09 May 2019 10:39:50 -0700 From: "Michael D Kinney" To: devel@edk2.groups.io Cc: Jordan Justen , Laszlo Ersek , Ard Biesheuvel Subject: [edk2-devel] [Patch] OvmfPkg/QemuVideoDxe: Remove dependency on OptionRomPkg Date: Thu, 9 May 2019 10:39:48 -0700 Message-Id: <20190509173948.13160-1-michael.d.kinney@intel.com> MIME-Version: 1.0 Precedence: Bulk List-Unsubscribe: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,michael.d.kinney@intel.com Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1557423596; bh=r71ScL2qGBRrbMiNLbBgSboCc7cKtjt5Q86o5KLACIw=; h=Cc:Date:From:Reply-To:Subject:To; b=Km9Molj6rjClDmgRuO+65Bfwv9R56eEPdTB0oB/4ZTvJtkNZiZkd7AY/7VPSZiYKWGX phvmmD+GPsBx+T5E2SHbcE6Z5ZprNzfS6rgjIB+HPpJksPUFQA5kWrs2PewiGdbPx2B8V gCuQCWBf++gXEp5A1YAmGUkYkpOqaXdUsGM= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Type: text/plain; charset="utf-8" Update the QemuVideoDxe driver to not depend on the OptionRomPkg to support moving OptionRomPkg to the edk2-platforms repository. The only dependency on the OptionRomPkg is the use of PcdDriverSupportedEfiVersion to set the version value in the EFI Driver Supported EFI Version Protocol. This protocol is intended for use in drivers for add-in devices, which does not apply to the QEMU integrated video controller. Since this protocol does not apply to QEMU environment, remove both the PCD and the installation of the EFI Driver Supported EFI Version Protocol. Cc: Jordan Justen Cc: Laszlo Ersek Cc: Ard Biesheuvel Signed-off-by: Michael D Kinney Reviewed-by: Laszlo Ersek --- OvmfPkg/QemuVideoDxe/Driver.c | 15 +-------------- OvmfPkg/QemuVideoDxe/DriverSupportedEfiVersion.c | 15 --------------- OvmfPkg/QemuVideoDxe/Qemu.h | 3 +-- OvmfPkg/QemuVideoDxe/QemuVideoDxe.inf | 6 +----- 4 files changed, 3 insertions(+), 36 deletions(-) delete mode 100644 OvmfPkg/QemuVideoDxe/DriverSupportedEfiVersion.c diff --git a/OvmfPkg/QemuVideoDxe/Driver.c b/OvmfPkg/QemuVideoDxe/Driver.c index 45bcfb8fd1..e8a613ef33 100644 --- a/OvmfPkg/QemuVideoDxe/Driver.c +++ b/OvmfPkg/QemuVideoDxe/Driver.c @@ -2,7 +2,7 @@ This driver is a sample implementation of the Graphics Output Protocol f= or the QEMU (Cirrus Logic 5446) video controller. =20 - Copyright (c) 2006 - 2010, Intel Corporation. All rights reserved.
+ Copyright (c) 2006 - 2019, Intel Corporation. All rights reserved.
=20 SPDX-License-Identifier: BSD-2-Clause-Patent =20 @@ -982,18 +982,5 @@ InitializeQemuVideo ( ); ASSERT_EFI_ERROR (Status); =20 - // - // Install EFI Driver Supported EFI Version Protocol required for - // EFI drivers that are on PCI and other plug in cards. - // - gQemuVideoDriverSupportedEfiVersion.FirmwareVersion =3D PcdGet32 (PcdDri= verSupportedEfiVersion); - Status =3D gBS->InstallMultipleProtocolInterfaces ( - &ImageHandle, - &gEfiDriverSupportedEfiVersionProtocolGuid, - &gQemuVideoDriverSupportedEfiVersion, - NULL - ); - ASSERT_EFI_ERROR (Status); - return Status; } diff --git a/OvmfPkg/QemuVideoDxe/DriverSupportedEfiVersion.c b/OvmfPkg/Qem= uVideoDxe/DriverSupportedEfiVersion.c deleted file mode 100644 index c06f1d73cf..0000000000 --- a/OvmfPkg/QemuVideoDxe/DriverSupportedEfiVersion.c +++ /dev/null @@ -1,15 +0,0 @@ -/** @file - Driver supported version protocol for the QEMU video driver. - - Copyright (c) 2007 - 2010, Intel Corporation. All rights reserved.
- - SPDX-License-Identifier: BSD-2-Clause-Patent - -**/ -#include "Qemu.h" - -EFI_DRIVER_SUPPORTED_EFI_VERSION_PROTOCOL gQemuVideoDriverSupportedEfiVers= ion =3D { - sizeof (EFI_DRIVER_SUPPORTED_EFI_VERSION_PROTOCOL), // Size of Protocol = structure. - 0 // Version number to= be filled at start up. -}; - diff --git a/OvmfPkg/QemuVideoDxe/Qemu.h b/OvmfPkg/QemuVideoDxe/Qemu.h index 2b64f1e2b0..87c933935f 100644 --- a/OvmfPkg/QemuVideoDxe/Qemu.h +++ b/OvmfPkg/QemuVideoDxe/Qemu.h @@ -1,7 +1,7 @@ /** @file QEMU Video Controller Driver =20 - Copyright (c) 2006 - 2016, Intel Corporation. All rights reserved.
+ Copyright (c) 2006 - 2019, Intel Corporation. All rights reserved.
SPDX-License-Identifier: BSD-2-Clause-Patent =20 **/ @@ -156,7 +156,6 @@ extern QEMU_VIDEO_BOCHS_MODES QemuV= ideoBochsModes[]; extern EFI_DRIVER_BINDING_PROTOCOL gQemuVideoDriverBinding; extern EFI_COMPONENT_NAME_PROTOCOL gQemuVideoComponentName; extern EFI_COMPONENT_NAME2_PROTOCOL gQemuVideoComponentName2; -extern EFI_DRIVER_SUPPORTED_EFI_VERSION_PROTOCOL gQemuVideoDriverSupporte= dEfiVersion; =20 // // Io Registers defined by VGA diff --git a/OvmfPkg/QemuVideoDxe/QemuVideoDxe.inf b/OvmfPkg/QemuVideoDxe/Q= emuVideoDxe.inf index 4e29b0c20f..fe8befd51d 100644 --- a/OvmfPkg/QemuVideoDxe/QemuVideoDxe.inf +++ b/OvmfPkg/QemuVideoDxe/QemuVideoDxe.inf @@ -2,7 +2,7 @@ # This driver is a sample implementation of the Graphics Output Protocol = for # the QEMU (Cirrus Logic 5446) video controller. # -# Copyright (c) 2006 - 2018, Intel Corporation. All rights reserved.
+# Copyright (c) 2006 - 2019, Intel Corporation. All rights reserved.
# # SPDX-License-Identifier: BSD-2-Clause-Patent # @@ -29,7 +29,6 @@ [Defines] [Sources.common] ComponentName.c Driver.c - DriverSupportedEfiVersion.c Gop.c Initialize.c Qemu.h @@ -41,7 +40,6 @@ [Sources.Ia32, Sources.X64] [Packages] MdePkg/MdePkg.dec MdeModulePkg/MdeModulePkg.dec - OptionRomPkg/OptionRomPkg.dec OvmfPkg/OvmfPkg.dec =20 [LibraryClasses] @@ -59,12 +57,10 @@ [LibraryClasses] UefiLib =20 [Protocols] - gEfiDriverSupportedEfiVersionProtocolGuid # PROTOCOL ALWAYS_PRODUCED gEfiGraphicsOutputProtocolGuid # PROTOCOL BY_START gEfiDevicePathProtocolGuid # PROTOCOL BY_START gEfiPciIoProtocolGuid # PROTOCOL TO_START =20 [Pcd] - gOptionRomPkgTokenSpaceGuid.PcdDriverSupportedEfiVersion gUefiOvmfPkgTokenSpaceGuid.PcdOvmfHostBridgePciDevId gEfiMdeModulePkgTokenSpaceGuid.PcdNullPointerDetectionPropertyMask --=20 2.21.0.windows.1 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#40373): https://edk2.groups.io/g/devel/message/40373 Mute This Topic: https://groups.io/mt/31566345/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-