ArmPlatformPkg: PL061 - drop pointless initialize function

PL061Initialize() did not actually do anything other than call
PL061Identify() - so cut out the middle man.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
Reviewed-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Reviewed-by: Ryan Harkin <ryan.harkin@linaro.org>

diff --git a/ArmPlatformPkg/Drivers/PL061GpioDxe/PL061Gpio.c b/ArmPlatformPkg/Drivers/PL061GpioDxe/PL061Gpio.c
index c8125fd0e08f261af88e9642f735690bb33ac29d..45897ca753a9052382bf8f9d817caa2c060e2e2c 100644 (file)
--- a/ArmPlatformPkg/Drivers/PL061GpioDxe/PL061Gpio.c
+++ b/ArmPlatformPkg/Drivers/PL061GpioDxe/PL061Gpio.c
@@ -57,31 +57,6 @@ PL061Identify (
   return EFI_SUCCESS;\r
 }\r
 \r
-EFI_STATUS\r
-PL061Initialize (\r
-  VOID\r
-  )\r
-{\r
-  EFI_STATUS  Status;\r
-\r
-  // Check if the PL061 GPIO module exists on board\r
-  Status = PL061Identify();\r
-  if (EFI_ERROR (Status)) {\r
-    Status = EFI_DEVICE_ERROR;\r
-    goto EXIT;\r
-  }\r
-\r
-  // Do other hardware initialisation things here as required\r
-\r
-  // Disable Interrupts\r
-  //if (MmioRead8 (PL061_GPIO_IE_REG) != 0) {\r
-  //   // Ensure interrupts are disabled\r
-  //}\r
-\r
-  EXIT:\r
-  return Status;\r
-}\r
-\r
 /**\r
 \r
 Routine Description:\r
@@ -298,7 +273,7 @@ PL061InstallProtocol (
   //\r
   ASSERT_PROTOCOL_ALREADY_INSTALLED (NULL, &gEmbeddedGpioProtocolGuid);\r
 \r
-  Status = PL061Initialize();\r
+  Status = PL061Identify();\r
   if (EFI_ERROR(Status)) {\r
     return EFI_DEVICE_ERROR;\r
   }\r