]> git.proxmox.com Git - mirror_edk2.git/blobdiff - OvmfPkg/SataControllerDxe/SataController.c
OvmfPkg: Apply uncrustify changes
[mirror_edk2.git] / OvmfPkg / SataControllerDxe / SataController.c
index a5b5f04d270881eb494c47c61b0dc83eeb3540bd..2b550b0a3ec0c74f52cb550b4e053be65819261e 100644 (file)
@@ -11,7 +11,7 @@
 ///\r
 /// EFI_DRIVER_BINDING_PROTOCOL instance\r
 ///\r
-EFI_DRIVER_BINDING_PROTOCOL gSataControllerDriverBinding = {\r
+EFI_DRIVER_BINDING_PROTOCOL  gSataControllerDriverBinding = {\r
   SataControllerSupported,\r
   SataControllerStart,\r
   SataControllerStop,\r
@@ -32,11 +32,11 @@ EFI_DRIVER_BINDING_PROTOCOL gSataControllerDriverBinding = {
 UINT32\r
 EFIAPI\r
 AhciReadReg (\r
-  IN EFI_PCI_IO_PROTOCOL    *PciIo,\r
-  IN UINT32                 Offset\r
+  IN EFI_PCI_IO_PROTOCOL  *PciIo,\r
+  IN UINT32               Offset\r
   )\r
 {\r
-  UINT32    Data;\r
+  UINT32  Data;\r
 \r
   ASSERT (PciIo != NULL);\r
 \r
@@ -46,7 +46,7 @@ AhciReadReg (
                PciIo,\r
                EfiPciIoWidthUint32,\r
                AHCI_BAR_INDEX,\r
-               (UINT64) Offset,\r
+               (UINT64)Offset,\r
                1,\r
                &Data\r
                );\r
@@ -65,9 +65,9 @@ AhciReadReg (
 VOID\r
 EFIAPI\r
 AhciWriteReg (\r
-  IN EFI_PCI_IO_PROTOCOL    *PciIo,\r
-  IN UINT32                 Offset,\r
-  IN UINT32                 Data\r
+  IN EFI_PCI_IO_PROTOCOL  *PciIo,\r
+  IN UINT32               Offset,\r
+  IN UINT32               Data\r
   )\r
 {\r
   ASSERT (PciIo != NULL);\r
@@ -76,7 +76,7 @@ AhciWriteReg (
                PciIo,\r
                EfiPciIoWidthUint32,\r
                AHCI_BAR_INDEX,\r
-               (UINT64) Offset,\r
+               (UINT64)Offset,\r
                1,\r
                &Data\r
                );\r
@@ -102,21 +102,20 @@ CalculateBestPioMode (
   OUT UINT16            *SelectedMode\r
   )\r
 {\r
-  UINT16    PioMode;\r
-  UINT16    AdvancedPioMode;\r
-  UINT16    Temp;\r
-  UINT16    Index;\r
-  UINT16    MinimumPioCycleTime;\r
+  UINT16  PioMode;\r
+  UINT16  AdvancedPioMode;\r
+  UINT16  Temp;\r
+  UINT16  Index;\r
+  UINT16  MinimumPioCycleTime;\r
 \r
   Temp = 0xff;\r
 \r
-  PioMode = (UINT8) (((ATA5_IDENTIFY_DATA *) (&(IdentifyData->AtaData)))->pio_cycle_timing >> 8);\r
+  PioMode = (UINT8)(((ATA5_IDENTIFY_DATA *)(&(IdentifyData->AtaData)))->pio_cycle_timing >> 8);\r
 \r
   //\r
   // See whether Identify Data word 64 - 70 are valid\r
   //\r
   if ((IdentifyData->AtaData.field_validity & 0x02) == 0x02) {\r
-\r
     AdvancedPioMode = IdentifyData->AtaData.advanced_pio_modes;\r
     DEBUG ((DEBUG_INFO, "CalculateBestPioMode: AdvancedPioMode = %x\n", AdvancedPioMode));\r
 \r
@@ -134,7 +133,7 @@ CalculateBestPioMode (
     // the best PIO Mode is the value in pio_cycle_timing.\r
     //\r
     if (Temp != 0xff) {\r
-      AdvancedPioMode = (UINT16) (Temp + 3);\r
+      AdvancedPioMode = (UINT16)(Temp + 3);\r
     } else {\r
       AdvancedPioMode = PioMode;\r
     }\r
@@ -142,16 +141,16 @@ CalculateBestPioMode (
     //\r
     // Limit the PIO mode to at most PIO4.\r
     //\r
-    PioMode = (UINT16) MIN (AdvancedPioMode, 4);\r
+    PioMode = (UINT16)MIN (AdvancedPioMode, 4);\r
 \r
     MinimumPioCycleTime = IdentifyData->AtaData.min_pio_cycle_time_with_flow_control;\r
 \r
     if (MinimumPioCycleTime <= 120) {\r
-      PioMode = (UINT16) MIN (4, PioMode);\r
+      PioMode = (UINT16)MIN (4, PioMode);\r
     } else if (MinimumPioCycleTime <= 180) {\r
-      PioMode = (UINT16) MIN (3, PioMode);\r
+      PioMode = (UINT16)MIN (3, PioMode);\r
     } else if (MinimumPioCycleTime <= 240) {\r
-      PioMode = (UINT16) MIN (2, PioMode);\r
+      PioMode = (UINT16)MIN (2, PioMode);\r
     } else {\r
       PioMode = 0;\r
     }\r
@@ -165,7 +164,7 @@ CalculateBestPioMode (
       }\r
 \r
       if (PioMode >= *DisPioMode) {\r
-        PioMode = (UINT16) (*DisPioMode - 1);\r
+        PioMode = (UINT16)(*DisPioMode - 1);\r
       }\r
     }\r
 \r
@@ -174,7 +173,6 @@ CalculateBestPioMode (
     } else {\r
       *SelectedMode = PioMode;  // ATA_PIO_MODE_2 to ATA_PIO_MODE_4;\r
     }\r
-\r
   } else {\r
     //\r
     // Identify Data word 64 - 70 are not valid\r
@@ -195,7 +193,6 @@ CalculateBestPioMode (
     } else {\r
       *SelectedMode = 2;        // ATA_PIO_MODE_2;\r
     }\r
-\r
   }\r
 \r
   return EFI_SUCCESS;\r
@@ -219,8 +216,8 @@ CalculateBestUdmaMode (
   OUT UINT16            *SelectedMode\r
   )\r
 {\r
-  UINT16    TempMode;\r
-  UINT16    DeviceUDmaMode;\r
+  UINT16  TempMode;\r
+  UINT16  DeviceUDmaMode;\r
 \r
   DeviceUDmaMode = 0;\r
 \r
@@ -234,7 +231,7 @@ CalculateBestUdmaMode (
   DeviceUDmaMode = IdentifyData->AtaData.ultra_dma_mode;\r
   DEBUG ((DEBUG_INFO, "CalculateBestUdmaMode: DeviceUDmaMode = %x\n", DeviceUDmaMode));\r
   DeviceUDmaMode &= 0x3f;\r
-  TempMode = 0;                 // initialize it to UDMA-0\r
+  TempMode        = 0;          // initialize it to UDMA-0\r
 \r
   while ((DeviceUDmaMode >>= 1) != 0) {\r
     TempMode++;\r
@@ -250,7 +247,7 @@ CalculateBestUdmaMode (
     }\r
 \r
     if (TempMode >= *DisUDmaMode) {\r
-      TempMode = (UINT16) (*DisUDmaMode - 1);\r
+      TempMode = (UINT16)(*DisUDmaMode - 1);\r
     }\r
   }\r
 \r
@@ -275,11 +272,11 @@ CalculateBestUdmaMode (
 EFI_STATUS\r
 EFIAPI\r
 InitializeSataControllerDriver (\r
-  IN EFI_HANDLE         ImageHandle,\r
-  IN EFI_SYSTEM_TABLE   *SystemTable\r
+  IN EFI_HANDLE        ImageHandle,\r
+  IN EFI_SYSTEM_TABLE  *SystemTable\r
   )\r
 {\r
-  EFI_STATUS    Status;\r
+  EFI_STATUS  Status;\r
 \r
   //\r
   // Install driver model protocol(s).\r
@@ -314,14 +311,14 @@ InitializeSataControllerDriver (
 EFI_STATUS\r
 EFIAPI\r
 SataControllerSupported (\r
-  IN EFI_DRIVER_BINDING_PROTOCOL    *This,\r
-  IN EFI_HANDLE                     Controller,\r
-  IN EFI_DEVICE_PATH_PROTOCOL       *RemainingDevicePath\r
+  IN EFI_DRIVER_BINDING_PROTOCOL  *This,\r
+  IN EFI_HANDLE                   Controller,\r
+  IN EFI_DEVICE_PATH_PROTOCOL     *RemainingDevicePath\r
   )\r
 {\r
-  EFI_STATUS            Status;\r
-  EFI_PCI_IO_PROTOCOL   *PciIo;\r
-  PCI_TYPE00            PciData;\r
+  EFI_STATUS           Status;\r
+  EFI_PCI_IO_PROTOCOL  *PciIo;\r
+  PCI_TYPE00           PciData;\r
 \r
   //\r
   // Attempt to open PCI I/O Protocol\r
@@ -329,7 +326,7 @@ SataControllerSupported (
   Status = gBS->OpenProtocol (\r
                   Controller,\r
                   &gEfiPciIoProtocolGuid,\r
-                  (VOID **) &PciIo,\r
+                  (VOID **)&PciIo,\r
                   This->DriverBindingHandle,\r
                   Controller,\r
                   EFI_OPEN_PROTOCOL_GET_PROTOCOL\r
@@ -377,9 +374,9 @@ SataControllerSupported (
 EFI_STATUS\r
 EFIAPI\r
 SataControllerStart (\r
-  IN EFI_DRIVER_BINDING_PROTOCOL    *This,\r
-  IN EFI_HANDLE                     Controller,\r
-  IN EFI_DEVICE_PATH_PROTOCOL       *RemainingDevicePath\r
+  IN EFI_DRIVER_BINDING_PROTOCOL  *This,\r
+  IN EFI_HANDLE                   Controller,\r
+  IN EFI_DEVICE_PATH_PROTOCOL     *RemainingDevicePath\r
   )\r
 {\r
   UINTN                             BailLogMask;\r
@@ -393,7 +390,7 @@ SataControllerStart (
 \r
   DEBUG ((DEBUG_INFO, "SataControllerStart START\n"));\r
 \r
-  BailLogMask = DEBUG_ERROR;\r
+  BailLogMask     = DEBUG_ERROR;\r
   SataPrivateData = NULL;\r
 \r
   //\r
@@ -402,7 +399,7 @@ SataControllerStart (
   Status = gBS->OpenProtocol (\r
                   Controller,\r
                   &gEfiPciIoProtocolGuid,\r
-                  (VOID **) &PciIo,\r
+                  (VOID **)&PciIo,\r
                   This->DriverBindingHandle,\r
                   Controller,\r
                   EFI_OPEN_PROTOCOL_BY_DRIVER\r
@@ -416,6 +413,7 @@ SataControllerStart (
       //\r
       BailLogMask = DEBUG_INFO;\r
     }\r
+\r
     goto Bail;\r
   }\r
 \r
@@ -423,13 +421,22 @@ SataControllerStart (
   // Save original PCI attributes, and enable IO space access, memory space\r
   // access, and Bus Master (DMA).\r
   //\r
-  Status = PciIo->Attributes (PciIo, EfiPciIoAttributeOperationGet, 0,\r
-                    &OriginalPciAttributes);\r
+  Status = PciIo->Attributes (\r
+                    PciIo,\r
+                    EfiPciIoAttributeOperationGet,\r
+                    0,\r
+                    &OriginalPciAttributes\r
+                    );\r
   if (EFI_ERROR (Status)) {\r
     goto ClosePciIo;\r
   }\r
-  Status = PciIo->Attributes (PciIo, EfiPciIoAttributeOperationEnable,\r
-                    EFI_PCI_DEVICE_ENABLE, NULL);\r
+\r
+  Status = PciIo->Attributes (\r
+                    PciIo,\r
+                    EfiPciIoAttributeOperationEnable,\r
+                    EFI_PCI_DEVICE_ENABLE,\r
+                    NULL\r
+                    );\r
   if (EFI_ERROR (Status)) {\r
     goto ClosePciIo;\r
   }\r
@@ -446,16 +453,16 @@ SataControllerStart (
   //\r
   // Initialize Sata Private Data\r
   //\r
-  SataPrivateData->Signature = SATA_CONTROLLER_SIGNATURE;\r
-  SataPrivateData->PciIo = PciIo;\r
-  SataPrivateData->OriginalPciAttributes = OriginalPciAttributes;\r
+  SataPrivateData->Signature              = SATA_CONTROLLER_SIGNATURE;\r
+  SataPrivateData->PciIo                  = PciIo;\r
+  SataPrivateData->OriginalPciAttributes  = OriginalPciAttributes;\r
   SataPrivateData->IdeInit.GetChannelInfo = IdeInitGetChannelInfo;\r
-  SataPrivateData->IdeInit.NotifyPhase = IdeInitNotifyPhase;\r
-  SataPrivateData->IdeInit.SubmitData = IdeInitSubmitData;\r
+  SataPrivateData->IdeInit.NotifyPhase    = IdeInitNotifyPhase;\r
+  SataPrivateData->IdeInit.SubmitData     = IdeInitSubmitData;\r
   SataPrivateData->IdeInit.DisqualifyMode = IdeInitDisqualifyMode;\r
-  SataPrivateData->IdeInit.CalculateMode = IdeInitCalculateMode;\r
-  SataPrivateData->IdeInit.SetTiming = IdeInitSetTiming;\r
-  SataPrivateData->IdeInit.EnumAll = SATA_ENUMER_ALL;\r
+  SataPrivateData->IdeInit.CalculateMode  = IdeInitCalculateMode;\r
+  SataPrivateData->IdeInit.SetTiming      = IdeInitSetTiming;\r
+  SataPrivateData->IdeInit.EnumAll        = SATA_ENUMER_ALL;\r
 \r
   Status = PciIo->Pci.Read (\r
                         PciIo,\r
@@ -470,22 +477,22 @@ SataControllerStart (
 \r
   if (IS_PCI_IDE (&PciData)) {\r
     SataPrivateData->IdeInit.ChannelCount = IDE_MAX_CHANNEL;\r
-    SataPrivateData->DeviceCount = IDE_MAX_DEVICES;\r
+    SataPrivateData->DeviceCount          = IDE_MAX_DEVICES;\r
   } else if (IS_PCI_SATADPA (&PciData)) {\r
     //\r
     // Read Host Capability Register(CAP) to get Number of Ports(NPS) and Supports Port Multiplier(SPM)\r
     //   NPS is 0's based value indicating the maximum number of ports supported by the HBA silicon.\r
     //   A maximum of 32 ports can be supported. A value of '0h', indicating one port, is the minimum requirement.\r
     //\r
-    Data32 = AhciReadReg (PciIo, R_AHCI_CAP);\r
-    SataPrivateData->IdeInit.ChannelCount = (UINT8) ((Data32 & B_AHCI_CAP_NPS) + 1);\r
-    SataPrivateData->DeviceCount = AHCI_MAX_DEVICES;\r
+    Data32                                = AhciReadReg (PciIo, R_AHCI_CAP);\r
+    SataPrivateData->IdeInit.ChannelCount = (UINT8)((Data32 & B_AHCI_CAP_NPS) + 1);\r
+    SataPrivateData->DeviceCount          = AHCI_MAX_DEVICES;\r
     if ((Data32 & B_AHCI_CAP_SPM) == B_AHCI_CAP_SPM) {\r
       SataPrivateData->DeviceCount = AHCI_MULTI_MAX_DEVICES;\r
     }\r
   }\r
 \r
-  ChannelDeviceCount = (UINTN) (SataPrivateData->IdeInit.ChannelCount) * (UINTN) (SataPrivateData->DeviceCount);\r
+  ChannelDeviceCount                 = (UINTN)(SataPrivateData->IdeInit.ChannelCount) * (UINTN)(SataPrivateData->DeviceCount);\r
   SataPrivateData->DisqualifiedModes = AllocateZeroPool ((sizeof (EFI_ATA_COLLECTIVE_MODE)) * ChannelDeviceCount);\r
   if (SataPrivateData->DisqualifiedModes == NULL) {\r
     Status = EFI_OUT_OF_RESOURCES;\r
@@ -534,8 +541,12 @@ FreeSataPrivateData:
   FreePool (SataPrivateData);\r
 \r
 RestorePciAttributes:\r
-  PciIo->Attributes (PciIo, EfiPciIoAttributeOperationSet,\r
-           OriginalPciAttributes, NULL);\r
+  PciIo->Attributes (\r
+           PciIo,\r
+           EfiPciIoAttributeOperationSet,\r
+           OriginalPciAttributes,\r
+           NULL\r
+           );\r
 \r
 ClosePciIo:\r
   gBS->CloseProtocol (\r
@@ -546,8 +557,11 @@ ClosePciIo:
          );\r
 \r
 Bail:\r
-  DEBUG ((BailLogMask, "SataControllerStart error return status = %r\n",\r
-    Status));\r
+  DEBUG ((\r
+    BailLogMask,\r
+    "SataControllerStart error return status = %r\n",\r
+    Status\r
+    ));\r
   return Status;\r
 }\r
 \r
@@ -566,10 +580,10 @@ Bail:
 EFI_STATUS\r
 EFIAPI\r
 SataControllerStop (\r
-  IN EFI_DRIVER_BINDING_PROTOCOL    *This,\r
-  IN EFI_HANDLE                     Controller,\r
-  IN UINTN                          NumberOfChildren,\r
-  IN EFI_HANDLE                     *ChildHandleBuffer\r
+  IN EFI_DRIVER_BINDING_PROTOCOL  *This,\r
+  IN EFI_HANDLE                   Controller,\r
+  IN UINTN                        NumberOfChildren,\r
+  IN EFI_HANDLE                   *ChildHandleBuffer\r
   )\r
 {\r
   EFI_STATUS                        Status;\r
@@ -584,7 +598,7 @@ SataControllerStop (
   Status = gBS->OpenProtocol (\r
                   Controller,\r
                   &gEfiIdeControllerInitProtocolGuid,\r
-                  (VOID **) &IdeInit,\r
+                  (VOID **)&IdeInit,\r
                   This->DriverBindingHandle,\r
                   Controller,\r
                   EFI_OPEN_PROTOCOL_GET_PROTOCOL\r
@@ -615,12 +629,15 @@ SataControllerStop (
   if (SataPrivateData->DisqualifiedModes != NULL) {\r
     FreePool (SataPrivateData->DisqualifiedModes);\r
   }\r
+\r
   if (SataPrivateData->IdentifyData != NULL) {\r
     FreePool (SataPrivateData->IdentifyData);\r
   }\r
+\r
   if (SataPrivateData->IdentifyValid != NULL) {\r
     FreePool (SataPrivateData->IdentifyValid);\r
   }\r
+\r
   FreePool (SataPrivateData);\r
 \r
   //\r
@@ -678,6 +695,7 @@ FlatDeviceIndex (
 //\r
 // Interface functions of IDE_CONTROLLER_INIT protocol\r
 //\r
+\r
 /**\r
   Returns the information about the specified IDE channel.\r
 \r
@@ -717,18 +735,19 @@ FlatDeviceIndex (
 EFI_STATUS\r
 EFIAPI\r
 IdeInitGetChannelInfo (\r
-  IN EFI_IDE_CONTROLLER_INIT_PROTOCOL   *This,\r
-  IN UINT8                              Channel,\r
-  OUT BOOLEAN                           *Enabled,\r
-  OUT UINT8                             *MaxDevices\r
+  IN EFI_IDE_CONTROLLER_INIT_PROTOCOL  *This,\r
+  IN UINT8                             Channel,\r
+  OUT BOOLEAN                          *Enabled,\r
+  OUT UINT8                            *MaxDevices\r
   )\r
 {\r
   EFI_SATA_CONTROLLER_PRIVATE_DATA  *SataPrivateData;\r
+\r
   SataPrivateData = SATA_CONTROLLER_PRIVATE_DATA_FROM_THIS (This);\r
   ASSERT (SataPrivateData != NULL);\r
 \r
   if (Channel < This->ChannelCount) {\r
-    *Enabled = TRUE;\r
+    *Enabled    = TRUE;\r
     *MaxDevices = SataPrivateData->DeviceCount;\r
     return EFI_SUCCESS;\r
   }\r
@@ -764,9 +783,9 @@ IdeInitGetChannelInfo (
 EFI_STATUS\r
 EFIAPI\r
 IdeInitNotifyPhase (\r
-  IN EFI_IDE_CONTROLLER_INIT_PROTOCOL   *This,\r
-  IN EFI_IDE_CONTROLLER_ENUM_PHASE      Phase,\r
-  IN UINT8                              Channel\r
+  IN EFI_IDE_CONTROLLER_INIT_PROTOCOL  *This,\r
+  IN EFI_IDE_CONTROLLER_ENUM_PHASE     Phase,\r
+  IN UINT8                             Channel\r
   )\r
 {\r
   return EFI_SUCCESS;\r
@@ -814,10 +833,10 @@ IdeInitNotifyPhase (
 EFI_STATUS\r
 EFIAPI\r
 IdeInitSubmitData (\r
-  IN EFI_IDE_CONTROLLER_INIT_PROTOCOL   *This,\r
-  IN UINT8                              Channel,\r
-  IN UINT8                              Device,\r
-  IN EFI_IDENTIFY_DATA                  *IdentifyData\r
+  IN EFI_IDE_CONTROLLER_INIT_PROTOCOL  *This,\r
+  IN UINT8                             Channel,\r
+  IN UINT8                             Device,\r
+  IN EFI_IDENTIFY_DATA                 *IdentifyData\r
   )\r
 {\r
   EFI_SATA_CONTROLLER_PRIVATE_DATA  *SataPrivateData;\r
@@ -893,10 +912,10 @@ IdeInitSubmitData (
 EFI_STATUS\r
 EFIAPI\r
 IdeInitDisqualifyMode (\r
-  IN EFI_IDE_CONTROLLER_INIT_PROTOCOL   *This,\r
-  IN UINT8                              Channel,\r
-  IN UINT8                              Device,\r
-  IN EFI_ATA_COLLECTIVE_MODE            *BadModes\r
+  IN EFI_IDE_CONTROLLER_INIT_PROTOCOL  *This,\r
+  IN UINT8                             Channel,\r
+  IN UINT8                             Device,\r
+  IN EFI_ATA_COLLECTIVE_MODE           *BadModes\r
   )\r
 {\r
   EFI_SATA_CONTROLLER_PRIVATE_DATA  *SataPrivateData;\r
@@ -981,10 +1000,10 @@ IdeInitDisqualifyMode (
 EFI_STATUS\r
 EFIAPI\r
 IdeInitCalculateMode (\r
-  IN EFI_IDE_CONTROLLER_INIT_PROTOCOL   *This,\r
-  IN UINT8                              Channel,\r
-  IN UINT8                              Device,\r
-  OUT EFI_ATA_COLLECTIVE_MODE           **SupportedModes\r
+  IN EFI_IDE_CONTROLLER_INIT_PROTOCOL  *This,\r
+  IN UINT8                             Channel,\r
+  IN UINT8                             Device,\r
+  OUT EFI_ATA_COLLECTIVE_MODE          **SupportedModes\r
   )\r
 {\r
   EFI_SATA_CONTROLLER_PRIVATE_DATA  *SataPrivateData;\r
@@ -1010,8 +1029,8 @@ IdeInitCalculateMode (
 \r
   DeviceIndex = FlatDeviceIndex (SataPrivateData, Channel, Device);\r
 \r
-  IdentifyData = &(SataPrivateData->IdentifyData[DeviceIndex]);\r
-  IdentifyValid = SataPrivateData->IdentifyValid[DeviceIndex];\r
+  IdentifyData      = &(SataPrivateData->IdentifyData[DeviceIndex]);\r
+  IdentifyValid     = SataPrivateData->IdentifyValid[DeviceIndex];\r
   DisqualifiedModes = &(SataPrivateData->DisqualifiedModes[DeviceIndex]);\r
 \r
   //\r
@@ -1023,32 +1042,32 @@ IdeInitCalculateMode (
   }\r
 \r
   Status = CalculateBestPioMode (\r
-            IdentifyData,\r
-            (DisqualifiedModes->PioMode.Valid ? ((UINT16 *) &(DisqualifiedModes->PioMode.Mode)) : NULL),\r
-            &SelectedMode\r
-            );\r
+             IdentifyData,\r
+             (DisqualifiedModes->PioMode.Valid ? ((UINT16 *)&(DisqualifiedModes->PioMode.Mode)) : NULL),\r
+             &SelectedMode\r
+             );\r
   if (!EFI_ERROR (Status)) {\r
     (*SupportedModes)->PioMode.Valid = TRUE;\r
-    (*SupportedModes)->PioMode.Mode = SelectedMode;\r
-\r
+    (*SupportedModes)->PioMode.Mode  = SelectedMode;\r
   } else {\r
     (*SupportedModes)->PioMode.Valid = FALSE;\r
   }\r
+\r
   DEBUG ((DEBUG_INFO, "IdeInitCalculateMode: PioMode = %x\n", (*SupportedModes)->PioMode.Mode));\r
 \r
   Status = CalculateBestUdmaMode (\r
-            IdentifyData,\r
-            (DisqualifiedModes->UdmaMode.Valid ? ((UINT16 *) &(DisqualifiedModes->UdmaMode.Mode)) : NULL),\r
-            &SelectedMode\r
-            );\r
+             IdentifyData,\r
+             (DisqualifiedModes->UdmaMode.Valid ? ((UINT16 *)&(DisqualifiedModes->UdmaMode.Mode)) : NULL),\r
+             &SelectedMode\r
+             );\r
 \r
   if (!EFI_ERROR (Status)) {\r
     (*SupportedModes)->UdmaMode.Valid = TRUE;\r
     (*SupportedModes)->UdmaMode.Mode  = SelectedMode;\r
-\r
   } else {\r
     (*SupportedModes)->UdmaMode.Valid = FALSE;\r
   }\r
+\r
   DEBUG ((DEBUG_INFO, "IdeInitCalculateMode: UdmaMode = %x\n", (*SupportedModes)->UdmaMode.Mode));\r
 \r
   //\r
@@ -1083,10 +1102,10 @@ IdeInitCalculateMode (
 EFI_STATUS\r
 EFIAPI\r
 IdeInitSetTiming (\r
-  IN EFI_IDE_CONTROLLER_INIT_PROTOCOL   *This,\r
-  IN UINT8                              Channel,\r
-  IN UINT8                              Device,\r
-  IN EFI_ATA_COLLECTIVE_MODE            *Modes\r
+  IN EFI_IDE_CONTROLLER_INIT_PROTOCOL  *This,\r
+  IN UINT8                             Channel,\r
+  IN UINT8                             Device,\r
+  IN EFI_ATA_COLLECTIVE_MODE           *Modes\r
   )\r
 {\r
   return EFI_SUCCESS;\r