OvmfPkg: make PlatformDebugLibIoPort a proper BASE library

Remove Uefi.h, which includes UefiSpec.h, and change the
return value to match the RETURN_STATUS type.

Contributed-under: TianoCore Contribution Agreement 1.1
Cc: Laszlo Ersek <lersek@redhat.com>
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Jordan Justen (Intel address) <jordan.l.justen@intel.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Tested-by: Laszlo Ersek <lersek@redhat.com>
Reviewed-by: Laszlo Ersek <lersek@redhat.com>

diff --git a/OvmfPkg/Library/PlatformDebugLibIoPort/DebugLib.c b/OvmfPkg/Library/PlatformDebugLibIoPort/DebugLib.c
index 5435767c1c69a8c0bef1fdf60500a013980a49b6..74f4d9c2d6314542f501e6a135bbed14577fb099 100644 (file)
--- a/OvmfPkg/Library/PlatformDebugLibIoPort/DebugLib.c
+++ b/OvmfPkg/Library/PlatformDebugLibIoPort/DebugLib.c
@@ -15,7 +15,6 @@
 **/\r
 \r
 #include <Base.h>\r
-#include <Uefi.h>\r
 #include <Library/DebugLib.h>\r
 #include <Library/BaseLib.h>\r
 #include <Library/IoLib.h>\r
@@ -32,7 +31,7 @@
 /**\r
   This constructor function does not have to do anything.\r
 \r
-  @retval EFI_SUCCESS   The constructor always returns RETURN_SUCCESS.\r
+  @retval RETURN_SUCCESS   The constructor always returns RETURN_SUCCESS.\r
 \r
 **/\r
 RETURN_STATUS\r
@@ -41,7 +40,7 @@ PlatformDebugLibIoPortConstructor (
   VOID\r
   )\r
 {\r
-  return EFI_SUCCESS;\r
+  return RETURN_SUCCESS;\r
 }\r
 \r
 /**\r