X-Git-Url: https://git.proxmox.com/?a=blobdiff_plain;f=IntelFrameworkModulePkg%2FBus%2FPci%2FIdeBusDxe%2FIde.h;h=bb2a4a75c3f536fb95ea4b048a1b684227ad287c;hb=75eccf9d4a1895f33ba1ada0ab9873f633e87031;hp=ec1f9e07a0917eda1fcc078a28284f04cb0d86ff;hpb=b0af4b695d94a4c35b86796446baabfa84575483;p=mirror_edk2.git

diff --git a/IntelFrameworkModulePkg/Bus/Pci/IdeBusDxe/Ide.h b/IntelFrameworkModulePkg/Bus/Pci/IdeBusDxe/Ide.h
index ec1f9e07a0..bb2a4a75c3 100644
--- a/IntelFrameworkModulePkg/Bus/Pci/IdeBusDxe/Ide.h
+++ b/IntelFrameworkModulePkg/Bus/Pci/IdeBusDxe/Ide.h
@@ -18,8 +18,8 @@
 
 **/
 
-#ifndef _IDE_H
-#define _IDE_H
+#ifndef _IDE_H_
+#define _IDE_H_
 
 //
 // Helper functions Prototype
@@ -31,7 +31,7 @@
   @param  Controller TODO: add argument description
   @param  Handle TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -39,8 +39,7 @@ DeRegisterIdeDevice (
   IN  EFI_DRIVER_BINDING_PROTOCOL    *This,
   IN  EFI_HANDLE                     Controller,
   IN  EFI_HANDLE                     Handle
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -50,7 +49,7 @@ DeRegisterIdeDevice (
   @param  ParentDevicePath TODO: add argument description
   @param  RemainingDevicePath TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -59,8 +58,7 @@ EnableIdeDevice (
   IN EFI_PCI_IO_PROTOCOL                 *PciIo,
   IN EFI_DEVICE_PATH_PROTOCOL            *ParentDevicePath,
   IN EFI_DEVICE_PATH_PROTOCOL            *RemainingDevicePath
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -68,15 +66,14 @@ EnableIdeDevice (
   @param  PciIo TODO: add argument description
   @param  Port TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 UINT8
 IDEReadPortB (
   IN  EFI_PCI_IO_PROTOCOL   *PciIo,
   IN  UINT16                Port
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -86,7 +83,7 @@ IDEReadPortB (
   @param  Count TODO: add argument description
   @param  Buffer TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 VOID
@@ -95,8 +92,7 @@ IDEReadPortWMultiple (
   IN  UINT16                Port,
   IN  UINTN                 Count,
   OUT  VOID                 *Buffer
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -105,7 +101,7 @@ IDEReadPortWMultiple (
   @param  Port TODO: add argument description
   @param  Data TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 VOID
@@ -113,8 +109,7 @@ IDEWritePortB (
   IN  EFI_PCI_IO_PROTOCOL   *PciIo,
   IN  UINT16                Port,
   IN  UINT8                 Data
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -123,7 +118,7 @@ IDEWritePortB (
   @param  Port TODO: add argument description
   @param  Data TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 VOID
@@ -131,8 +126,7 @@ IDEWritePortW (
   IN  EFI_PCI_IO_PROTOCOL   *PciIo,
   IN  UINT16                Port,
   IN  UINT16                Data
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -142,7 +136,7 @@ IDEWritePortW (
   @param  Count TODO: add argument description
   @param  Buffer TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 VOID
@@ -151,8 +145,7 @@ IDEWritePortWMultiple (
   IN  UINT16                Port,
   IN  UINTN                 Count,
   IN  VOID                  *Buffer
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -160,43 +153,40 @@ IDEWritePortWMultiple (
   @param  PciIo TODO: add argument description
   @param  IdeRegsBaseAddr TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 GetIdeRegistersBaseAddr (
   IN  EFI_PCI_IO_PROTOCOL         *PciIo,
   OUT IDE_REGISTERS_BASE_ADDR     *IdeRegsBaseAddr
-  )
-;
+  );
 
 /**
   TODO: Add function description
 
   @param  IdeDev TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 ReassignIdeResources (
   IN  IDE_BLK_IO_DEV  *IdeDev
-  )
-;
+  );
 
 /**
   TODO: Add function description
 
   @param  IdeDev TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 DiscoverIdeDevice (
   IN IDE_BLK_IO_DEV *IdeDev
-  )
-;
+  );
 
 /**
   This interface is used to initialize all state data related to the
@@ -208,22 +198,20 @@ DiscoverIdeDevice (
 EFI_STATUS
 InitializeIDEChannelData (
   VOID
-  )
-;
+  );
 
 /**
   TODO: Add function description
 
   @param  IdeDev TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 DetectIDEController (
   IN  IDE_BLK_IO_DEV  *IdeDev
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -231,15 +219,14 @@ DetectIDEController (
   @param  IdeDev TODO: add argument description
   @param  TimeoutInMilliSeconds TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 DRQClear (
   IN  IDE_BLK_IO_DEV  *IdeDev,
   IN  UINTN           TimeoutInMilliSeconds
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -247,15 +234,14 @@ DRQClear (
   @param  IdeDev TODO: add argument description
   @param  TimeoutInMilliSeconds TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 DRQClear2 (
   IN  IDE_BLK_IO_DEV  *IdeDev,
   IN  UINTN           TimeoutInMilliSeconds
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -263,15 +249,14 @@ DRQClear2 (
   @param  IdeDev TODO: add argument description
   @param  TimeoutInMilliSeconds TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 DRQReady (
   IN  IDE_BLK_IO_DEV  *IdeDev,
   IN  UINTN           TimeoutInMilliSeconds
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -279,15 +264,14 @@ DRQReady (
   @param  IdeDev TODO: add argument description
   @param  TimeoutInMilliSeconds TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 DRQReady2 (
   IN  IDE_BLK_IO_DEV  *IdeDev,
   IN  UINTN           TimeoutInMilliSeconds
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -295,15 +279,14 @@ DRQReady2 (
   @param  IdeDev TODO: add argument description
   @param  TimeoutInMilliSeconds TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 WaitForBSYClear (
   IN  IDE_BLK_IO_DEV  *IdeDev,
   IN  UINTN           TimeoutInMilliSeconds
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -311,15 +294,14 @@ WaitForBSYClear (
   @param  IdeDev TODO: add argument description
   @param  TimeoutInMilliSeconds TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 WaitForBSYClear2 (
   IN  IDE_BLK_IO_DEV  *IdeDev,
   IN  UINTN           TimeoutInMilliSeconds
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -327,15 +309,14 @@ WaitForBSYClear2 (
   @param  IdeDev TODO: add argument description
   @param  DelayInMilliSeconds TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 DRDYReady (
   IN  IDE_BLK_IO_DEV  *IdeDev,
   IN  UINTN           DelayInMilliSeconds
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -343,15 +324,14 @@ DRDYReady (
   @param  IdeDev TODO: add argument description
   @param  DelayInMilliSeconds TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 DRDYReady2 (
   IN  IDE_BLK_IO_DEV  *IdeDev,
   IN  UINTN           DelayInMilliSeconds
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -360,7 +340,7 @@ DRDYReady2 (
   @param  Source TODO: add argument description
   @param  Size TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 VOID
@@ -368,8 +348,7 @@ SwapStringChars (
   IN CHAR8  *Destination,
   IN CHAR8  *Source,
   IN UINT32 Size
-  )
-;
+  );
 
 //
 //  ATA device functions' prototype
@@ -379,28 +358,26 @@ SwapStringChars (
 
   @param  IdeDev TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 ATAIdentify (
   IN  IDE_BLK_IO_DEV  *IdeDev
-  )
-;
+  );
 
 /**
   TODO: Add function description
 
   @param  IdeDev TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 VOID
 PrintAtaModuleName (
   IN  IDE_BLK_IO_DEV  *IdeDev
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -415,7 +392,7 @@ PrintAtaModuleName (
   @param  CylinderLsb TODO: add argument description
   @param  CylinderMsb TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -429,8 +406,7 @@ AtaPioDataIn (
   IN  UINT8           SectorNumber,
   IN  UINT8           CylinderLsb,
   IN  UINT8           CylinderMsb
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -445,7 +421,7 @@ AtaPioDataIn (
   @param  CylinderLsb TODO: add argument description
   @param  CylinderMsb TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -459,22 +435,20 @@ AtaPioDataOut (
   IN  UINT8           SectorNumber,
   IN  UINT8           CylinderLsb,
   IN  UINT8           CylinderMsb
-  )
-;
+  );
 
 /**
   TODO: Add function description
 
   @param  IdeDev TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 CheckErrorStatus (
   IN  IDE_BLK_IO_DEV  *IdeDev
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -484,7 +458,7 @@ CheckErrorStatus (
   @param  Lba TODO: add argument description
   @param  NumberOfBlocks TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -493,8 +467,7 @@ AtaReadSectors (
   IN  VOID            *DataBuffer,
   IN  EFI_LBA         Lba,
   IN  UINTN           NumberOfBlocks
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -504,7 +477,7 @@ AtaReadSectors (
   @param  Lba TODO: add argument description
   @param  NumberOfBlocks TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -513,22 +486,20 @@ AtaWriteSectors (
   IN  VOID            *BufferData,
   IN  EFI_LBA         Lba,
   IN  UINTN           NumberOfBlocks
-  )
-;
+  );
 
 /**
   TODO: Add function description
 
   @param  IdeDev TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 AtaSoftReset (
   IN  IDE_BLK_IO_DEV  *IdeDev
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -539,7 +510,7 @@ AtaSoftReset (
   @param  BufferSize TODO: add argument description
   @param  Buffer TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -549,8 +520,7 @@ AtaBlkIoReadBlocks (
   IN EFI_LBA          LBA,
   IN UINTN            BufferSize,
   OUT VOID            *Buffer
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -561,7 +531,7 @@ AtaBlkIoReadBlocks (
   @param  BufferSize TODO: add argument description
   @param  Buffer TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -571,8 +541,7 @@ AtaBlkIoWriteBlocks (
   IN EFI_LBA          LBA,
   IN UINTN            BufferSize,
   OUT VOID            *Buffer
-  )
-;
+  );
 
 //
 // ATAPI device functions' prototype
@@ -582,28 +551,26 @@ AtaBlkIoWriteBlocks (
 
   @param  IdeDev TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 ATAPIIdentify (
   IN  IDE_BLK_IO_DEV  *IdeDev
-  )
-;
+  );
 
 /**
   TODO: Add function description
 
   @param  IdeDev TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 AtapiInquiry (
   IN  IDE_BLK_IO_DEV  *IdeDev
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -614,7 +581,7 @@ AtapiInquiry (
   @param  ByteCount TODO: add argument description
   @param  TimeOut TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -624,8 +591,7 @@ AtapiPacketCommandIn (
   IN  UINT16                *Buffer,
   IN  UINT32                ByteCount,
   IN  UINTN                 TimeOut
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -636,7 +602,7 @@ AtapiPacketCommandIn (
   @param  ByteCount TODO: add argument description
   @param  TimeOut TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -646,8 +612,7 @@ AtapiPacketCommandOut (
   IN  UINT16                *Buffer,
   IN  UINT32                ByteCount,
   IN  UINTN                 TimeOut
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -658,7 +623,7 @@ AtapiPacketCommandOut (
   @param  Read TODO: add argument description
   @param  TimeOut TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -668,24 +633,22 @@ PioReadWriteData (
   IN  UINT32          ByteCount,
   IN  BOOLEAN         Read,
   IN  UINTN           TimeOut
-  )
-;
+  );
 
 /**
   TODO: Add function description
 
-  @param  IdeDev TODO: add argument description
-  @param  IdeDev TODO: add argument description
+  @param  IdeDev  TODO: add argument description
+  @param  SResult TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 AtapiTestUnitReady (
   IN  IDE_BLK_IO_DEV  *IdeDev,
-  OUT UINTN           *SenseCount
-  )
-;
+  OUT SENSE_RESULT	  *SResult  
+  );
 
 /**
   TODO: Add function description
@@ -693,31 +656,29 @@ AtapiTestUnitReady (
   @param  IdeDev TODO: add argument description
   @param  SenseCounts TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 AtapiRequestSense (
   IN  IDE_BLK_IO_DEV  *IdeDev,
   OUT UINTN           *SenseCounts
-  )
-;
+  );
 
 /**
   TODO: Add function description
 
-  @param  IdeDev TODO: add argument description
-  @param  IdeDev TODO: add argument description
+  @param  IdeDev  TODO: add argument description
+  @param  SResult TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 AtapiReadCapacity (
   IN  IDE_BLK_IO_DEV  *IdeDev,
-  OUT UINTN           *SenseCount
-  )
-;
+  OUT SENSE_RESULT	  *SResult  
+  );
 
 /**
   TODO: Add function description
@@ -725,15 +686,14 @@ AtapiReadCapacity (
   @param  IdeDev TODO: add argument description
   @param  MediaChange TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 AtapiDetectMedia (
   IN  IDE_BLK_IO_DEV  *IdeDev,
   OUT BOOLEAN         *MediaChange
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -743,7 +703,7 @@ AtapiDetectMedia (
   @param  Lba TODO: add argument description
   @param  NumberOfBlocks TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -752,8 +712,7 @@ AtapiReadSectors (
   IN  VOID            *Buffer,
   IN  EFI_LBA         Lba,
   IN  UINTN           NumberOfBlocks
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -763,7 +722,7 @@ AtapiReadSectors (
   @param  Lba TODO: add argument description
   @param  NumberOfBlocks TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -772,22 +731,20 @@ AtapiWriteSectors (
   IN  VOID            *Buffer,
   IN  EFI_LBA         Lba,
   IN  UINTN           NumberOfBlocks
-  )
-;
+  );
 
 /**
   TODO: Add function description
 
   @param  IdeDev TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 AtapiSoftReset (
   IN  IDE_BLK_IO_DEV  *IdeDev
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -798,7 +755,7 @@ AtapiSoftReset (
   @param  BufferSize TODO: add argument description
   @param  Buffer TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -808,8 +765,7 @@ AtapiBlkIoReadBlocks (
   IN EFI_LBA          LBA,
   IN UINTN            BufferSize,
   OUT VOID            *Buffer
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -820,7 +776,7 @@ AtapiBlkIoReadBlocks (
   @param  BufferSize TODO: add argument description
   @param  Buffer TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -830,8 +786,7 @@ AtapiBlkIoWriteBlocks (
   IN EFI_LBA          LBA,
   IN UINTN            BufferSize,
   OUT VOID            *Buffer
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -840,7 +795,7 @@ AtapiBlkIoWriteBlocks (
   @param  SenseCount TODO: add argument description
   @param  Result TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -848,22 +803,20 @@ ParseSenseData (
   IN IDE_BLK_IO_DEV     *IdeDev,
   IN UINTN              SenseCount,
   OUT SENSE_RESULT      *Result
-  )
-;
+  );
 
 /**
   TODO: Add function description
 
   @param  IdeDev TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 AtapiReadPendingData (
   IN IDE_BLK_IO_DEV     *IdeDev
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -871,29 +824,27 @@ AtapiReadPendingData (
   @param  IdeDev TODO: add argument description
   @param  WriteProtected TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 IsLS120orZipWriteProtected (
   IN  IDE_BLK_IO_DEV    *IdeDev,
   OUT BOOLEAN           *WriteProtected
-  )
-;
+  );
 
 /**
   TODO: Add function description
 
   @param  IdeBlkIoDevice TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 VOID
 ReleaseIdeResources (
   IN  IDE_BLK_IO_DEV  *IdeBlkIoDevice
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -901,15 +852,14 @@ ReleaseIdeResources (
   @param  IdeDev TODO: add argument description
   @param  TransferMode TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 SetDeviceTransferMode (
   IN IDE_BLK_IO_DEV       *IdeDev,
   IN ATA_TRANSFER_MODE    *TransferMode
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -917,15 +867,14 @@ SetDeviceTransferMode (
   @param  IdeDev TODO: add argument description
   @param  NativeMaxAddress TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 ReadNativeMaxAddress (
   IN  IDE_BLK_IO_DEV                *IdeDev,
   OUT EFI_LBA                       *NativeMaxAddress
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -934,7 +883,7 @@ ReadNativeMaxAddress (
   @param  MaxAddress TODO: add argument description
   @param  bVolatile TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -942,8 +891,7 @@ SetMaxAddress (
   IN  IDE_BLK_IO_DEV                *IdeDev,
   IN  EFI_LBA                       MaxAddress,
   IN  BOOLEAN                       bVolatile
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -957,7 +905,7 @@ SetMaxAddress (
   @param  LbaMiddle TODO: add argument description
   @param  LbaHigh TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -970,8 +918,7 @@ AtaNonDataCommandIn (
   IN  UINT8           LbaLow,
   IN  UINT8           LbaMiddle,
   IN  UINT8           LbaHigh
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -983,7 +930,7 @@ AtaNonDataCommandIn (
   @param  SectorCount TODO: add argument description
   @param  LbaAddress TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -994,8 +941,7 @@ AtaNonDataCommandInExt (
   IN  UINT16          Feature,
   IN  UINT16          SectorCount,
   IN  EFI_LBA         LbaAddress
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -1005,7 +951,7 @@ AtaNonDataCommandInExt (
   @param  StartLba TODO: add argument description
   @param  NumberOfBlocks TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -1014,8 +960,7 @@ AtaReadSectorsExt (
   IN  VOID            *DataBuffer,
   IN  EFI_LBA         StartLba,
   IN  UINTN           NumberOfBlocks
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -1025,7 +970,7 @@ AtaReadSectorsExt (
   @param  StartLba TODO: add argument description
   @param  NumberOfBlocks TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -1034,8 +979,7 @@ AtaWriteSectorsExt (
   IN  VOID            *DataBuffer,
   IN  EFI_LBA         StartLba,
   IN  UINTN           NumberOfBlocks
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -1045,7 +989,7 @@ AtaWriteSectorsExt (
   @param  StartLba TODO: add argument description
   @param  NumberOfBlocks TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -1054,8 +998,7 @@ AtaUdmaReadExt (
   IN  VOID            *DataBuffer,
   IN  EFI_LBA         StartLba,
   IN  UINTN           NumberOfBlocks
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -1065,7 +1008,7 @@ AtaUdmaReadExt (
   @param  StartLba TODO: add argument description
   @param  NumberOfBlocks TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -1074,8 +1017,7 @@ AtaUdmaRead (
   IN  VOID            *DataBuffer,
   IN  EFI_LBA         StartLba,
   IN  UINTN           NumberOfBlocks
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -1085,7 +1027,7 @@ AtaUdmaRead (
   @param  StartLba TODO: add argument description
   @param  NumberOfBlocks TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -1094,8 +1036,7 @@ AtaUdmaWriteExt (
   IN  VOID            *DataBuffer,
   IN  EFI_LBA         StartLba,
   IN  UINTN           NumberOfBlocks
-  )
-;
+  );
 
 /**
   Perform an ATA Udma operation (Read, ReadExt, Write, WriteExt).
@@ -1129,8 +1070,7 @@ DoAtaUdma (
   IN  EFI_LBA             StartLba,
   IN  UINTN               NumberOfBlocks,
   IN  ATA_UDMA_OPERATION  UdmaOp
-  )
-;
+  );
 
 
 /**
@@ -1141,7 +1081,7 @@ DoAtaUdma (
   @param  StartLba TODO: add argument description
   @param  NumberOfBlocks TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -1150,8 +1090,7 @@ AtaUdmaWrite (
   IN  VOID            *DataBuffer,
   IN  EFI_LBA         StartLba,
   IN  UINTN           NumberOfBlocks
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -1163,7 +1102,7 @@ AtaUdmaWrite (
   @param  SectorCount TODO: add argument description
   @param  LbaAddress TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -1174,8 +1113,7 @@ AtaCommandIssueExt (
   IN  UINT16          Feature,
   IN  UINT16          SectorCount,
   IN  EFI_LBA         LbaAddress
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -1187,7 +1125,7 @@ AtaCommandIssueExt (
   @param  SectorCount TODO: add argument description
   @param  LbaAddress TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -1198,22 +1136,20 @@ AtaCommandIssue (
   IN  UINT16          Feature,
   IN  UINT16          SectorCount,
   IN  EFI_LBA         LbaAddress
-  )
-;
+  );
 
 /**
   TODO: Add function description
 
   @param  IdeDev TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 AtaAtapi6Identify (
   IN  IDE_BLK_IO_DEV  *IdeDev
-  )
-;
+  );
 
 
 /**
@@ -1221,14 +1157,29 @@ AtaAtapi6Identify (
 
   @param  IdeDev TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 VOID
 AtaSMARTSupport (
   IN  IDE_BLK_IO_DEV  *IdeDev
-  )
-;
+  );
+
+
+/**
+  Enable Long Physical Sector Feature for ATA device.
+
+  @param   IdeDev  The IDE device data
+
+  @retval  EFI_SUCCESS      The ATA device supports Long Physical Sector feature
+                            and corresponding fields in BlockIo structure is updated.
+  @retval  EFI_UNSUPPORTED  The device is not ATA device or Long Physical Sector
+                            feature is not supported.
+**/
+EFI_STATUS
+AtaEnableLongPhysicalSector (
+  IN  IDE_BLK_IO_DEV  *IdeDev
+  );
 
 /**
   TODO: Add function description
@@ -1240,7 +1191,7 @@ AtaSMARTSupport (
   @param  StartLba TODO: add argument description
   @param  SectorCount TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -1251,8 +1202,7 @@ AtaPioDataInExt (
   IN  UINT8           AtaCommand,
   IN  EFI_LBA         StartLba,
   IN  UINT16          SectorCount
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -1264,7 +1214,7 @@ AtaPioDataInExt (
   @param  StartLba TODO: add argument description
   @param  SectorCount TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
@@ -1275,8 +1225,7 @@ AtaPioDataOutExt (
   IN  UINT8           AtaCommand,
   IN  EFI_LBA         StartLba,
   IN  UINT16          SectorCount
-  )
-;
+  );
 
 /**
   TODO: Add function description
@@ -1284,37 +1233,33 @@ AtaPioDataOutExt (
   @param  IdeDev TODO: add argument description
   @param  DriveParameters TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 SetDriveParameters (
   IN IDE_BLK_IO_DEV       *IdeDev,
   IN ATA_DRIVE_PARMS      *DriveParameters
-  )
-;
+  );
 
 /**
   TODO: Add function description
 
   @param  IdeDev TODO: add argument description
 
-  TODO: add return values
+  TODO: add return values.
 
 **/
 EFI_STATUS
 EnableInterrupt (
   IN IDE_BLK_IO_DEV       *IdeDev
-  )
-;
+  );
 
 /**
   Clear pending IDE interrupt before OS loader/kernel take control of the IDE device.
 
-  @param[in]  Event   Pointer to this event
-  @param[in]  Context Event hanlder private data
-
-  @retval  EFI_SUCCESS - Interrupt cleared
+  @param[in]  Event   Pointer to this event.
+  @param[in]  Context Event hanlder private data.
 
 **/
 VOID
@@ -1322,7 +1267,6 @@ EFIAPI
 ClearInterrupt (
   IN EFI_EVENT  Event,
   IN VOID       *Context
-  )
-;
+  );
 
 #endif