]> git.proxmox.com Git - mirror_edk2.git/blobdiff - OvmfPkg/SataControllerDxe/SataController.c
OvmfPkg: Apply uncrustify changes
[mirror_edk2.git] / OvmfPkg / SataControllerDxe / SataController.c
index ba21717cad81416b3677b731551d899949796617..2b550b0a3ec0c74f52cb550b4e053be65819261e 100644 (file)
@@ -2,13 +2,7 @@
   This driver module produces IDE_CONTROLLER_INIT protocol for Sata Controllers.\r
 \r
   Copyright (c) 2011, Intel Corporation. All rights reserved.<BR>\r
-  This program and the accompanying materials\r
-  are licensed and made available under the terms and conditions of the BSD License\r
-  which accompanies this distribution.  The full text of the license may be found at\r
-  http://opensource.org/licenses/bsd-license.php\r
-\r
-  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
-  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
+  SPDX-License-Identifier: BSD-2-Clause-Patent\r
 \r
 **/\r
 \r
@@ -17,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
@@ -38,21 +32,21 @@ 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
+\r
   Data = 0;\r
 \r
   PciIo->Mem.Read (\r
                PciIo,\r
                EfiPciIoWidthUint32,\r
                AHCI_BAR_INDEX,\r
-               (UINT64) Offset,\r
+               (UINT64)Offset,\r
                1,\r
                &Data\r
                );\r
@@ -71,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
@@ -82,7 +76,7 @@ AhciWriteReg (
                PciIo,\r
                EfiPciIoWidthUint32,\r
                AHCI_BAR_INDEX,\r
-               (UINT64) Offset,\r
+               (UINT64)Offset,\r
                1,\r
                &Data\r
                );\r
@@ -108,23 +102,22 @@ 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 ((EFI_D_INFO, "CalculateBestPioMode: AdvancedPioMode = %x\n", AdvancedPioMode));\r
+    DEBUG ((DEBUG_INFO, "CalculateBestPioMode: AdvancedPioMode = %x\n", AdvancedPioMode));\r
 \r
     for (Index = 0; Index < 8; Index++) {\r
       if ((AdvancedPioMode & 0x01) != 0) {\r
@@ -140,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
@@ -148,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
@@ -171,7 +164,7 @@ CalculateBestPioMode (
       }\r
 \r
       if (PioMode >= *DisPioMode) {\r
-        PioMode = (UINT16) (*DisPioMode - 1);\r
+        PioMode = (UINT16)(*DisPioMode - 1);\r
       }\r
     }\r
 \r
@@ -180,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
@@ -201,7 +193,6 @@ CalculateBestPioMode (
     } else {\r
       *SelectedMode = 2;        // ATA_PIO_MODE_2;\r
     }\r
-\r
   }\r
 \r
   return EFI_SUCCESS;\r
@@ -225,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
@@ -238,9 +229,9 @@ CalculateBestUdmaMode (
   }\r
 \r
   DeviceUDmaMode = IdentifyData->AtaData.ultra_dma_mode;\r
-  DEBUG ((EFI_D_INFO, "CalculateBestUdmaMode: DeviceUDmaMode = %x\n", DeviceUDmaMode));\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
@@ -256,7 +247,7 @@ CalculateBestUdmaMode (
     }\r
 \r
     if (TempMode >= *DisUDmaMode) {\r
-      TempMode = (UINT16) (*DisUDmaMode - 1);\r
+      TempMode = (UINT16)(*DisUDmaMode - 1);\r
     }\r
   }\r
 \r
@@ -271,9 +262,9 @@ CalculateBestUdmaMode (
 /**\r
   The Entry Point of module. It follows the standard UEFI driver model.\r
 \r
-  @param[in] ImageHandle    The firmware allocated handle for the EFI image.  \r
+  @param[in] ImageHandle    The firmware allocated handle for the EFI image.\r
   @param[in] SystemTable    A pointer to the EFI System Table.\r
-  \r
+\r
   @retval EFI_SUCCESS   The entry point is executed successfully.\r
   @retval other         Some error occurs when executing this entry point.\r
 \r
@@ -281,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
@@ -320,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
@@ -335,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
@@ -367,7 +358,7 @@ SataControllerSupported (
 }\r
 \r
 /**\r
-  This routine is called right after the .Supported() called and \r
+  This routine is called right after the .Supported() called and\r
   Start this driver on ControllerHandle.\r
 \r
   @param This                   Protocol instance pointer.\r
@@ -383,11 +374,12 @@ 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
   EFI_STATUS                        Status;\r
   EFI_PCI_IO_PROTOCOL               *PciIo;\r
   UINT64                            OriginalPciAttributes;\r
@@ -396,8 +388,9 @@ SataControllerStart (
   UINT32                            Data32;\r
   UINTN                             ChannelDeviceCount;\r
 \r
-  DEBUG ((EFI_D_INFO, "SataControllerStart START\n"));\r
+  DEBUG ((DEBUG_INFO, "SataControllerStart START\n"));\r
 \r
+  BailLogMask     = DEBUG_ERROR;\r
   SataPrivateData = NULL;\r
 \r
   //\r
@@ -406,12 +399,21 @@ 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
                   );\r
   if (EFI_ERROR (Status)) {\r
+    if (Status == EFI_ALREADY_STARTED) {\r
+      //\r
+      // This is an expected condition for OpenProtocol() / BY_DRIVER, in a\r
+      // DriverBindingStart() member function; degrade the log mask to\r
+      // DEBUG_INFO.\r
+      //\r
+      BailLogMask = DEBUG_INFO;\r
+    }\r
+\r
     goto Bail;\r
   }\r
 \r
@@ -419,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
@@ -442,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
@@ -466,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
@@ -514,7 +525,7 @@ SataControllerStart (
     goto FreeIdentifyValid;\r
   }\r
 \r
-  DEBUG ((EFI_D_INFO, "SataControllerStart END status = %r\n", Status));\r
+  DEBUG ((DEBUG_INFO, "SataControllerStart END status = %r\n", Status));\r
   return Status;\r
 \r
 FreeIdentifyValid:\r
@@ -530,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
@@ -542,7 +557,11 @@ ClosePciIo:
          );\r
 \r
 Bail:\r
-  DEBUG ((EFI_D_ERROR, "SataControllerStart error return status = %r\n", Status));\r
+  DEBUG ((\r
+    BailLogMask,\r
+    "SataControllerStart error return status = %r\n",\r
+    Status\r
+    ));\r
   return Status;\r
 }\r
 \r
@@ -561,15 +580,17 @@ 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
   EFI_IDE_CONTROLLER_INIT_PROTOCOL  *IdeInit;\r
   EFI_SATA_CONTROLLER_PRIVATE_DATA  *SataPrivateData;\r
+  EFI_PCI_IO_PROTOCOL               *PciIo;\r
+  UINT64                            OriginalPciAttributes;\r
 \r
   //\r
   // Open the produced protocol\r
@@ -577,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
@@ -589,6 +610,9 @@ SataControllerStop (
   SataPrivateData = SATA_CONTROLLER_PRIVATE_DATA_FROM_THIS (IdeInit);\r
   ASSERT (SataPrivateData != NULL);\r
 \r
+  PciIo                 = SataPrivateData->PciIo;\r
+  OriginalPciAttributes = SataPrivateData->OriginalPciAttributes;\r
+\r
   //\r
   // Uninstall the IDE Controller Init Protocol from this instance\r
   //\r
@@ -602,28 +626,29 @@ SataControllerStop (
     return Status;\r
   }\r
 \r
-  if (SataPrivateData != NULL) {\r
-    if (SataPrivateData->DisqualifiedModes != NULL) {\r
-      FreePool (SataPrivateData->DisqualifiedModes);\r
-    }\r
-    if (SataPrivateData->IdentifyData != NULL) {\r
-      FreePool (SataPrivateData->IdentifyData);\r
-    }\r
-    if (SataPrivateData->IdentifyValid != NULL) {\r
-      FreePool (SataPrivateData->IdentifyValid);\r
-    }\r
-    FreePool (SataPrivateData);\r
+  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
   // Restore original PCI attributes\r
   //\r
-  SataPrivateData->PciIo->Attributes (\r
-                            SataPrivateData->PciIo,\r
-                            EfiPciIoAttributeOperationSet,\r
-                            SataPrivateData->OriginalPciAttributes,\r
-                            NULL\r
-                            );\r
+  PciIo->Attributes (\r
+           PciIo,\r
+           EfiPciIoAttributeOperationSet,\r
+           OriginalPciAttributes,\r
+           NULL\r
+           );\r
 \r
   //\r
   // Close protocols opened by Sata Controller driver\r
@@ -670,36 +695,37 @@ FlatDeviceIndex (
 //\r
 // Interface functions of IDE_CONTROLLER_INIT protocol\r
 //\r
+\r
 /**\r
   Returns the information about the specified IDE channel.\r
-  \r
+\r
   This function can be used to obtain information about a particular IDE channel.\r
-  The driver entity uses this information during the enumeration process. \r
-  \r
-  If Enabled is set to FALSE, the driver entity will not scan the channel. Note \r
+  The driver entity uses this information during the enumeration process.\r
+\r
+  If Enabled is set to FALSE, the driver entity will not scan the channel. Note\r
   that it will not prevent an operating system driver from scanning the channel.\r
-  \r
-  For most of today's controllers, MaxDevices will either be 1 or 2. For SATA \r
-  controllers, this value will always be 1. SATA configurations can contain SATA \r
+\r
+  For most of today's controllers, MaxDevices will either be 1 or 2. For SATA\r
+  controllers, this value will always be 1. SATA configurations can contain SATA\r
   port multipliers. SATA port multipliers behave like SATA bridges and can support\r
-  up to 16 devices on the other side. If a SATA port out of the IDE controller \r
-  is connected to a port multiplier, MaxDevices will be set to the number of SATA \r
-  devices that the port multiplier supports. Because today's port multipliers \r
-  support up to fifteen SATA devices, this number can be as large as fifteen. The IDE  \r
-  bus driver is required to scan for the presence of port multipliers behind an SATA \r
-  controller and enumerate up to MaxDevices number of devices behind the port \r
-  multiplier.    \r
-  \r
-  In this context, the devices behind a port multiplier constitute a channel.  \r
-  \r
+  up to 16 devices on the other side. If a SATA port out of the IDE controller\r
+  is connected to a port multiplier, MaxDevices will be set to the number of SATA\r
+  devices that the port multiplier supports. Because today's port multipliers\r
+  support up to fifteen SATA devices, this number can be as large as fifteen. The IDE\r
+  bus driver is required to scan for the presence of port multipliers behind an SATA\r
+  controller and enumerate up to MaxDevices number of devices behind the port\r
+  multiplier.\r
+\r
+  In this context, the devices behind a port multiplier constitute a channel.\r
+\r
   @param[in]  This         The pointer to the EFI_IDE_CONTROLLER_INIT_PROTOCOL instance.\r
   @param[in]  Channel      Zero-based channel number.\r
-  @param[out] Enabled      TRUE if this channel is enabled.  Disabled channels \r
+  @param[out] Enabled      TRUE if this channel is enabled.  Disabled channels\r
                            are not scanned to see if any devices are present.\r
   @param[out] MaxDevices   The maximum number of IDE devices that the bus driver\r
-                           can expect on this channel.  For the ATA/ATAPI \r
-                           specification, version 6, this number will either be \r
-                           one or two. For Serial ATA (SATA) configurations with a \r
+                           can expect on this channel.  For the ATA/ATAPI\r
+                           specification, version 6, this number will either be\r
+                           one or two. For Serial ATA (SATA) configurations with a\r
                            port multiplier, this number can be as large as fifteen.\r
 \r
   @retval EFI_SUCCESS             Information was returned without any errors.\r
@@ -709,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
@@ -732,13 +759,13 @@ IdeInitGetChannelInfo (
 /**\r
   The notifications from the driver entity that it is about to enter a certain\r
   phase of the IDE channel enumeration process.\r
-  \r
-  This function can be used to notify the IDE controller driver to perform \r
-  specific actions, including any chipset-specific initialization, so that the \r
-  chipset is ready to enter the next phase. Seven notification points are defined \r
-  at this time. \r
-  \r
-  More synchronization points may be added as required in the future.  \r
+\r
+  This function can be used to notify the IDE controller driver to perform\r
+  specific actions, including any chipset-specific initialization, so that the\r
+  chipset is ready to enter the next phase. Seven notification points are defined\r
+  at this time.\r
+\r
+  More synchronization points may be added as required in the future.\r
 \r
   @param[in] This      The pointer to the EFI_IDE_CONTROLLER_INIT_PROTOCOL instance.\r
   @param[in] Phase     The phase during enumeration.\r
@@ -747,18 +774,18 @@ IdeInitGetChannelInfo (
   @retval EFI_SUCCESS             The notification was accepted without any errors.\r
   @retval EFI_UNSUPPORTED         Phase is not supported.\r
   @retval EFI_INVALID_PARAMETER   Channel is invalid (Channel >= ChannelCount).\r
-  @retval EFI_NOT_READY           This phase cannot be entered at this time; for \r
-                                  example, an attempt was made to enter a Phase \r
-                                  without having entered one or more previous \r
+  @retval EFI_NOT_READY           This phase cannot be entered at this time; for\r
+                                  example, an attempt was made to enter a Phase\r
+                                  without having entered one or more previous\r
                                   Phase.\r
 \r
 **/\r
 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
@@ -767,32 +794,32 @@ IdeInitNotifyPhase (
 /**\r
   Submits the device information to the IDE controller driver.\r
 \r
-  This function is used by the driver entity to pass detailed information about \r
-  a particular device to the IDE controller driver. The driver entity obtains \r
+  This function is used by the driver entity to pass detailed information about\r
+  a particular device to the IDE controller driver. The driver entity obtains\r
   this information by issuing an ATA or ATAPI IDENTIFY_DEVICE command. IdentifyData\r
-  is the pointer to the response data buffer. The IdentifyData buffer is owned \r
-  by the driver entity, and the IDE controller driver must make a local copy \r
-  of the entire buffer or parts of the buffer as needed. The original IdentifyData \r
+  is the pointer to the response data buffer. The IdentifyData buffer is owned\r
+  by the driver entity, and the IDE controller driver must make a local copy\r
+  of the entire buffer or parts of the buffer as needed. The original IdentifyData\r
   buffer pointer may not be valid when\r
-  \r
+\r
     - EFI_IDE_CONTROLLER_INIT_PROTOCOL.CalculateMode() or\r
     - EFI_IDE_CONTROLLER_INIT_PROTOCOL.DisqualifyMode() is called at a later point.\r
-    \r
-  The IDE controller driver may consult various fields of EFI_IDENTIFY_DATA to \r
-  compute the optimum mode for the device. These fields are not limited to the \r
-  timing information. For example, an implementation of the IDE controller driver \r
-  may examine the vendor and type/mode field to match known bad drives.  \r
-  \r
-  The driver entity may submit drive information in any order, as long as it \r
-  submits information for all the devices belonging to the enumeration group \r
+\r
+  The IDE controller driver may consult various fields of EFI_IDENTIFY_DATA to\r
+  compute the optimum mode for the device. These fields are not limited to the\r
+  timing information. For example, an implementation of the IDE controller driver\r
+  may examine the vendor and type/mode field to match known bad drives.\r
+\r
+  The driver entity may submit drive information in any order, as long as it\r
+  submits information for all the devices belonging to the enumeration group\r
   before EFI_IDE_CONTROLLER_INIT_PROTOCOL.CalculateMode() is called for any device\r
   in that enumeration group. If a device is absent, EFI_IDE_CONTROLLER_INIT_PROTOCOL.SubmitData()\r
-  should be called with IdentifyData set to NULL.  The IDE controller driver may \r
-  not have any other mechanism to know whether a device is present or not. Therefore, \r
-  setting IdentifyData to NULL does not constitute an error condition. \r
-  EFI_IDE_CONTROLLER_INIT_PROTOCOL.SubmitData() can be called only once for a \r
-  given (Channel, Device) pair.  \r
-    \r
+  should be called with IdentifyData set to NULL.  The IDE controller driver may\r
+  not have any other mechanism to know whether a device is present or not. Therefore,\r
+  setting IdentifyData to NULL does not constitute an error condition.\r
+  EFI_IDE_CONTROLLER_INIT_PROTOCOL.SubmitData() can be called only once for a\r
+  given (Channel, Device) pair.\r
+\r
   @param[in] This           A pointer to the EFI_IDE_CONTROLLER_INIT_PROTOCOL instance.\r
   @param[in] Channel        Zero-based channel number.\r
   @param[in] Device         Zero-based device number on the Channel.\r
@@ -806,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
@@ -845,31 +872,31 @@ IdeInitSubmitData (
 /**\r
   Disqualifies specific modes for an IDE device.\r
 \r
-  This function allows the driver entity or other drivers (such as platform \r
+  This function allows the driver entity or other drivers (such as platform\r
   drivers) to reject certain timing modes and request the IDE controller driver\r
-  to recalculate modes. This function allows the driver entity and the IDE \r
-  controller driver to negotiate the timings on a per-device basis. This function \r
-  is useful in the case of drives that lie about their capabilities. An example \r
-  is when the IDE device fails to accept the timing modes that are calculated \r
+  to recalculate modes. This function allows the driver entity and the IDE\r
+  controller driver to negotiate the timings on a per-device basis. This function\r
+  is useful in the case of drives that lie about their capabilities. An example\r
+  is when the IDE device fails to accept the timing modes that are calculated\r
   by the IDE controller driver based on the response to the Identify Drive command.\r
 \r
-  If the driver entity does not want to limit the ATA timing modes and leave that \r
-  decision to the IDE controller driver, it can either not call this function for \r
-  the given device or call this function and set the Valid flag to FALSE for all \r
+  If the driver entity does not want to limit the ATA timing modes and leave that\r
+  decision to the IDE controller driver, it can either not call this function for\r
+  the given device or call this function and set the Valid flag to FALSE for all\r
   modes that are listed in EFI_ATA_COLLECTIVE_MODE.\r
-  \r
-  The driver entity may disqualify modes for a device in any order and any number \r
+\r
+  The driver entity may disqualify modes for a device in any order and any number\r
   of times.\r
-  \r
-  This function can be called multiple times to invalidate multiple modes of the \r
-  same type (e.g., Programmed Input/Output [PIO] modes 3 and 4). See the ATA/ATAPI \r
-  specification for more information on PIO modes.  \r
-  \r
+\r
+  This function can be called multiple times to invalidate multiple modes of the\r
+  same type (e.g., Programmed Input/Output [PIO] modes 3 and 4). See the ATA/ATAPI\r
+  specification for more information on PIO modes.\r
+\r
   For Serial ATA (SATA) controllers, this member function can be used to disqualify\r
   a higher transfer rate mode on a given channel. For example, a platform driver\r
-  may inform the IDE controller driver to not use second-generation (Gen2) speeds \r
+  may inform the IDE controller driver to not use second-generation (Gen2) speeds\r
   for a certain SATA drive.\r
-  \r
+\r
   @param[in] This       The pointer to the EFI_IDE_CONTROLLER_INIT_PROTOCOL instance.\r
   @param[in] Channel    The zero-based channel number.\r
   @param[in] Device     The zero-based device number on the Channel.\r
@@ -880,15 +907,15 @@ IdeInitSubmitData (
   @retval EFI_INVALID_PARAMETER   Channel is invalid (Channel >= ChannelCount).\r
   @retval EFI_INVALID_PARAMETER   Device is invalid.\r
   @retval EFI_INVALID_PARAMETER   IdentifyData is NULL.\r
-                                \r
+\r
 **/\r
 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
@@ -920,39 +947,39 @@ IdeInitDisqualifyMode (
   Returns the information about the optimum modes for the specified IDE device.\r
 \r
   This function is used by the driver entity to obtain the optimum ATA modes for\r
-  a specific device.  The IDE controller driver takes into account the following \r
+  a specific device.  The IDE controller driver takes into account the following\r
   while calculating the mode:\r
     - The IdentifyData inputs to EFI_IDE_CONTROLLER_INIT_PROTOCOL.SubmitData()\r
     - The BadModes inputs to EFI_IDE_CONTROLLER_INIT_PROTOCOL.DisqualifyMode()\r
 \r
-  The driver entity is required to call EFI_IDE_CONTROLLER_INIT_PROTOCOL.SubmitData() \r
-  for all the devices that belong to an enumeration group before calling \r
-  EFI_IDE_CONTROLLER_INIT_PROTOCOL.CalculateMode() for any device in the same group.  \r
-  \r
-  The IDE controller driver will use controller- and possibly platform-specific \r
-  algorithms to arrive at SupportedModes.  The IDE controller may base its \r
-  decision on user preferences and other considerations as well. This function \r
-  may be called multiple times because the driver entity may renegotiate the mode \r
+  The driver entity is required to call EFI_IDE_CONTROLLER_INIT_PROTOCOL.SubmitData()\r
+  for all the devices that belong to an enumeration group before calling\r
+  EFI_IDE_CONTROLLER_INIT_PROTOCOL.CalculateMode() for any device in the same group.\r
+\r
+  The IDE controller driver will use controller- and possibly platform-specific\r
+  algorithms to arrive at SupportedModes.  The IDE controller may base its\r
+  decision on user preferences and other considerations as well. This function\r
+  may be called multiple times because the driver entity may renegotiate the mode\r
   with the IDE controller driver using EFI_IDE_CONTROLLER_INIT_PROTOCOL.DisqualifyMode().\r
-    \r
-  The driver entity may collect timing information for various devices in any \r
+\r
+  The driver entity may collect timing information for various devices in any\r
   order. The driver entity is responsible for making sure that all the dependencies\r
-  are satisfied. For example, the SupportedModes information for device A that \r
-  was previously returned may become stale after a call to \r
+  are satisfied. For example, the SupportedModes information for device A that\r
+  was previously returned may become stale after a call to\r
   EFI_IDE_CONTROLLER_INIT_PROTOCOL.DisqualifyMode() for device B.\r
-  \r
-  The buffer SupportedModes is allocated by the callee because the caller does \r
-  not necessarily know the size of the buffer. The type EFI_ATA_COLLECTIVE_MODE \r
-  is defined in a way that allows for future extensibility and can be of variable \r
-  length. This memory pool should be deallocated by the caller when it is no \r
-  longer necessary.  \r
-  \r
-  The IDE controller driver for a Serial ATA (SATA) controller can use this \r
-  member function to force a lower speed (first-generation [Gen1] speeds on a \r
-  second-generation [Gen2]-capable hardware).  The IDE controller driver can \r
-  also allow the driver entity to stay with the speed that has been negotiated \r
+\r
+  The buffer SupportedModes is allocated by the callee because the caller does\r
+  not necessarily know the size of the buffer. The type EFI_ATA_COLLECTIVE_MODE\r
+  is defined in a way that allows for future extensibility and can be of variable\r
+  length. This memory pool should be deallocated by the caller when it is no\r
+  longer necessary.\r
+\r
+  The IDE controller driver for a Serial ATA (SATA) controller can use this\r
+  member function to force a lower speed (first-generation [Gen1] speeds on a\r
+  second-generation [Gen2]-capable hardware).  The IDE controller driver can\r
+  also allow the driver entity to stay with the speed that has been negotiated\r
   by the physical layer.\r
-  \r
+\r
   @param[in]  This             The pointer to the EFI_IDE_CONTROLLER_INIT_PROTOCOL instance.\r
   @param[in]  Channel          A zero-based channel number.\r
   @param[in]  Device           A zero-based device number on the Channel.\r
@@ -960,23 +987,23 @@ IdeInitDisqualifyMode (
 \r
   @retval EFI_SUCCESS             SupportedModes was returned.\r
   @retval EFI_INVALID_PARAMETER   Channel is invalid (Channel >= ChannelCount).\r
-  @retval EFI_INVALID_PARAMETER   Device is invalid. \r
+  @retval EFI_INVALID_PARAMETER   Device is invalid.\r
   @retval EFI_INVALID_PARAMETER   SupportedModes is NULL.\r
-  @retval EFI_NOT_READY           Modes cannot be calculated due to a lack of \r
-                                  data.  This error may happen if \r
-                                  EFI_IDE_CONTROLLER_INIT_PROTOCOL.SubmitData() \r
-                                  and EFI_IDE_CONTROLLER_INIT_PROTOCOL.DisqualifyData() \r
-                                  were not called for at least one drive in the \r
+  @retval EFI_NOT_READY           Modes cannot be calculated due to a lack of\r
+                                  data.  This error may happen if\r
+                                  EFI_IDE_CONTROLLER_INIT_PROTOCOL.SubmitData()\r
+                                  and EFI_IDE_CONTROLLER_INIT_PROTOCOL.DisqualifyData()\r
+                                  were not called for at least one drive in the\r
                                   same enumeration group.\r
 \r
 **/\r
 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
@@ -1002,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
@@ -1015,33 +1042,33 @@ 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
-  DEBUG ((EFI_D_INFO, "IdeInitCalculateMode: PioMode = %x\n", (*SupportedModes)->PioMode.Mode));\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
-  DEBUG ((EFI_D_INFO, "IdeInitCalculateMode: UdmaMode = %x\n", (*SupportedModes)->UdmaMode.Mode));\r
+\r
+  DEBUG ((DEBUG_INFO, "IdeInitCalculateMode: UdmaMode = %x\n", (*SupportedModes)->UdmaMode.Mode));\r
 \r
   //\r
   // The modes other than PIO and UDMA are not supported\r
@@ -1053,9 +1080,9 @@ IdeInitCalculateMode (
   Commands the IDE controller driver to program the IDE controller hardware\r
   so that the specified device can operate at the specified mode.\r
 \r
-  This function is used by the driver entity to instruct the IDE controller \r
-  driver to program the IDE controller hardware to the specified modes. This \r
-  function can be called only once for a particular device. For a Serial ATA \r
+  This function is used by the driver entity to instruct the IDE controller\r
+  driver to program the IDE controller hardware to the specified modes. This\r
+  function can be called only once for a particular device. For a Serial ATA\r
   (SATA) Advanced Host Controller Interface (AHCI) controller, no controller-\r
   specific programming may be required.\r
 \r
@@ -1075,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