I noticed that the default value used for the driver supported efi version protocol was 0x2000a - UEFI 2.1. The attached patch bumps it to 2.5 and updates the NvmExpress driver to use the Pcd from OptionRomPkg.

--
Bruce
From 67c38a99bd343902bf4e68e429338d7fd75f5f68 Mon Sep 17 00:00:00 2001
From: Bruce Cran <bruce.c...@gmail.com>
Date: Thu, 28 May 2015 19:11:53 -0600
Subject: [PATCH] Bump the default driver supported efi version value to UEFI
 2.5

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Bruce Cran <bruce.c...@gmail.com>
---
 MdeModulePkg/Bus/Pci/NvmExpressDxe/NvmExpress.c      | 2 +-
 MdeModulePkg/Bus/Pci/NvmExpressDxe/NvmExpressDxe.inf | 6 +++++-
 OptionRomPkg/OptionRomPkg.dec                        | 2 +-
 OptionRomPkg/OptionRomPkg.dsc                        | 2 +-
 4 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/MdeModulePkg/Bus/Pci/NvmExpressDxe/NvmExpress.c 
b/MdeModulePkg/Bus/Pci/NvmExpressDxe/NvmExpress.c
index 99f89f1..18d1eb0 100644
--- a/MdeModulePkg/Bus/Pci/NvmExpressDxe/NvmExpress.c
+++ b/MdeModulePkg/Bus/Pci/NvmExpressDxe/NvmExpress.c
@@ -1059,7 +1059,7 @@ NvmExpressDriverEntry (
   // Install EFI Driver Supported EFI Version Protocol required for
   // EFI drivers that are on PCI and other plug in cards.
   //
-  gNvmExpressDriverSupportedEfiVersion.FirmwareVersion = 0x00020028;
+  gNvmExpressDriverSupportedEfiVersion.FirmwareVersion = PcdGet32 
(PcdDriverSupportedEfiVersion);
   Status = gBS->InstallMultipleProtocolInterfaces (
                   &ImageHandle,
                   &gEfiDriverSupportedEfiVersionProtocolGuid,
diff --git a/MdeModulePkg/Bus/Pci/NvmExpressDxe/NvmExpressDxe.inf 
b/MdeModulePkg/Bus/Pci/NvmExpressDxe/NvmExpressDxe.inf
index f73d6af..62a9577 100644
--- a/MdeModulePkg/Bus/Pci/NvmExpressDxe/NvmExpressDxe.inf
+++ b/MdeModulePkg/Bus/Pci/NvmExpressDxe/NvmExpressDxe.inf
@@ -48,6 +48,7 @@
 
 [Packages]
   MdePkg/MdePkg.dec
+  OptionRomPkg/OptionRomPkg.dec
 
 [LibraryClasses]
   BaseMemoryLib
@@ -70,9 +71,12 @@
   gEfiDiskInfoProtocolGuid                    ## BY_START
   gEfiDriverSupportedEfiVersionProtocolGuid   ## PRODUCES
 
+[Pcd]
+  gOptionRomPkgTokenSpaceGuid.PcdDriverSupportedEfiVersion
+
 # [Event]
 # EVENT_TYPE_RELATIVE_TIMER ## SOMETIMES_CONSUMES
 #
 
 [UserExtensions.TianoCore."ExtraFiles"]
-  NvmExpressDxeExtra.uni
\ No newline at end of file
+  NvmExpressDxeExtra.uni
diff --git a/OptionRomPkg/OptionRomPkg.dec b/OptionRomPkg/OptionRomPkg.dec
index ea4d57b..de3af87 100644
--- a/OptionRomPkg/OptionRomPkg.dec
+++ b/OptionRomPkg/OptionRomPkg.dec
@@ -43,5 +43,5 @@
   gOptionRomPkgTokenSpaceGuid.PcdSupportUga|TRUE|BOOLEAN|0x00010005
 
 [PcdsFixedAtBuild, PcdsPatchableInModule]
-  
gOptionRomPkgTokenSpaceGuid.PcdDriverSupportedEfiVersion|0x0002000a|UINT32|0x00010003
+  
gOptionRomPkgTokenSpaceGuid.PcdDriverSupportedEfiVersion|0x00020032|UINT32|0x00010003
 
diff --git a/OptionRomPkg/OptionRomPkg.dsc b/OptionRomPkg/OptionRomPkg.dsc
index 03592f8..860568e 100644
--- a/OptionRomPkg/OptionRomPkg.dsc
+++ b/OptionRomPkg/OptionRomPkg.dsc
@@ -76,7 +76,7 @@
   gEfiMdePkgTokenSpaceGuid.PcdMaximumUnicodeStringLength|0x0
   gEfiMdePkgTokenSpaceGuid.PcdMaximumAsciiStringLength|0x0
   gEfiMdePkgTokenSpaceGuid.PcdMaximumLinkedListLength|0x0
-  gOptionRomPkgTokenSpaceGuid.PcdDriverSupportedEfiVersion|0x0002000a # 
EFI_2_10_SYSTEM_TABLE_REVISION
+  gOptionRomPkgTokenSpaceGuid.PcdDriverSupportedEfiVersion|0x00020032 # 
EFI_2_50_SYSTEM_TABLE_REVISION
 
 
###################################################################################################
 #
-- 
2.4.1

------------------------------------------------------------------------------
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to