]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Bus/Ata/AtaAtapiPassThru/IdeMode.c
MdeModulePkg: Apply uncrustify changes
[mirror_edk2.git] / MdeModulePkg / Bus / Ata / AtaAtapiPassThru / IdeMode.c
index 9a6d0e60e42ca58dd1b4d6737ccab065a80e965d..75403886e44a1df2c778dee3a95f62c3154b9d56 100644 (file)
 UINT8\r
 EFIAPI\r
 IdeReadPortB (\r
-  IN  EFI_PCI_IO_PROTOCOL   *PciIo,\r
-  IN  UINT16                Port\r
+  IN  EFI_PCI_IO_PROTOCOL  *PciIo,\r
+  IN  UINT16               Port\r
   )\r
 {\r
-  UINT8 Data;\r
+  UINT8  Data;\r
 \r
   ASSERT (PciIo != NULL);\r
 \r
@@ -35,7 +35,7 @@ IdeReadPortB (
               PciIo,\r
               EfiPciIoWidthUint8,\r
               EFI_PCI_IO_PASS_THROUGH_BAR,\r
-              (UINT64) Port,\r
+              (UINT64)Port,\r
               1,\r
               &Data\r
               );\r
@@ -52,9 +52,9 @@ IdeReadPortB (
 VOID\r
 EFIAPI\r
 IdeWritePortB (\r
-  IN  EFI_PCI_IO_PROTOCOL   *PciIo,\r
-  IN  UINT16                Port,\r
-  IN  UINT8                 Data\r
+  IN  EFI_PCI_IO_PROTOCOL  *PciIo,\r
+  IN  UINT16               Port,\r
+  IN  UINT8                Data\r
   )\r
 {\r
   ASSERT (PciIo != NULL);\r
@@ -66,7 +66,7 @@ IdeWritePortB (
               PciIo,\r
               EfiPciIoWidthUint8,\r
               EFI_PCI_IO_PASS_THROUGH_BAR,\r
-              (UINT64) Port,\r
+              (UINT64)Port,\r
               1,\r
               &Data\r
               );\r
@@ -82,9 +82,9 @@ IdeWritePortB (
 VOID\r
 EFIAPI\r
 IdeWritePortW (\r
-  IN  EFI_PCI_IO_PROTOCOL   *PciIo,\r
-  IN  UINT16                Port,\r
-  IN  UINT16                Data\r
+  IN  EFI_PCI_IO_PROTOCOL  *PciIo,\r
+  IN  UINT16               Port,\r
+  IN  UINT16               Data\r
   )\r
 {\r
   ASSERT (PciIo != NULL);\r
@@ -96,7 +96,7 @@ IdeWritePortW (
               PciIo,\r
               EfiPciIoWidthUint16,\r
               EFI_PCI_IO_PASS_THROUGH_BAR,\r
-              (UINT64) Port,\r
+              (UINT64)Port,\r
               1,\r
               &Data\r
               );\r
@@ -112,9 +112,9 @@ IdeWritePortW (
 VOID\r
 EFIAPI\r
 IdeWritePortDW (\r
-  IN  EFI_PCI_IO_PROTOCOL   *PciIo,\r
-  IN  UINT16                Port,\r
-  IN  UINT32                Data\r
+  IN  EFI_PCI_IO_PROTOCOL  *PciIo,\r
+  IN  UINT16               Port,\r
+  IN  UINT32               Data\r
   )\r
 {\r
   ASSERT (PciIo != NULL);\r
@@ -126,7 +126,7 @@ IdeWritePortDW (
               PciIo,\r
               EfiPciIoWidthUint32,\r
               EFI_PCI_IO_PASS_THROUGH_BAR,\r
-              (UINT64) Port,\r
+              (UINT64)Port,\r
               1,\r
               &Data\r
               );\r
@@ -146,10 +146,10 @@ IdeWritePortDW (
 VOID\r
 EFIAPI\r
 IdeWritePortWMultiple (\r
-  IN  EFI_PCI_IO_PROTOCOL   *PciIo,\r
-  IN  UINT16                Port,\r
-  IN  UINTN                 Count,\r
-  IN  VOID                  *Buffer\r
+  IN  EFI_PCI_IO_PROTOCOL  *PciIo,\r
+  IN  UINT16               Port,\r
+  IN  UINTN                Count,\r
+  IN  VOID                 *Buffer\r
   )\r
 {\r
   ASSERT (PciIo  != NULL);\r
@@ -162,11 +162,10 @@ IdeWritePortWMultiple (
               PciIo,\r
               EfiPciIoWidthFifoUint16,\r
               EFI_PCI_IO_PASS_THROUGH_BAR,\r
-              (UINT64) Port,\r
+              (UINT64)Port,\r
               Count,\r
-              (UINT16 *) Buffer\r
+              (UINT16 *)Buffer\r
               );\r
-\r
 }\r
 \r
 /**\r
@@ -183,10 +182,10 @@ IdeWritePortWMultiple (
 VOID\r
 EFIAPI\r
 IdeReadPortWMultiple (\r
-  IN  EFI_PCI_IO_PROTOCOL   *PciIo,\r
-  IN  UINT16                Port,\r
-  IN  UINTN                 Count,\r
-  IN  VOID                  *Buffer\r
+  IN  EFI_PCI_IO_PROTOCOL  *PciIo,\r
+  IN  UINT16               Port,\r
+  IN  UINTN                Count,\r
+  IN  VOID                 *Buffer\r
   )\r
 {\r
   ASSERT (PciIo  != NULL);\r
@@ -199,11 +198,10 @@ IdeReadPortWMultiple (
               PciIo,\r
               EfiPciIoWidthFifoUint16,\r
               EFI_PCI_IO_PASS_THROUGH_BAR,\r
-              (UINT64) Port,\r
+              (UINT64)Port,\r
               Count,\r
-              (UINT16 *) Buffer\r
+              (UINT16 *)Buffer\r
               );\r
-\r
 }\r
 \r
 /**\r
@@ -219,12 +217,12 @@ IdeReadPortWMultiple (
 VOID\r
 EFIAPI\r
 DumpAllIdeRegisters (\r
-  IN     EFI_PCI_IO_PROTOCOL      *PciIo,\r
-  IN     EFI_IDE_REGISTERS        *IdeRegisters,\r
-  IN OUT EFI_ATA_STATUS_BLOCK     *AtaStatusBlock\r
+  IN     EFI_PCI_IO_PROTOCOL   *PciIo,\r
+  IN     EFI_IDE_REGISTERS     *IdeRegisters,\r
+  IN OUT EFI_ATA_STATUS_BLOCK  *AtaStatusBlock\r
   )\r
 {\r
-  EFI_ATA_STATUS_BLOCK StatusBlock;\r
+  EFI_ATA_STATUS_BLOCK  StatusBlock;\r
 \r
   ASSERT (PciIo != NULL);\r
   ASSERT (IdeRegisters != NULL);\r
@@ -284,6 +282,7 @@ DumpAllIdeRegisters (
       DEBUG ((DEBUG_ERROR, "CheckRegisterStatus()-- %02x : Error : Address Mark Not Found\n", StatusBlock.AtaError));\r
     }\r
   }\r
+\r
   DEBUG_CODE_END ();\r
 }\r
 \r
@@ -301,11 +300,11 @@ DumpAllIdeRegisters (
 EFI_STATUS\r
 EFIAPI\r
 CheckStatusRegister (\r
-  IN  EFI_PCI_IO_PROTOCOL      *PciIo,\r
-  IN  EFI_IDE_REGISTERS        *IdeRegisters\r
+  IN  EFI_PCI_IO_PROTOCOL  *PciIo,\r
+  IN  EFI_IDE_REGISTERS    *IdeRegisters\r
   )\r
 {\r
-  UINT8           StatusRegister;\r
+  UINT8  StatusRegister;\r
 \r
   ASSERT (PciIo != NULL);\r
   ASSERT (IdeRegisters != NULL);\r
@@ -319,6 +318,7 @@ CheckStatusRegister (
       return EFI_DEVICE_ERROR;\r
     }\r
   }\r
+\r
   return EFI_SUCCESS;\r
 }\r
 \r
@@ -342,14 +342,14 @@ CheckStatusRegister (
 EFI_STATUS\r
 EFIAPI\r
 DRQClear (\r
-  IN  EFI_PCI_IO_PROTOCOL       *PciIo,\r
-  IN  EFI_IDE_REGISTERS         *IdeRegisters,\r
-  IN  UINT64                    Timeout\r
+  IN  EFI_PCI_IO_PROTOCOL  *PciIo,\r
+  IN  EFI_IDE_REGISTERS    *IdeRegisters,\r
+  IN  UINT64               Timeout\r
   )\r
 {\r
-  UINT64  Delay;\r
-  UINT8   StatusRegister;\r
-  BOOLEAN InfiniteWait;\r
+  UINT64   Delay;\r
+  UINT8    StatusRegister;\r
+  BOOLEAN  InfiniteWait;\r
 \r
   ASSERT (PciIo != NULL);\r
   ASSERT (IdeRegisters != NULL);\r
@@ -360,7 +360,7 @@ DRQClear (
     InfiniteWait = FALSE;\r
   }\r
 \r
-  Delay = DivU64x32(Timeout, 1000) + 1;\r
+  Delay = DivU64x32 (Timeout, 1000) + 1;\r
   do {\r
     StatusRegister = IdeReadPortB (PciIo, IdeRegisters->CmdOrStatus);\r
 \r
@@ -381,11 +381,11 @@ DRQClear (
     MicroSecondDelay (100);\r
 \r
     Delay--;\r
-\r
   } while (InfiniteWait || (Delay > 0));\r
 \r
   return EFI_TIMEOUT;\r
 }\r
+\r
 /**\r
   This function is used to poll for the DRQ bit clear in the Alternate\r
   Status Register. DRQ is cleared when the device is finished\r
@@ -410,9 +410,9 @@ DRQClear2 (
   IN  UINT64               Timeout\r
   )\r
 {\r
-  UINT64  Delay;\r
-  UINT8   AltRegister;\r
-  BOOLEAN InfiniteWait;\r
+  UINT64   Delay;\r
+  UINT8    AltRegister;\r
+  BOOLEAN  InfiniteWait;\r
 \r
   ASSERT (PciIo != NULL);\r
   ASSERT (IdeRegisters != NULL);\r
@@ -423,7 +423,7 @@ DRQClear2 (
     InfiniteWait = FALSE;\r
   }\r
 \r
-  Delay = DivU64x32(Timeout, 1000) + 1;\r
+  Delay = DivU64x32 (Timeout, 1000) + 1;\r
   do {\r
     AltRegister = IdeReadPortB (PciIo, IdeRegisters->AltOrDev);\r
 \r
@@ -444,7 +444,6 @@ DRQClear2 (
     MicroSecondDelay (100);\r
 \r
     Delay--;\r
-\r
   } while (InfiniteWait || (Delay > 0));\r
 \r
   return EFI_TIMEOUT;\r
@@ -485,10 +484,10 @@ DRQReady (
   IN  UINT64               Timeout\r
   )\r
 {\r
-  UINT64  Delay;\r
-  UINT8   StatusRegister;\r
-  UINT8   ErrorRegister;\r
-  BOOLEAN InfiniteWait;\r
+  UINT64   Delay;\r
+  UINT8    StatusRegister;\r
+  UINT8    ErrorRegister;\r
+  BOOLEAN  InfiniteWait;\r
 \r
   ASSERT (PciIo != NULL);\r
   ASSERT (IdeRegisters != NULL);\r
@@ -499,7 +498,7 @@ DRQReady (
     InfiniteWait = FALSE;\r
   }\r
 \r
-  Delay = DivU64x32(Timeout, 1000) + 1;\r
+  Delay = DivU64x32 (Timeout, 1000) + 1;\r
   do {\r
     //\r
     // Read Status Register will clear interrupt\r
@@ -516,6 +515,7 @@ DRQReady (
         if ((ErrorRegister & ATA_ERRREG_ABRT) == ATA_ERRREG_ABRT) {\r
           return EFI_ABORTED;\r
         }\r
+\r
         return EFI_DEVICE_ERROR;\r
       }\r
 \r
@@ -536,6 +536,7 @@ DRQReady (
 \r
   return EFI_TIMEOUT;\r
 }\r
+\r
 /**\r
   This function is used to poll for the DRQ bit set in the Alternate Status Register.\r
   DRQ is set when the device is ready to transfer data. So this function is called after\r
@@ -569,10 +570,10 @@ DRQReady2 (
   IN  UINT64               Timeout\r
   )\r
 {\r
-  UINT64  Delay;\r
-  UINT8   AltRegister;\r
-  UINT8   ErrorRegister;\r
-  BOOLEAN InfiniteWait;\r
+  UINT64   Delay;\r
+  UINT8    AltRegister;\r
+  UINT8    ErrorRegister;\r
+  BOOLEAN  InfiniteWait;\r
 \r
   ASSERT (PciIo != NULL);\r
   ASSERT (IdeRegisters != NULL);\r
@@ -583,7 +584,7 @@ DRQReady2 (
     InfiniteWait = FALSE;\r
   }\r
 \r
-  Delay = DivU64x32(Timeout, 1000) + 1;\r
+  Delay = DivU64x32 (Timeout, 1000) + 1;\r
 \r
   do {\r
     //\r
@@ -600,6 +601,7 @@ DRQReady2 (
         if ((ErrorRegister & ATA_ERRREG_ABRT) == ATA_ERRREG_ABRT) {\r
           return EFI_ABORTED;\r
         }\r
+\r
         return EFI_DEVICE_ERROR;\r
       }\r
 \r
@@ -621,9 +623,6 @@ DRQReady2 (
   return EFI_TIMEOUT;\r
 }\r
 \r
-\r
-\r
-\r
 /**\r
   This function is used to poll for the BSY bit clear in the Status Register. BSY\r
   is clear when the device is not busy. Every command must be sent after device is not busy.\r
@@ -645,9 +644,9 @@ WaitForBSYClear (
   IN  UINT64               Timeout\r
   )\r
 {\r
-  UINT64  Delay;\r
-  UINT8   StatusRegister;\r
-  BOOLEAN InfiniteWait;\r
+  UINT64   Delay;\r
+  UINT8    StatusRegister;\r
+  BOOLEAN  InfiniteWait;\r
 \r
   ASSERT (PciIo != NULL);\r
   ASSERT (IdeRegisters != NULL);\r
@@ -658,7 +657,7 @@ WaitForBSYClear (
     InfiniteWait = FALSE;\r
   }\r
 \r
-  Delay = DivU64x32(Timeout, 1000) + 1;\r
+  Delay = DivU64x32 (Timeout, 1000) + 1;\r
   do {\r
     StatusRegister = IdeReadPortB (PciIo, IdeRegisters->CmdOrStatus);\r
 \r
@@ -672,13 +671,11 @@ WaitForBSYClear (
     MicroSecondDelay (100);\r
 \r
     Delay--;\r
-\r
   } while (InfiniteWait || (Delay > 0));\r
 \r
   return EFI_TIMEOUT;\r
 }\r
 \r
-\r
 /**\r
   Get IDE i/o port registers' base addresses by mode.\r
 \r
@@ -729,15 +726,15 @@ WaitForBSYClear (
 EFI_STATUS\r
 EFIAPI\r
 GetIdeRegisterIoAddr (\r
-  IN     EFI_PCI_IO_PROTOCOL         *PciIo,\r
-  IN OUT EFI_IDE_REGISTERS           *IdeRegisters\r
+  IN     EFI_PCI_IO_PROTOCOL  *PciIo,\r
+  IN OUT EFI_IDE_REGISTERS    *IdeRegisters\r
   )\r
 {\r
-  EFI_STATUS        Status;\r
-  PCI_TYPE00        PciData;\r
-  UINT16            CommandBlockBaseAddr;\r
-  UINT16            ControlBlockBaseAddr;\r
-  UINT16            BusMasterBaseAddr;\r
+  EFI_STATUS  Status;\r
+  PCI_TYPE00  PciData;\r
+  UINT16      CommandBlockBaseAddr;\r
+  UINT16      ControlBlockBaseAddr;\r
+  UINT16      BusMasterBaseAddr;\r
 \r
   if ((PciIo == NULL) || (IdeRegisters == NULL)) {\r
     return EFI_INVALID_PARAMETER;\r
@@ -755,7 +752,7 @@ GetIdeRegisterIoAddr (
     return Status;\r
   }\r
 \r
-  BusMasterBaseAddr    = (UINT16) ((PciData.Device.Bar[4] & 0x0000fff0));\r
+  BusMasterBaseAddr = (UINT16)((PciData.Device.Bar[4] & 0x0000fff0));\r
 \r
   if ((PciData.Hdr.ClassCode[0] & IDE_PRIMARY_OPERATING_MODE) == 0) {\r
     CommandBlockBaseAddr = 0x1f0;\r
@@ -764,26 +761,27 @@ GetIdeRegisterIoAddr (
     //\r
     // The BARs should be of IO type\r
     //\r
-    if ((PciData.Device.Bar[0] & BIT0) == 0 ||\r
-        (PciData.Device.Bar[1] & BIT0) == 0) {\r
+    if (((PciData.Device.Bar[0] & BIT0) == 0) ||\r
+        ((PciData.Device.Bar[1] & BIT0) == 0))\r
+    {\r
       return EFI_UNSUPPORTED;\r
     }\r
 \r
-    CommandBlockBaseAddr = (UINT16) (PciData.Device.Bar[0] & 0x0000fff8);\r
-    ControlBlockBaseAddr = (UINT16) ((PciData.Device.Bar[1] & 0x0000fffc) + 2);\r
+    CommandBlockBaseAddr = (UINT16)(PciData.Device.Bar[0] & 0x0000fff8);\r
+    ControlBlockBaseAddr = (UINT16)((PciData.Device.Bar[1] & 0x0000fffc) + 2);\r
   }\r
 \r
   //\r
   // Calculate IDE primary channel I/O register base address.\r
   //\r
   IdeRegisters[EfiIdePrimary].Data              = CommandBlockBaseAddr;\r
-  IdeRegisters[EfiIdePrimary].ErrOrFeature      = (UINT16) (CommandBlockBaseAddr + 0x01);\r
-  IdeRegisters[EfiIdePrimary].SectorCount       = (UINT16) (CommandBlockBaseAddr + 0x02);\r
-  IdeRegisters[EfiIdePrimary].SectorNumber      = (UINT16) (CommandBlockBaseAddr + 0x03);\r
-  IdeRegisters[EfiIdePrimary].CylinderLsb       = (UINT16) (CommandBlockBaseAddr + 0x04);\r
-  IdeRegisters[EfiIdePrimary].CylinderMsb       = (UINT16) (CommandBlockBaseAddr + 0x05);\r
-  IdeRegisters[EfiIdePrimary].Head              = (UINT16) (CommandBlockBaseAddr + 0x06);\r
-  IdeRegisters[EfiIdePrimary].CmdOrStatus       = (UINT16) (CommandBlockBaseAddr + 0x07);\r
+  IdeRegisters[EfiIdePrimary].ErrOrFeature      = (UINT16)(CommandBlockBaseAddr + 0x01);\r
+  IdeRegisters[EfiIdePrimary].SectorCount       = (UINT16)(CommandBlockBaseAddr + 0x02);\r
+  IdeRegisters[EfiIdePrimary].SectorNumber      = (UINT16)(CommandBlockBaseAddr + 0x03);\r
+  IdeRegisters[EfiIdePrimary].CylinderLsb       = (UINT16)(CommandBlockBaseAddr + 0x04);\r
+  IdeRegisters[EfiIdePrimary].CylinderMsb       = (UINT16)(CommandBlockBaseAddr + 0x05);\r
+  IdeRegisters[EfiIdePrimary].Head              = (UINT16)(CommandBlockBaseAddr + 0x06);\r
+  IdeRegisters[EfiIdePrimary].CmdOrStatus       = (UINT16)(CommandBlockBaseAddr + 0x07);\r
   IdeRegisters[EfiIdePrimary].AltOrDev          = ControlBlockBaseAddr;\r
   IdeRegisters[EfiIdePrimary].BusMasterBaseAddr = BusMasterBaseAddr;\r
 \r
@@ -794,33 +792,33 @@ GetIdeRegisterIoAddr (
     //\r
     // The BARs should be of IO type\r
     //\r
-    if ((PciData.Device.Bar[2] & BIT0) == 0 ||\r
-        (PciData.Device.Bar[3] & BIT0) == 0) {\r
+    if (((PciData.Device.Bar[2] & BIT0) == 0) ||\r
+        ((PciData.Device.Bar[3] & BIT0) == 0))\r
+    {\r
       return EFI_UNSUPPORTED;\r
     }\r
 \r
-    CommandBlockBaseAddr = (UINT16) (PciData.Device.Bar[2] & 0x0000fff8);\r
-    ControlBlockBaseAddr = (UINT16) ((PciData.Device.Bar[3] & 0x0000fffc) + 2);\r
+    CommandBlockBaseAddr = (UINT16)(PciData.Device.Bar[2] & 0x0000fff8);\r
+    ControlBlockBaseAddr = (UINT16)((PciData.Device.Bar[3] & 0x0000fffc) + 2);\r
   }\r
 \r
   //\r
   // Calculate IDE secondary channel I/O register base address.\r
   //\r
   IdeRegisters[EfiIdeSecondary].Data              = CommandBlockBaseAddr;\r
-  IdeRegisters[EfiIdeSecondary].ErrOrFeature      = (UINT16) (CommandBlockBaseAddr + 0x01);\r
-  IdeRegisters[EfiIdeSecondary].SectorCount       = (UINT16) (CommandBlockBaseAddr + 0x02);\r
-  IdeRegisters[EfiIdeSecondary].SectorNumber      = (UINT16) (CommandBlockBaseAddr + 0x03);\r
-  IdeRegisters[EfiIdeSecondary].CylinderLsb       = (UINT16) (CommandBlockBaseAddr + 0x04);\r
-  IdeRegisters[EfiIdeSecondary].CylinderMsb       = (UINT16) (CommandBlockBaseAddr + 0x05);\r
-  IdeRegisters[EfiIdeSecondary].Head              = (UINT16) (CommandBlockBaseAddr + 0x06);\r
-  IdeRegisters[EfiIdeSecondary].CmdOrStatus       = (UINT16) (CommandBlockBaseAddr + 0x07);\r
+  IdeRegisters[EfiIdeSecondary].ErrOrFeature      = (UINT16)(CommandBlockBaseAddr + 0x01);\r
+  IdeRegisters[EfiIdeSecondary].SectorCount       = (UINT16)(CommandBlockBaseAddr + 0x02);\r
+  IdeRegisters[EfiIdeSecondary].SectorNumber      = (UINT16)(CommandBlockBaseAddr + 0x03);\r
+  IdeRegisters[EfiIdeSecondary].CylinderLsb       = (UINT16)(CommandBlockBaseAddr + 0x04);\r
+  IdeRegisters[EfiIdeSecondary].CylinderMsb       = (UINT16)(CommandBlockBaseAddr + 0x05);\r
+  IdeRegisters[EfiIdeSecondary].Head              = (UINT16)(CommandBlockBaseAddr + 0x06);\r
+  IdeRegisters[EfiIdeSecondary].CmdOrStatus       = (UINT16)(CommandBlockBaseAddr + 0x07);\r
   IdeRegisters[EfiIdeSecondary].AltOrDev          = ControlBlockBaseAddr;\r
-  IdeRegisters[EfiIdeSecondary].BusMasterBaseAddr = (UINT16) (BusMasterBaseAddr + 0x8);\r
+  IdeRegisters[EfiIdeSecondary].BusMasterBaseAddr = (UINT16)(BusMasterBaseAddr + 0x8);\r
 \r
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
 /**\r
   Send ATA Ext command into device with NON_DATA protocol.\r
 \r
@@ -836,10 +834,10 @@ GetIdeRegisterIoAddr (
 EFI_STATUS\r
 EFIAPI\r
 AtaIssueCommand (\r
-  IN  EFI_PCI_IO_PROTOCOL       *PciIo,\r
-  IN  EFI_IDE_REGISTERS         *IdeRegisters,\r
-  IN  EFI_ATA_COMMAND_BLOCK     *AtaCommandBlock,\r
-  IN  UINT64                    Timeout\r
+  IN  EFI_PCI_IO_PROTOCOL    *PciIo,\r
+  IN  EFI_IDE_REGISTERS      *IdeRegisters,\r
+  IN  EFI_ATA_COMMAND_BLOCK  *AtaCommandBlock,\r
+  IN  UINT64                 Timeout\r
   )\r
 {\r
   EFI_STATUS  Status;\r
@@ -861,7 +859,7 @@ AtaIssueCommand (
   //\r
   // Select device (bit4), set LBA mode(bit6) (use 0xe0 for compatibility)\r
   //\r
-  IdeWritePortB (PciIo, IdeRegisters->Head, (UINT8) (0xe0 | DeviceHead));\r
+  IdeWritePortB (PciIo, IdeRegisters->Head, (UINT8)(0xe0 | DeviceHead));\r
 \r
   //\r
   // set all the command parameters\r
@@ -934,15 +932,15 @@ AtaIssueCommand (
 EFI_STATUS\r
 EFIAPI\r
 AtaPioDataInOut (\r
-  IN     EFI_PCI_IO_PROTOCOL       *PciIo,\r
-  IN     EFI_IDE_REGISTERS         *IdeRegisters,\r
-  IN OUT VOID                      *Buffer,\r
-  IN     UINT64                    ByteCount,\r
-  IN     BOOLEAN                   Read,\r
-  IN     EFI_ATA_COMMAND_BLOCK     *AtaCommandBlock,\r
-  IN OUT EFI_ATA_STATUS_BLOCK      *AtaStatusBlock,\r
-  IN     UINT64                    Timeout,\r
-  IN     ATA_NONBLOCK_TASK         *Task\r
+  IN     EFI_PCI_IO_PROTOCOL    *PciIo,\r
+  IN     EFI_IDE_REGISTERS      *IdeRegisters,\r
+  IN OUT VOID                   *Buffer,\r
+  IN     UINT64                 ByteCount,\r
+  IN     BOOLEAN                Read,\r
+  IN     EFI_ATA_COMMAND_BLOCK  *AtaCommandBlock,\r
+  IN OUT EFI_ATA_STATUS_BLOCK   *AtaStatusBlock,\r
+  IN     UINT64                 Timeout,\r
+  IN     ATA_NONBLOCK_TASK      *Task\r
   )\r
 {\r
   UINTN       WordCount;\r
@@ -963,7 +961,7 @@ AtaPioDataInOut (
     goto Exit;\r
   }\r
 \r
-  Buffer16 = (UINT16 *) Buffer;\r
+  Buffer16 = (UINT16 *)Buffer;\r
 \r
   //\r
   // According to PIO data in protocol, host can perform a series of reads to\r
@@ -987,7 +985,7 @@ AtaPioDataInOut (
   //\r
   WordCount = 0;\r
 \r
-  while (WordCount < RShiftU64(ByteCount, 1)) {\r
+  while (WordCount < RShiftU64 (ByteCount, 1)) {\r
     //\r
     // Poll DRQ bit set, data transfer can be performed only when DRQ is ready\r
     //\r
@@ -1000,8 +998,8 @@ AtaPioDataInOut (
     //\r
     // Get the byte count for one series of read\r
     //\r
-    if ((WordCount + Increment) > RShiftU64(ByteCount, 1)) {\r
-      Increment = (UINTN)(RShiftU64(ByteCount, 1) - WordCount);\r
+    if ((WordCount + Increment) > RShiftU64 (ByteCount, 1)) {\r
+      Increment = (UINTN)(RShiftU64 (ByteCount, 1) - WordCount);\r
     }\r
 \r
     if (Read) {\r
@@ -1069,12 +1067,12 @@ Exit:
 EFI_STATUS\r
 EFIAPI\r
 AtaNonDataCommandIn (\r
-  IN     EFI_PCI_IO_PROTOCOL       *PciIo,\r
-  IN     EFI_IDE_REGISTERS         *IdeRegisters,\r
-  IN     EFI_ATA_COMMAND_BLOCK     *AtaCommandBlock,\r
-  IN OUT EFI_ATA_STATUS_BLOCK      *AtaStatusBlock,\r
-  IN     UINT64                    Timeout,\r
-  IN     ATA_NONBLOCK_TASK         *Task\r
+  IN     EFI_PCI_IO_PROTOCOL    *PciIo,\r
+  IN     EFI_IDE_REGISTERS      *IdeRegisters,\r
+  IN     EFI_ATA_COMMAND_BLOCK  *AtaCommandBlock,\r
+  IN OUT EFI_ATA_STATUS_BLOCK   *AtaStatusBlock,\r
+  IN     UINT64                 Timeout,\r
+  IN     ATA_NONBLOCK_TASK      *Task\r
   )\r
 {\r
   EFI_STATUS  Status;\r
@@ -1133,16 +1131,16 @@ Exit:
 **/\r
 EFI_STATUS\r
 AtaUdmStatusWait (\r
-  IN  EFI_PCI_IO_PROTOCOL       *PciIo,\r
-  IN  EFI_IDE_REGISTERS         *IdeRegisters,\r
-  IN  UINT64                    Timeout\r
- )\r
+  IN  EFI_PCI_IO_PROTOCOL  *PciIo,\r
+  IN  EFI_IDE_REGISTERS    *IdeRegisters,\r
+  IN  UINT64               Timeout\r
 )\r
 {\r
-  UINT8                         RegisterValue;\r
-  EFI_STATUS                    Status;\r
-  UINT16                        IoPortForBmis;\r
-  UINT64                        Delay;\r
-  BOOLEAN                       InfiniteWait;\r
+  UINT8       RegisterValue;\r
+  EFI_STATUS  Status;\r
+  UINT16      IoPortForBmis;\r
+  UINT64      Delay;\r
+  BOOLEAN     InfiniteWait;\r
 \r
   if (Timeout == 0) {\r
     InfiniteWait = TRUE;\r
@@ -1159,7 +1157,7 @@ AtaUdmStatusWait (
       break;\r
     }\r
 \r
-    IoPortForBmis = (UINT16) (IdeRegisters->BusMasterBaseAddr + BMIS_OFFSET);\r
+    IoPortForBmis = (UINT16)(IdeRegisters->BusMasterBaseAddr + BMIS_OFFSET);\r
     RegisterValue = IdeReadPortB (PciIo, IoPortForBmis);\r
     if (((RegisterValue & BMIS_ERROR) != 0) || (Timeout == 0)) {\r
       DEBUG ((DEBUG_ERROR, "ATA UDMA operation fails\n"));\r
@@ -1171,6 +1169,7 @@ AtaUdmStatusWait (
       Status = EFI_SUCCESS;\r
       break;\r
     }\r
+\r
     //\r
     // Stall for 100 microseconds.\r
     //\r
@@ -1197,14 +1196,14 @@ AtaUdmStatusWait (
 **/\r
 EFI_STATUS\r
 AtaUdmStatusCheck (\r
-  IN     EFI_PCI_IO_PROTOCOL        *PciIo,\r
-  IN     ATA_NONBLOCK_TASK          *Task,\r
-  IN     EFI_IDE_REGISTERS          *IdeRegisters\r
- )\r
+  IN     EFI_PCI_IO_PROTOCOL  *PciIo,\r
+  IN     ATA_NONBLOCK_TASK    *Task,\r
+  IN     EFI_IDE_REGISTERS    *IdeRegisters\r
 )\r
 {\r
-  UINT8          RegisterValue;\r
-  UINT16         IoPortForBmis;\r
-  EFI_STATUS     Status;\r
+  UINT8       RegisterValue;\r
+  UINT16      IoPortForBmis;\r
+  EFI_STATUS  Status;\r
 \r
   Task->RetryTimes--;\r
 \r
@@ -1213,7 +1212,7 @@ AtaUdmStatusCheck (
     return EFI_DEVICE_ERROR;\r
   }\r
 \r
-  IoPortForBmis = (UINT16) (IdeRegisters->BusMasterBaseAddr + BMIS_OFFSET);\r
+  IoPortForBmis = (UINT16)(IdeRegisters->BusMasterBaseAddr + BMIS_OFFSET);\r
   RegisterValue = IdeReadPortB (PciIo, IoPortForBmis);\r
 \r
   if ((RegisterValue & BMIS_ERROR) != 0) {\r
@@ -1273,36 +1272,36 @@ AtaUdmaInOut (
   IN     ATA_NONBLOCK_TASK             *Task\r
   )\r
 {\r
-  EFI_STATUS                    Status;\r
-  UINT16                        IoPortForBmic;\r
-  UINT16                        IoPortForBmis;\r
-  UINT16                        IoPortForBmid;\r
-\r
-  UINTN                         PrdTableSize;\r
-  EFI_PHYSICAL_ADDRESS          PrdTableMapAddr;\r
-  VOID                          *PrdTableMap;\r
-  EFI_PHYSICAL_ADDRESS          PrdTableBaseAddr;\r
-  EFI_ATA_DMA_PRD               *TempPrdBaseAddr;\r
-  UINTN                         PrdTableNum;\r
-\r
-  UINT8                         RegisterValue;\r
-  UINTN                         PageCount;\r
-  UINTN                         ByteCount;\r
-  UINTN                         ByteRemaining;\r
-  UINT8                         DeviceControl;\r
-\r
-  VOID                          *BufferMap;\r
-  EFI_PHYSICAL_ADDRESS          BufferMapAddress;\r
-  EFI_PCI_IO_PROTOCOL_OPERATION PciIoOperation;\r
-\r
-  UINT8                         DeviceHead;\r
-  EFI_PCI_IO_PROTOCOL           *PciIo;\r
-  EFI_TPL                       OldTpl;\r
-\r
-  UINTN                         AlignmentMask;\r
-  UINTN                         RealPageCount;\r
-  EFI_PHYSICAL_ADDRESS          BaseAddr;\r
-  EFI_PHYSICAL_ADDRESS          BaseMapAddr;\r
+  EFI_STATUS  Status;\r
+  UINT16      IoPortForBmic;\r
+  UINT16      IoPortForBmis;\r
+  UINT16      IoPortForBmid;\r
+\r
+  UINTN                 PrdTableSize;\r
+  EFI_PHYSICAL_ADDRESS  PrdTableMapAddr;\r
+  VOID                  *PrdTableMap;\r
+  EFI_PHYSICAL_ADDRESS  PrdTableBaseAddr;\r
+  EFI_ATA_DMA_PRD       *TempPrdBaseAddr;\r
+  UINTN                 PrdTableNum;\r
+\r
+  UINT8  RegisterValue;\r
+  UINTN  PageCount;\r
+  UINTN  ByteCount;\r
+  UINTN  ByteRemaining;\r
+  UINT8  DeviceControl;\r
+\r
+  VOID                           *BufferMap;\r
+  EFI_PHYSICAL_ADDRESS           BufferMapAddress;\r
+  EFI_PCI_IO_PROTOCOL_OPERATION  PciIoOperation;\r
+\r
+  UINT8                DeviceHead;\r
+  EFI_PCI_IO_PROTOCOL  *PciIo;\r
+  EFI_TPL              OldTpl;\r
+\r
+  UINTN                 AlignmentMask;\r
+  UINTN                 RealPageCount;\r
+  EFI_PHYSICAL_ADDRESS  BaseAddr;\r
+  EFI_PHYSICAL_ADDRESS  BaseMapAddr;\r
 \r
   Status        = EFI_SUCCESS;\r
   PrdTableMap   = NULL;\r
@@ -1329,6 +1328,7 @@ AtaUdmaInOut (
     //\r
     MicroSecondDelay (1000);\r
   }\r
+\r
   gBS->RestoreTPL (OldTpl);\r
 \r
   //\r
@@ -1341,9 +1341,9 @@ AtaUdmaInOut (
   //\r
   // Set relevant IO Port address.\r
   //\r
-  IoPortForBmic = (UINT16) (IdeRegisters->BusMasterBaseAddr + BMIC_OFFSET);\r
-  IoPortForBmis = (UINT16) (IdeRegisters->BusMasterBaseAddr + BMIS_OFFSET);\r
-  IoPortForBmid = (UINT16) (IdeRegisters->BusMasterBaseAddr + BMID_OFFSET);\r
+  IoPortForBmic = (UINT16)(IdeRegisters->BusMasterBaseAddr + BMIC_OFFSET);\r
+  IoPortForBmis = (UINT16)(IdeRegisters->BusMasterBaseAddr + BMIS_OFFSET);\r
+  IoPortForBmid = (UINT16)(IdeRegisters->BusMasterBaseAddr + BMID_OFFSET);\r
 \r
   //\r
   // For Blocking mode, start the command.\r
@@ -1355,7 +1355,7 @@ AtaUdmaInOut (
     // Calculate the number of PRD entry.\r
     // Every entry in PRD table can specify a 64K memory region.\r
     //\r
-    PrdTableNum   = (UINTN)(RShiftU64(DataLength, 16) + 1);\r
+    PrdTableNum = (UINTN)(RShiftU64 (DataLength, 16) + 1);\r
 \r
     //\r
     // Make sure that the memory region of PRD table is not cross 64K boundary\r
@@ -1378,14 +1378,14 @@ AtaUdmaInOut (
     //\r
     ASSERT (RealPageCount > PageCount);\r
 \r
-    Status    = PciIo->AllocateBuffer (\r
-                         PciIo,\r
-                         AllocateAnyPages,\r
-                         EfiBootServicesData,\r
-                         RealPageCount,\r
-                         (VOID **)&BaseAddr,\r
-                         0\r
-                         );\r
+    Status = PciIo->AllocateBuffer (\r
+                      PciIo,\r
+                      AllocateAnyPages,\r
+                      EfiBootServicesData,\r
+                      RealPageCount,\r
+                      (VOID **)&BaseAddr,\r
+                      0\r
+                      );\r
     if (EFI_ERROR (Status)) {\r
       return EFI_OUT_OF_RESOURCES;\r
     }\r
@@ -1394,7 +1394,7 @@ AtaUdmaInOut (
     Status    = PciIo->Map (\r
                          PciIo,\r
                          EfiPciIoOperationBusMasterCommonBuffer,\r
-                         (VOID*)(UINTN)BaseAddr,\r
+                         (VOID *)(UINTN)BaseAddr,\r
                          &ByteCount,\r
                          &BaseMapAddr,\r
                          &PrdTableMap\r
@@ -1405,18 +1405,18 @@ AtaUdmaInOut (
       // it means the DMA operation may be broken into several discontinuous smaller chunks.\r
       // Can't handle this case.\r
       //\r
-      PciIo->FreeBuffer (PciIo, RealPageCount, (VOID*)(UINTN)BaseAddr);\r
+      PciIo->FreeBuffer (PciIo, RealPageCount, (VOID *)(UINTN)BaseAddr);\r
       return EFI_OUT_OF_RESOURCES;\r
     }\r
 \r
-    ZeroMem ((VOID *) ((UINTN) BaseAddr), ByteCount);\r
+    ZeroMem ((VOID *)((UINTN)BaseAddr), ByteCount);\r
 \r
     //\r
     // Calculate the 64K align address as PRD Table base address.\r
     //\r
     AlignmentMask    = SIZE_64KB - 1;\r
-    PrdTableBaseAddr = ((UINTN) BaseAddr + AlignmentMask) & ~AlignmentMask;\r
-    PrdTableMapAddr  = ((UINTN) BaseMapAddr + AlignmentMask) & ~AlignmentMask;\r
+    PrdTableBaseAddr = ((UINTN)BaseAddr + AlignmentMask) & ~AlignmentMask;\r
+    PrdTableMapAddr  = ((UINTN)BaseMapAddr + AlignmentMask) & ~AlignmentMask;\r
 \r
     //\r
     // Map the host address of DataBuffer to DMA master address.\r
@@ -1438,7 +1438,7 @@ AtaUdmaInOut (
                          );\r
     if (EFI_ERROR (Status) || (ByteCount != DataLength)) {\r
       PciIo->Unmap (PciIo, PrdTableMap);\r
-      PciIo->FreeBuffer (PciIo, RealPageCount, (VOID*)(UINTN)BaseAddr);\r
+      PciIo->FreeBuffer (PciIo, RealPageCount, (VOID *)(UINTN)BaseAddr);\r
       return EFI_OUT_OF_RESOURCES;\r
     }\r
 \r
@@ -1452,17 +1452,17 @@ AtaUdmaInOut (
     // Fill the PRD table with appropriate bus master address of data buffer and data length.\r
     //\r
     ByteRemaining   = ByteCount;\r
-    TempPrdBaseAddr = (EFI_ATA_DMA_PRD*)(UINTN)PrdTableBaseAddr;\r
+    TempPrdBaseAddr = (EFI_ATA_DMA_PRD *)(UINTN)PrdTableBaseAddr;\r
     while (ByteRemaining != 0) {\r
       if (ByteRemaining <= 0x10000) {\r
-        TempPrdBaseAddr->RegionBaseAddr = (UINT32) ((UINTN) BufferMapAddress);\r
-        TempPrdBaseAddr->ByteCount      = (UINT16) ByteRemaining;\r
+        TempPrdBaseAddr->RegionBaseAddr = (UINT32)((UINTN)BufferMapAddress);\r
+        TempPrdBaseAddr->ByteCount      = (UINT16)ByteRemaining;\r
         TempPrdBaseAddr->EndOfTable     = 0x8000;\r
         break;\r
       }\r
 \r
-      TempPrdBaseAddr->RegionBaseAddr = (UINT32) ((UINTN) BufferMapAddress);\r
-      TempPrdBaseAddr->ByteCount      = (UINT16) 0x0;\r
+      TempPrdBaseAddr->RegionBaseAddr = (UINT32)((UINTN)BufferMapAddress);\r
+      TempPrdBaseAddr->ByteCount      = (UINT16)0x0;\r
 \r
       ByteRemaining    -= 0x10000;\r
       BufferMapAddress += 0x10000;\r
@@ -1485,7 +1485,7 @@ AtaUdmaInOut (
     //\r
     // Read BMIS register and clear ERROR and INTR bit\r
     //\r
-    RegisterValue  = IdeReadPortB(PciIo, IoPortForBmis);\r
+    RegisterValue  = IdeReadPortB (PciIo, IoPortForBmis);\r
     RegisterValue |= (BMIS_INTERRUPT | BMIS_ERROR);\r
     IdeWritePortB (PciIo, IoPortForBmis, RegisterValue);\r
 \r
@@ -1497,18 +1497,19 @@ AtaUdmaInOut (
     //\r
     // Set BMIC register to identify the operation direction\r
     //\r
-    RegisterValue = IdeReadPortB(PciIo, IoPortForBmic);\r
+    RegisterValue = IdeReadPortB (PciIo, IoPortForBmic);\r
     if (Read) {\r
       RegisterValue |= BMIC_NREAD;\r
     } else {\r
-      RegisterValue &= ~((UINT8) BMIC_NREAD);\r
+      RegisterValue &= ~((UINT8)BMIC_NREAD);\r
     }\r
+\r
     IdeWritePortB (PciIo, IoPortForBmic, RegisterValue);\r
 \r
     if (Task != NULL) {\r
       Task->Map            = BufferMap;\r
       Task->TableMap       = PrdTableMap;\r
-      Task->MapBaseAddress = (EFI_ATA_DMA_PRD*)(UINTN)BaseAddr;\r
+      Task->MapBaseAddress = (EFI_ATA_DMA_PRD *)(UINTN)BaseAddr;\r
       Task->PageCount      = RealPageCount;\r
       Task->IsStart        = TRUE;\r
     }\r
@@ -1528,13 +1529,13 @@ AtaUdmaInOut (
       Status = EFI_DEVICE_ERROR;\r
       goto Exit;\r
     }\r
+\r
     //\r
     // Set START bit of BMIC register\r
     //\r
-    RegisterValue  = IdeReadPortB(PciIo, IoPortForBmic);\r
+    RegisterValue  = IdeReadPortB (PciIo, IoPortForBmic);\r
     RegisterValue |= BMIC_START;\r
-    IdeWritePortB(PciIo, IoPortForBmic, RegisterValue);\r
-\r
+    IdeWritePortB (PciIo, IoPortForBmic, RegisterValue);\r
   }\r
 \r
   //\r
@@ -1552,7 +1553,7 @@ AtaUdmaInOut (
   // out, or a error has been happened, it needs to clear the register and free\r
   // buffer.\r
   //\r
-  if ((Task == NULL) || Status != EFI_NOT_READY) {\r
+  if ((Task == NULL) || (Status != EFI_NOT_READY)) {\r
     //\r
     // Read BMIS register and clear ERROR and INTR bit\r
     //\r
@@ -1563,13 +1564,13 @@ AtaUdmaInOut (
     //\r
     // Read Status Register of IDE device to clear interrupt\r
     //\r
-    RegisterValue  = IdeReadPortB(PciIo, IdeRegisters->CmdOrStatus);\r
+    RegisterValue = IdeReadPortB (PciIo, IdeRegisters->CmdOrStatus);\r
 \r
     //\r
     // Clear START bit of BMIC register\r
     //\r
-    RegisterValue  = IdeReadPortB(PciIo, IoPortForBmic);\r
-    RegisterValue &= ~((UINT8) BMIC_START);\r
+    RegisterValue  = IdeReadPortB (PciIo, IoPortForBmic);\r
+    RegisterValue &= ~((UINT8)BMIC_START);\r
     IdeWritePortB (PciIo, IoPortForBmic, RegisterValue);\r
 \r
     //\r
@@ -1582,21 +1583,20 @@ AtaUdmaInOut (
     // Stall for 10 milliseconds.\r
     //\r
     MicroSecondDelay (10000);\r
-\r
   }\r
 \r
 Exit:\r
   //\r
   // Free all allocated resource\r
   //\r
-  if ((Task == NULL) || Status != EFI_NOT_READY) {\r
+  if ((Task == NULL) || (Status != EFI_NOT_READY)) {\r
     if (Task != NULL) {\r
       PciIo->Unmap (PciIo, Task->TableMap);\r
       PciIo->FreeBuffer (PciIo, Task->PageCount, Task->MapBaseAddress);\r
       PciIo->Unmap (PciIo, Task->Map);\r
     } else {\r
       PciIo->Unmap (PciIo, PrdTableMap);\r
-      PciIo->FreeBuffer (PciIo, RealPageCount, (VOID*)(UINTN)BaseAddr);\r
+      PciIo->FreeBuffer (PciIo, RealPageCount, (VOID *)(UINTN)BaseAddr);\r
       PciIo->Unmap (PciIo, BufferMap);\r
     }\r
 \r
@@ -1622,12 +1622,12 @@ Exit:
 EFI_STATUS\r
 EFIAPI\r
 AtaPacketReadPendingData (\r
-  IN  EFI_PCI_IO_PROTOCOL       *PciIo,\r
-  IN  EFI_IDE_REGISTERS         *IdeRegisters\r
+  IN  EFI_PCI_IO_PROTOCOL  *PciIo,\r
+  IN  EFI_IDE_REGISTERS    *IdeRegisters\r
   )\r
 {\r
-  UINT8     AltRegister;\r
-  UINT16    TempWordBuffer;\r
+  UINT8   AltRegister;\r
+  UINT16  TempWordBuffer;\r
 \r
   AltRegister = IdeReadPortB (PciIo, IdeRegisters->AltOrDev);\r
   if ((AltRegister & ATA_STSREG_BSY) == ATA_STSREG_BSY) {\r
@@ -1646,6 +1646,7 @@ AtaPacketReadPendingData (
       TempWordBuffer = IdeReadPortB (PciIo, IdeRegisters->AltOrDev);\r
     }\r
   }\r
+\r
   return EFI_SUCCESS;\r
 }\r
 \r
@@ -1670,12 +1671,12 @@ AtaPacketReadPendingData (
 EFI_STATUS\r
 EFIAPI\r
 AtaPacketReadWrite (\r
-  IN     EFI_PCI_IO_PROTOCOL       *PciIo,\r
-  IN     EFI_IDE_REGISTERS         *IdeRegisters,\r
-  IN OUT VOID                      *Buffer,\r
-  IN OUT UINT32                    *ByteCount,\r
-  IN     BOOLEAN                   Read,\r
-  IN     UINT64                    Timeout\r
+  IN     EFI_PCI_IO_PROTOCOL  *PciIo,\r
+  IN     EFI_IDE_REGISTERS    *IdeRegisters,\r
+  IN OUT VOID                 *Buffer,\r
+  IN OUT UINT32               *ByteCount,\r
+  IN     BOOLEAN              Read,\r
+  IN     UINT64               Timeout\r
   )\r
 {\r
   UINT32      RequiredWordCount;\r
@@ -1803,17 +1804,17 @@ AtaPacketReadWrite (
 EFI_STATUS\r
 EFIAPI\r
 AtaPacketCommandExecute (\r
-  IN  EFI_PCI_IO_PROTOCOL                           *PciIo,\r
-  IN  EFI_IDE_REGISTERS                             *IdeRegisters,\r
-  IN  UINT8                                         Channel,\r
-  IN  UINT8                                         Device,\r
-  IN  EFI_EXT_SCSI_PASS_THRU_SCSI_REQUEST_PACKET    *Packet\r
+  IN  EFI_PCI_IO_PROTOCOL                         *PciIo,\r
+  IN  EFI_IDE_REGISTERS                           *IdeRegisters,\r
+  IN  UINT8                                       Channel,\r
+  IN  UINT8                                       Device,\r
+  IN  EFI_EXT_SCSI_PASS_THRU_SCSI_REQUEST_PACKET  *Packet\r
   )\r
 {\r
-  EFI_ATA_COMMAND_BLOCK       AtaCommandBlock;\r
-  EFI_STATUS                  Status;\r
-  UINT8                       Count;\r
-  UINT8                       PacketCommand[12];\r
+  EFI_ATA_COMMAND_BLOCK  AtaCommandBlock;\r
+  EFI_STATUS             Status;\r
+  UINT8                  Count;\r
+  UINT8                  PacketCommand[12];\r
 \r
   ZeroMem (&AtaCommandBlock, sizeof (EFI_ATA_COMMAND_BLOCK));\r
 \r
@@ -1836,9 +1837,9 @@ AtaPacketCommandExecute (
   // set the transfersize to ATAPI_MAX_BYTE_COUNT to let the device\r
   // determine how many data should be transferred.\r
   //\r
-  AtaCommandBlock.AtaCylinderLow  = (UINT8) (ATAPI_MAX_BYTE_COUNT & 0x00ff);\r
-  AtaCommandBlock.AtaCylinderHigh = (UINT8) (ATAPI_MAX_BYTE_COUNT >> 8);\r
-  AtaCommandBlock.AtaDeviceHead   = (UINT8) (Device << 0x4);\r
+  AtaCommandBlock.AtaCylinderLow  = (UINT8)(ATAPI_MAX_BYTE_COUNT & 0x00ff);\r
+  AtaCommandBlock.AtaCylinderHigh = (UINT8)(ATAPI_MAX_BYTE_COUNT >> 8);\r
+  AtaCommandBlock.AtaDeviceHead   = (UINT8)(Device << 0x4);\r
   AtaCommandBlock.AtaCommand      = ATA_CMD_PACKET;\r
 \r
   IdeWritePortB (PciIo, IdeRegisters->Head, (UINT8)(0xe0 | (Device << 0x4)));\r
@@ -1864,7 +1865,7 @@ AtaPacketCommandExecute (
   // Send out ATAPI command packet\r
   //\r
   for (Count = 0; Count < 6; Count++) {\r
-    IdeWritePortW (PciIo, IdeRegisters->Data, *((UINT16*)PacketCommand + Count));\r
+    IdeWritePortW (PciIo, IdeRegisters->Data, *((UINT16 *)PacketCommand + Count));\r
     //\r
     // Stall for 10 microseconds.\r
     //\r
@@ -1897,7 +1898,6 @@ AtaPacketCommandExecute (
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Set the calculated Best transfer mode to a detected device.\r
 \r
@@ -1922,8 +1922,8 @@ SetDeviceTransferMode (
   IN OUT EFI_ATA_STATUS_BLOCK          *AtaStatusBlock\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
-  EFI_ATA_COMMAND_BLOCK   AtaCommandBlock;\r
+  EFI_STATUS             Status;\r
+  EFI_ATA_COMMAND_BLOCK  AtaCommandBlock;\r
 \r
   ZeroMem (&AtaCommandBlock, sizeof (EFI_ATA_COMMAND_BLOCK));\r
 \r
@@ -1971,14 +1971,14 @@ SetDriveParameters (
   IN OUT EFI_ATA_STATUS_BLOCK          *AtaStatusBlock\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
-  EFI_ATA_COMMAND_BLOCK   AtaCommandBlock;\r
+  EFI_STATUS             Status;\r
+  EFI_ATA_COMMAND_BLOCK  AtaCommandBlock;\r
 \r
   ZeroMem (&AtaCommandBlock, sizeof (EFI_ATA_COMMAND_BLOCK));\r
 \r
   AtaCommandBlock.AtaCommand     = ATA_CMD_INIT_DRIVE_PARAM;\r
   AtaCommandBlock.AtaSectorCount = DriveParameters->Sector;\r
-  AtaCommandBlock.AtaDeviceHead  = (UINT8) ((Device << 0x4) + DriveParameters->Heads);\r
+  AtaCommandBlock.AtaDeviceHead  = (UINT8)((Device << 0x4) + DriveParameters->Heads);\r
 \r
   //\r
   // Send Init drive parameters\r
@@ -2032,10 +2032,10 @@ IdeAtaSmartReturnStatusCheck (
   IN OUT EFI_ATA_STATUS_BLOCK          *AtaStatusBlock\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
-  EFI_ATA_COMMAND_BLOCK   AtaCommandBlock;\r
-  UINT8                   LBAMid;\r
-  UINT8                   LBAHigh;\r
+  EFI_STATUS             Status;\r
+  EFI_ATA_COMMAND_BLOCK  AtaCommandBlock;\r
+  UINT8                  LBAMid;\r
+  UINT8                  LBAHigh;\r
 \r
   ZeroMem (&AtaCommandBlock, sizeof (EFI_ATA_COMMAND_BLOCK));\r
 \r
@@ -2043,7 +2043,7 @@ IdeAtaSmartReturnStatusCheck (
   AtaCommandBlock.AtaFeatures     = ATA_SMART_RETURN_STATUS;\r
   AtaCommandBlock.AtaCylinderLow  = ATA_CONSTANT_4F;\r
   AtaCommandBlock.AtaCylinderHigh = ATA_CONSTANT_C2;\r
-  AtaCommandBlock.AtaDeviceHead   = (UINT8) ((Device << 0x4) | 0xe0);\r
+  AtaCommandBlock.AtaDeviceHead   = (UINT8)((Device << 0x4) | 0xe0);\r
 \r
   //\r
   // Send S.M.A.R.T Read Return Status command to device\r
@@ -2079,18 +2079,18 @@ IdeAtaSmartReturnStatusCheck (
     //\r
     DEBUG ((DEBUG_INFO, "The S.M.A.R.T threshold exceeded condition is not detected\n"));\r
     REPORT_STATUS_CODE (\r
-          EFI_PROGRESS_CODE,\r
-          (EFI_IO_BUS_ATA_ATAPI | EFI_IOB_ATA_BUS_SMART_UNDERTHRESHOLD)\r
-          );\r
+      EFI_PROGRESS_CODE,\r
+      (EFI_IO_BUS_ATA_ATAPI | EFI_IOB_ATA_BUS_SMART_UNDERTHRESHOLD)\r
+      );\r
   } else if ((LBAMid == 0xf4) && (LBAHigh == 0x2c)) {\r
     //\r
     // The threshold exceeded condition is detected by the device\r
     //\r
     DEBUG ((DEBUG_INFO, "The S.M.A.R.T threshold exceeded condition is detected\n"));\r
     REPORT_STATUS_CODE (\r
-         EFI_PROGRESS_CODE,\r
-         (EFI_IO_BUS_ATA_ATAPI | EFI_IOB_ATA_BUS_SMART_OVERTHRESHOLD)\r
-         );\r
+      EFI_PROGRESS_CODE,\r
+      (EFI_IO_BUS_ATA_ATAPI | EFI_IOB_ATA_BUS_SMART_OVERTHRESHOLD)\r
+      );\r
   }\r
 \r
   return EFI_SUCCESS;\r
@@ -2116,8 +2116,8 @@ IdeAtaSmartSupport (
   IN OUT EFI_ATA_STATUS_BLOCK          *AtaStatusBlock\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
-  EFI_ATA_COMMAND_BLOCK   AtaCommandBlock;\r
+  EFI_STATUS             Status;\r
+  EFI_ATA_COMMAND_BLOCK  AtaCommandBlock;\r
 \r
   //\r
   // Detect if the device supports S.M.A.R.T.\r
@@ -2126,8 +2126,12 @@ IdeAtaSmartSupport (
     //\r
     // S.M.A.R.T is not supported by the device\r
     //\r
-    DEBUG ((DEBUG_INFO, "S.M.A.R.T feature is not supported at [%a] channel [%a] device!\n",\r
-            (Channel == 1) ? "secondary" : "primary", (Device == 1) ? "slave" : "master"));\r
+    DEBUG ((\r
+      DEBUG_INFO,\r
+      "S.M.A.R.T feature is not supported at [%a] channel [%a] device!\n",\r
+      (Channel == 1) ? "secondary" : "primary",\r
+      (Device == 1) ? "slave" : "master"\r
+      ));\r
     REPORT_STATUS_CODE (\r
       EFI_ERROR_CODE | EFI_ERROR_MINOR,\r
       (EFI_IO_BUS_ATA_ATAPI | EFI_IOB_ATA_BUS_SMART_NOTSUPPORTED)\r
@@ -2137,7 +2141,6 @@ IdeAtaSmartSupport (
     // Check if the feature is enabled. If not, then enable S.M.A.R.T.\r
     //\r
     if ((IdentifyData->AtaData.command_set_feature_enb_85 & 0x0001) != 0x0001) {\r
-\r
       REPORT_STATUS_CODE (\r
         EFI_PROGRESS_CODE,\r
         (EFI_IO_BUS_ATA_ATAPI | EFI_IOB_ATA_BUS_SMART_DISABLE)\r
@@ -2149,7 +2152,7 @@ IdeAtaSmartSupport (
       AtaCommandBlock.AtaFeatures     = ATA_SMART_ENABLE_OPERATION;\r
       AtaCommandBlock.AtaCylinderLow  = ATA_CONSTANT_4F;\r
       AtaCommandBlock.AtaCylinderHigh = ATA_CONSTANT_C2;\r
-      AtaCommandBlock.AtaDeviceHead   = (UINT8) ((Device << 0x4) | 0xe0);\r
+      AtaCommandBlock.AtaDeviceHead   = (UINT8)((Device << 0x4) | 0xe0);\r
 \r
       //\r
       // Send S.M.A.R.T Enable command to device\r
@@ -2174,7 +2177,7 @@ IdeAtaSmartSupport (
         AtaCommandBlock.AtaSectorCount  = 0xF1;\r
         AtaCommandBlock.AtaCylinderLow  = ATA_CONSTANT_4F;\r
         AtaCommandBlock.AtaCylinderHigh = ATA_CONSTANT_C2;\r
-        AtaCommandBlock.AtaDeviceHead   = (UINT8) ((Device << 0x4) | 0xe0);\r
+        AtaCommandBlock.AtaDeviceHead   = (UINT8)((Device << 0x4) | 0xe0);\r
 \r
         Status = AtaNonDataCommandIn (\r
                    Instance->PciIo,\r
@@ -2195,15 +2198,17 @@ IdeAtaSmartSupport (
       }\r
     }\r
 \r
-    DEBUG ((DEBUG_INFO, "Enabled S.M.A.R.T feature at [%a] channel [%a] device!\n",\r
-           (Channel == 1) ? "secondary" : "primary", (Device == 1) ? "slave" : "master"));\r
-\r
+    DEBUG ((\r
+      DEBUG_INFO,\r
+      "Enabled S.M.A.R.T feature at [%a] channel [%a] device!\n",\r
+      (Channel == 1) ? "secondary" : "primary",\r
+      (Device == 1) ? "slave" : "master"\r
+      ));\r
   }\r
 \r
-  return ;\r
+  return;\r
 }\r
 \r
-\r
 /**\r
   Sends out an ATA Identify Command to the specified device.\r
 \r
@@ -2315,7 +2320,7 @@ AtaIdentifyPacket (
   Status = AtaPioDataInOut (\r
              Instance->PciIo,\r
              &Instance->IdeRegisters[Channel],\r
-             (VOID *) Buffer,\r
+             (VOID *)Buffer,\r
              sizeof (EFI_IDENTIFY_DATA),\r
              TRUE,\r
              &AtaCommandBlock,\r
@@ -2327,7 +2332,6 @@ AtaIdentifyPacket (
   return Status;\r
 }\r
 \r
-\r
 /**\r
   This function is used for detect whether the IDE device exists in the\r
   specified Channel as the specified Device Number.\r
@@ -2357,22 +2361,22 @@ DetectAndConfigIdeDevice (
   IN  UINT8                         IdeChannel\r
   )\r
 {\r
-  EFI_STATUS                        Status;\r
-  UINT8                             SectorCountReg;\r
-  UINT8                             LBALowReg;\r
-  UINT8                             LBAMidReg;\r
-  UINT8                             LBAHighReg;\r
-  EFI_ATA_DEVICE_TYPE               DeviceType;\r
-  UINT8                             IdeDevice;\r
-  EFI_IDE_REGISTERS                 *IdeRegisters;\r
-  EFI_IDENTIFY_DATA                 Buffer;\r
+  EFI_STATUS           Status;\r
+  UINT8                SectorCountReg;\r
+  UINT8                LBALowReg;\r
+  UINT8                LBAMidReg;\r
+  UINT8                LBAHighReg;\r
+  EFI_ATA_DEVICE_TYPE  DeviceType;\r
+  UINT8                IdeDevice;\r
+  EFI_IDE_REGISTERS    *IdeRegisters;\r
+  EFI_IDENTIFY_DATA    Buffer;\r
 \r
   EFI_IDE_CONTROLLER_INIT_PROTOCOL  *IdeInit;\r
   EFI_PCI_IO_PROTOCOL               *PciIo;\r
 \r
-  EFI_ATA_COLLECTIVE_MODE           *SupportedModes;\r
-  EFI_ATA_TRANSFER_MODE             TransferMode;\r
-  EFI_ATA_DRIVE_PARMS               DriveParameters;\r
+  EFI_ATA_COLLECTIVE_MODE  *SupportedModes;\r
+  EFI_ATA_TRANSFER_MODE    TransferMode;\r
+  EFI_ATA_DRIVE_PARMS      DriveParameters;\r
 \r
   IdeRegisters = &Instance->IdeRegisters[IdeChannel];\r
   IdeInit      = Instance->IdeControllerInit;\r
@@ -2392,7 +2396,7 @@ DetectAndConfigIdeDevice (
 \r
     Status = WaitForBSYClear (PciIo, IdeRegisters, 350000000);\r
     if (EFI_ERROR (Status)) {\r
-      DEBUG((DEBUG_ERROR, "New detecting method: Send Execute Diagnostic Command: WaitForBSYClear: Status: %d\n", Status));\r
+      DEBUG ((DEBUG_ERROR, "New detecting method: Send Execute Diagnostic Command: WaitForBSYClear: Status: %d\n", Status));\r
       continue;\r
     }\r
 \r
@@ -2453,9 +2457,13 @@ DetectAndConfigIdeDevice (
       continue;\r
     }\r
 \r
-    DEBUG ((DEBUG_INFO, "[%a] channel [%a] [%a] device\n",\r
-            (IdeChannel == 1) ? "secondary" : "primary  ", (IdeDevice == 1) ? "slave " : "master",\r
-            DeviceType == EfiIdeCdrom ? "cdrom   " : "harddisk"));\r
+    DEBUG ((\r
+      DEBUG_INFO,\r
+      "[%a] channel [%a] [%a] device\n",\r
+      (IdeChannel == 1) ? "secondary" : "primary  ",\r
+      (IdeDevice == 1) ? "slave " : "master",\r
+      DeviceType == EfiIdeCdrom ? "cdrom   " : "harddisk"\r
+      ));\r
     //\r
     // If the device is a hard disk, then try to enable S.M.A.R.T feature\r
     //\r
@@ -2497,9 +2505,9 @@ DetectAndConfigIdeDevice (
       TransferMode.ModeCategory = EFI_ATA_MODE_FLOW_PIO;\r
     }\r
 \r
-    TransferMode.ModeNumber = (UINT8) (SupportedModes->PioMode.Mode);\r
+    TransferMode.ModeNumber = (UINT8)(SupportedModes->PioMode.Mode);\r
 \r
-    if (SupportedModes->ExtModeCount == 0){\r
+    if (SupportedModes->ExtModeCount == 0) {\r
       Status = SetDeviceTransferMode (Instance, IdeChannel, IdeDevice, &TransferMode, NULL);\r
 \r
       if (EFI_ERROR (Status)) {\r
@@ -2516,8 +2524,8 @@ DetectAndConfigIdeDevice (
     //\r
     if (SupportedModes->UdmaMode.Valid) {\r
       TransferMode.ModeCategory = EFI_ATA_MODE_UDMA;\r
-      TransferMode.ModeNumber   = (UINT8) (SupportedModes->UdmaMode.Mode);\r
-      Status = SetDeviceTransferMode (Instance, IdeChannel, IdeDevice, &TransferMode, NULL);\r
+      TransferMode.ModeNumber   = (UINT8)(SupportedModes->UdmaMode.Mode);\r
+      Status                    = SetDeviceTransferMode (Instance, IdeChannel, IdeDevice, &TransferMode, NULL);\r
 \r
       if (EFI_ERROR (Status)) {\r
         DEBUG ((DEBUG_ERROR, "Set transfer Mode Fail, Status = %r\n", Status));\r
@@ -2525,8 +2533,8 @@ DetectAndConfigIdeDevice (
       }\r
     } else if (SupportedModes->MultiWordDmaMode.Valid) {\r
       TransferMode.ModeCategory = EFI_ATA_MODE_MDMA;\r
-      TransferMode.ModeNumber   = (UINT8) SupportedModes->MultiWordDmaMode.Mode;\r
-      Status = SetDeviceTransferMode (Instance, IdeChannel, IdeDevice, &TransferMode, NULL);\r
+      TransferMode.ModeNumber   = (UINT8)SupportedModes->MultiWordDmaMode.Mode;\r
+      Status                    = SetDeviceTransferMode (Instance, IdeChannel, IdeDevice, &TransferMode, NULL);\r
 \r
       if (EFI_ERROR (Status)) {\r
         DEBUG ((DEBUG_ERROR, "Set transfer Mode Fail, Status = %r\n", Status));\r
@@ -2543,9 +2551,9 @@ DetectAndConfigIdeDevice (
       //\r
       // Init driver parameters\r
       //\r
-      DriveParameters.Sector         = (UINT8) ((ATA5_IDENTIFY_DATA *)(&Buffer.AtaData))->sectors_per_track;\r
-      DriveParameters.Heads          = (UINT8) (((ATA5_IDENTIFY_DATA *)(&Buffer.AtaData))->heads - 1);\r
-      DriveParameters.MultipleSector = (UINT8) ((ATA5_IDENTIFY_DATA *)(&Buffer.AtaData))->multi_sector_cmd_max_sct_cnt;\r
+      DriveParameters.Sector         = (UINT8)((ATA5_IDENTIFY_DATA *)(&Buffer.AtaData))->sectors_per_track;\r
+      DriveParameters.Heads          = (UINT8)(((ATA5_IDENTIFY_DATA *)(&Buffer.AtaData))->heads - 1);\r
+      DriveParameters.MultipleSector = (UINT8)((ATA5_IDENTIFY_DATA *)(&Buffer.AtaData))->multi_sector_cmd_max_sct_cnt;\r
 \r
       Status = SetDriveParameters (Instance, IdeChannel, IdeDevice, &DriveParameters, NULL);\r
     }\r
@@ -2568,10 +2576,10 @@ DetectAndConfigIdeDevice (
       REPORT_STATUS_CODE (EFI_PROGRESS_CODE, (EFI_PERIPHERAL_FIXED_MEDIA | EFI_P_PC_ENABLE));\r
     }\r
   }\r
+\r
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
 /**\r
   Initialize ATA host controller at IDE mode.\r
 \r
@@ -2583,7 +2591,7 @@ DetectAndConfigIdeDevice (
 EFI_STATUS\r
 EFIAPI\r
 IdeModeInitialization (\r
-  IN  ATA_ATAPI_PASS_THRU_INSTANCE    *Instance\r
+  IN  ATA_ATAPI_PASS_THRU_INSTANCE  *Instance\r
   )\r
 {\r
   EFI_STATUS                        Status;\r