]> git.proxmox.com Git - mirror_edk2.git/blobdiff - NetworkPkg/IScsiDxe/IScsiMisc.c
NetworkPkg: Apply uncrustify changes
[mirror_edk2.git] / NetworkPkg / IScsiDxe / IScsiMisc.c
index 4069547867513f50e97799e1294e9068835cd806..b3ea90158f51848776568cbfe2ca849f5efc2d4f 100644 (file)
@@ -21,22 +21,21 @@ GLOBAL_REMOVE_IF_UNREFERENCED CONST CHAR8  IScsiHexString[] = "0123456789ABCDEFa
 **/\r
 VOID\r
 IScsiStrTrim (\r
-  IN OUT CHAR16   *Str,\r
-  IN     CHAR16   CharC\r
+  IN OUT CHAR16  *Str,\r
+  IN     CHAR16  CharC\r
   )\r
 {\r
   CHAR16  *Pointer1;\r
   CHAR16  *Pointer2;\r
 \r
   if (*Str == 0) {\r
-    return ;\r
+    return;\r
   }\r
 \r
   //\r
   // Trim off the leading and trailing characters c\r
   //\r
   for (Pointer1 = Str; (*Pointer1 != 0) && (*Pointer1 == CharC); Pointer1++) {\r
-    ;\r
   }\r
 \r
   Pointer2 = Str;\r
@@ -47,18 +46,18 @@ IScsiStrTrim (
     }\r
   } else {\r
     while (*Pointer1 != 0) {\r
-    *Pointer2 = *Pointer1;\r
-    Pointer1++;\r
-    Pointer2++;\r
+      *Pointer2 = *Pointer1;\r
+      Pointer1++;\r
+      Pointer2++;\r
     }\r
+\r
     *Pointer2 = 0;\r
   }\r
 \r
-\r
-  for (Pointer1 = Str + StrLen(Str) - 1; Pointer1 >= Str && *Pointer1 == CharC; Pointer1--) {\r
-    ;\r
+  for (Pointer1 = Str + StrLen (Str) - 1; Pointer1 >= Str && *Pointer1 == CharC; Pointer1--) {\r
   }\r
-  if  (Pointer1 !=  Str + StrLen(Str) - 1) {\r
+\r
+  if (Pointer1 !=  Str + StrLen (Str) - 1) {\r
     *(Pointer1 + 1) = 0;\r
   }\r
 }\r
@@ -101,10 +100,9 @@ IScsiGetSubnetMaskPrefixLength (
     Len++;\r
   }\r
 \r
-  return (UINT8) (32 - Len);\r
+  return (UINT8)(32 - Len);\r
 }\r
 \r
-\r
 /**\r
   Convert the hexadecimal encoded LUN string into the 64-bit LUN.\r
 \r
@@ -128,15 +126,15 @@ IScsiAsciiStrToLun (
 \r
   ZeroMem (Lun, 8);\r
   ZeroMem (TemStr, 2);\r
-  ZeroMem ((UINT8 *) Value, sizeof (Value));\r
+  ZeroMem ((UINT8 *)Value, sizeof (Value));\r
   SizeStr    = AsciiStrLen (Str);\r
   IndexValue = 0;\r
   IndexNum   = 0;\r
 \r
-  for (Index = 0; Index < SizeStr; Index ++) {\r
+  for (Index = 0; Index < SizeStr; Index++) {\r
     TemStr[0] = Str[Index];\r
-    TemValue = (UINT8) AsciiStrHexToUint64 (TemStr);\r
-    if (TemValue == 0 && TemStr[0] != '0') {\r
+    TemValue  = (UINT8)AsciiStrHexToUint64 (TemStr);\r
+    if ((TemValue == 0) && (TemStr[0] != '0')) {\r
       if ((TemStr[0] != '-') || (IndexNum == 0)) {\r
         //\r
         // Invalid Lun Char.\r
@@ -155,6 +153,7 @@ IScsiAsciiStrToLun (
         //\r
         return EFI_INVALID_PARAMETER;\r
       }\r
+\r
       //\r
       // Restart str index for the next lun value.\r
       //\r
@@ -172,11 +171,11 @@ IScsiAsciiStrToLun (
     //\r
     // Combine UINT16 value.\r
     //\r
-    Value[IndexValue] = (UINT16) ((Value[IndexValue] << 4) + TemValue);\r
+    Value[IndexValue] = (UINT16)((Value[IndexValue] << 4) + TemValue);\r
   }\r
 \r
-  for (Index = 0; Index <= IndexValue; Index ++) {\r
-    *((UINT16 *) &Lun[Index * 2]) =  HTONS (Value[Index]);\r
+  for (Index = 0; Index <= IndexValue; Index++) {\r
+    *((UINT16 *)&Lun[Index * 2]) =  HTONS (Value[Index]);\r
   }\r
 \r
   return EFI_SUCCESS;\r
@@ -201,26 +200,26 @@ IScsiLunToUnicodeStr (
   TempStr = Str;\r
 \r
   for (Index = 0; Index < 4; Index++) {\r
-\r
     if ((Lun[2 * Index] | Lun[2 * Index + 1]) == 0) {\r
       CopyMem (TempStr, L"0-", sizeof (L"0-"));\r
     } else {\r
-      TempStr[0]  = (CHAR16) IScsiHexString[Lun[2 * Index] >> 4];\r
-      TempStr[1]  = (CHAR16) IScsiHexString[Lun[2 * Index] & 0x0F];\r
-      TempStr[2]  = (CHAR16) IScsiHexString[Lun[2 * Index + 1] >> 4];\r
-      TempStr[3]  = (CHAR16) IScsiHexString[Lun[2 * Index + 1] & 0x0F];\r
-      TempStr[4]  = L'-';\r
-      TempStr[5]  = 0;\r
+      TempStr[0] = (CHAR16)IScsiHexString[Lun[2 * Index] >> 4];\r
+      TempStr[1] = (CHAR16)IScsiHexString[Lun[2 * Index] & 0x0F];\r
+      TempStr[2] = (CHAR16)IScsiHexString[Lun[2 * Index + 1] >> 4];\r
+      TempStr[3] = (CHAR16)IScsiHexString[Lun[2 * Index + 1] & 0x0F];\r
+      TempStr[4] = L'-';\r
+      TempStr[5] = 0;\r
 \r
       IScsiStrTrim (TempStr, L'0');\r
     }\r
 \r
     TempStr += StrLen (TempStr);\r
   }\r
+\r
   //\r
   // Remove the last '-'\r
   //\r
-  ASSERT (StrLen(Str) >= 1);\r
+  ASSERT (StrLen (Str) >= 1);\r
   Str[StrLen (Str) - 1] = 0;\r
 \r
   for (Index = StrLen (Str) - 1; Index > 1; Index = Index - 2) {\r
@@ -246,26 +245,24 @@ IScsiLunToUnicodeStr (
 **/\r
 EFI_STATUS\r
 IScsiAsciiStrToIp (\r
-  IN  CHAR8             *Str,\r
-  IN  UINT8             IpMode,\r
-  OUT EFI_IP_ADDRESS    *Ip\r
+  IN  CHAR8           *Str,\r
+  IN  UINT8           IpMode,\r
+  OUT EFI_IP_ADDRESS  *Ip\r
   )\r
 {\r
-  EFI_STATUS            Status;\r
+  EFI_STATUS  Status;\r
 \r
-  if (IpMode == IP_MODE_IP4 || IpMode == IP_MODE_AUTOCONFIG_IP4) {\r
+  if ((IpMode == IP_MODE_IP4) || (IpMode == IP_MODE_AUTOCONFIG_IP4)) {\r
     return NetLibAsciiStrToIp4 (Str, &Ip->v4);\r
-\r
-  } else if (IpMode == IP_MODE_IP6 || IpMode == IP_MODE_AUTOCONFIG_IP6) {\r
+  } else if ((IpMode == IP_MODE_IP6) || (IpMode == IP_MODE_AUTOCONFIG_IP6)) {\r
     return NetLibAsciiStrToIp6 (Str, &Ip->v6);\r
-\r
   } else if (IpMode == IP_MODE_AUTOCONFIG) {\r
     Status = NetLibAsciiStrToIp4 (Str, &Ip->v4);\r
     if (!EFI_ERROR (Status)) {\r
       return Status;\r
     }\r
-    return NetLibAsciiStrToIp6 (Str, &Ip->v6);\r
 \r
+    return NetLibAsciiStrToIp6 (Str, &Ip->v6);\r
   }\r
 \r
   return EFI_INVALID_PARAMETER;\r
@@ -292,14 +289,14 @@ IScsiMacAddrToStr (
   CHAR16  *String;\r
 \r
   for (Index = 0; Index < Len; Index++) {\r
-    Str[3 * Index]      = (CHAR16) IScsiHexString[(Mac->Addr[Index] >> 4) & 0x0F];\r
-    Str[3 * Index + 1]  = (CHAR16) IScsiHexString[Mac->Addr[Index] & 0x0F];\r
-    Str[3 * Index + 2]  = L':';\r
+    Str[3 * Index]     = (CHAR16)IScsiHexString[(Mac->Addr[Index] >> 4) & 0x0F];\r
+    Str[3 * Index + 1] = (CHAR16)IScsiHexString[Mac->Addr[Index] & 0x0F];\r
+    Str[3 * Index + 2] = L':';\r
   }\r
 \r
-  String = &Str[3 * Index - 1] ;\r
+  String = &Str[3 * Index - 1];\r
   if (VlanId != 0) {\r
-    String += UnicodeSPrint (String, 6 * sizeof (CHAR16), L"\\%04x", (UINTN) VlanId);\r
+    String += UnicodeSPrint (String, 6 * sizeof (CHAR16), L"\\%04x", (UINTN)VlanId);\r
   }\r
 \r
   *String = L'\0';\r
@@ -322,15 +319,15 @@ IScsiMacAddrToStr (
 **/\r
 EFI_STATUS\r
 IScsiBinToHex (\r
-  IN     UINT8  *BinBuffer,\r
-  IN     UINT32 BinLength,\r
-  IN OUT CHAR8  *HexStr,\r
-  IN OUT UINT32 *HexLength\r
+  IN     UINT8   *BinBuffer,\r
+  IN     UINT32  BinLength,\r
+  IN OUT CHAR8   *HexStr,\r
+  IN OUT UINT32  *HexLength\r
   )\r
 {\r
-  UINT32 HexLengthMin;\r
-  UINT32 HexLengthProvided;\r
-  UINT32 Index;\r
+  UINT32  HexLengthMin;\r
+  UINT32  HexLengthProvided;\r
+  UINT32  Index;\r
 \r
   if ((HexStr == NULL) || (BinBuffer == NULL) || (BinLength == 0)) {\r
     return EFI_INVALID_PARAMETER;\r
@@ -340,12 +337,13 @@ IScsiBinToHex (
   // Safely calculate: HexLengthMin := BinLength * 2 + 3.\r
   //\r
   if (RETURN_ERROR (SafeUint32Mult (BinLength, 2, &HexLengthMin)) ||\r
-      RETURN_ERROR (SafeUint32Add (HexLengthMin, 3, &HexLengthMin))) {\r
+      RETURN_ERROR (SafeUint32Add (HexLengthMin, 3, &HexLengthMin)))\r
+  {\r
     return EFI_BAD_BUFFER_SIZE;\r
   }\r
 \r
   HexLengthProvided = *HexLength;\r
-  *HexLength = HexLengthMin;\r
+  *HexLength        = HexLengthMin;\r
   if (HexLengthProvided < HexLengthMin) {\r
     return EFI_BUFFER_TOO_SMALL;\r
   }\r
@@ -366,7 +364,6 @@ IScsiBinToHex (
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
 /**\r
   Convert the hexadecimal string into a binary encoded buffer.\r
 \r
@@ -385,9 +382,9 @@ IScsiBinToHex (
 **/\r
 EFI_STATUS\r
 IScsiHexToBin (\r
-  IN OUT UINT8  *BinBuffer,\r
-  IN OUT UINT32 *BinLength,\r
-  IN     CHAR8  *HexStr\r
+  IN OUT UINT8   *BinBuffer,\r
+  IN OUT UINT32  *BinLength,\r
+  IN     CHAR8   *HexStr\r
   )\r
 {\r
   UINTN   BinLengthMin;\r
@@ -411,9 +408,10 @@ IScsiHexToBin (
   //\r
   // Reject an empty hex string; reject a stray nibble.\r
   //\r
-  if (Length == 0 || Length % 2 != 0) {\r
+  if ((Length == 0) || (Length % 2 != 0)) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
+\r
   //\r
   // Check if the caller provides enough room for the decoded blob.\r
   //\r
@@ -421,31 +419,33 @@ IScsiHexToBin (
   if (BinLengthMin > MAX_UINT32) {\r
     return EFI_BAD_BUFFER_SIZE;\r
   }\r
+\r
   BinLengthProvided = *BinLength;\r
-  *BinLength = (UINT32)BinLengthMin;\r
+  *BinLength        = (UINT32)BinLengthMin;\r
   if (BinLengthProvided < BinLengthMin) {\r
     return EFI_BUFFER_TOO_SMALL;\r
   }\r
 \r
-  for (Index = 0; Index < Length; Index ++) {\r
+  for (Index = 0; Index < Length; Index++) {\r
     TemStr[0] = HexStr[Index];\r
-    Digit = (UINT8) AsciiStrHexToUint64 (TemStr);\r
-    if (Digit == 0 && TemStr[0] != '0') {\r
+    Digit     = (UINT8)AsciiStrHexToUint64 (TemStr);\r
+    if ((Digit == 0) && (TemStr[0] != '0')) {\r
       //\r
       // Invalid Hex Char.\r
       //\r
       return EFI_INVALID_PARAMETER;\r
     }\r
+\r
     if ((Index & 1) == 0) {\r
-      BinBuffer [Index/2] = Digit;\r
+      BinBuffer[Index/2] = Digit;\r
     } else {\r
-      BinBuffer [Index/2] = (UINT8) ((BinBuffer [Index/2] << 4) + Digit);\r
+      BinBuffer[Index/2] = (UINT8)((BinBuffer[Index/2] << 4) + Digit);\r
     }\r
   }\r
+\r
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
 /**\r
   Convert the decimal-constant string or hex-constant string into a numerical value.\r
 \r
@@ -468,7 +468,6 @@ IScsiNetNtoi (
   return AsciiStrDecimalToUintn (Str);\r
 }\r
 \r
-\r
 /**\r
   Generate random numbers.\r
 \r
@@ -486,12 +485,11 @@ IScsiGenRandom (
 \r
   while (RandLength > 0) {\r
     Random  = NET_RANDOM (NetRandomInitSeed ());\r
-    *Rand++ = (UINT8) (Random);\r
+    *Rand++ = (UINT8)(Random);\r
     RandLength--;\r
   }\r
 }\r
 \r
-\r
 /**\r
   Check whether UNDI protocol supports IPv6.\r
 \r
@@ -505,22 +503,22 @@ IScsiGenRandom (
 **/\r
 EFI_STATUS\r
 IScsiCheckIpv6Support (\r
-  IN  EFI_HANDLE                   ControllerHandle,\r
-  IN  EFI_HANDLE                   Image,\r
-  OUT BOOLEAN                      *Ipv6Support\r
+  IN  EFI_HANDLE  ControllerHandle,\r
+  IN  EFI_HANDLE  Image,\r
+  OUT BOOLEAN     *Ipv6Support\r
   )\r
 {\r
-  EFI_HANDLE                       Handle;\r
-  EFI_ADAPTER_INFORMATION_PROTOCOL *Aip;\r
-  EFI_STATUS                       Status;\r
-  EFI_GUID                         *InfoTypesBuffer;\r
-  UINTN                            InfoTypeBufferCount;\r
-  UINTN                            TypeIndex;\r
-  BOOLEAN                          Supported;\r
-  VOID                             *InfoBlock;\r
-  UINTN                            InfoBlockSize;\r
-\r
-  EFI_NETWORK_INTERFACE_IDENTIFIER_PROTOCOL *Nii;\r
+  EFI_HANDLE                        Handle;\r
+  EFI_ADAPTER_INFORMATION_PROTOCOL  *Aip;\r
+  EFI_STATUS                        Status;\r
+  EFI_GUID                          *InfoTypesBuffer;\r
+  UINTN                             InfoTypeBufferCount;\r
+  UINTN                             TypeIndex;\r
+  BOOLEAN                           Supported;\r
+  VOID                              *InfoBlock;\r
+  UINTN                             InfoBlockSize;\r
+\r
+  EFI_NETWORK_INTERFACE_IDENTIFIER_PROTOCOL  *Nii;\r
 \r
   ASSERT (Ipv6Support != NULL);\r
 \r
@@ -530,7 +528,7 @@ IScsiCheckIpv6Support (
   Status = gBS->OpenProtocol (\r
                   ControllerHandle,\r
                   &gEfiNetworkInterfaceIdentifierProtocolGuid_31,\r
-                  (VOID **) &Nii,\r
+                  (VOID **)&Nii,\r
                   Image,\r
                   ControllerHandle,\r
                   EFI_OPEN_PROTOCOL_GET_PROTOCOL\r
@@ -552,16 +550,16 @@ IScsiCheckIpv6Support (
   Status = gBS->HandleProtocol (\r
                   Handle,\r
                   &gEfiAdapterInformationProtocolGuid,\r
-                  (VOID *) &Aip\r
+                  (VOID *)&Aip\r
                   );\r
-  if (EFI_ERROR (Status) || Aip == NULL) {\r
+  if (EFI_ERROR (Status) || (Aip == NULL)) {\r
     return EFI_NOT_FOUND;\r
   }\r
 \r
   InfoTypesBuffer     = NULL;\r
   InfoTypeBufferCount = 0;\r
-  Status = Aip->GetSupportedTypes (Aip, &InfoTypesBuffer, &InfoTypeBufferCount);\r
-  if (EFI_ERROR (Status) || InfoTypesBuffer == NULL) {\r
+  Status              = Aip->GetSupportedTypes (Aip, &InfoTypesBuffer, &InfoTypeBufferCount);\r
+  if (EFI_ERROR (Status) || (InfoTypesBuffer == NULL)) {\r
     FreePool (InfoTypesBuffer);\r
     return EFI_NOT_FOUND;\r
   }\r
@@ -584,13 +582,13 @@ IScsiCheckIpv6Support (
   //\r
   InfoBlock     = NULL;\r
   InfoBlockSize = 0;\r
-  Status = Aip->GetInformation (Aip, &gEfiAdapterInfoUndiIpv6SupportGuid, &InfoBlock, &InfoBlockSize);\r
-  if (EFI_ERROR (Status) || InfoBlock == NULL) {\r
+  Status        = Aip->GetInformation (Aip, &gEfiAdapterInfoUndiIpv6SupportGuid, &InfoBlock, &InfoBlockSize);\r
+  if (EFI_ERROR (Status) || (InfoBlock == NULL)) {\r
     FreePool (InfoBlock);\r
     return EFI_NOT_FOUND;\r
   }\r
 \r
-  *Ipv6Support = ((EFI_ADAPTER_INFO_UNDI_IPV6_SUPPORT *) InfoBlock)->Ipv6Support;\r
+  *Ipv6Support = ((EFI_ADAPTER_INFO_UNDI_IPV6_SUPPORT *)InfoBlock)->Ipv6Support;\r
   FreePool (InfoBlock);\r
 \r
   return EFI_SUCCESS;\r
@@ -613,12 +611,12 @@ IScsiAddNic (
   IN EFI_HANDLE  Image\r
   )\r
 {\r
-  EFI_STATUS                  Status;\r
-  ISCSI_NIC_INFO              *NicInfo;\r
-  LIST_ENTRY                  *Entry;\r
-  EFI_MAC_ADDRESS             MacAddr;\r
-  UINTN                       HwAddressSize;\r
-  UINT16                      VlanId;\r
+  EFI_STATUS       Status;\r
+  ISCSI_NIC_INFO   *NicInfo;\r
+  LIST_ENTRY       *Entry;\r
+  EFI_MAC_ADDRESS  MacAddr;\r
+  UINTN            HwAddressSize;\r
+  UINT16           VlanId;\r
 \r
   //\r
   // Get MAC address of this network device.\r
@@ -638,9 +636,10 @@ IScsiAddNic (
   //\r
   NET_LIST_FOR_EACH (Entry, &mPrivate->NicInfoList) {\r
     NicInfo = NET_LIST_USER_STRUCT (Entry, ISCSI_NIC_INFO, Link);\r
-    if (NicInfo->HwAddressSize == HwAddressSize &&\r
-        CompareMem (&NicInfo->PermanentAddress, MacAddr.Addr, HwAddressSize) == 0 &&\r
-        NicInfo->VlanId == VlanId) {\r
+    if ((NicInfo->HwAddressSize == HwAddressSize) &&\r
+        (CompareMem (&NicInfo->PermanentAddress, MacAddr.Addr, HwAddressSize) == 0) &&\r
+        (NicInfo->VlanId == VlanId))\r
+    {\r
       mPrivate->CurrentNic = NicInfo->NicIndex;\r
 \r
       //\r
@@ -672,10 +671,10 @@ IScsiAddNic (
   }\r
 \r
   CopyMem (&NicInfo->PermanentAddress, MacAddr.Addr, HwAddressSize);\r
-  NicInfo->HwAddressSize  = (UINT32) HwAddressSize;\r
-  NicInfo->VlanId         = VlanId;\r
-  NicInfo->NicIndex       = (UINT8) (mPrivate->MaxNic + 1);\r
-  mPrivate->MaxNic        = NicInfo->NicIndex;\r
+  NicInfo->HwAddressSize = (UINT32)HwAddressSize;\r
+  NicInfo->VlanId        = VlanId;\r
+  NicInfo->NicIndex      = (UINT8)(mPrivate->MaxNic + 1);\r
+  mPrivate->MaxNic       = NicInfo->NicIndex;\r
 \r
   //\r
   // Set IPv6 available flag.\r
@@ -706,7 +705,6 @@ IScsiAddNic (
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
 /**\r
   Delete the recorded NIC info from global structure. Also delete corresponding\r
   attempts.\r
@@ -722,15 +720,15 @@ IScsiRemoveNic (
   IN EFI_HANDLE  Controller\r
   )\r
 {\r
-  EFI_STATUS                  Status;\r
-  ISCSI_NIC_INFO              *NicInfo;\r
-  LIST_ENTRY                  *Entry;\r
-  LIST_ENTRY                  *NextEntry;\r
-  ISCSI_ATTEMPT_CONFIG_NVDATA *AttemptConfigData;\r
-  ISCSI_NIC_INFO              *ThisNic;\r
-  EFI_MAC_ADDRESS             MacAddr;\r
-  UINTN                       HwAddressSize;\r
-  UINT16                      VlanId;\r
+  EFI_STATUS                   Status;\r
+  ISCSI_NIC_INFO               *NicInfo;\r
+  LIST_ENTRY                   *Entry;\r
+  LIST_ENTRY                   *NextEntry;\r
+  ISCSI_ATTEMPT_CONFIG_NVDATA  *AttemptConfigData;\r
+  ISCSI_NIC_INFO               *ThisNic;\r
+  EFI_MAC_ADDRESS              MacAddr;\r
+  UINTN                        HwAddressSize;\r
+  UINT16                       VlanId;\r
 \r
   //\r
   // Get MAC address of this network device.\r
@@ -752,10 +750,10 @@ IScsiRemoveNic (
 \r
   NET_LIST_FOR_EACH (Entry, &mPrivate->NicInfoList) {\r
     NicInfo = NET_LIST_USER_STRUCT (Entry, ISCSI_NIC_INFO, Link);\r
-    if (NicInfo->HwAddressSize == HwAddressSize &&\r
-        CompareMem (&NicInfo->PermanentAddress, MacAddr.Addr, HwAddressSize) == 0 &&\r
-        NicInfo->VlanId == VlanId) {\r
-\r
+    if ((NicInfo->HwAddressSize == HwAddressSize) &&\r
+        (CompareMem (&NicInfo->PermanentAddress, MacAddr.Addr, HwAddressSize) == 0) &&\r
+        (NicInfo->VlanId == VlanId))\r
+    {\r
       ThisNic = NicInfo;\r
       break;\r
     }\r
@@ -780,16 +778,15 @@ IScsiRemoveNic (
       RemoveEntryList (&AttemptConfigData->Link);\r
       mPrivate->AttemptCount--;\r
 \r
-      if (AttemptConfigData->SessionConfigData.Enabled == ISCSI_ENABLED_FOR_MPIO && mPrivate->MpioCount > 0) {\r
+      if ((AttemptConfigData->SessionConfigData.Enabled == ISCSI_ENABLED_FOR_MPIO) && (mPrivate->MpioCount > 0)) {\r
         if (--mPrivate->MpioCount == 0) {\r
           mPrivate->EnableMpio = FALSE;\r
         }\r
 \r
-        if (AttemptConfigData->AuthenticationType == ISCSI_AUTH_TYPE_KRB && mPrivate->Krb5MpioCount > 0) {\r
+        if ((AttemptConfigData->AuthenticationType == ISCSI_AUTH_TYPE_KRB) && (mPrivate->Krb5MpioCount > 0)) {\r
           mPrivate->Krb5MpioCount--;\r
         }\r
-\r
-      } else if (AttemptConfigData->SessionConfigData.Enabled == ISCSI_ENABLED && mPrivate->SinglePathCount > 0) {\r
+      } else if ((AttemptConfigData->SessionConfigData.Enabled == ISCSI_ENABLED) && (mPrivate->SinglePathCount > 0)) {\r
         mPrivate->SinglePathCount--;\r
 \r
         if (mPrivate->ValidSinglePathCount > 0) {\r
@@ -815,19 +812,19 @@ IScsiRemoveNic (
 **/\r
 EFI_STATUS\r
 IScsiCreateAttempts (\r
-  IN UINTN            AttemptNum\r
-)\r
+  IN UINTN  AttemptNum\r
+  )\r
 {\r
-  ISCSI_ATTEMPT_CONFIG_NVDATA   *AttemptConfigData;\r
-  ISCSI_SESSION_CONFIG_NVDATA   *ConfigData;\r
-  UINT8                         *AttemptConfigOrder;\r
-  UINTN                         AttemptConfigOrderSize;\r
-  UINT8                         *AttemptOrderTmp;\r
-  UINTN                         TotalNumber;\r
-  UINT8                         Index;\r
-  EFI_STATUS                    Status;\r
+  ISCSI_ATTEMPT_CONFIG_NVDATA  *AttemptConfigData;\r
+  ISCSI_SESSION_CONFIG_NVDATA  *ConfigData;\r
+  UINT8                        *AttemptConfigOrder;\r
+  UINTN                        AttemptConfigOrderSize;\r
+  UINT8                        *AttemptOrderTmp;\r
+  UINTN                        TotalNumber;\r
+  UINT8                        Index;\r
+  EFI_STATUS                   Status;\r
 \r
-  for (Index = 1; Index <= AttemptNum; Index ++) {\r
+  for (Index = 1; Index <= AttemptNum; Index++) {\r
     //\r
     // Get the initialized attempt order. This is used to essure creating attempts by order.\r
     //\r
@@ -841,6 +838,7 @@ IScsiCreateAttempts (
       Status = EFI_SUCCESS;\r
       break;\r
     }\r
+\r
     TotalNumber++;\r
 \r
     //\r
@@ -851,6 +849,7 @@ IScsiCreateAttempts (
       if (AttemptConfigOrder != NULL) {\r
         FreePool (AttemptConfigOrder);\r
       }\r
+\r
       return EFI_OUT_OF_RESOURCES;\r
     }\r
 \r
@@ -872,10 +871,14 @@ IScsiCreateAttempts (
                     );\r
     FreePool (AttemptConfigOrder);\r
     if (EFI_ERROR (Status)) {\r
-      DEBUG ((DEBUG_ERROR,\r
+      DEBUG ((\r
+        DEBUG_ERROR,\r
         "%a: Failed to set 'InitialAttemptOrder' with Guid (%g): "\r
         "%r\n",\r
-        __FUNCTION__, &gIScsiConfigGuid, Status));\r
+        __FUNCTION__,\r
+        &gIScsiConfigGuid,\r
+        Status\r
+        ));\r
       return Status;\r
     }\r
 \r
@@ -886,6 +889,7 @@ IScsiCreateAttempts (
     if (AttemptConfigData == NULL) {\r
       return EFI_OUT_OF_RESOURCES;\r
     }\r
+\r
     ConfigData                    = &AttemptConfigData->SessionConfigData;\r
     ConfigData->TargetPort        = ISCSI_WELL_KNOWN_PORT;\r
     ConfigData->ConnectTimeout    = CONNECT_DEFAULT_TIMEOUT;\r
@@ -903,9 +907,9 @@ IScsiCreateAttempts (
     //\r
     UnicodeSPrint (\r
       mPrivate->PortString,\r
-      (UINTN) ISCSI_NAME_IFR_MAX_SIZE,\r
+      (UINTN)ISCSI_NAME_IFR_MAX_SIZE,\r
       L"Attempt %d",\r
-      (UINTN) AttemptConfigData->AttemptConfigIndex\r
+      (UINTN)AttemptConfigData->AttemptConfigIndex\r
       );\r
     UnicodeStrToAsciiStrS (mPrivate->PortString, AttemptConfigData->AttemptName, ATTEMPT_NAME_SIZE);\r
 \r
@@ -918,10 +922,14 @@ IScsiCreateAttempts (
                     );\r
     FreePool (AttemptConfigData);\r
     if (EFI_ERROR (Status)) {\r
-        DEBUG ((DEBUG_ERROR,\r
-          "%a: Failed to set variable (mPrivate->PortString) with Guid (%g): "\r
-          "%r\n",\r
-          __FUNCTION__, &gEfiIScsiInitiatorNameProtocolGuid, Status));\r
+      DEBUG ((\r
+        DEBUG_ERROR,\r
+        "%a: Failed to set variable (mPrivate->PortString) with Guid (%g): "\r
+        "%r\n",\r
+        __FUNCTION__,\r
+        &gEfiIScsiInitiatorNameProtocolGuid,\r
+        Status\r
+        ));\r
       return Status;\r
     }\r
   }\r
@@ -941,18 +949,18 @@ IScsiCreateAttempts (
 **/\r
 EFI_STATUS\r
 IScsiCreateKeywords (\r
-  IN UINTN            KeywordNum\r
-)\r
+  IN UINTN  KeywordNum\r
+  )\r
 {\r
-  VOID                          *StartOpCodeHandle;\r
-  EFI_IFR_GUID_LABEL            *StartLabel;\r
-  VOID                          *EndOpCodeHandle;\r
-  EFI_IFR_GUID_LABEL            *EndLabel;\r
-  UINTN                         Index;\r
-  EFI_STRING_ID                 StringToken;\r
-  CHAR16                        StringId[64];\r
-  CHAR16                        KeywordId[32];\r
-  EFI_STATUS                    Status;\r
+  VOID                *StartOpCodeHandle;\r
+  EFI_IFR_GUID_LABEL  *StartLabel;\r
+  VOID                *EndOpCodeHandle;\r
+  EFI_IFR_GUID_LABEL  *EndLabel;\r
+  UINTN               Index;\r
+  EFI_STRING_ID       StringToken;\r
+  CHAR16              StringId[64];\r
+  CHAR16              KeywordId[32];\r
+  EFI_STATUS          Status;\r
 \r
   Status = IScsiCreateOpCode (\r
              KEYWORD_ENTRY_LABEL,\r
@@ -965,7 +973,7 @@ IScsiCreateKeywords (
     return EFI_OUT_OF_RESOURCES;\r
   }\r
 \r
-  for (Index = 1; Index <= KeywordNum; Index ++) {\r
+  for (Index = 1; Index <= KeywordNum; Index++) {\r
     //\r
     // Create iSCSIAttemptName Keyword.\r
     //\r
@@ -980,9 +988,9 @@ IScsiCreateKeywords (
     HiiSetString (mCallbackInfo->RegisteredHandle, StringToken, KeywordId, "x-UEFI-ns");\r
     HiiCreateStringOpCode (\r
       StartOpCodeHandle,\r
-      (EFI_QUESTION_ID) (ATTEMPT_ATTEMPT_NAME_QUESTION_ID + (Index - 1)),\r
+      (EFI_QUESTION_ID)(ATTEMPT_ATTEMPT_NAME_QUESTION_ID + (Index - 1)),\r
       CONFIGURATION_VARSTORE_ID,\r
-      (UINT16) (ATTEMPT_ATTEMPT_NAME_VAR_OFFSET + ATTEMPT_NAME_SIZE * (Index - 1) * sizeof (CHAR16)),\r
+      (UINT16)(ATTEMPT_ATTEMPT_NAME_VAR_OFFSET + ATTEMPT_NAME_SIZE * (Index - 1) * sizeof (CHAR16)),\r
       StringToken,\r
       StringToken,\r
       EFI_IFR_FLAG_READ_ONLY,\r
@@ -1006,9 +1014,9 @@ IScsiCreateKeywords (
     HiiSetString (mCallbackInfo->RegisteredHandle, StringToken, KeywordId, "x-UEFI-ns");\r
     HiiCreateNumericOpCode (\r
       StartOpCodeHandle,\r
-      (EFI_QUESTION_ID) (ATTEMPT_BOOTENABLE_QUESTION_ID + (Index - 1)),\r
+      (EFI_QUESTION_ID)(ATTEMPT_BOOTENABLE_QUESTION_ID + (Index - 1)),\r
       CONFIGURATION_VARSTORE_ID,\r
-      (UINT16) (ATTEMPT_BOOTENABLE_VAR_OFFSET + (Index - 1)),\r
+      (UINT16)(ATTEMPT_BOOTENABLE_VAR_OFFSET + (Index - 1)),\r
       StringToken,\r
       StringToken,\r
       0,\r
@@ -1033,9 +1041,9 @@ IScsiCreateKeywords (
     HiiSetString (mCallbackInfo->RegisteredHandle, StringToken, KeywordId, "x-UEFI-ns");\r
     HiiCreateNumericOpCode (\r
       StartOpCodeHandle,\r
-      (EFI_QUESTION_ID) (ATTEMPT_ADDRESS_TYPE_QUESTION_ID + (Index - 1)),\r
+      (EFI_QUESTION_ID)(ATTEMPT_ADDRESS_TYPE_QUESTION_ID + (Index - 1)),\r
       CONFIGURATION_VARSTORE_ID,\r
-      (UINT16) (ATTEMPT_ADDRESS_TYPE_VAR_OFFSET + (Index - 1)),\r
+      (UINT16)(ATTEMPT_ADDRESS_TYPE_VAR_OFFSET + (Index - 1)),\r
       StringToken,\r
       StringToken,\r
       0,\r
@@ -1060,9 +1068,9 @@ IScsiCreateKeywords (
     HiiSetString (mCallbackInfo->RegisteredHandle, StringToken, KeywordId, "x-UEFI-ns");\r
     HiiCreateNumericOpCode (\r
       StartOpCodeHandle,\r
-      (EFI_QUESTION_ID) (ATTEMPT_CONNECT_RETRY_QUESTION_ID + (Index - 1)),\r
+      (EFI_QUESTION_ID)(ATTEMPT_CONNECT_RETRY_QUESTION_ID + (Index - 1)),\r
       CONFIGURATION_VARSTORE_ID,\r
-      (UINT16) (ATTEMPT_CONNECT_RETRY_VAR_OFFSET + (Index - 1)),\r
+      (UINT16)(ATTEMPT_CONNECT_RETRY_VAR_OFFSET + (Index - 1)),\r
       StringToken,\r
       StringToken,\r
       0,\r
@@ -1087,9 +1095,9 @@ IScsiCreateKeywords (
     HiiSetString (mCallbackInfo->RegisteredHandle, StringToken, KeywordId, "x-UEFI-ns");\r
     HiiCreateNumericOpCode (\r
       StartOpCodeHandle,\r
-      (EFI_QUESTION_ID) (ATTEMPT_CONNECT_TIMEOUT_QUESTION_ID + (Index - 1)),\r
+      (EFI_QUESTION_ID)(ATTEMPT_CONNECT_TIMEOUT_QUESTION_ID + (Index - 1)),\r
       CONFIGURATION_VARSTORE_ID,\r
-      (UINT16) (ATTEMPT_CONNECT_TIMEOUT_VAR_OFFSET + 2 * (Index - 1)),\r
+      (UINT16)(ATTEMPT_CONNECT_TIMEOUT_VAR_OFFSET + 2 * (Index - 1)),\r
       StringToken,\r
       StringToken,\r
       0,\r
@@ -1114,9 +1122,9 @@ IScsiCreateKeywords (
     HiiSetString (mCallbackInfo->RegisteredHandle, StringToken, KeywordId, "x-UEFI-ns");\r
     HiiCreateStringOpCode (\r
       StartOpCodeHandle,\r
-      (EFI_QUESTION_ID) (ATTEMPT_ISID_QUESTION_ID + (Index - 1)),\r
+      (EFI_QUESTION_ID)(ATTEMPT_ISID_QUESTION_ID + (Index - 1)),\r
       CONFIGURATION_VARSTORE_ID,\r
-      (UINT16) (ATTEMPT_ISID_VAR_OFFSET + sizeof (KEYWORD_STR) * (Index - 1)),\r
+      (UINT16)(ATTEMPT_ISID_VAR_OFFSET + sizeof (KEYWORD_STR) * (Index - 1)),\r
       StringToken,\r
       STRING_TOKEN (STR_ISCSI_ISID_HELP),\r
       0,\r
@@ -1139,19 +1147,19 @@ IScsiCreateKeywords (
     UnicodeSPrint (KeywordId, sizeof (KeywordId), L"iSCSIInitiatorInfoViaDHCP:%d", Index);\r
     HiiSetString (mCallbackInfo->RegisteredHandle, StringToken, KeywordId, "x-UEFI-ns");\r
     HiiCreateNumericOpCode (\r
-    StartOpCodeHandle,\r
-    (EFI_QUESTION_ID) (ATTEMPT_INITIATOR_VIA_DHCP_QUESTION_ID + (Index - 1)),\r
-    CONFIGURATION_VARSTORE_ID,\r
-    (UINT16) (ATTEMPT_INITIATOR_VIA_DHCP_VAR_OFFSET + (Index - 1)),\r
-    StringToken,\r
-    StringToken,\r
-    0,\r
-    0,\r
-    0,\r
-    1,\r
-    0,\r
-    NULL\r
-    );\r
+      StartOpCodeHandle,\r
+      (EFI_QUESTION_ID)(ATTEMPT_INITIATOR_VIA_DHCP_QUESTION_ID + (Index - 1)),\r
+      CONFIGURATION_VARSTORE_ID,\r
+      (UINT16)(ATTEMPT_INITIATOR_VIA_DHCP_VAR_OFFSET + (Index - 1)),\r
+      StringToken,\r
+      StringToken,\r
+      0,\r
+      0,\r
+      0,\r
+      1,\r
+      0,\r
+      NULL\r
+      );\r
 \r
     //\r
     // Create iSCSIInitiatorIpAddress Keyword.\r
@@ -1167,9 +1175,9 @@ IScsiCreateKeywords (
     HiiSetString (mCallbackInfo->RegisteredHandle, StringToken, KeywordId, "x-UEFI-ns");\r
     HiiCreateStringOpCode (\r
       StartOpCodeHandle,\r
-      (EFI_QUESTION_ID) (ATTEMPT_INITIATOR_IP_ADDRESS_QUESTION_ID + (Index - 1)),\r
+      (EFI_QUESTION_ID)(ATTEMPT_INITIATOR_IP_ADDRESS_QUESTION_ID + (Index - 1)),\r
       CONFIGURATION_VARSTORE_ID,\r
-      (UINT16) (ATTEMPT_INITIATOR_IP_ADDRESS_VAR_OFFSET + sizeof (KEYWORD_STR) * (Index - 1)),\r
+      (UINT16)(ATTEMPT_INITIATOR_IP_ADDRESS_VAR_OFFSET + sizeof (KEYWORD_STR) * (Index - 1)),\r
       StringToken,\r
       StringToken,\r
       0,\r
@@ -1193,9 +1201,9 @@ IScsiCreateKeywords (
     HiiSetString (mCallbackInfo->RegisteredHandle, StringToken, KeywordId, "x-UEFI-ns");\r
     HiiCreateStringOpCode (\r
       StartOpCodeHandle,\r
-      (EFI_QUESTION_ID) (ATTEMPT_INITIATOR_NET_MASK_QUESTION_ID + (Index - 1)),\r
+      (EFI_QUESTION_ID)(ATTEMPT_INITIATOR_NET_MASK_QUESTION_ID + (Index - 1)),\r
       CONFIGURATION_VARSTORE_ID,\r
-      (UINT16) (ATTEMPT_INITIATOR_NET_MASK_VAR_OFFSET + sizeof (KEYWORD_STR) * (Index - 1)),\r
+      (UINT16)(ATTEMPT_INITIATOR_NET_MASK_VAR_OFFSET + sizeof (KEYWORD_STR) * (Index - 1)),\r
       StringToken,\r
       StringToken,\r
       0,\r
@@ -1219,9 +1227,9 @@ IScsiCreateKeywords (
     HiiSetString (mCallbackInfo->RegisteredHandle, StringToken, KeywordId, "x-UEFI-ns");\r
     HiiCreateStringOpCode (\r
       StartOpCodeHandle,\r
-      (EFI_QUESTION_ID) (ATTEMPT_INITIATOR_GATE_WAY_QUESTION_ID + (Index - 1)),\r
+      (EFI_QUESTION_ID)(ATTEMPT_INITIATOR_GATE_WAY_QUESTION_ID + (Index - 1)),\r
       CONFIGURATION_VARSTORE_ID,\r
-      (UINT16) (ATTEMPT_INITIATOR_GATE_WAY_VAR_OFFSET + sizeof (KEYWORD_STR) * (Index - 1)),\r
+      (UINT16)(ATTEMPT_INITIATOR_GATE_WAY_VAR_OFFSET + sizeof (KEYWORD_STR) * (Index - 1)),\r
       StringToken,\r
       StringToken,\r
       0,\r
@@ -1244,19 +1252,19 @@ IScsiCreateKeywords (
     UnicodeSPrint (KeywordId, sizeof (KeywordId), L"iSCSITargetInfoViaDHCP:%d", Index);\r
     HiiSetString (mCallbackInfo->RegisteredHandle, StringToken, KeywordId, "x-UEFI-ns");\r
     HiiCreateNumericOpCode (\r
-    StartOpCodeHandle,\r
-    (EFI_QUESTION_ID) (ATTEMPT_TARGET_VIA_DHCP_QUESTION_ID + (Index - 1)),\r
-    CONFIGURATION_VARSTORE_ID,\r
-    (UINT16) (ATTEMPT_TARGET_VIA_DHCP_VAR_OFFSET + (Index - 1)),\r
-    StringToken,\r
-    StringToken,\r
-    0,\r
-    0,\r
-    0,\r
-    1,\r
-    0,\r
-    NULL\r
-    );\r
+      StartOpCodeHandle,\r
+      (EFI_QUESTION_ID)(ATTEMPT_TARGET_VIA_DHCP_QUESTION_ID + (Index - 1)),\r
+      CONFIGURATION_VARSTORE_ID,\r
+      (UINT16)(ATTEMPT_TARGET_VIA_DHCP_VAR_OFFSET + (Index - 1)),\r
+      StringToken,\r
+      StringToken,\r
+      0,\r
+      0,\r
+      0,\r
+      1,\r
+      0,\r
+      NULL\r
+      );\r
 \r
     //\r
     // Create iSCSITargetTcpPort Keyword.\r
@@ -1272,9 +1280,9 @@ IScsiCreateKeywords (
     HiiSetString (mCallbackInfo->RegisteredHandle, StringToken, KeywordId, "x-UEFI-ns");\r
     HiiCreateNumericOpCode (\r
       StartOpCodeHandle,\r
-      (EFI_QUESTION_ID) (ATTEMPT_TARGET_TCP_PORT_QUESTION_ID + (Index - 1)),\r
+      (EFI_QUESTION_ID)(ATTEMPT_TARGET_TCP_PORT_QUESTION_ID + (Index - 1)),\r
       CONFIGURATION_VARSTORE_ID,\r
-      (UINT16) (ATTEMPT_TARGET_TCP_PORT_VAR_OFFSET + 2 * (Index - 1)),\r
+      (UINT16)(ATTEMPT_TARGET_TCP_PORT_VAR_OFFSET + 2 * (Index - 1)),\r
       StringToken,\r
       StringToken,\r
       0,\r
@@ -1299,9 +1307,9 @@ IScsiCreateKeywords (
     HiiSetString (mCallbackInfo->RegisteredHandle, StringToken, KeywordId, "x-UEFI-ns");\r
     HiiCreateStringOpCode (\r
       StartOpCodeHandle,\r
-      (EFI_QUESTION_ID) (ATTEMPT_TARGET_NAME_QUESTION_ID + (Index - 1)),\r
+      (EFI_QUESTION_ID)(ATTEMPT_TARGET_NAME_QUESTION_ID + (Index - 1)),\r
       CONFIGURATION_VARSTORE_ID,\r
-      (UINT16) (ATTEMPT_TARGET_NAME_VAR_OFFSET + sizeof (KEYWORD_STR) * (Index - 1)),\r
+      (UINT16)(ATTEMPT_TARGET_NAME_VAR_OFFSET + sizeof (KEYWORD_STR) * (Index - 1)),\r
       StringToken,\r
       StringToken,\r
       0,\r
@@ -1325,9 +1333,9 @@ IScsiCreateKeywords (
     HiiSetString (mCallbackInfo->RegisteredHandle, StringToken, KeywordId, "x-UEFI-ns");\r
     HiiCreateStringOpCode (\r
       StartOpCodeHandle,\r
-      (EFI_QUESTION_ID) (ATTEMPT_TARGET_IP_ADDRESS_QUESTION_ID + (Index - 1)),\r
+      (EFI_QUESTION_ID)(ATTEMPT_TARGET_IP_ADDRESS_QUESTION_ID + (Index - 1)),\r
       CONFIGURATION_VARSTORE_ID,\r
-      (UINT16) (ATTEMPT_TARGET_IP_ADDRESS_VAR_OFFSET + sizeof (KEYWORD_STR) * (Index - 1)),\r
+      (UINT16)(ATTEMPT_TARGET_IP_ADDRESS_VAR_OFFSET + sizeof (KEYWORD_STR) * (Index - 1)),\r
       StringToken,\r
       StringToken,\r
       0,\r
@@ -1351,9 +1359,9 @@ IScsiCreateKeywords (
     HiiSetString (mCallbackInfo->RegisteredHandle, StringToken, KeywordId, "x-UEFI-ns");\r
     HiiCreateStringOpCode (\r
       StartOpCodeHandle,\r
-      (EFI_QUESTION_ID) (ATTEMPT_LUN_QUESTION_ID + (Index - 1)),\r
+      (EFI_QUESTION_ID)(ATTEMPT_LUN_QUESTION_ID + (Index - 1)),\r
       CONFIGURATION_VARSTORE_ID,\r
-      (UINT16) (ATTEMPT_LUN_VAR_OFFSET + sizeof (KEYWORD_STR) * (Index - 1)),\r
+      (UINT16)(ATTEMPT_LUN_VAR_OFFSET + sizeof (KEYWORD_STR) * (Index - 1)),\r
       StringToken,\r
       StringToken,\r
       0,\r
@@ -1376,19 +1384,19 @@ IScsiCreateKeywords (
     UnicodeSPrint (KeywordId, sizeof (KeywordId), L"iSCSIAuthenticationMethod:%d", Index);\r
     HiiSetString (mCallbackInfo->RegisteredHandle, StringToken, KeywordId, "x-UEFI-ns");\r
     HiiCreateNumericOpCode (\r
-    StartOpCodeHandle,\r
-    (EFI_QUESTION_ID) (ATTEMPT_AUTHENTICATION_METHOD_QUESTION_ID + (Index - 1)),\r
-    CONFIGURATION_VARSTORE_ID,\r
-    (UINT16) (ATTEMPT_AUTHENTICATION_METHOD_VAR_OFFSET + (Index - 1)),\r
-    StringToken,\r
-    StringToken,\r
-    0,\r
-    0,\r
-    0,\r
-    1,\r
-    0,\r
-    NULL\r
-    );\r
+      StartOpCodeHandle,\r
+      (EFI_QUESTION_ID)(ATTEMPT_AUTHENTICATION_METHOD_QUESTION_ID + (Index - 1)),\r
+      CONFIGURATION_VARSTORE_ID,\r
+      (UINT16)(ATTEMPT_AUTHENTICATION_METHOD_VAR_OFFSET + (Index - 1)),\r
+      StringToken,\r
+      StringToken,\r
+      0,\r
+      0,\r
+      0,\r
+      1,\r
+      0,\r
+      NULL\r
+      );\r
 \r
     //\r
     // Create iSCSIChapType Keyword.\r
@@ -1403,19 +1411,19 @@ IScsiCreateKeywords (
     UnicodeSPrint (KeywordId, sizeof (KeywordId), L"iSCSIChapType:%d", Index);\r
     HiiSetString (mCallbackInfo->RegisteredHandle, StringToken, KeywordId, "x-UEFI-ns");\r
     HiiCreateNumericOpCode (\r
-    StartOpCodeHandle,\r
-    (EFI_QUESTION_ID) (ATTEMPT_CHARTYPE_QUESTION_ID + (Index - 1)),\r
-    CONFIGURATION_VARSTORE_ID,\r
-    (UINT16) (ATTEMPT_CHARTYPE_VAR_OFFSET + (Index - 1)),\r
-    StringToken,\r
-    StringToken,\r
-    0,\r
-    0,\r
-    0,\r
-    1,\r
-    0,\r
-    NULL\r
-    );\r
+      StartOpCodeHandle,\r
+      (EFI_QUESTION_ID)(ATTEMPT_CHARTYPE_QUESTION_ID + (Index - 1)),\r
+      CONFIGURATION_VARSTORE_ID,\r
+      (UINT16)(ATTEMPT_CHARTYPE_VAR_OFFSET + (Index - 1)),\r
+      StringToken,\r
+      StringToken,\r
+      0,\r
+      0,\r
+      0,\r
+      1,\r
+      0,\r
+      NULL\r
+      );\r
 \r
     //\r
     // Create iSCSIChapUsername Keyword.\r
@@ -1431,9 +1439,9 @@ IScsiCreateKeywords (
     HiiSetString (mCallbackInfo->RegisteredHandle, StringToken, KeywordId, "x-UEFI-ns");\r
     HiiCreateStringOpCode (\r
       StartOpCodeHandle,\r
-      (EFI_QUESTION_ID) (ATTEMPT_CHAR_USER_NAME_QUESTION_ID + (Index - 1)),\r
+      (EFI_QUESTION_ID)(ATTEMPT_CHAR_USER_NAME_QUESTION_ID + (Index - 1)),\r
       CONFIGURATION_VARSTORE_ID,\r
-      (UINT16) (ATTEMPT_CHAR_USER_NAME_VAR_OFFSET + sizeof (KEYWORD_STR) * (Index - 1)),\r
+      (UINT16)(ATTEMPT_CHAR_USER_NAME_VAR_OFFSET + sizeof (KEYWORD_STR) * (Index - 1)),\r
       StringToken,\r
       StringToken,\r
       0,\r
@@ -1457,9 +1465,9 @@ IScsiCreateKeywords (
     HiiSetString (mCallbackInfo->RegisteredHandle, StringToken, KeywordId, "x-UEFI-ns");\r
     HiiCreateStringOpCode (\r
       StartOpCodeHandle,\r
-      (EFI_QUESTION_ID) (ATTEMPT_CHAR_SECRET_QUESTION_ID + (Index - 1)),\r
+      (EFI_QUESTION_ID)(ATTEMPT_CHAR_SECRET_QUESTION_ID + (Index - 1)),\r
       CONFIGURATION_VARSTORE_ID,\r
-      (UINT16) (ATTEMPT_CHAR_SECRET_VAR_OFFSET + sizeof (KEYWORD_STR) * (Index - 1)),\r
+      (UINT16)(ATTEMPT_CHAR_SECRET_VAR_OFFSET + sizeof (KEYWORD_STR) * (Index - 1)),\r
       StringToken,\r
       StringToken,\r
       0,\r
@@ -1483,9 +1491,9 @@ IScsiCreateKeywords (
     HiiSetString (mCallbackInfo->RegisteredHandle, StringToken, KeywordId, "x-UEFI-ns");\r
     HiiCreateStringOpCode (\r
       StartOpCodeHandle,\r
-      (EFI_QUESTION_ID) (ATTEMPT_CHAR_REVERSE_USER_NAME_QUESTION_ID + (Index - 1)),\r
+      (EFI_QUESTION_ID)(ATTEMPT_CHAR_REVERSE_USER_NAME_QUESTION_ID + (Index - 1)),\r
       CONFIGURATION_VARSTORE_ID,\r
-      (UINT16) (ATTEMPT_CHAR_REVERSE_USER_NAME_VAR_OFFSET + sizeof (KEYWORD_STR) * (Index - 1)),\r
+      (UINT16)(ATTEMPT_CHAR_REVERSE_USER_NAME_VAR_OFFSET + sizeof (KEYWORD_STR) * (Index - 1)),\r
       StringToken,\r
       StringToken,\r
       0,\r
@@ -1509,9 +1517,9 @@ IScsiCreateKeywords (
     HiiSetString (mCallbackInfo->RegisteredHandle, StringToken, KeywordId, "x-UEFI-ns");\r
     HiiCreateStringOpCode (\r
       StartOpCodeHandle,\r
-      (EFI_QUESTION_ID) (ATTEMPT_CHAR_REVERSE_SECRET_QUESTION_ID + (Index - 1)),\r
+      (EFI_QUESTION_ID)(ATTEMPT_CHAR_REVERSE_SECRET_QUESTION_ID + (Index - 1)),\r
       CONFIGURATION_VARSTORE_ID,\r
-      (UINT16) (ATTEMPT_CHAR_REVERSE_SECRET_VAR_OFFSET + sizeof (KEYWORD_STR) * (Index - 1)),\r
+      (UINT16)(ATTEMPT_CHAR_REVERSE_SECRET_VAR_OFFSET + sizeof (KEYWORD_STR) * (Index - 1)),\r
       StringToken,\r
       StringToken,\r
       0,\r
@@ -1544,12 +1552,12 @@ IScsiCreateKeywords (
 VOID\r
 IScsiCleanAttemptVariable (\r
   IN   VOID\r
-)\r
+  )\r
 {\r
-  ISCSI_ATTEMPT_CONFIG_NVDATA   *AttemptConfigData;\r
-  UINT8                         *AttemptConfigOrder;\r
-  UINTN                         AttemptConfigOrderSize;\r
-  UINTN                         Index;\r
+  ISCSI_ATTEMPT_CONFIG_NVDATA  *AttemptConfigData;\r
+  UINT8                        *AttemptConfigOrder;\r
+  UINTN                        AttemptConfigOrderSize;\r
+  UINTN                        Index;\r
 \r
   //\r
   // Get the initialized attempt order.\r
@@ -1559,14 +1567,14 @@ IScsiCleanAttemptVariable (
                          &gIScsiConfigGuid,\r
                          &AttemptConfigOrderSize\r
                          );\r
-  if (AttemptConfigOrder == NULL || AttemptConfigOrderSize == 0) {\r
+  if ((AttemptConfigOrder == NULL) || (AttemptConfigOrderSize == 0)) {\r
     return;\r
   }\r
 \r
   for (Index = 1; Index < AttemptConfigOrderSize / sizeof (UINT8); Index++) {\r
     UnicodeSPrint (\r
       mPrivate->PortString,\r
-      (UINTN) ISCSI_NAME_IFR_MAX_SIZE,\r
+      (UINTN)ISCSI_NAME_IFR_MAX_SIZE,\r
       L"Attempt %d",\r
       Index\r
       );\r
@@ -1574,7 +1582,7 @@ IScsiCleanAttemptVariable (
     GetVariable2 (\r
       mPrivate->PortString,\r
       &gEfiIScsiInitiatorNameProtocolGuid,\r
-      (VOID**)&AttemptConfigData,\r
+      (VOID **)&AttemptConfigData,\r
       NULL\r
       );\r
 \r
@@ -1588,6 +1596,7 @@ IScsiCleanAttemptVariable (
              );\r
     }\r
   }\r
+\r
   return;\r
 }\r
 \r
@@ -1601,11 +1610,11 @@ IScsiCleanAttemptVariable (
 **/\r
 ISCSI_NIC_INFO *\r
 IScsiGetNicInfoByIndex (\r
-  IN UINT8      NicIndex\r
+  IN UINT8  NicIndex\r
   )\r
 {\r
-  LIST_ENTRY        *Entry;\r
-  ISCSI_NIC_INFO    *NicInfo;\r
+  LIST_ENTRY      *Entry;\r
+  ISCSI_NIC_INFO  *NicInfo;\r
 \r
   NET_LIST_FOR_EACH (Entry, &mPrivate->NicInfoList) {\r
     NicInfo = NET_LIST_USER_STRUCT (Entry, ISCSI_NIC_INFO, Link);\r
@@ -1617,7 +1626,6 @@ IScsiGetNicInfoByIndex (
   return NULL;\r
 }\r
 \r
-\r
 /**\r
   Get the NIC's PCI location and return it according to the composited\r
   format defined in iSCSI Boot Firmware Table.\r
@@ -1647,7 +1655,7 @@ IScsiGetNICPciLocation (
   Status = gBS->HandleProtocol (\r
                   Controller,\r
                   &gEfiDevicePathProtocolGuid,\r
-                  (VOID **) &DevicePath\r
+                  (VOID **)&DevicePath\r
                   );\r
   if (EFI_ERROR (Status)) {\r
     return 0;\r
@@ -1662,7 +1670,7 @@ IScsiGetNICPciLocation (
     return 0;\r
   }\r
 \r
-  Status = gBS->HandleProtocol (PciIoHandle, &gEfiPciIoProtocolGuid, (VOID **) &PciIo);\r
+  Status = gBS->HandleProtocol (PciIoHandle, &gEfiPciIoProtocolGuid, (VOID **)&PciIo);\r
   if (EFI_ERROR (Status)) {\r
     return 0;\r
   }\r
@@ -1672,10 +1680,9 @@ IScsiGetNICPciLocation (
     return 0;\r
   }\r
 \r
-  return (UINT16) ((*Bus << 8) | (*Device << 3) | *Function);\r
+  return (UINT16)((*Bus << 8) | (*Device << 3) | *Function);\r
 }\r
 \r
-\r
 /**\r
   Read the EFI variable (VendorGuid/Name) and return a dynamically allocated\r
   buffer, and the size of the buffer. If failure, return NULL.\r
@@ -1691,9 +1698,9 @@ IScsiGetNICPciLocation (
 **/\r
 VOID *\r
 IScsiGetVariableAndSize (\r
-  IN  CHAR16              *Name,\r
-  IN  EFI_GUID            *VendorGuid,\r
-  OUT UINTN               *VariableSize\r
+  IN  CHAR16    *Name,\r
+  IN  EFI_GUID  *VendorGuid,\r
+  OUT UINTN     *VariableSize\r
   )\r
 {\r
   EFI_STATUS  Status;\r
@@ -1705,8 +1712,8 @@ IScsiGetVariableAndSize (
   //\r
   // Pass in a zero size buffer to find the required buffer size.\r
   //\r
-  BufferSize  = 0;\r
-  Status      = gRT->GetVariable (Name, VendorGuid, NULL, &BufferSize, Buffer);\r
+  BufferSize = 0;\r
+  Status     = gRT->GetVariable (Name, VendorGuid, NULL, &BufferSize, Buffer);\r
   if (Status == EFI_BUFFER_TOO_SMALL) {\r
     //\r
     // Allocate the buffer to return\r
@@ -1715,6 +1722,7 @@ IScsiGetVariableAndSize (
     if (Buffer == NULL) {\r
       return NULL;\r
     }\r
+\r
     //\r
     // Read variable into the allocated buffer.\r
     //\r
@@ -1728,7 +1736,6 @@ IScsiGetVariableAndSize (
   return Buffer;\r
 }\r
 \r
-\r
 /**\r
   Create the iSCSI driver data.\r
 \r
@@ -1745,8 +1752,8 @@ IScsiCreateDriverData (
   IN EFI_HANDLE  Controller\r
   )\r
 {\r
-  ISCSI_DRIVER_DATA *Private;\r
-  EFI_STATUS        Status;\r
+  ISCSI_DRIVER_DATA  *Private;\r
+  EFI_STATUS         Status;\r
 \r
   Private = AllocateZeroPool (sizeof (ISCSI_DRIVER_DATA));\r
   if (Private == NULL) {\r
@@ -1776,7 +1783,7 @@ IScsiCreateDriverData (
   }\r
 \r
   Private->ExtScsiPassThruHandle = NULL;\r
-  CopyMem(&Private->IScsiExtScsiPassThru, &gIScsiExtScsiPassThruProtocolTemplate, sizeof(EFI_EXT_SCSI_PASS_THRU_PROTOCOL));\r
+  CopyMem (&Private->IScsiExtScsiPassThru, &gIScsiExtScsiPassThruProtocolTemplate, sizeof (EFI_EXT_SCSI_PASS_THRU_PROTOCOL));\r
 \r
   //\r
   // 0 is designated to the TargetId, so use another value for the AdapterId.\r
@@ -1789,7 +1796,6 @@ IScsiCreateDriverData (
   return Private;\r
 }\r
 \r
-\r
 /**\r
   Clean the iSCSI driver data.\r
 \r
@@ -1804,7 +1810,7 @@ IScsiCleanDriverData (
   IN ISCSI_DRIVER_DATA  *Private\r
   )\r
 {\r
-  EFI_STATUS            Status;\r
+  EFI_STATUS  Status;\r
 \r
   Status = EFI_SUCCESS;\r
 \r
@@ -1859,24 +1865,24 @@ IScsiDhcpIsConfigured (
   IN UINT8       IpVersion\r
   )\r
 {\r
-  ISCSI_ATTEMPT_CONFIG_NVDATA *AttemptTmp;\r
-  UINT8                       *AttemptConfigOrder;\r
-  UINTN                       AttemptConfigOrderSize;\r
-  UINTN                       Index;\r
-  EFI_STATUS                  Status;\r
-  EFI_MAC_ADDRESS             MacAddr;\r
-  UINTN                       HwAddressSize;\r
-  UINT16                      VlanId;\r
-  CHAR16                      MacString[ISCSI_MAX_MAC_STRING_LEN];\r
-  CHAR16                      AttemptMacString[ISCSI_MAX_MAC_STRING_LEN];\r
-  CHAR16                      AttemptName[ISCSI_NAME_IFR_MAX_SIZE];\r
+  ISCSI_ATTEMPT_CONFIG_NVDATA  *AttemptTmp;\r
+  UINT8                        *AttemptConfigOrder;\r
+  UINTN                        AttemptConfigOrderSize;\r
+  UINTN                        Index;\r
+  EFI_STATUS                   Status;\r
+  EFI_MAC_ADDRESS              MacAddr;\r
+  UINTN                        HwAddressSize;\r
+  UINT16                       VlanId;\r
+  CHAR16                       MacString[ISCSI_MAX_MAC_STRING_LEN];\r
+  CHAR16                       AttemptMacString[ISCSI_MAX_MAC_STRING_LEN];\r
+  CHAR16                       AttemptName[ISCSI_NAME_IFR_MAX_SIZE];\r
 \r
   AttemptConfigOrder = IScsiGetVariableAndSize (\r
                          L"AttemptOrder",\r
                          &gIScsiConfigGuid,\r
                          &AttemptConfigOrderSize\r
                          );\r
-  if (AttemptConfigOrder == NULL || AttemptConfigOrderSize == 0) {\r
+  if ((AttemptConfigOrder == NULL) || (AttemptConfigOrderSize == 0)) {\r
     return FALSE;\r
   }\r
 \r
@@ -1884,29 +1890,30 @@ IScsiDhcpIsConfigured (
   // Get MAC address of this network device.\r
   //\r
   Status = NetLibGetMacAddress (Controller, &MacAddr, &HwAddressSize);\r
-  if(EFI_ERROR (Status)) {\r
+  if (EFI_ERROR (Status)) {\r
     return FALSE;\r
   }\r
+\r
   //\r
   // Get VLAN ID of this network device.\r
   //\r
   VlanId = NetLibGetVlanId (Controller);\r
-  IScsiMacAddrToStr (&MacAddr, (UINT32) HwAddressSize, VlanId, MacString);\r
+  IScsiMacAddrToStr (&MacAddr, (UINT32)HwAddressSize, VlanId, MacString);\r
 \r
   for (Index = 0; Index < AttemptConfigOrderSize / sizeof (UINT8); Index++) {\r
     UnicodeSPrint (\r
       AttemptName,\r
-      (UINTN) 128,\r
+      (UINTN)128,\r
       L"Attempt %d",\r
-      (UINTN) AttemptConfigOrder[Index]\r
+      (UINTN)AttemptConfigOrder[Index]\r
       );\r
     Status = GetVariable2 (\r
                AttemptName,\r
                &gEfiIScsiInitiatorNameProtocolGuid,\r
-               (VOID**)&AttemptTmp,\r
+               (VOID **)&AttemptTmp,\r
                NULL\r
                );\r
-    if(AttemptTmp == NULL || EFI_ERROR (Status)) {\r
+    if ((AttemptTmp == NULL) || EFI_ERROR (Status)) {\r
       continue;\r
     }\r
 \r
@@ -1917,21 +1924,23 @@ IScsiDhcpIsConfigured (
       continue;\r
     }\r
 \r
-    if (AttemptTmp->SessionConfigData.IpMode != IP_MODE_AUTOCONFIG &&\r
-        AttemptTmp->SessionConfigData.IpMode != ((IpVersion == IP_VERSION_4) ? IP_MODE_IP4 : IP_MODE_IP6)) {\r
+    if ((AttemptTmp->SessionConfigData.IpMode != IP_MODE_AUTOCONFIG) &&\r
+        (AttemptTmp->SessionConfigData.IpMode != ((IpVersion == IP_VERSION_4) ? IP_MODE_IP4 : IP_MODE_IP6)))\r
+    {\r
       FreePool (AttemptTmp);\r
       continue;\r
     }\r
 \r
     AsciiStrToUnicodeStrS (AttemptTmp->MacString, AttemptMacString, sizeof (AttemptMacString) / sizeof (AttemptMacString[0]));\r
 \r
-    if (AttemptTmp->Actived == ISCSI_ACTIVE_DISABLED || StrCmp (MacString, AttemptMacString)) {\r
+    if ((AttemptTmp->Actived == ISCSI_ACTIVE_DISABLED) || StrCmp (MacString, AttemptMacString)) {\r
       continue;\r
     }\r
 \r
-    if(AttemptTmp->SessionConfigData.IpMode == IP_MODE_AUTOCONFIG ||\r
-       AttemptTmp->SessionConfigData.InitiatorInfoFromDhcp == TRUE ||\r
-       AttemptTmp->SessionConfigData.TargetInfoFromDhcp == TRUE) {\r
+    if ((AttemptTmp->SessionConfigData.IpMode == IP_MODE_AUTOCONFIG) ||\r
+        (AttemptTmp->SessionConfigData.InitiatorInfoFromDhcp == TRUE) ||\r
+        (AttemptTmp->SessionConfigData.TargetInfoFromDhcp == TRUE))\r
+    {\r
       FreePool (AttemptTmp);\r
       FreePool (AttemptConfigOrder);\r
       return TRUE;\r
@@ -1958,24 +1967,24 @@ IScsiDnsIsConfigured (
   IN EFI_HANDLE  Controller\r
   )\r
 {\r
-  ISCSI_ATTEMPT_CONFIG_NVDATA *AttemptTmp;\r
-  UINT8                       *AttemptConfigOrder;\r
-  UINTN                       AttemptConfigOrderSize;\r
-  UINTN                       Index;\r
-  EFI_STATUS                  Status;\r
-  EFI_MAC_ADDRESS             MacAddr;\r
-  UINTN                       HwAddressSize;\r
-  UINT16                      VlanId;\r
-  CHAR16                      AttemptMacString[ISCSI_MAX_MAC_STRING_LEN];\r
-  CHAR16                      MacString[ISCSI_MAX_MAC_STRING_LEN];\r
-  CHAR16                      AttemptName[ISCSI_NAME_IFR_MAX_SIZE];\r
+  ISCSI_ATTEMPT_CONFIG_NVDATA  *AttemptTmp;\r
+  UINT8                        *AttemptConfigOrder;\r
+  UINTN                        AttemptConfigOrderSize;\r
+  UINTN                        Index;\r
+  EFI_STATUS                   Status;\r
+  EFI_MAC_ADDRESS              MacAddr;\r
+  UINTN                        HwAddressSize;\r
+  UINT16                       VlanId;\r
+  CHAR16                       AttemptMacString[ISCSI_MAX_MAC_STRING_LEN];\r
+  CHAR16                       MacString[ISCSI_MAX_MAC_STRING_LEN];\r
+  CHAR16                       AttemptName[ISCSI_NAME_IFR_MAX_SIZE];\r
 \r
   AttemptConfigOrder = IScsiGetVariableAndSize (\r
                          L"AttemptOrder",\r
                          &gIScsiConfigGuid,\r
                          &AttemptConfigOrderSize\r
                          );\r
-  if (AttemptConfigOrder == NULL || AttemptConfigOrderSize == 0) {\r
+  if ((AttemptConfigOrder == NULL) || (AttemptConfigOrderSize == 0)) {\r
     return FALSE;\r
   }\r
 \r
@@ -1983,30 +1992,31 @@ IScsiDnsIsConfigured (
   // Get MAC address of this network device.\r
   //\r
   Status = NetLibGetMacAddress (Controller, &MacAddr, &HwAddressSize);\r
-  if(EFI_ERROR (Status)) {\r
+  if (EFI_ERROR (Status)) {\r
     return FALSE;\r
   }\r
+\r
   //\r
   // Get VLAN ID of this network device.\r
   //\r
   VlanId = NetLibGetVlanId (Controller);\r
-  IScsiMacAddrToStr (&MacAddr, (UINT32) HwAddressSize, VlanId, MacString);\r
+  IScsiMacAddrToStr (&MacAddr, (UINT32)HwAddressSize, VlanId, MacString);\r
 \r
   for (Index = 0; Index < AttemptConfigOrderSize / sizeof (UINT8); Index++) {\r
     UnicodeSPrint (\r
       AttemptName,\r
-      (UINTN) 128,\r
+      (UINTN)128,\r
       L"Attempt %d",\r
-      (UINTN) AttemptConfigOrder[Index]\r
+      (UINTN)AttemptConfigOrder[Index]\r
       );\r
 \r
     Status = GetVariable2 (\r
                AttemptName,\r
                &gEfiIScsiInitiatorNameProtocolGuid,\r
-               (VOID**)&AttemptTmp,\r
+               (VOID **)&AttemptTmp,\r
                NULL\r
                );\r
-    if(AttemptTmp == NULL || EFI_ERROR (Status)) {\r
+    if ((AttemptTmp == NULL) || EFI_ERROR (Status)) {\r
       continue;\r
     }\r
 \r
@@ -2014,7 +2024,7 @@ IScsiDnsIsConfigured (
 \r
     AsciiStrToUnicodeStrS (AttemptTmp->MacString, AttemptMacString, sizeof (AttemptMacString) / sizeof (AttemptMacString[0]));\r
 \r
-    if (AttemptTmp->SessionConfigData.Enabled == ISCSI_DISABLED || StrCmp (MacString, AttemptMacString)) {\r
+    if ((AttemptTmp->SessionConfigData.Enabled == ISCSI_DISABLED) || StrCmp (MacString, AttemptMacString)) {\r
       FreePool (AttemptTmp);\r
       continue;\r
     }\r
@@ -2027,12 +2037,10 @@ IScsiDnsIsConfigured (
       FreePool (AttemptTmp);\r
       continue;\r
     }\r
-\r
   }\r
 \r
   FreePool (AttemptConfigOrder);\r
   return FALSE;\r
-\r
 }\r
 \r
 /**\r
@@ -2050,17 +2058,17 @@ IScsiGetConfigData (
   IN ISCSI_DRIVER_DATA  *Private\r
   )\r
 {\r
-  EFI_STATUS                  Status;\r
-  CHAR16                      MacString[ISCSI_MAX_MAC_STRING_LEN];\r
-  CHAR16                      AttemptMacString[ISCSI_MAX_MAC_STRING_LEN];\r
-  UINTN                       Index;\r
-  ISCSI_NIC_INFO              *NicInfo;\r
-  ISCSI_ATTEMPT_CONFIG_NVDATA *AttemptConfigData;\r
-  ISCSI_ATTEMPT_CONFIG_NVDATA *AttemptTmp;\r
-  UINT8                       *AttemptConfigOrder;\r
-  UINTN                       AttemptConfigOrderSize;\r
-  CHAR16                      IScsiMode[64];\r
-  CHAR16                      IpMode[64];\r
+  EFI_STATUS                   Status;\r
+  CHAR16                       MacString[ISCSI_MAX_MAC_STRING_LEN];\r
+  CHAR16                       AttemptMacString[ISCSI_MAX_MAC_STRING_LEN];\r
+  UINTN                        Index;\r
+  ISCSI_NIC_INFO               *NicInfo;\r
+  ISCSI_ATTEMPT_CONFIG_NVDATA  *AttemptConfigData;\r
+  ISCSI_ATTEMPT_CONFIG_NVDATA  *AttemptTmp;\r
+  UINT8                        *AttemptConfigOrder;\r
+  UINTN                        AttemptConfigOrderSize;\r
+  CHAR16                       IScsiMode[64];\r
+  CHAR16                       IpMode[64];\r
 \r
   //\r
   // There should be at least one attempt configured.\r
@@ -2070,19 +2078,19 @@ IScsiGetConfigData (
                          &gIScsiConfigGuid,\r
                          &AttemptConfigOrderSize\r
                          );\r
-  if (AttemptConfigOrder == NULL || AttemptConfigOrderSize == 0) {\r
+  if ((AttemptConfigOrder == NULL) || (AttemptConfigOrderSize == 0)) {\r
     return EFI_NOT_FOUND;\r
   }\r
 \r
   //\r
   // Get the iSCSI Initiator Name.\r
   //\r
-  mPrivate->InitiatorNameLength  = ISCSI_NAME_MAX_SIZE;\r
-  Status = gIScsiInitiatorName.Get (\r
-                                 &gIScsiInitiatorName,\r
-                                 &mPrivate->InitiatorNameLength,\r
-                                 mPrivate->InitiatorName\r
-                                 );\r
+  mPrivate->InitiatorNameLength = ISCSI_NAME_MAX_SIZE;\r
+  Status                        = gIScsiInitiatorName.Get (\r
+                                                        &gIScsiInitiatorName,\r
+                                                        &mPrivate->InitiatorNameLength,\r
+                                                        mPrivate->InitiatorName\r
+                                                        );\r
   if (EFI_ERROR (Status)) {\r
     return Status;\r
   }\r
@@ -2091,27 +2099,29 @@ IScsiGetConfigData (
   // Get the normal configuration.\r
   //\r
   for (Index = 0; Index < AttemptConfigOrderSize / sizeof (UINT8); Index++) {\r
-\r
     //\r
     // Check whether the attempt exists in AttemptConfig.\r
     //\r
     AttemptTmp = IScsiConfigGetAttemptByConfigIndex (AttemptConfigOrder[Index]);\r
-    if (AttemptTmp != NULL && AttemptTmp->SessionConfigData.Enabled == ISCSI_DISABLED) {\r
+    if ((AttemptTmp != NULL) && (AttemptTmp->SessionConfigData.Enabled == ISCSI_DISABLED)) {\r
       continue;\r
-    } else if (AttemptTmp != NULL && AttemptTmp->SessionConfigData.Enabled != ISCSI_DISABLED) {\r
+    } else if ((AttemptTmp != NULL) && (AttemptTmp->SessionConfigData.Enabled != ISCSI_DISABLED)) {\r
       //\r
       // Check the autoconfig path to see whether it should be retried.\r
       //\r
-      if (AttemptTmp->SessionConfigData.IpMode == IP_MODE_AUTOCONFIG &&\r
-          !AttemptTmp->AutoConfigureSuccess) {\r
+      if ((AttemptTmp->SessionConfigData.IpMode == IP_MODE_AUTOCONFIG) &&\r
+          !AttemptTmp->AutoConfigureSuccess)\r
+      {\r
         if (mPrivate->Ipv6Flag &&\r
-            AttemptTmp->AutoConfigureMode == IP_MODE_AUTOCONFIG_IP6) {\r
+            (AttemptTmp->AutoConfigureMode == IP_MODE_AUTOCONFIG_IP6))\r
+        {\r
           //\r
           // Autoconfigure for IP6 already attempted but failed. Do not try again.\r
           //\r
           continue;\r
         } else if (!mPrivate->Ipv6Flag &&\r
-                   AttemptTmp->AutoConfigureMode == IP_MODE_AUTOCONFIG_IP4) {\r
+                   (AttemptTmp->AutoConfigureMode == IP_MODE_AUTOCONFIG_IP4))\r
+        {\r
           //\r
           // Autoconfigure for IP4  already attempted but failed. Do not try again.\r
           //\r
@@ -2121,7 +2131,7 @@ IScsiGetConfigData (
           // Try another approach for this autoconfigure path.\r
           //\r
           AttemptTmp->AutoConfigureMode =\r
-            (UINT8) (mPrivate->Ipv6Flag ? IP_MODE_AUTOCONFIG_IP6 : IP_MODE_AUTOCONFIG_IP4);\r
+            (UINT8)(mPrivate->Ipv6Flag ? IP_MODE_AUTOCONFIG_IP6 : IP_MODE_AUTOCONFIG_IP4);\r
           AttemptTmp->SessionConfigData.InitiatorInfoFromDhcp = TRUE;\r
           AttemptTmp->SessionConfigData.TargetInfoFromDhcp    = TRUE;\r
           AttemptTmp->DhcpSuccess                             = FALSE;\r
@@ -2146,9 +2156,9 @@ IScsiGetConfigData (
           //\r
           UnicodeSPrint (\r
             mPrivate->PortString,\r
-            (UINTN) ISCSI_NAME_IFR_MAX_SIZE,\r
+            (UINTN)ISCSI_NAME_IFR_MAX_SIZE,\r
             L"Attempt %d",\r
-            (UINTN) AttemptTmp->AttemptConfigIndex\r
+            (UINTN)AttemptTmp->AttemptConfigIndex\r
             );\r
 \r
           gRT->SetVariable (\r
@@ -2163,7 +2173,8 @@ IScsiGetConfigData (
         }\r
       } else if (AttemptTmp->SessionConfigData.InitiatorInfoFromDhcp &&\r
                  !AttemptTmp->ValidPath &&\r
-                 AttemptTmp->NicIndex == mPrivate->CurrentNic) {\r
+                 (AttemptTmp->NicIndex == mPrivate->CurrentNic))\r
+      {\r
         //\r
         // If the attempt associates with the current NIC, we can\r
         // get DHCP information for already added, but failed, attempt.\r
@@ -2186,9 +2197,9 @@ IScsiGetConfigData (
         //\r
         UnicodeSPrint (\r
           mPrivate->PortString,\r
-          (UINTN) ISCSI_NAME_IFR_MAX_SIZE,\r
+          (UINTN)ISCSI_NAME_IFR_MAX_SIZE,\r
           L"Attempt %d",\r
-          (UINTN) AttemptTmp->AttemptConfigIndex\r
+          (UINTN)AttemptTmp->AttemptConfigIndex\r
           );\r
 \r
         gRT->SetVariable (\r
@@ -2200,7 +2211,6 @@ IScsiGetConfigData (
                );\r
 \r
         continue;\r
-\r
       } else {\r
         continue;\r
       }\r
@@ -2215,21 +2225,22 @@ IScsiGetConfigData (
     IScsiMacAddrToStr (&NicInfo->PermanentAddress, NicInfo->HwAddressSize, NicInfo->VlanId, MacString);\r
     UnicodeSPrint (\r
       mPrivate->PortString,\r
-      (UINTN) ISCSI_NAME_IFR_MAX_SIZE,\r
+      (UINTN)ISCSI_NAME_IFR_MAX_SIZE,\r
       L"Attempt %d",\r
-      (UINTN) AttemptConfigOrder[Index]\r
+      (UINTN)AttemptConfigOrder[Index]\r
       );\r
 \r
     GetVariable2 (\r
       mPrivate->PortString,\r
       &gEfiIScsiInitiatorNameProtocolGuid,\r
-      (VOID**)&AttemptConfigData,\r
+      (VOID **)&AttemptConfigData,\r
       NULL\r
       );\r
     AsciiStrToUnicodeStrS (AttemptConfigData->MacString, AttemptMacString, sizeof (AttemptMacString) / sizeof (AttemptMacString[0]));\r
 \r
-    if (AttemptConfigData == NULL || AttemptConfigData->Actived == ISCSI_ACTIVE_DISABLED ||\r
-        StrCmp (MacString, AttemptMacString)) {\r
+    if ((AttemptConfigData == NULL) || (AttemptConfigData->Actived == ISCSI_ACTIVE_DISABLED) ||\r
+        StrCmp (MacString, AttemptMacString))\r
+    {\r
       continue;\r
     }\r
 \r
@@ -2237,7 +2248,7 @@ IScsiGetConfigData (
 \r
     AttemptConfigData->NicIndex      = NicInfo->NicIndex;\r
     AttemptConfigData->DhcpSuccess   = FALSE;\r
-    AttemptConfigData->ValidiBFTPath = (BOOLEAN) (mPrivate->EnableMpio ? TRUE : FALSE);\r
+    AttemptConfigData->ValidiBFTPath = (BOOLEAN)(mPrivate->EnableMpio ? TRUE : FALSE);\r
     AttemptConfigData->ValidPath     = FALSE;\r
 \r
     if (AttemptConfigData->SessionConfigData.IpMode == IP_MODE_AUTOCONFIG) {\r
@@ -2245,26 +2256,28 @@ IScsiGetConfigData (
       AttemptConfigData->SessionConfigData.TargetInfoFromDhcp    = TRUE;\r
 \r
       AttemptConfigData->AutoConfigureMode =\r
-        (UINT8) (mPrivate->Ipv6Flag ? IP_MODE_AUTOCONFIG_IP6 : IP_MODE_AUTOCONFIG_IP4);\r
+        (UINT8)(mPrivate->Ipv6Flag ? IP_MODE_AUTOCONFIG_IP6 : IP_MODE_AUTOCONFIG_IP4);\r
       AttemptConfigData->AutoConfigureSuccess = FALSE;\r
     }\r
 \r
     //\r
     // Get some information from dhcp server.\r
     //\r
-    if (AttemptConfigData->SessionConfigData.Enabled != ISCSI_DISABLED &&\r
-        AttemptConfigData->SessionConfigData.InitiatorInfoFromDhcp) {\r
-\r
+    if ((AttemptConfigData->SessionConfigData.Enabled != ISCSI_DISABLED) &&\r
+        AttemptConfigData->SessionConfigData.InitiatorInfoFromDhcp)\r
+    {\r
       if (!mPrivate->Ipv6Flag &&\r
-          (AttemptConfigData->SessionConfigData.IpMode == IP_MODE_IP4 ||\r
-           AttemptConfigData->AutoConfigureMode == IP_MODE_AUTOCONFIG_IP4)) {\r
+          ((AttemptConfigData->SessionConfigData.IpMode == IP_MODE_IP4) ||\r
+           (AttemptConfigData->AutoConfigureMode == IP_MODE_AUTOCONFIG_IP4)))\r
+      {\r
         Status = IScsiDoDhcp (Private->Image, Private->Controller, AttemptConfigData);\r
         if (!EFI_ERROR (Status)) {\r
           AttemptConfigData->DhcpSuccess = TRUE;\r
         }\r
       } else if (mPrivate->Ipv6Flag &&\r
-                (AttemptConfigData->SessionConfigData.IpMode == IP_MODE_IP6 ||\r
-                 AttemptConfigData->AutoConfigureMode == IP_MODE_AUTOCONFIG_IP6)) {\r
+                 ((AttemptConfigData->SessionConfigData.IpMode == IP_MODE_IP6) ||\r
+                  (AttemptConfigData->AutoConfigureMode == IP_MODE_AUTOCONFIG_IP6)))\r
+      {\r
         Status = IScsiDoDhcp6 (Private->Image, Private->Controller, AttemptConfigData);\r
         if (!EFI_ERROR (Status)) {\r
           AttemptConfigData->DhcpSuccess = TRUE;\r
@@ -2276,9 +2289,9 @@ IScsiGetConfigData (
       //\r
       UnicodeSPrint (\r
         mPrivate->PortString,\r
-        (UINTN) ISCSI_NAME_IFR_MAX_SIZE,\r
+        (UINTN)ISCSI_NAME_IFR_MAX_SIZE,\r
         L"Attempt %d",\r
-        (UINTN) AttemptConfigData->AttemptConfigIndex\r
+        (UINTN)AttemptConfigData->AttemptConfigIndex\r
         );\r
 \r
       gRT->SetVariable (\r
@@ -2312,7 +2325,7 @@ IScsiGetConfigData (
 \r
     UnicodeSPrint (\r
       mPrivate->PortString,\r
-      (UINTN) ISCSI_NAME_IFR_MAX_SIZE,\r
+      (UINTN)ISCSI_NAME_IFR_MAX_SIZE,\r
       L"MAC: %s, PFA: Bus %d | Dev %d | Func %d, iSCSI mode: %s, IP version: %s",\r
       MacString,\r
       NicInfo->BusNumber,\r
@@ -2377,13 +2390,13 @@ IScsiGetConfigData (
     if (mPrivate->SinglePathCount == 0) {\r
       return EFI_NOT_FOUND;\r
     }\r
+\r
     mPrivate->ValidSinglePathCount = mPrivate->SinglePathCount;\r
   }\r
 \r
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
 /**\r
   Get the device path of the iSCSI tcp connection and update it.\r
 \r
@@ -2395,7 +2408,7 @@ IScsiGetConfigData (
 **/\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 IScsiGetTcpConnDevicePath (\r
-  IN ISCSI_SESSION      *Session\r
+  IN ISCSI_SESSION  *Session\r
   )\r
 {\r
   ISCSI_CONNECTION          *Conn;\r
@@ -2418,28 +2431,29 @@ IScsiGetTcpConnDevicePath (
   Status = gBS->HandleProtocol (\r
                   Conn->TcpIo.Handle,\r
                   &gEfiDevicePathProtocolGuid,\r
-                  (VOID **) &DevicePath\r
+                  (VOID **)&DevicePath\r
                   );\r
   if (EFI_ERROR (Status)) {\r
     return NULL;\r
   }\r
+\r
   //\r
   // Duplicate it.\r
   //\r
-  DevicePath  = DuplicateDevicePath (DevicePath);\r
+  DevicePath = DuplicateDevicePath (DevicePath);\r
   if (DevicePath == NULL) {\r
     return NULL;\r
   }\r
 \r
-  DPathNode   = (EFI_DEV_PATH *) DevicePath;\r
+  DPathNode = (EFI_DEV_PATH *)DevicePath;\r
 \r
   while (!IsDevicePathEnd (&DPathNode->DevPath)) {\r
     if (DevicePathType (&DPathNode->DevPath) == MESSAGING_DEVICE_PATH) {\r
-      if (!Conn->Ipv6Flag && DevicePathSubType (&DPathNode->DevPath) == MSG_IPv4_DP) {\r
-        DPathNode->Ipv4.LocalPort       = 0;\r
+      if (!Conn->Ipv6Flag && (DevicePathSubType (&DPathNode->DevPath) == MSG_IPv4_DP)) {\r
+        DPathNode->Ipv4.LocalPort = 0;\r
 \r
         DPathNode->Ipv4.StaticIpAddress =\r
-          (BOOLEAN) (!Session->ConfigData->SessionConfigData.InitiatorInfoFromDhcp);\r
+          (BOOLEAN)(!Session->ConfigData->SessionConfigData.InitiatorInfoFromDhcp);\r
 \r
         //\r
         //  Add a judgement here to support previous versions of IPv4_DEVICE_PATH.\r
@@ -2451,7 +2465,6 @@ IScsiGetTcpConnDevicePath (
         PathLen = DevicePathNodeLength (&DPathNode->Ipv4);\r
 \r
         if (PathLen == IP4_NODE_LEN_NEW_VERSIONS) {\r
-\r
           IP4_COPY_ADDRESS (\r
             &DPathNode->Ipv4.GatewayIpAddress,\r
             &Session->ConfigData->SessionConfigData.Gateway\r
@@ -2464,8 +2477,8 @@ IScsiGetTcpConnDevicePath (
         }\r
 \r
         break;\r
-      } else if (Conn->Ipv6Flag && DevicePathSubType (&DPathNode->DevPath) == MSG_IPv6_DP) {\r
-        DPathNode->Ipv6.LocalPort       = 0;\r
+      } else if (Conn->Ipv6Flag && (DevicePathSubType (&DPathNode->DevPath) == MSG_IPv6_DP)) {\r
+        DPathNode->Ipv6.LocalPort = 0;\r
 \r
         //\r
         //  Add a judgement here to support previous versions of IPv6_DEVICE_PATH.\r
@@ -2478,32 +2491,28 @@ IScsiGetTcpConnDevicePath (
         PathLen = DevicePathNodeLength (&DPathNode->Ipv6);\r
 \r
         if (PathLen == IP6_NODE_LEN_NEW_VERSIONS ) {\r
-\r
           DPathNode->Ipv6.IpAddressOrigin = 0;\r
           DPathNode->Ipv6.PrefixLength    = IP6_PREFIX_LENGTH;\r
           ZeroMem (&DPathNode->Ipv6.GatewayIpAddress, sizeof (EFI_IPv6_ADDRESS));\r
-        }\r
-        else if (PathLen == IP6_NODE_LEN_OLD_VERSIONS) {\r
-\r
+        } else if (PathLen == IP6_NODE_LEN_OLD_VERSIONS) {\r
           //\r
           //  StaticIPAddress is a field in old versions of IPv6_DEVICE_PATH, while ignored in new\r
           //  version. Set StaticIPAddress through its' offset in old IPv6_DEVICE_PATH.\r
           //\r
           *((UINT8 *)(&DPathNode->Ipv6) + IP6_OLD_IPADDRESS_OFFSET) =\r
-            (BOOLEAN) (!Session->ConfigData->SessionConfigData.InitiatorInfoFromDhcp);\r
+            (BOOLEAN)(!Session->ConfigData->SessionConfigData.InitiatorInfoFromDhcp);\r
         }\r
 \r
         break;\r
       }\r
     }\r
 \r
-    DPathNode = (EFI_DEV_PATH *) NextDevicePathNode (&DPathNode->DevPath);\r
+    DPathNode = (EFI_DEV_PATH *)NextDevicePathNode (&DPathNode->DevPath);\r
   }\r
 \r
   return DevicePath;\r
 }\r
 \r
-\r
 /**\r
   Abort the session when the transition from BS to RT is initiated.\r
 \r
@@ -2518,9 +2527,9 @@ IScsiOnExitBootService (
   IN VOID       *Context\r
   )\r
 {\r
-  ISCSI_DRIVER_DATA *Private;\r
+  ISCSI_DRIVER_DATA  *Private;\r
 \r
-  Private = (ISCSI_DRIVER_DATA *) Context;\r
+  Private = (ISCSI_DRIVER_DATA *)Context;\r
 \r
   gBS->CloseEvent (Private->ExitBootServiceEvent);\r
   Private->ExitBootServiceEvent = NULL;\r
@@ -2556,14 +2565,14 @@ IScsiOnExitBootService (
 EFI_STATUS\r
 EFIAPI\r
 IScsiTestManagedDevice (\r
-  IN  EFI_HANDLE       ControllerHandle,\r
-  IN  EFI_HANDLE       DriverBindingHandle,\r
-  IN  EFI_GUID         *ProtocolGuid\r
+  IN  EFI_HANDLE  ControllerHandle,\r
+  IN  EFI_HANDLE  DriverBindingHandle,\r
+  IN  EFI_GUID    *ProtocolGuid\r
   )\r
 {\r
-  EFI_STATUS     Status;\r
-  VOID           *ManagedInterface;\r
-  EFI_HANDLE     NicControllerHandle;\r
+  EFI_STATUS  Status;\r
+  VOID        *ManagedInterface;\r
+  EFI_HANDLE  NicControllerHandle;\r
 \r
   ASSERT (ProtocolGuid != NULL);\r
 \r
@@ -2574,7 +2583,7 @@ IScsiTestManagedDevice (
 \r
   Status = gBS->OpenProtocol (\r
                   ControllerHandle,\r
-                  (EFI_GUID *) ProtocolGuid,\r
+                  (EFI_GUID *)ProtocolGuid,\r
                   &ManagedInterface,\r
                   DriverBindingHandle,\r
                   NicControllerHandle,\r
@@ -2583,7 +2592,7 @@ IScsiTestManagedDevice (
   if (!EFI_ERROR (Status)) {\r
     gBS->CloseProtocol (\r
            ControllerHandle,\r
-           (EFI_GUID *) ProtocolGuid,\r
+           (EFI_GUID *)ProtocolGuid,\r
            DriverBindingHandle,\r
            NicControllerHandle\r
            );\r