ArmPlatformPkg: PL061 - drop pointless initialize function
authorLeif Lindholm <leif.lindholm@linaro.org>
Wed, 10 Feb 2016 16:38:26 +0000 (16:38 +0000)
committerLeif Lindholm <leif.lindholm@linaro.org>
Fri, 26 Feb 2016 16:50:22 +0000 (16:50 +0000)
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>
ArmPlatformPkg/Drivers/PL061GpioDxe/PL061Gpio.c

index c8125fd..45897ca 100644 (file)
@@ -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