]> git.proxmox.com Git - mirror_edk2.git/blobdiff - ArmPlatformPkg/Drivers/PL180MciDxe/PL180Mci.c
ArmPlatformPkg/PL180MciDxe: Fixed coding style
[mirror_edk2.git] / ArmPlatformPkg / Drivers / PL180MciDxe / PL180Mci.c
index b79845dccb713bfcd480d96b41d867e32188891f..313f2af9fdc568b083e7346a694370c74ed900a9 100644 (file)
@@ -1,7 +1,7 @@
 /** @file\r
   This file implement the MMC Host Protocol for the ARM PrimeCell PL180.\r
 \r
-  Copyright (c) 2011, ARM Limited. All rights reserved.\r
+  Copyright (c) 2011-2012, ARM Limited. All rights reserved.\r
   \r
   This program and the accompanying materials                          \r
   are licensed and made available under the terms and conditions of the BSD License         \r
@@ -27,12 +27,15 @@ EFI_MMC_HOST_PROTOCOL     *gpMmcHost;
 #define MMCI0_POW2_BLOCKLEN     9\r
 #define MMCI0_TIMEOUT           1000\r
 \r
+#define SYS_MCI_CARDIN          BIT0\r
+#define SYS_MCI_WPROT           BIT1\r
+\r
 BOOLEAN\r
 MciIsPowerOn (\r
   VOID\r
   )\r
 {\r
-  return ((MmioRead32(MCI_POWER_CONTROL_REG) & 0x3) == MCI_POWER_ON);\r
+  return ((MmioRead32 (MCI_POWER_CONTROL_REG) & MCI_POWER_ON) == MCI_POWER_ON);\r
 }\r
 \r
 EFI_STATUS\r
@@ -40,7 +43,7 @@ MciInitialize (
   VOID\r
   )\r
 {\r
-  MCI_TRACE("MciInitialize()");\r
+  MCI_TRACE ("MciInitialize()");\r
   return EFI_SUCCESS;\r
 }\r
 \r
@@ -49,7 +52,7 @@ MciIsCardPresent (
   IN EFI_MMC_HOST_PROTOCOL     *This\r
   )\r
 {\r
-  return (MmioRead32(FixedPcdGet32(PcdPL180SysMciRegAddress)) & 1);\r
+  return (MmioRead32 (FixedPcdGet32 (PcdPL180SysMciRegAddress)) & SYS_MCI_CARDIN);\r
 }\r
 \r
 BOOLEAN\r
@@ -57,7 +60,7 @@ MciIsReadOnly (
   IN EFI_MMC_HOST_PROTOCOL     *This\r
   )\r
 {\r
-  return (MmioRead32(FixedPcdGet32(PcdPL180SysMciRegAddress)) & 2);\r
+  return (MmioRead32 (FixedPcdGet32 (PcdPL180SysMciRegAddress)) & SYS_MCI_WPROT);\r
 }\r
 \r
 #if 0\r
@@ -92,13 +95,13 @@ MciPrepareDataPath (
   )\r
 {\r
   // Set Data Length & Data Timer\r
-  MmioWrite32 (MCI_DATA_TIMER_REG,0xFFFFFFF);\r
-  MmioWrite32 (MCI_DATA_LENGTH_REG,MMCI0_BLOCKLEN);\r
+  MmioWrite32 (MCI_DATA_TIMER_REG, 0xFFFFFFF);\r
+  MmioWrite32 (MCI_DATA_LENGTH_REG, MMCI0_BLOCKLEN);\r
 \r
 #ifndef USE_STREAM\r
-  //Note: we are using a hardcoded BlockLen (=512). If we decide to use a variable size, we could\r
-  // compute the pow2 of BlockLen with the above function GetPow2BlockLen()\r
-  MmioWrite32 (MCI_DATA_CTL_REG, MCI_DATACTL_ENABLE |  MCI_DATACTL_DMA_ENABLE | TransferDirection | (MMCI0_POW2_BLOCKLEN << 4));\r
+  //Note: we are using a hardcoded BlockLen (==512). If we decide to use a variable size, we could\r
+  // compute the pow2 of BlockLen with the above function GetPow2BlockLen ()\r
+  MmioWrite32 (MCI_DATA_CTL_REG, MCI_DATACTL_ENABLE | MCI_DATACTL_DMA_ENABLE | TransferDirection | (MMCI0_POW2_BLOCKLEN << 4));\r
 #else\r
   MmioWrite32 (MCI_DATA_CTL_REG, MCI_DATACTL_ENABLE | MCI_DATACTL_DMA_ENABLE | TransferDirection | MCI_DATACTL_STREAM_TRANS);\r
 #endif\r
@@ -111,21 +114,21 @@ MciSendCommand (
   IN UINT32                     Argument\r
   )\r
 {\r
-  UINT32 Status;\r
-  UINT32 Cmd;\r
-  UINTN  RetVal;\r
-  UINTN  CmdCtrlReg;\r
+  UINT32  Status;\r
+  UINT32  Cmd;\r
+  UINTN   RetVal;\r
+  UINTN   CmdCtrlReg;\r
 \r
   RetVal = EFI_SUCCESS;\r
 \r
   if ((MmcCmd == MMC_CMD17) || (MmcCmd == MMC_CMD11)) {\r
-    MciPrepareDataPath(MCI_DATACTL_CARD_TO_CONT);\r
+    MciPrepareDataPath (MCI_DATACTL_CARD_TO_CONT);\r
   } else if ((MmcCmd == MMC_CMD24) || (MmcCmd == MMC_CMD20)) {\r
-    MciPrepareDataPath(MCI_DATACTL_CONT_TO_CARD);\r
+    MciPrepareDataPath (MCI_DATACTL_CONT_TO_CARD);\r
   }\r
 \r
   // Create Command for PL180\r
-  Cmd = (MMC_GET_INDX(MmcCmd) & INDX_MASK)  | MCI_CPSM_ENABLED;\r
+  Cmd = (MMC_GET_INDX (MmcCmd) & INDX_MASK)  | MCI_CPSM_ENABLE;\r
   if (MmcCmd & MMC_CMD_WAIT_RESPONSE) {\r
     Cmd |= MCI_CPSM_WAIT_RESPONSE;\r
   }\r
@@ -135,29 +138,29 @@ MciSendCommand (
   }\r
 \r
   // Clear Status register static flags\r
-  MmioWrite32(MCI_CLEAR_STATUS_REG,0x7FF);\r
+  MmioWrite32 (MCI_CLEAR_STATUS_REG, MCI_CLR_ALL_STATUS);\r
 \r
-  //Write to command argument register\r
-  MmioWrite32(MCI_ARGUMENT_REG,Argument);\r
+  // Write to command argument register\r
+  MmioWrite32 (MCI_ARGUMENT_REG, Argument);\r
 \r
-  //Write to command register\r
-  MmioWrite32(MCI_COMMAND_REG,Cmd);\r
+  // Write to command register\r
+  MmioWrite32 (MCI_COMMAND_REG, Cmd);\r
 \r
   if (Cmd & MCI_CPSM_WAIT_RESPONSE) {\r
-    Status = MmioRead32(MCI_STATUS_REG);\r
+    Status = MmioRead32 (MCI_STATUS_REG);\r
     while (!(Status & (MCI_STATUS_CMD_RESPEND | MCI_STATUS_CMD_CMDCRCFAIL | MCI_STATUS_CMD_CMDTIMEOUT | MCI_STATUS_CMD_START_BIT_ERROR))) {\r
       Status = MmioRead32(MCI_STATUS_REG);\r
     }\r
 \r
     if ((Status & MCI_STATUS_CMD_START_BIT_ERROR)) {\r
-      DEBUG ((EFI_D_ERROR, "MciSendCommand(CmdIndex:%d) Start bit Error! Response:0x%X Status:0x%x\n",(Cmd & 0x3F),MmioRead32(MCI_RESPONSE0_REG),Status));\r
+      DEBUG ((EFI_D_ERROR, "MciSendCommand(CmdIndex:%d) Start bit Error! Response:0x%X Status:0x%x\n", (Cmd & 0x3F), MmioRead32 (MCI_RESPONSE0_REG), Status));\r
       RetVal = EFI_NO_RESPONSE;\r
       goto Exit;\r
     } else if ((Status & MCI_STATUS_CMD_CMDTIMEOUT)) {\r
-      //DEBUG ((EFI_D_ERROR, "MciSendCommand(CmdIndex:%d) TIMEOUT! Response:0x%X Status:0x%x\n",(Cmd & 0x3F),MmioRead32(MCI_RESPONSE0_REG),Status));\r
+      //DEBUG ((EFI_D_ERROR, "MciSendCommand(CmdIndex:%d) TIMEOUT! Response:0x%X Status:0x%x\n", (Cmd & 0x3F), MmioRead32 (MCI_RESPONSE0_REG), Status));\r
       RetVal = EFI_TIMEOUT;\r
       goto Exit;\r
-    } else if ((!(MmcCmd & MMC_CMD_NO_CRC_RESPONSE)) && (Status & MCI_STATUS_CMD_CMDCRCFAIL)) {\r
+    } else if ((! (MmcCmd & MMC_CMD_NO_CRC_RESPONSE)) && (Status & MCI_STATUS_CMD_CMDCRCFAIL)) {\r
       // The CMD1 and response type R3 do not contain CRC. We should ignore the CRC failed Status.\r
       RetVal = EFI_CRC_ERROR;\r
       goto Exit;\r
@@ -191,10 +194,10 @@ MciSendCommand (
   }\r
 \r
 Exit:\r
-       // Disable Command Path\r
-       CmdCtrlReg = MmioRead32(MCI_COMMAND_REG);\r
-       MmioWrite32(MCI_COMMAND_REG, (CmdCtrlReg & ~MCI_CPSM_ENABLED));\r
-       return RetVal;\r
+  // Disable Command Path\r
+  CmdCtrlReg = MmioRead32 (MCI_COMMAND_REG);\r
+  MmioWrite32 (MCI_COMMAND_REG, (CmdCtrlReg & ~MCI_CPSM_ENABLE));\r
+  return RetVal;\r
 }\r
 \r
 EFI_STATUS\r
@@ -208,16 +211,18 @@ MciReceiveResponse (
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
-  if ((Type == MMC_RESPONSE_TYPE_R1) || (Type == MMC_RESPONSE_TYPE_R1b) ||\r
-      (Type == MMC_RESPONSE_TYPE_R3) || (Type == MMC_RESPONSE_TYPE_R6) ||\r
-      (Type == MMC_RESPONSE_TYPE_R7))\r
+  if (   (Type == MMC_RESPONSE_TYPE_R1)\r
+      || (Type == MMC_RESPONSE_TYPE_R1b)\r
+      || (Type == MMC_RESPONSE_TYPE_R3)\r
+      || (Type == MMC_RESPONSE_TYPE_R6)\r
+      || (Type == MMC_RESPONSE_TYPE_R7))\r
   {\r
-    Buffer[0] = MmioRead32(MCI_RESPONSE3_REG);\r
+    Buffer[0] = MmioRead32 (MCI_RESPONSE3_REG);\r
   } else if (Type == MMC_RESPONSE_TYPE_R2) {\r
-    Buffer[0] = MmioRead32(MCI_RESPONSE0_REG);\r
-    Buffer[1] = MmioRead32(MCI_RESPONSE1_REG);\r
-    Buffer[2] = MmioRead32(MCI_RESPONSE2_REG);\r
-    Buffer[3] = MmioRead32(MCI_RESPONSE3_REG);\r
+    Buffer[0] = MmioRead32 (MCI_RESPONSE0_REG);\r
+    Buffer[1] = MmioRead32 (MCI_RESPONSE1_REG);\r
+    Buffer[2] = MmioRead32 (MCI_RESPONSE2_REG);\r
+    Buffer[3] = MmioRead32 (MCI_RESPONSE3_REG);\r
   }\r
 \r
   return EFI_SUCCESS;\r
@@ -244,7 +249,7 @@ MciReadBlockData (
   Finish = MMCI0_BLOCKLEN / 4;\r
   do {\r
     // Read the Status flags\r
-    Status = MmioRead32(MCI_STATUS_REG);\r
+    Status = MmioRead32 (MCI_STATUS_REG);\r
 \r
     // Do eight reads if possible else a single read\r
     if (Status & MCI_STATUS_CMD_RXFIFOHALFFULL) {\r
@@ -269,16 +274,16 @@ MciReadBlockData (
       Loop++;\r
     } else {\r
       //Check for error conditions and timeouts\r
-      if(Status & MCI_STATUS_CMD_DATATIMEOUT) {\r
-        DEBUG ((EFI_D_ERROR, "MciReadBlockData(): TIMEOUT! Response:0x%X Status:0x%x\n",MmioRead32(MCI_RESPONSE0_REG),Status));\r
+      if (Status & MCI_STATUS_CMD_DATATIMEOUT) {\r
+        DEBUG ((EFI_D_ERROR, "MciReadBlockData(): TIMEOUT! Response:0x%X Status:0x%x\n", MmioRead32 (MCI_RESPONSE0_REG), Status));\r
         RetVal = EFI_TIMEOUT;\r
         break;\r
-      } else if(Status & MCI_STATUS_CMD_DATACRCFAIL) {\r
-        DEBUG ((EFI_D_ERROR, "MciReadBlockData(): CRC Error! Response:0x%X Status:0x%x\n",MmioRead32(MCI_RESPONSE0_REG),Status));\r
+      } else if (Status & MCI_STATUS_CMD_DATACRCFAIL) {\r
+        DEBUG ((EFI_D_ERROR, "MciReadBlockData(): CRC Error! Response:0x%X Status:0x%x\n", MmioRead32 (MCI_RESPONSE0_REG), Status));\r
         RetVal = EFI_CRC_ERROR;\r
         break;\r
-      } else if(Status & MCI_STATUS_CMD_START_BIT_ERROR) {\r
-        DEBUG ((EFI_D_ERROR, "MciReadBlockData(): Start-bit Error! Response:0x%X Status:0x%x\n",MmioRead32(MCI_RESPONSE0_REG),Status));\r
+      } else if (Status & MCI_STATUS_CMD_START_BIT_ERROR) {\r
+        DEBUG ((EFI_D_ERROR, "MciReadBlockData(): Start-bit Error! Response:0x%X Status:0x%x\n", MmioRead32 (MCI_RESPONSE0_REG), Status));\r
         RetVal = EFI_NO_RESPONSE;\r
         break;\r
       }\r
@@ -289,14 +294,14 @@ MciReadBlockData (
     }\r
   } while ((Loop < Finish));\r
 \r
-       //Clear Status flags\r
-       MmioWrite32(MCI_CLEAR_STATUS_REG, 0x7FF);\r
+  // Clear Status flags\r
+  MmioWrite32 (MCI_CLEAR_STATUS_REG, MCI_CLR_ALL_STATUS);\r
 \r
-       //Disable Data path\r
-       DataCtrlReg = MmioRead32(MCI_DATA_CTL_REG);\r
-       MmioWrite32(MCI_DATA_CTL_REG, (DataCtrlReg & 0xFE));\r
+  //Disable Data path\r
+  DataCtrlReg = MmioRead32 (MCI_DATA_CTL_REG);\r
+  MmioWrite32 (MCI_DATA_CTL_REG, (DataCtrlReg & MCI_DATACTL_DISABLE_MASK));\r
 \r
-       return RetVal;\r
+  return RetVal;\r
 }\r
 \r
 EFI_STATUS\r
@@ -322,7 +327,7 @@ MciWriteBlockData (
   Timer  = MMCI0_TIMEOUT * 100;\r
   do {\r
     // Read the Status flags\r
-    Status = MmioRead32(MCI_STATUS_REG);\r
+    Status = MmioRead32 (MCI_STATUS_REG);\r
 \r
     // Do eight writes if possible else a single write\r
     if (Status & MCI_STATUS_CMD_TXFIFOHALFEMPTY) {\r
@@ -346,16 +351,16 @@ MciWriteBlockData (
         MmioWrite32(MCI_FIFO_REG, Buffer[Loop]);\r
         Loop++;\r
     } else {\r
-      //Check for error conditions and timeouts\r
-      if(Status & MCI_STATUS_CMD_DATATIMEOUT) {\r
-        DEBUG ((EFI_D_ERROR, "MciWriteBlockData(): TIMEOUT! Response:0x%X Status:0x%x\n",MmioRead32(MCI_RESPONSE0_REG),Status));\r
+      // Check for error conditions and timeouts\r
+      if (Status & MCI_STATUS_CMD_DATATIMEOUT) {\r
+        DEBUG ((EFI_D_ERROR, "MciWriteBlockData(): TIMEOUT! Response:0x%X Status:0x%x\n", MmioRead32 (MCI_RESPONSE0_REG), Status));\r
         RetVal = EFI_TIMEOUT;\r
         goto Exit;\r
-      } else if(Status & MCI_STATUS_CMD_DATACRCFAIL) {\r
-        DEBUG ((EFI_D_ERROR, "MciWriteBlockData(): CRC Error! Response:0x%X Status:0x%x\n",MmioRead32(MCI_RESPONSE0_REG),Status));\r
+      } else if (Status & MCI_STATUS_CMD_DATACRCFAIL) {\r
+        DEBUG ((EFI_D_ERROR, "MciWriteBlockData(): CRC Error! Response:0x%X Status:0x%x\n", MmioRead32 (MCI_RESPONSE0_REG), Status));\r
         RetVal = EFI_CRC_ERROR;\r
         goto Exit;\r
-      } else if(Status & MCI_STATUS_CMD_TX_UNDERRUN) {\r
+      } else if (Status & MCI_STATUS_CMD_TX_UNDERRUN) {\r
         DEBUG ((EFI_D_ERROR, "MciWriteBlockData(): TX buffer Underrun! Response:0x%X Status:0x%x, Number of bytes written 0x%x\n",MmioRead32(MCI_RESPONSE0_REG),Status, Loop));\r
         RetVal = EFI_BUFFER_TOO_SMALL;\r
         ASSERT(0);\r
@@ -365,37 +370,37 @@ MciWriteBlockData (
   } while (Loop < Finish);\r
 \r
   // Wait for FIFO to drain\r
-  Timer = MMCI0_TIMEOUT * 60;\r
-  Status = MmioRead32(MCI_STATUS_REG);\r
+  Timer  = MMCI0_TIMEOUT * 60;\r
+  Status = MmioRead32 (MCI_STATUS_REG);\r
 #ifndef USE_STREAM\r
   // Single block\r
-  while (((Status & MCI_STATUS_CMD_TXDONE) != MCI_STATUS_CMD_TXDONE) && Timer) {\r
+  while (((Status & MCI_STATUS_TXDONE) != MCI_STATUS_TXDONE) && Timer) {\r
 #else\r
   // Stream\r
   while (((Status & MCI_STATUS_CMD_DATAEND) != MCI_STATUS_CMD_DATAEND) && Timer) {\r
 #endif\r
     NanoSecondDelay(10);\r
-    Status = MmioRead32(MCI_STATUS_REG);\r
+    Status = MmioRead32 (MCI_STATUS_REG);\r
     Timer--;\r
   }\r
 \r
-  if(Timer == 0) {\r
+  if (Timer == 0) {\r
     DEBUG ((EFI_D_ERROR, "MciWriteBlockData(): Data End timeout Number of bytes written 0x%x\n",Loop));\r
-    ASSERT(Timer > 0);\r
+    ASSERT (Timer > 0);\r
     return EFI_TIMEOUT;\r
   }\r
 \r
-  //Clear Status flags\r
-  MmioWrite32(MCI_CLEAR_STATUS_REG, 0x7FF);\r
+  // Clear Status flags\r
+  MmioWrite32 (MCI_CLEAR_STATUS_REG, MCI_CLR_ALL_STATUS);\r
   if (Timer == 0) {\r
     RetVal = EFI_TIMEOUT;\r
   }\r
 \r
 Exit:\r
-       //Disable Data path\r
-       DataCtrlReg = MmioRead32(MCI_DATA_CTL_REG);\r
-       MmioWrite32(MCI_DATA_CTL_REG, (DataCtrlReg & 0xFE));\r
-       return RetVal;\r
+  // Disable Data path\r
+  DataCtrlReg = MmioRead32 (MCI_DATA_CTL_REG);\r
+  MmioWrite32 (MCI_DATA_CTL_REG, (DataCtrlReg & MCI_DATACTL_DISABLE_MASK));\r
+  return RetVal;\r
 }\r
 \r
 EFI_STATUS\r
@@ -406,59 +411,59 @@ MciNotifyState (
 {\r
   UINT32      Data32;\r
 \r
-  switch(State) {\r
+  switch (State) {\r
   case MmcInvalidState:\r
-    ASSERT(0);\r
+    ASSERT (0);\r
     break;\r
   case MmcHwInitializationState:\r
     // If device already turn on then restart it\r
-    Data32 = MmioRead32(MCI_POWER_CONTROL_REG);\r
+    Data32 = MmioRead32 (MCI_POWER_CONTROL_REG);\r
     if ((Data32 & 0x2) == MCI_POWER_UP) {\r
-      MCI_TRACE("MciNotifyState(MmcHwInitializationState): TurnOff MCI");\r
+      MCI_TRACE ("MciNotifyState(MmcHwInitializationState): TurnOff MCI");\r
 \r
       // Turn off\r
-      MmioWrite32(MCI_CLOCK_CONTROL_REG, 0);\r
-      MmioWrite32(MCI_POWER_CONTROL_REG, 0);\r
-      MicroSecondDelay(100);\r
+      MmioWrite32 (MCI_CLOCK_CONTROL_REG, 0);\r
+      MmioWrite32 (MCI_POWER_CONTROL_REG, 0);\r
+      MicroSecondDelay (100);\r
     }\r
 \r
-    MCI_TRACE("MciNotifyState(MmcHwInitializationState): TurnOn MCI");\r
+    MCI_TRACE ("MciNotifyState(MmcHwInitializationState): TurnOn MCI");\r
     // Setup clock\r
     //  - 0x1D = 29 => should be the clock divider to be less than 400kHz at MCLK = 24Mhz\r
-    MmioWrite32(MCI_CLOCK_CONTROL_REG,0x1D | MCI_CLOCK_ENABLE | MCI_CLOCK_POWERSAVE);\r
+    MmioWrite32 (MCI_CLOCK_CONTROL_REG, 0x1D | MCI_CLOCK_ENABLE | MCI_CLOCK_POWERSAVE);\r
     //MmioWrite32(MCI_CLOCK_CONTROL_REG,0x1D | MCI_CLOCK_ENABLE);\r
 \r
     // Set the voltage\r
-    MmioWrite32(MCI_POWER_CONTROL_REG,MCI_POWER_OPENDRAIN | (15<<2));\r
-    MmioWrite32(MCI_POWER_CONTROL_REG,MCI_POWER_ROD | MCI_POWER_OPENDRAIN | (15<<2) | MCI_POWER_UP);\r
-    MicroSecondDelay(10);\r
-    MmioWrite32(MCI_POWER_CONTROL_REG,MCI_POWER_ROD | MCI_POWER_OPENDRAIN | (15<<2) | MCI_POWER_ON);\r
-    MicroSecondDelay(100);\r
+    MmioWrite32 (MCI_POWER_CONTROL_REG, MCI_POWER_OPENDRAIN | (15<<2));\r
+    MmioWrite32 (MCI_POWER_CONTROL_REG, MCI_POWER_ROD | MCI_POWER_OPENDRAIN | (15<<2) | MCI_POWER_UP);\r
+    MicroSecondDelay (10);\r
+    MmioWrite32 (MCI_POWER_CONTROL_REG, MCI_POWER_ROD | MCI_POWER_OPENDRAIN | (15<<2) | MCI_POWER_ON);\r
+    MicroSecondDelay (100);\r
 \r
     // Set Data Length & Data Timer\r
-    MmioWrite32(MCI_DATA_TIMER_REG,0xFFFFF);\r
-    MmioWrite32(MCI_DATA_LENGTH_REG,8);\r
+    MmioWrite32 (MCI_DATA_TIMER_REG, 0xFFFFF);\r
+    MmioWrite32 (MCI_DATA_LENGTH_REG, 8);\r
 \r
-    ASSERT((MmioRead32(MCI_POWER_CONTROL_REG) & 0x3) == MCI_POWER_ON);\r
+    ASSERT ((MmioRead32 (MCI_POWER_CONTROL_REG) & 0x3) == MCI_POWER_ON);\r
     break;\r
   case MmcIdleState:\r
-    MCI_TRACE("MciNotifyState(MmcIdleState)");\r
+    MCI_TRACE ("MciNotifyState(MmcIdleState)");\r
     break;\r
   case MmcReadyState:\r
-    MCI_TRACE("MciNotifyState(MmcReadyState)");\r
+    MCI_TRACE ("MciNotifyState(MmcReadyState)");\r
     break;\r
   case MmcIdentificationState:\r
-    MCI_TRACE("MciNotifyState(MmcIdentificationState)");\r
+    MCI_TRACE ("MciNotifyState (MmcIdentificationState)");\r
     break;\r
   case MmcStandByState:{\r
     volatile UINT32 PwrCtrlReg;\r
-    MCI_TRACE("MciNotifyState(MmcStandByState)");\r
+    MCI_TRACE ("MciNotifyState (MmcStandByState)");\r
 \r
     // Enable MCICMD push-pull drive\r
-    PwrCtrlReg = MmioRead32(MCI_POWER_CONTROL_REG);\r
+    PwrCtrlReg = MmioRead32 (MCI_POWER_CONTROL_REG);\r
     //Disable Open Drain output\r
-    PwrCtrlReg &=~(MCI_POWER_OPENDRAIN);\r
-    MmioWrite32(MCI_POWER_CONTROL_REG,PwrCtrlReg);\r
+    PwrCtrlReg &= ~ (MCI_POWER_OPENDRAIN);\r
+    MmioWrite32 (MCI_POWER_CONTROL_REG, PwrCtrlReg);\r
 \r
     // Set MMCI0 clock to 4MHz (24MHz may be possible with cache enabled)\r
     //\r
@@ -471,22 +476,22 @@ MciNotifyState (
     break;\r
   }\r
   case MmcTransferState:\r
-    //MCI_TRACE("MciNotifyState(MmcTransferState)");\r
+    //MCI_TRACE ("MciNotifyState(MmcTransferState)");\r
     break;\r
   case MmcSendingDataState:\r
-    MCI_TRACE("MciNotifyState(MmcSendingDataState)");\r
+    MCI_TRACE ("MciNotifyState(MmcSendingDataState)");\r
     break;\r
   case MmcReceiveDataState:\r
-    MCI_TRACE("MciNotifyState(MmcReceiveDataState)");\r
+    MCI_TRACE ("MciNotifyState(MmcReceiveDataState)");\r
     break;\r
   case MmcProgrammingState:\r
-    MCI_TRACE("MciNotifyState(MmcProgrammingState)");\r
+    MCI_TRACE ("MciNotifyState(MmcProgrammingState)");\r
     break;\r
   case MmcDisconnectState:\r
-    MCI_TRACE("MciNotifyState(MmcDisconnectState)");\r
+    MCI_TRACE ("MciNotifyState(MmcDisconnectState)");\r
     break;\r
   default:\r
-    ASSERT(0);\r
+    ASSERT (0);\r
   }\r
   return EFI_SUCCESS;\r
 }\r
@@ -501,8 +506,8 @@ MciBuildDevicePath (
 {\r
   EFI_DEVICE_PATH_PROTOCOL    *NewDevicePathNode;\r
 \r
-  NewDevicePathNode = CreateDeviceNode(HARDWARE_DEVICE_PATH,HW_VENDOR_DP,sizeof(VENDOR_DEVICE_PATH));\r
-  CopyGuid(&((VENDOR_DEVICE_PATH*)NewDevicePathNode)->Guid,&mPL180MciDevicePathGuid);\r
+  NewDevicePathNode = CreateDeviceNode (HARDWARE_DEVICE_PATH, HW_VENDOR_DP, sizeof (VENDOR_DEVICE_PATH));\r
+  CopyGuid (& ((VENDOR_DEVICE_PATH*)NewDevicePathNode)->Guid, &mPL180MciDevicePathGuid);\r
 \r
   *DevicePath = NewDevicePathNode;\r
   return EFI_SUCCESS;\r
@@ -527,9 +532,11 @@ PL180MciDxeInitialize (
   )\r
 {\r
   EFI_STATUS    Status;\r
-  EFI_HANDLE    Handle = NULL;\r
+  EFI_HANDLE    Handle;\r
+\r
+  Handle = NULL;\r
 \r
-  MCI_TRACE("PL180MciDxeInitialize()");\r
+  MCI_TRACE ("PL180MciDxeInitialize()");\r
 \r
   //Publish Component Name, BlockIO protocol interfaces\r
   Status = gBS->InstallMultipleProtocolInterfaces (\r