On 2015/9/19 2:41, Laszlo Ersek wrote:
Ray,

On 09/16/15 04:57, Laszlo Ersek wrote:
Cc: Alexander Graf <[email protected]>
Cc: Reza Jelveh <[email protected]>
Cc: Jordan Justen <[email protected]>
Cc: Ruiyu Ni <[email protected]>
Cc: Hannes Reinecke <[email protected]>
Cc: Gabriel L. Somlo <[email protected]>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Laszlo Ersek <[email protected]>
---
  DuetPkg/SataControllerDxe/SataController.h |  2 +-
  DuetPkg/SataControllerDxe/SataController.c | 22 ++++++++++----------
  2 files changed, 12 insertions(+), 12 deletions(-)

can you please check this patch and the next one? They're quite simple.

If you don't have time for them, I can do the same just in the OvmfPkg
copy -- I just didn't want to start with leaving DuetPkg behind, so I
figured I'd fix up the origin before the copy.

Thanks
Laszlo

diff --git a/DuetPkg/SataControllerDxe/SataController.h 
b/DuetPkg/SataControllerDxe/SataController.h
index d2a929f..a6c6c16 100644
--- a/DuetPkg/SataControllerDxe/SataController.h
+++ b/DuetPkg/SataControllerDxe/SataController.h
@@ -95,7 +95,7 @@ typedef struct _EFI_SATA_CONTROLLER_PRIVATE_DATA {
    // From ATA/ATAPI spec, if a mode is not supported,
    // the modes higher than it is also not supported
    //
-  EFI_ATA_COLLECTIVE_MODE           *DisqulifiedModes;
+  EFI_ATA_COLLECTIVE_MODE           *DisqualifiedModes;

    //
    // A copy of EFI_IDENTIFY_DATA data for each attached SATA device and its 
flag
diff --git a/DuetPkg/SataControllerDxe/SataController.c 
b/DuetPkg/SataControllerDxe/SataController.c
index 1807c88..e78915a 100644
--- a/DuetPkg/SataControllerDxe/SataController.c
+++ b/DuetPkg/SataControllerDxe/SataController.c
@@ -464,8 +464,8 @@ SataControllerStart (
    }

    ChannelDeviceCount = (UINTN) (SataPrivateData->IdeInit.ChannelCount) * (UINTN) 
(SataPrivateData->DeviceCount);
-  SataPrivateData->DisqulifiedModes = AllocateZeroPool ((sizeof 
(EFI_ATA_COLLECTIVE_MODE)) * ChannelDeviceCount);
-  if (SataPrivateData->DisqulifiedModes == NULL) {
+  SataPrivateData->DisqualifiedModes = AllocateZeroPool ((sizeof 
(EFI_ATA_COLLECTIVE_MODE)) * ChannelDeviceCount);
+  if (SataPrivateData->DisqualifiedModes == NULL) {
      Status = EFI_OUT_OF_RESOURCES;
      goto Done;
    }
@@ -502,8 +502,8 @@ Done:
            Controller
            );
      if (SataPrivateData != NULL) {
-      if (SataPrivateData->DisqulifiedModes != NULL) {
-        FreePool (SataPrivateData->DisqulifiedModes);
+      if (SataPrivateData->DisqualifiedModes != NULL) {
+        FreePool (SataPrivateData->DisqualifiedModes);
        }
        if (SataPrivateData->IdentifyData != NULL) {
          FreePool (SataPrivateData->IdentifyData);
@@ -577,8 +577,8 @@ SataControllerStop (
    }

    if (SataPrivateData != NULL) {
-    if (SataPrivateData->DisqulifiedModes != NULL) {
-      FreePool (SataPrivateData->DisqulifiedModes);
+    if (SataPrivateData->DisqualifiedModes != NULL) {
+      FreePool (SataPrivateData->DisqualifiedModes);
      }
      if (SataPrivateData->IdentifyData != NULL) {
        FreePool (SataPrivateData->IdentifyData);
@@ -833,7 +833,7 @@ IdeInitDisqualifyMode (
    // if a mode is not supported, the modes higher than it is also not 
supported.
    //
    CopyMem (
-    &(SataPrivateData->DisqulifiedModes[Channel * Device]),
+    &(SataPrivateData->DisqualifiedModes[Channel * Device]),
      BadModes,
      sizeof (EFI_ATA_COLLECTIVE_MODE)
      );
@@ -907,7 +907,7 @@ IdeInitCalculateMode (
    EFI_SATA_CONTROLLER_PRIVATE_DATA  *SataPrivateData;
    EFI_IDENTIFY_DATA                 *IdentifyData;
    BOOLEAN                           IdentifyValid;
-  EFI_ATA_COLLECTIVE_MODE           *DisqulifiedModes;
+  EFI_ATA_COLLECTIVE_MODE           *DisqualifiedModes;
    UINT16                            SelectedMode;
    EFI_STATUS                        Status;

@@ -926,7 +926,7 @@ IdeInitCalculateMode (

    IdentifyData = &(SataPrivateData->IdentifyData[Channel * Device]);
    IdentifyValid = SataPrivateData->IdentifyValid[Channel * Device];
-  DisqulifiedModes = &(SataPrivateData->DisqulifiedModes[Channel * Device]);
+  DisqualifiedModes = &(SataPrivateData->DisqualifiedModes[Channel * Device]);

    //
    // Make sure we've got the valid identify data of the device from 
SubmitData()
@@ -938,7 +938,7 @@ IdeInitCalculateMode (

    Status = CalculateBestPioMode (
              IdentifyData,
-            (DisqulifiedModes->PioMode.Valid ? ((UINT16 *) 
&(DisqulifiedModes->PioMode.Mode)) : NULL),
+            (DisqualifiedModes->PioMode.Valid ? ((UINT16 *) 
&(DisqualifiedModes->PioMode.Mode)) : NULL),
              &SelectedMode
              );
    if (!EFI_ERROR (Status)) {
@@ -952,7 +952,7 @@ IdeInitCalculateMode (

    Status = CalculateBestUdmaMode (
              IdentifyData,
-            (DisqulifiedModes->UdmaMode.Valid ? ((UINT16 *) 
&(DisqulifiedModes->UdmaMode.Mode)) : NULL),
+            (DisqualifiedModes->UdmaMode.Valid ? ((UINT16 *) 
&(DisqualifiedModes->UdmaMode.Mode)) : NULL),
              &SelectedMode
              );


Reviewed-by: Ruiyu Ni <[email protected]>

_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to