]> git.proxmox.com Git - mirror_edk2.git/blobdiff - NetworkPkg/Ip4Dxe/Ip4Config2Impl.c
NetworkPkg: Apply uncrustify changes
[mirror_edk2.git] / NetworkPkg / Ip4Dxe / Ip4Config2Impl.c
index 9cfad5916d1037cc8451cf65f8fd8023a78eede0..0d9f8a8fe62bceb5afc311eb17a6e03717494f5b 100644 (file)
@@ -10,7 +10,7 @@
 \r
 #include "Ip4Impl.h"\r
 \r
-LIST_ENTRY  mIp4Config2InstanceList = {&mIp4Config2InstanceList, &mIp4Config2InstanceList};\r
+LIST_ENTRY  mIp4Config2InstanceList = { &mIp4Config2InstanceList, &mIp4Config2InstanceList };\r
 \r
 /**\r
   The event process routine when the DHCPv4 service binding protocol is installed\r
@@ -41,9 +41,9 @@ Ip4Config2DestroyDhcp4 (
   IN OUT IP4_CONFIG2_INSTANCE  *Instance\r
   )\r
 {\r
-  IP4_SERVICE                 *IpSb;\r
-  EFI_STATUS                  Status;\r
-  EFI_DHCP4_PROTOCOL          *Dhcp4;\r
+  IP4_SERVICE         *IpSb;\r
+  EFI_STATUS          Status;\r
+  EFI_DHCP4_PROTOCOL  *Dhcp4;\r
 \r
   Dhcp4 = Instance->Dhcp4;\r
   ASSERT (Dhcp4 != NULL);\r
@@ -90,12 +90,12 @@ Ip4Config2DestroyDhcp4 (
 **/\r
 VOID\r
 Ip4Config2OnPolicyChanged (\r
-  IN IP4_SERVICE            *IpSb,\r
-  IN EFI_IP4_CONFIG2_POLICY NewPolicy\r
+  IN IP4_SERVICE             *IpSb,\r
+  IN EFI_IP4_CONFIG2_POLICY  NewPolicy\r
   )\r
 {\r
-  IP4_INTERFACE   *IpIf;\r
-  IP4_ROUTE_TABLE *RouteTable;\r
+  IP4_INTERFACE    *IpIf;\r
+  IP4_ROUTE_TABLE  *RouteTable;\r
 \r
   //\r
   // Currently there are only two policies: static and dhcp. Regardless of\r
@@ -122,21 +122,21 @@ Ip4Config2OnPolicyChanged (
   //\r
   IpIf = Ip4CreateInterface (IpSb->Mnp, IpSb->Controller, IpSb->Image);\r
   if (IpIf == NULL) {\r
-    return ;\r
+    return;\r
   }\r
 \r
   RouteTable = Ip4CreateRouteTable ();\r
   if (RouteTable == NULL) {\r
     Ip4FreeInterface (IpIf, NULL);\r
-    return ;\r
+    return;\r
   }\r
 \r
-  IpSb->DefaultInterface  = IpIf;\r
+  IpSb->DefaultInterface = IpIf;\r
   InsertHeadList (&IpSb->Interfaces, &IpIf->Link);\r
   IpSb->DefaultRouteTable = RouteTable;\r
   Ip4ReceiveFrame (IpIf, NULL, Ip4AccpetFrame, IpSb);\r
 \r
-  if (IpSb->State == IP4_SERVICE_CONFIGED || IpSb->State == IP4_SERVICE_STARTED) {\r
+  if ((IpSb->State == IP4_SERVICE_CONFIGED) || (IpSb->State == IP4_SERVICE_STARTED)) {\r
     IpSb->State = IP4_SERVICE_UNSTARTED;\r
   }\r
 \r
@@ -146,7 +146,6 @@ Ip4Config2OnPolicyChanged (
   if (NewPolicy == Ip4Config2PolicyDhcp) {\r
     Ip4StartAutoConfig (&IpSb->Ip4Config2Instance);\r
   }\r
-\r
 }\r
 \r
 /**\r
@@ -161,12 +160,12 @@ Ip4Config2OnPolicyChanged (
 EFI_STATUS\r
 EFIAPI\r
 Ip4Config2SignalEvent (\r
-  IN NET_MAP                *Map,\r
-  IN NET_MAP_ITEM           *Item,\r
-  IN VOID                   *Arg\r
+  IN NET_MAP       *Map,\r
+  IN NET_MAP_ITEM  *Item,\r
+  IN VOID          *Arg\r
   )\r
 {\r
-  gBS->SignalEvent ((EFI_EVENT) Item->Key);\r
+  gBS->SignalEvent ((EFI_EVENT)Item->Key);\r
 \r
   return EFI_SUCCESS;\r
 }\r
@@ -187,17 +186,17 @@ Ip4Config2SignalEvent (
 **/\r
 EFI_STATUS\r
 Ip4Config2ReadConfigData (\r
-  IN     CHAR16               *VarName,\r
-  IN OUT IP4_CONFIG2_INSTANCE *Instance\r
+  IN     CHAR16                *VarName,\r
+  IN OUT IP4_CONFIG2_INSTANCE  *Instance\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
-  UINTN                   VarSize;\r
-  IP4_CONFIG2_VARIABLE    *Variable;\r
-  IP4_CONFIG2_DATA_ITEM   *DataItem;\r
-  UINTN                   Index;\r
-  IP4_CONFIG2_DATA_RECORD DataRecord;\r
-  CHAR8                   *Data;\r
+  EFI_STATUS               Status;\r
+  UINTN                    VarSize;\r
+  IP4_CONFIG2_VARIABLE     *Variable;\r
+  IP4_CONFIG2_DATA_ITEM    *DataItem;\r
+  UINTN                    Index;\r
+  IP4_CONFIG2_DATA_RECORD  DataRecord;\r
+  CHAR8                    *Data;\r
 \r
   //\r
   // Try to read the configuration variable.\r
@@ -227,7 +226,7 @@ Ip4Config2ReadConfigData (
                     &VarSize,\r
                     Variable\r
                     );\r
-    if (EFI_ERROR (Status) || (UINT16) (~NetblockChecksum ((UINT8 *) Variable, (UINT32) VarSize)) != 0) {\r
+    if (EFI_ERROR (Status) || ((UINT16)(~NetblockChecksum ((UINT8 *)Variable, (UINT32)VarSize)) != 0)) {\r
       //\r
       // GetVariable still error or the variable is corrupted.\r
       // Fall back to the default value.\r
@@ -249,15 +248,14 @@ Ip4Config2ReadConfigData (
       return EFI_NOT_FOUND;\r
     }\r
 \r
-\r
     for (Index = 0; Index < Variable->DataRecordCount; Index++) {\r
-\r
       CopyMem (&DataRecord, &Variable->DataRecord[Index], sizeof (DataRecord));\r
 \r
       DataItem = &Instance->DataItem[DataRecord.DataType];\r
       if (DATA_ATTRIB_SET (DataItem->Attribute, DATA_ATTRIB_SIZE_FIXED) &&\r
           (DataItem->DataSize != DataRecord.DataSize)\r
-          ) {\r
+          )\r
+      {\r
         //\r
         // Perhaps a corrupted data record...\r
         //\r
@@ -277,7 +275,7 @@ Ip4Config2ReadConfigData (
         }\r
       }\r
 \r
-      Data = (CHAR8 *) Variable + DataRecord.Offset;\r
+      Data = (CHAR8 *)Variable + DataRecord.Offset;\r
       CopyMem (DataItem->Data.Ptr, Data, DataRecord.DataSize);\r
 \r
       DataItem->DataSize = DataRecord.DataSize;\r
@@ -303,25 +301,23 @@ Ip4Config2ReadConfigData (
 **/\r
 EFI_STATUS\r
 Ip4Config2WriteConfigData (\r
-  IN CHAR16               *VarName,\r
-  IN IP4_CONFIG2_INSTANCE *Instance\r
+  IN CHAR16                *VarName,\r
+  IN IP4_CONFIG2_INSTANCE  *Instance\r
   )\r
 {\r
-  UINTN                   Index;\r
-  UINTN                   VarSize;\r
-  IP4_CONFIG2_DATA_ITEM   *DataItem;\r
-  IP4_CONFIG2_VARIABLE    *Variable;\r
-  IP4_CONFIG2_DATA_RECORD *DataRecord;\r
-  CHAR8                   *Heap;\r
-  EFI_STATUS              Status;\r
+  UINTN                    Index;\r
+  UINTN                    VarSize;\r
+  IP4_CONFIG2_DATA_ITEM    *DataItem;\r
+  IP4_CONFIG2_VARIABLE     *Variable;\r
+  IP4_CONFIG2_DATA_RECORD  *DataRecord;\r
+  CHAR8                    *Heap;\r
+  EFI_STATUS               Status;\r
 \r
   VarSize = sizeof (IP4_CONFIG2_VARIABLE) - sizeof (IP4_CONFIG2_DATA_RECORD);\r
 \r
   for (Index = 0; Index < Ip4Config2DataTypeMaximum; Index++) {\r
-\r
     DataItem = &Instance->DataItem[Index];\r
     if (!DATA_ATTRIB_SET (DataItem->Attribute, DATA_ATTRIB_VOLATILE) && !EFI_ERROR (DataItem->Status)) {\r
-\r
       VarSize += sizeof (IP4_CONFIG2_DATA_RECORD) + DataItem->DataSize;\r
     }\r
   }\r
@@ -331,28 +327,26 @@ Ip4Config2WriteConfigData (
     return EFI_OUT_OF_RESOURCES;\r
   }\r
 \r
-  Heap                      = (CHAR8 *) Variable + VarSize;\r
+  Heap                      = (CHAR8 *)Variable + VarSize;\r
   Variable->DataRecordCount = 0;\r
 \r
   for (Index = 0; Index < Ip4Config2DataTypeMaximum; Index++) {\r
-\r
     DataItem = &Instance->DataItem[Index];\r
     if (!DATA_ATTRIB_SET (DataItem->Attribute, DATA_ATTRIB_VOLATILE) && !EFI_ERROR (DataItem->Status)) {\r
-\r
       Heap -= DataItem->DataSize;\r
       CopyMem (Heap, DataItem->Data.Ptr, DataItem->DataSize);\r
 \r
       DataRecord           = &Variable->DataRecord[Variable->DataRecordCount];\r
-      DataRecord->DataType = (EFI_IP4_CONFIG2_DATA_TYPE) Index;\r
-      DataRecord->DataSize = (UINT32) DataItem->DataSize;\r
-      DataRecord->Offset   = (UINT16) (Heap - (CHAR8 *) Variable);\r
+      DataRecord->DataType = (EFI_IP4_CONFIG2_DATA_TYPE)Index;\r
+      DataRecord->DataSize = (UINT32)DataItem->DataSize;\r
+      DataRecord->Offset   = (UINT16)(Heap - (CHAR8 *)Variable);\r
 \r
       Variable->DataRecordCount++;\r
     }\r
   }\r
 \r
   Variable->Checksum = 0;\r
-  Variable->Checksum = (UINT16) ~NetblockChecksum ((UINT8 *) Variable, (UINT32) VarSize);\r
+  Variable->Checksum = (UINT16) ~NetblockChecksum ((UINT8 *)Variable, (UINT32)VarSize);\r
 \r
   Status = gRT->SetVariable (\r
                   VarName,\r
@@ -367,7 +361,6 @@ Ip4Config2WriteConfigData (
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Build a EFI_IP4_ROUTE_TABLE to be returned to the caller of GetModeData.\r
   The EFI_IP4_ROUTE_TABLE is clumsy to use in the internal operation of the\r
@@ -382,14 +375,14 @@ Ip4Config2WriteConfigData (
 **/\r
 EFI_STATUS\r
 Ip4Config2BuildDefaultRouteTable (\r
-  IN  IP4_SERVICE               *IpSb,\r
-  OUT EFI_IP4_ROUTE_TABLE       *Table\r
+  IN  IP4_SERVICE          *IpSb,\r
+  OUT EFI_IP4_ROUTE_TABLE  *Table\r
   )\r
 {\r
-  LIST_ENTRY                *Entry;\r
-  IP4_ROUTE_ENTRY           *RtEntry;\r
-  UINT32                    Count;\r
-  INT32                     Index;\r
+  LIST_ENTRY       *Entry;\r
+  IP4_ROUTE_ENTRY  *RtEntry;\r
+  UINT32           Count;\r
+  INT32            Index;\r
 \r
   if (IpSb->DefaultRouteTable == NULL) {\r
     return EFI_NOT_FOUND;\r
@@ -410,7 +403,6 @@ Ip4Config2BuildDefaultRouteTable (
   Count = 0;\r
 \r
   for (Index = IP4_MASK_MAX; Index >= 0; Index--) {\r
-\r
     NET_LIST_FOR_EACH (Entry, &(IpSb->DefaultRouteTable->RouteArea[Index])) {\r
       RtEntry = NET_LIST_USER_STRUCT (Entry, IP4_ROUTE_ENTRY, Link);\r
 \r
@@ -420,7 +412,6 @@ Ip4Config2BuildDefaultRouteTable (
 \r
       Count++;\r
     }\r
-\r
   }\r
 \r
   return EFI_SUCCESS;\r
@@ -443,13 +434,13 @@ Ip4Config2OnDhcp4SbInstalled (
 {\r
   IP4_CONFIG2_INSTANCE  *Instance;\r
 \r
-  Instance = (IP4_CONFIG2_INSTANCE *) Context;\r
+  Instance = (IP4_CONFIG2_INSTANCE *)Context;\r
 \r
   if ((Instance->Dhcp4Handle != NULL) || (Instance->Policy != Ip4Config2PolicyDhcp)) {\r
     //\r
     // The DHCP4 child is already created or the policy is no longer DHCP.\r
     //\r
-    return ;\r
+    return;\r
   }\r
 \r
   Ip4StartAutoConfig (Instance);\r
@@ -468,18 +459,18 @@ Ip4Config2OnDhcp4SbInstalled (
 **/\r
 EFI_STATUS\r
 Ip4Config2SetDefaultAddr (\r
-  IN IP4_SERVICE            *IpSb,\r
-  IN IP4_ADDR               StationAddress,\r
-  IN IP4_ADDR               SubnetMask\r
+  IN IP4_SERVICE  *IpSb,\r
+  IN IP4_ADDR     StationAddress,\r
+  IN IP4_ADDR     SubnetMask\r
   )\r
 {\r
-  EFI_STATUS                Status;\r
-  IP4_INTERFACE             *IpIf;\r
-  IP4_PROTOCOL              *Ip4Instance;\r
-  EFI_ARP_PROTOCOL          *Arp;\r
-  LIST_ENTRY                *Entry;\r
-  IP4_ADDR                  Subnet;\r
-  IP4_ROUTE_TABLE           *RouteTable;\r
+  EFI_STATUS        Status;\r
+  IP4_INTERFACE     *IpIf;\r
+  IP4_PROTOCOL      *Ip4Instance;\r
+  EFI_ARP_PROTOCOL  *Arp;\r
+  LIST_ENTRY        *Entry;\r
+  IP4_ADDR          Subnet;\r
+  IP4_ROUTE_TABLE   *RouteTable;\r
 \r
   IpIf = IpSb->DefaultInterface;\r
   ASSERT (IpIf != NULL);\r
@@ -515,7 +506,7 @@ Ip4Config2SetDefaultAddr (
       return EFI_OUT_OF_RESOURCES;\r
     }\r
 \r
-    IpSb->DefaultInterface  = IpIf;\r
+    IpSb->DefaultInterface = IpIf;\r
     InsertHeadList (&IpSb->Interfaces, &IpIf->Link);\r
     IpSb->DefaultRouteTable = RouteTable;\r
     Ip4ReceiveFrame (IpIf, NULL, Ip4AccpetFrame, IpSb);\r
@@ -539,14 +530,14 @@ Ip4Config2SetDefaultAddr (
     Arp = NULL;\r
     NET_LIST_FOR_EACH (Entry, &IpIf->IpInstances) {\r
       Ip4Instance = NET_LIST_USER_STRUCT_S (Entry, IP4_PROTOCOL, AddrLink, IP4_PROTOCOL_SIGNATURE);\r
-      Status = gBS->OpenProtocol (\r
-                      IpIf->ArpHandle,\r
-                      &gEfiArpProtocolGuid,\r
-                      (VOID **) &Arp,\r
-                      gIp4DriverBinding.DriverBindingHandle,\r
-                      Ip4Instance->Handle,\r
-                      EFI_OPEN_PROTOCOL_BY_CHILD_CONTROLLER\r
-                      );\r
+      Status      = gBS->OpenProtocol (\r
+                           IpIf->ArpHandle,\r
+                           &gEfiArpProtocolGuid,\r
+                           (VOID **)&Arp,\r
+                           gIp4DriverBinding.DriverBindingHandle,\r
+                           Ip4Instance->Handle,\r
+                           EFI_OPEN_PROTOCOL_BY_CHILD_CONTROLLER\r
+                           );\r
       if (EFI_ERROR (Status)) {\r
         return Status;\r
       }\r
@@ -565,7 +556,7 @@ Ip4Config2SetDefaultAddr (
     IP4_ALLZERO_ADDRESS\r
     );\r
 \r
-  IpSb->State = IP4_SERVICE_CONFIGED;\r
+  IpSb->State    = IP4_SERVICE_CONFIGED;\r
   IpSb->Reconfig = FALSE;\r
 \r
   return EFI_SUCCESS;\r
@@ -585,14 +576,14 @@ Ip4Config2SetDefaultAddr (
 **/\r
 EFI_STATUS\r
 Ip4Config2SetDefaultIf (\r
-  IN IP4_CONFIG2_INSTANCE   *Instance,\r
-  IN IP4_ADDR               StationAddress,\r
-  IN IP4_ADDR               SubnetMask,\r
-  IN IP4_ADDR               GatewayAddress\r
+  IN IP4_CONFIG2_INSTANCE  *Instance,\r
+  IN IP4_ADDR              StationAddress,\r
+  IN IP4_ADDR              SubnetMask,\r
+  IN IP4_ADDR              GatewayAddress\r
   )\r
 {\r
-  EFI_STATUS                Status;\r
-  IP4_SERVICE               *IpSb;\r
+  EFI_STATUS   Status;\r
+  IP4_SERVICE  *IpSb;\r
 \r
   IpSb = IP4_SERVICE_FROM_IP4_CONFIG2_INSTANCE (Instance);\r
 \r
@@ -623,7 +614,6 @@ Ip4Config2SetDefaultIf (
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
 /**\r
   Release all the DHCP related resources.\r
 \r
@@ -634,10 +624,10 @@ Ip4Config2SetDefaultIf (
 **/\r
 VOID\r
 Ip4Config2CleanDhcp4 (\r
-  IN IP4_CONFIG2_INSTANCE   *Instance\r
+  IN IP4_CONFIG2_INSTANCE  *Instance\r
   )\r
 {\r
-  IP4_SERVICE               *IpSb;\r
+  IP4_SERVICE  *IpSb;\r
 \r
   IpSb = IP4_SERVICE_FROM_IP4_CONFIG2_INSTANCE (Instance);\r
 \r
@@ -645,11 +635,11 @@ Ip4Config2CleanDhcp4 (
     Instance->Dhcp4->Stop (Instance->Dhcp4);\r
 \r
     gBS->CloseProtocol (\r
-          Instance->Dhcp4Handle,\r
-          &gEfiDhcp4ProtocolGuid,\r
-          IpSb->Image,\r
-          IpSb->Controller\r
-          );\r
+           Instance->Dhcp4Handle,\r
+           &gEfiDhcp4ProtocolGuid,\r
+           IpSb->Image,\r
+           IpSb->Controller\r
+           );\r
 \r
     Instance->Dhcp4 = NULL;\r
   }\r
@@ -692,28 +682,28 @@ Ip4Config2CleanDhcp4 (
 **/\r
 EFI_STATUS\r
 Ip4Config2SetDnsServerWorker (\r
-  IN IP4_CONFIG2_INSTANCE    *Instance,\r
-  IN UINTN                   DataSize,\r
-  IN VOID                    *Data\r
+  IN IP4_CONFIG2_INSTANCE  *Instance,\r
+  IN UINTN                 DataSize,\r
+  IN VOID                  *Data\r
   )\r
 {\r
-  UINTN                 OldIndex;\r
-  UINTN                 NewIndex;\r
-  EFI_IPv4_ADDRESS      *OldDns;\r
-  EFI_IPv4_ADDRESS      *NewDns;\r
-  UINTN                 OldDnsCount;\r
-  UINTN                 NewDnsCount;\r
-  IP4_CONFIG2_DATA_ITEM *Item;\r
-  BOOLEAN               OneAdded;\r
-  VOID                  *Tmp;\r
-  IP4_ADDR              DnsAddress;\r
+  UINTN                  OldIndex;\r
+  UINTN                  NewIndex;\r
+  EFI_IPv4_ADDRESS       *OldDns;\r
+  EFI_IPv4_ADDRESS       *NewDns;\r
+  UINTN                  OldDnsCount;\r
+  UINTN                  NewDnsCount;\r
+  IP4_CONFIG2_DATA_ITEM  *Item;\r
+  BOOLEAN                OneAdded;\r
+  VOID                   *Tmp;\r
+  IP4_ADDR               DnsAddress;\r
 \r
   if ((DataSize % sizeof (EFI_IPv4_ADDRESS) != 0) || (DataSize == 0)) {\r
     return EFI_BAD_BUFFER_SIZE;\r
   }\r
 \r
   Item        = &Instance->DataItem[Ip4Config2DataTypeDnsServer];\r
-  NewDns      = (EFI_IPv4_ADDRESS *) Data;\r
+  NewDns      = (EFI_IPv4_ADDRESS *)Data;\r
   OldDns      = Item->Data.DnsServers;\r
   NewDnsCount = DataSize / sizeof (EFI_IPv4_ADDRESS);\r
   OldDnsCount = Item->DataSize / sizeof (EFI_IPv4_ADDRESS);\r
@@ -737,6 +727,7 @@ Ip4Config2SetDnsServerWorker (
       if (Tmp != NULL) {\r
         FreePool (Tmp);\r
       }\r
+\r
       return EFI_INVALID_PARAMETER;\r
     }\r
 \r
@@ -773,6 +764,7 @@ Ip4Config2SetDnsServerWorker (
       if (Item->Data.Ptr != NULL) {\r
         FreePool (Item->Data.Ptr);\r
       }\r
+\r
       Item->Data.Ptr = Tmp;\r
     }\r
 \r
@@ -783,8 +775,6 @@ Ip4Config2SetDnsServerWorker (
   }\r
 }\r
 \r
-\r
-\r
 /**\r
   Callback function when DHCP process finished. It will save the\r
   retrieved IP configure parameter from DHCP to the NVRam.\r
@@ -798,21 +788,21 @@ Ip4Config2SetDnsServerWorker (
 VOID\r
 EFIAPI\r
 Ip4Config2OnDhcp4Complete (\r
-  IN EFI_EVENT              Event,\r
-  IN VOID                   *Context\r
+  IN EFI_EVENT  Event,\r
+  IN VOID       *Context\r
   )\r
 {\r
-  IP4_CONFIG2_INSTANCE      *Instance;\r
-  EFI_DHCP4_MODE_DATA       Dhcp4Mode;\r
-  EFI_STATUS                Status;\r
-  IP4_ADDR                  StationAddress;\r
-  IP4_ADDR                  SubnetMask;\r
-  IP4_ADDR                  GatewayAddress;\r
-  UINT32                    Index;\r
-  UINT32                    OptionCount;\r
-  EFI_DHCP4_PACKET_OPTION   **OptionList;\r
-\r
-  Instance = (IP4_CONFIG2_INSTANCE *) Context;\r
+  IP4_CONFIG2_INSTANCE     *Instance;\r
+  EFI_DHCP4_MODE_DATA      Dhcp4Mode;\r
+  EFI_STATUS               Status;\r
+  IP4_ADDR                 StationAddress;\r
+  IP4_ADDR                 SubnetMask;\r
+  IP4_ADDR                 GatewayAddress;\r
+  UINT32                   Index;\r
+  UINT32                   OptionCount;\r
+  EFI_DHCP4_PACKET_OPTION  **OptionList;\r
+\r
+  Instance = (IP4_CONFIG2_INSTANCE *)Context;\r
   ASSERT (Instance->Dhcp4 != NULL);\r
 \r
   //\r
@@ -826,7 +816,7 @@ Ip4Config2OnDhcp4Complete (
 \r
   if (Dhcp4Mode.State == Dhcp4Bound) {\r
     StationAddress = EFI_NTOHL (Dhcp4Mode.ClientAddress);\r
-    SubnetMask = EFI_NTOHL (Dhcp4Mode.SubnetMask);\r
+    SubnetMask     = EFI_NTOHL (Dhcp4Mode.SubnetMask);\r
     GatewayAddress = EFI_NTOHL (Dhcp4Mode.RouterAddress);\r
 \r
     Status = Ip4Config2SetDefaultIf (Instance, StationAddress, SubnetMask, GatewayAddress);\r
@@ -840,7 +830,7 @@ Ip4Config2OnDhcp4Complete (
     OptionCount = 0;\r
     OptionList  = NULL;\r
 \r
-    Status      = Instance->Dhcp4->Parse (Instance->Dhcp4, Dhcp4Mode.ReplyPacket, &OptionCount, OptionList);\r
+    Status = Instance->Dhcp4->Parse (Instance->Dhcp4, Dhcp4Mode.ReplyPacket, &OptionCount, OptionList);\r
     if (Status != EFI_BUFFER_TOO_SMALL) {\r
       goto Exit;\r
     }\r
@@ -880,7 +870,6 @@ Exit:
   DispatchDpc ();\r
 }\r
 \r
-\r
 /**\r
   Start the DHCP configuration for this IP service instance.\r
   It will locates the EFI_IP4_CONFIG2_PROTOCOL, then start the\r
@@ -894,15 +883,15 @@ Exit:
 **/\r
 EFI_STATUS\r
 Ip4StartAutoConfig (\r
-  IN IP4_CONFIG2_INSTANCE   *Instance\r
+  IN IP4_CONFIG2_INSTANCE  *Instance\r
   )\r
 {\r
-  IP4_SERVICE                    *IpSb;\r
-  EFI_DHCP4_PROTOCOL             *Dhcp4;\r
-  EFI_DHCP4_MODE_DATA            Dhcp4Mode;\r
-  EFI_DHCP4_PACKET_OPTION        *OptionList[1];\r
-  IP4_CONFIG2_DHCP4_OPTION       ParaList;\r
-  EFI_STATUS                     Status;\r
+  IP4_SERVICE               *IpSb;\r
+  EFI_DHCP4_PROTOCOL        *Dhcp4;\r
+  EFI_DHCP4_MODE_DATA       Dhcp4Mode;\r
+  EFI_DHCP4_PACKET_OPTION   *OptionList[1];\r
+  IP4_CONFIG2_DHCP4_OPTION  ParaList;\r
+  EFI_STATUS                Status;\r
 \r
   IpSb = IP4_SERVICE_FROM_IP4_CONFIG2_INSTANCE (Instance);\r
 \r
@@ -934,7 +923,7 @@ Ip4StartAutoConfig (
                                        &gEfiDhcp4ServiceBindingProtocolGuid,\r
                                        TPL_CALLBACK,\r
                                        Ip4Config2OnDhcp4SbInstalled,\r
-                                       (VOID *) Instance,\r
+                                       (VOID *)Instance,\r
                                        &Instance->Registration\r
                                        );\r
     }\r
@@ -951,7 +940,7 @@ Ip4StartAutoConfig (
   Status = gBS->OpenProtocol (\r
                   Instance->Dhcp4Handle,\r
                   &gEfiDhcp4ProtocolGuid,\r
-                  (VOID **) &Instance->Dhcp4,\r
+                  (VOID **)&Instance->Dhcp4,\r
                   IpSb->Image,\r
                   IpSb->Controller,\r
                   EFI_OPEN_PROTOCOL_BY_DRIVER\r
@@ -1048,8 +1037,6 @@ Ip4StartAutoConfig (
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
-\r
 /**\r
   The work function is to get the interface information of the communication\r
   device this IP4_CONFIG2_INSTANCE manages.\r
@@ -1069,16 +1056,16 @@ Ip4StartAutoConfig (
 **/\r
 EFI_STATUS\r
 Ip4Config2GetIfInfo (\r
-  IN IP4_CONFIG2_INSTANCE *Instance,\r
-  IN OUT UINTN            *DataSize,\r
-  IN VOID                 *Data      OPTIONAL\r
+  IN IP4_CONFIG2_INSTANCE  *Instance,\r
+  IN OUT UINTN             *DataSize,\r
+  IN VOID                  *Data      OPTIONAL\r
   )\r
 {\r
-  IP4_SERVICE                    *IpSb;\r
-  UINTN                          Length;\r
-  IP4_CONFIG2_DATA_ITEM          *Item;\r
-  EFI_IP4_CONFIG2_INTERFACE_INFO *IfInfo;\r
-  IP4_ADDR                       Address;\r
+  IP4_SERVICE                     *IpSb;\r
+  UINTN                           Length;\r
+  IP4_CONFIG2_DATA_ITEM           *Item;\r
+  EFI_IP4_CONFIG2_INTERFACE_INFO  *IfInfo;\r
+  IP4_ADDR                        Address;\r
 \r
   IpSb   = IP4_SERVICE_FROM_IP4_CONFIG2_INSTANCE (Instance);\r
   Length = sizeof (EFI_IP4_CONFIG2_INTERFACE_INFO);\r
@@ -1095,8 +1082,8 @@ Ip4Config2GetIfInfo (
   //\r
   // Copy the fixed size part of the interface info.\r
   //\r
-  Item = &Instance->DataItem[Ip4Config2DataTypeInterfaceInfo];\r
-  IfInfo = (EFI_IP4_CONFIG2_INTERFACE_INFO *) Data;\r
+  Item   = &Instance->DataItem[Ip4Config2DataTypeInterfaceInfo];\r
+  IfInfo = (EFI_IP4_CONFIG2_INTERFACE_INFO *)Data;\r
   CopyMem (IfInfo, Item->Data.Ptr, sizeof (EFI_IP4_CONFIG2_INTERFACE_INFO));\r
 \r
   //\r
@@ -1111,7 +1098,7 @@ Ip4Config2GetIfInfo (
 \r
   if (IpSb->DefaultRouteTable != NULL) {\r
     IfInfo->RouteTableSize = IpSb->DefaultRouteTable->TotalNum;\r
-    IfInfo->RouteTable   = (EFI_IP4_ROUTE_TABLE *) ((UINT8 *) Data + sizeof (EFI_IP4_CONFIG2_INTERFACE_INFO));\r
+    IfInfo->RouteTable     = (EFI_IP4_ROUTE_TABLE *)((UINT8 *)Data + sizeof (EFI_IP4_CONFIG2_INTERFACE_INFO));\r
 \r
     Ip4Config2BuildDefaultRouteTable (IpSb, IfInfo->RouteTable);\r
   }\r
@@ -1137,27 +1124,27 @@ Ip4Config2GetIfInfo (
 **/\r
 EFI_STATUS\r
 Ip4Config2SetPolicy (\r
-  IN IP4_CONFIG2_INSTANCE *Instance,\r
-  IN UINTN                DataSize,\r
-  IN VOID                 *Data\r
+  IN IP4_CONFIG2_INSTANCE  *Instance,\r
+  IN UINTN                 DataSize,\r
+  IN VOID                  *Data\r
   )\r
 {\r
-  EFI_IP4_CONFIG2_POLICY NewPolicy;\r
-  IP4_CONFIG2_DATA_ITEM  *DataItem;\r
-  IP4_SERVICE            *IpSb;\r
+  EFI_IP4_CONFIG2_POLICY  NewPolicy;\r
+  IP4_CONFIG2_DATA_ITEM   *DataItem;\r
+  IP4_SERVICE             *IpSb;\r
 \r
   if (DataSize != sizeof (EFI_IP4_CONFIG2_POLICY)) {\r
     return EFI_BAD_BUFFER_SIZE;\r
   }\r
 \r
-  NewPolicy = *((EFI_IP4_CONFIG2_POLICY *) Data);\r
+  NewPolicy = *((EFI_IP4_CONFIG2_POLICY *)Data);\r
 \r
   if (NewPolicy >= Ip4Config2PolicyMax) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
   if (NewPolicy == Instance->Policy) {\r
-    if (NewPolicy != Ip4Config2PolicyDhcp || Instance->DhcpSuccess) {\r
+    if ((NewPolicy != Ip4Config2PolicyDhcp) || Instance->DhcpSuccess) {\r
       return EFI_ABORTED;\r
     }\r
   } else {\r
@@ -1165,28 +1152,31 @@ Ip4Config2SetPolicy (
     // The policy is changed. Clean the ManualAddress, Gateway and DnsServers,\r
     // shrink the variable data size, and fire up all the related events.\r
     //\r
-    DataItem           = &Instance->DataItem[Ip4Config2DataTypeManualAddress];\r
+    DataItem = &Instance->DataItem[Ip4Config2DataTypeManualAddress];\r
     if (DataItem->Data.Ptr != NULL) {\r
       FreePool (DataItem->Data.Ptr);\r
     }\r
+\r
     DataItem->Data.Ptr = NULL;\r
     DataItem->DataSize = 0;\r
     DataItem->Status   = EFI_NOT_FOUND;\r
     NetMapIterate (&DataItem->EventMap, Ip4Config2SignalEvent, NULL);\r
 \r
-    DataItem           = &Instance->DataItem[Ip4Config2DataTypeGateway];\r
+    DataItem = &Instance->DataItem[Ip4Config2DataTypeGateway];\r
     if (DataItem->Data.Ptr != NULL) {\r
       FreePool (DataItem->Data.Ptr);\r
     }\r
+\r
     DataItem->Data.Ptr = NULL;\r
     DataItem->DataSize = 0;\r
     DataItem->Status   = EFI_NOT_FOUND;\r
     NetMapIterate (&DataItem->EventMap, Ip4Config2SignalEvent, NULL);\r
 \r
-    DataItem           = &Instance->DataItem[Ip4Config2DataTypeDnsServer];\r
+    DataItem = &Instance->DataItem[Ip4Config2DataTypeDnsServer];\r
     if (DataItem->Data.Ptr != NULL) {\r
       FreePool (DataItem->Data.Ptr);\r
     }\r
+\r
     DataItem->Data.Ptr = NULL;\r
     DataItem->DataSize = 0;\r
     DataItem->Status   = EFI_NOT_FOUND;\r
@@ -1244,20 +1234,20 @@ Ip4Config2SetPolicy (
 **/\r
 EFI_STATUS\r
 Ip4Config2SetManualAddress (\r
-  IN IP4_CONFIG2_INSTANCE *Instance,\r
-  IN UINTN                DataSize,\r
-  IN VOID                 *Data\r
+  IN IP4_CONFIG2_INSTANCE  *Instance,\r
+  IN UINTN                 DataSize,\r
+  IN VOID                  *Data\r
   )\r
 {\r
-  EFI_IP4_CONFIG2_MANUAL_ADDRESS NewAddress;\r
-  IP4_CONFIG2_DATA_ITEM          *DataItem;\r
-  EFI_STATUS                     Status;\r
-  IP4_ADDR                       StationAddress;\r
-  IP4_ADDR                       SubnetMask;\r
-  VOID                           *Ptr;\r
-  IP4_SERVICE                    *IpSb;\r
-  IP4_INTERFACE                  *IpIf;\r
-  IP4_ROUTE_TABLE                *RouteTable;\r
+  EFI_IP4_CONFIG2_MANUAL_ADDRESS  NewAddress;\r
+  IP4_CONFIG2_DATA_ITEM           *DataItem;\r
+  EFI_STATUS                      Status;\r
+  IP4_ADDR                        StationAddress;\r
+  IP4_ADDR                        SubnetMask;\r
+  VOID                            *Ptr;\r
+  IP4_SERVICE                     *IpSb;\r
+  IP4_INTERFACE                   *IpIf;\r
+  IP4_ROUTE_TABLE                 *RouteTable;\r
 \r
   DataItem   = NULL;\r
   Status     = EFI_SUCCESS;\r
@@ -1279,11 +1269,11 @@ Ip4Config2SetManualAddress (
 \r
   DataItem = &Instance->DataItem[Ip4Config2DataTypeManualAddress];\r
 \r
-  if (Data != NULL && DataSize != 0) {\r
-    NewAddress = *((EFI_IP4_CONFIG2_MANUAL_ADDRESS *) Data);\r
+  if ((Data != NULL) && (DataSize != 0)) {\r
+    NewAddress = *((EFI_IP4_CONFIG2_MANUAL_ADDRESS *)Data);\r
 \r
     StationAddress = EFI_NTOHL (NewAddress.Address);\r
-    SubnetMask = EFI_NTOHL (NewAddress.SubnetMask);\r
+    SubnetMask     = EFI_NTOHL (NewAddress.SubnetMask);\r
 \r
     //\r
     // Check whether the StationAddress/SubnetMask pair is valid.\r
@@ -1310,14 +1300,15 @@ Ip4Config2SetManualAddress (
     DataItem->Status   = EFI_NOT_READY;\r
 \r
     IpSb->Reconfig = TRUE;\r
-    Status = Ip4Config2SetDefaultAddr (IpSb, StationAddress, SubnetMask);\r
+    Status         = Ip4Config2SetDefaultAddr (IpSb, StationAddress, SubnetMask);\r
 \r
     DataItem->Status = Status;\r
 \r
-    if (EFI_ERROR (DataItem->Status) && DataItem->Status != EFI_NOT_READY) {\r
+    if (EFI_ERROR (DataItem->Status) && (DataItem->Status != EFI_NOT_READY)) {\r
       if (Ptr != NULL) {\r
         FreePool (Ptr);\r
       }\r
+\r
       DataItem->Data.Ptr = NULL;\r
     }\r
   } else {\r
@@ -1327,6 +1318,7 @@ Ip4Config2SetManualAddress (
     if (DataItem->Data.Ptr != NULL) {\r
       FreePool (DataItem->Data.Ptr);\r
     }\r
+\r
     DataItem->Data.Ptr = NULL;\r
     DataItem->DataSize = 0;\r
     DataItem->Status   = EFI_NOT_FOUND;\r
@@ -1362,7 +1354,7 @@ Ip4Config2SetManualAddress (
       return EFI_OUT_OF_RESOURCES;\r
     }\r
 \r
-    IpSb->DefaultInterface  = IpIf;\r
+    IpSb->DefaultInterface = IpIf;\r
     InsertHeadList (&IpSb->Interfaces, &IpIf->Link);\r
     IpSb->DefaultRouteTable = RouteTable;\r
     Ip4ReceiveFrame (IpIf, NULL, Ip4AccpetFrame, IpSb);\r
@@ -1370,7 +1362,7 @@ Ip4Config2SetManualAddress (
     //\r
     // Reset the State to unstarted.\r
     //\r
-    if (IpSb->State == IP4_SERVICE_CONFIGED || IpSb->State == IP4_SERVICE_STARTED) {\r
+    if ((IpSb->State == IP4_SERVICE_CONFIGED) || (IpSb->State == IP4_SERVICE_STARTED)) {\r
       IpSb->State = IP4_SERVICE_UNSTARTED;\r
     }\r
   }\r
@@ -1402,30 +1394,30 @@ Ip4Config2SetManualAddress (
 **/\r
 EFI_STATUS\r
 Ip4Config2SetGateway (\r
-  IN IP4_CONFIG2_INSTANCE *Instance,\r
-  IN UINTN                DataSize,\r
-  IN VOID                 *Data\r
+  IN IP4_CONFIG2_INSTANCE  *Instance,\r
+  IN UINTN                 DataSize,\r
+  IN VOID                  *Data\r
   )\r
 {\r
-  IP4_SERVICE                    *IpSb;\r
-  IP4_CONFIG2_DATA_ITEM          *DataItem;\r
-  IP4_ADDR                       Gateway;\r
-\r
-  UINTN                 Index1;\r
-  UINTN                 Index2;\r
-  EFI_IPv4_ADDRESS      *OldGateway;\r
-  EFI_IPv4_ADDRESS      *NewGateway;\r
-  UINTN                 OldGatewayCount;\r
-  UINTN                 NewGatewayCount;\r
-  BOOLEAN               OneRemoved;\r
-  BOOLEAN               OneAdded;\r
-  VOID                  *Tmp;\r
-\r
-  OldGateway      = NULL;\r
-  NewGateway      = NULL;\r
-  OneRemoved      = FALSE;\r
-  OneAdded        = FALSE;\r
-  Tmp             = NULL;\r
+  IP4_SERVICE            *IpSb;\r
+  IP4_CONFIG2_DATA_ITEM  *DataItem;\r
+  IP4_ADDR               Gateway;\r
+\r
+  UINTN             Index1;\r
+  UINTN             Index2;\r
+  EFI_IPv4_ADDRESS  *OldGateway;\r
+  EFI_IPv4_ADDRESS  *NewGateway;\r
+  UINTN             OldGatewayCount;\r
+  UINTN             NewGatewayCount;\r
+  BOOLEAN           OneRemoved;\r
+  BOOLEAN           OneAdded;\r
+  VOID              *Tmp;\r
+\r
+  OldGateway = NULL;\r
+  NewGateway = NULL;\r
+  OneRemoved = FALSE;\r
+  OneAdded   = FALSE;\r
+  Tmp        = NULL;\r
 \r
   if ((DataSize != 0) && (DataSize % sizeof (EFI_IPv4_ADDRESS) != 0)) {\r
     return EFI_BAD_BUFFER_SIZE;\r
@@ -1455,14 +1447,15 @@ Ip4Config2SetGateway (
     OneRemoved = TRUE;\r
   }\r
 \r
-  if (Data != NULL && DataSize != 0) {\r
-    NewGateway      = (EFI_IPv4_ADDRESS *) Data;\r
+  if ((Data != NULL) && (DataSize != 0)) {\r
+    NewGateway      = (EFI_IPv4_ADDRESS *)Data;\r
     NewGatewayCount = DataSize / sizeof (EFI_IPv4_ADDRESS);\r
     for (Index1 = 0; Index1 < NewGatewayCount; Index1++) {\r
       CopyMem (&Gateway, NewGateway + Index1, sizeof (IP4_ADDR));\r
 \r
       if ((IpSb->DefaultInterface->SubnetMask != 0) &&\r
-          !NetIp4IsUnicast (NTOHL (Gateway), IpSb->DefaultInterface->SubnetMask)) {\r
+          !NetIp4IsUnicast (NTOHL (Gateway), IpSb->DefaultInterface->SubnetMask))\r
+      {\r
         return EFI_INVALID_PARAMETER;\r
       }\r
 \r
@@ -1505,6 +1498,7 @@ Ip4Config2SetGateway (
         if (DataItem->Data.Ptr != NULL) {\r
           FreePool (DataItem->Data.Ptr);\r
         }\r
+\r
         DataItem->Data.Ptr = Tmp;\r
       }\r
 \r
@@ -1519,6 +1513,7 @@ Ip4Config2SetGateway (
     if (DataItem->Data.Ptr != NULL) {\r
       FreePool (DataItem->Data.Ptr);\r
     }\r
+\r
     DataItem->Data.Ptr = NULL;\r
     DataItem->DataSize = 0;\r
     DataItem->Status   = EFI_NOT_FOUND;\r
@@ -1551,13 +1546,13 @@ Ip4Config2SetGateway (
 **/\r
 EFI_STATUS\r
 Ip4Config2SetDnsServer (\r
-  IN IP4_CONFIG2_INSTANCE *Instance,\r
-  IN UINTN                DataSize,\r
-  IN VOID                 *Data\r
+  IN IP4_CONFIG2_INSTANCE  *Instance,\r
+  IN UINTN                 DataSize,\r
+  IN VOID                  *Data\r
   )\r
 {\r
-  EFI_STATUS                     Status;\r
-  IP4_CONFIG2_DATA_ITEM          *Item;\r
+  EFI_STATUS             Status;\r
+  IP4_CONFIG2_DATA_ITEM  *Item;\r
 \r
   Status = EFI_SUCCESS;\r
   Item   = NULL;\r
@@ -1572,7 +1567,7 @@ Ip4Config2SetDnsServer (
     REMOVE_DATA_ATTRIB (Item->Attribute, DATA_ATTRIB_VOLATILE);\r
   }\r
 \r
-  if (Data != NULL && DataSize != 0) {\r
+  if ((Data != NULL) && (DataSize != 0)) {\r
     Status = Ip4Config2SetDnsServerWorker (Instance, DataSize, Data);\r
   } else {\r
     //\r
@@ -1581,6 +1576,7 @@ Ip4Config2SetDnsServer (
     if (Item->Data.Ptr != NULL) {\r
       FreePool (Item->Data.Ptr);\r
     }\r
+\r
     Item->Data.Ptr = NULL;\r
     Item->DataSize = 0;\r
     Item->Status   = EFI_NOT_FOUND;\r
@@ -1599,8 +1595,8 @@ Ip4Config2SetDnsServer (
 **/\r
 VOID\r
 Ip4Config2InitIfInfo (\r
-  IN  IP4_SERVICE                    *IpSb,\r
-  OUT EFI_IP4_CONFIG2_INTERFACE_INFO *IfInfo\r
+  IN  IP4_SERVICE                     *IpSb,\r
+  OUT EFI_IP4_CONFIG2_INTERFACE_INFO  *IfInfo\r
   )\r
 {\r
   UnicodeSPrint (\r
@@ -1608,15 +1604,13 @@ Ip4Config2InitIfInfo (
     EFI_IP4_CONFIG2_INTERFACE_INFO_NAME_SIZE,\r
     L"eth%d",\r
     IpSb->Ip4Config2Instance.IfIndex\r
-  );\r
+    );\r
 \r
   IfInfo->IfType        = IpSb->SnpMode.IfType;\r
   IfInfo->HwAddressSize = IpSb->SnpMode.HwAddressSize;\r
   CopyMem (&IfInfo->HwAddress, &IpSb->SnpMode.CurrentAddress, IfInfo->HwAddressSize);\r
 }\r
 \r
-\r
-\r
 /**\r
   Set the configuration for the EFI IPv4 network stack running on the communication\r
   device this EFI_IP4_CONFIG2_PROTOCOL instance manages.\r
@@ -1673,12 +1667,12 @@ EfiIp4Config2SetData (
   IN VOID                       *Data\r
   )\r
 {\r
-  EFI_TPL              OldTpl;\r
-  EFI_STATUS           Status;\r
-  IP4_CONFIG2_INSTANCE *Instance;\r
-  IP4_SERVICE          *IpSb;\r
+  EFI_TPL               OldTpl;\r
+  EFI_STATUS            Status;\r
+  IP4_CONFIG2_INSTANCE  *Instance;\r
+  IP4_SERVICE           *IpSb;\r
 \r
-  if ((This == NULL) || (Data == NULL && DataSize != 0) || (Data != NULL && DataSize == 0)) {\r
+  if ((This == NULL) || ((Data == NULL) && (DataSize != 0)) || ((Data != NULL) && (DataSize == 0))) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -1690,19 +1684,16 @@ EfiIp4Config2SetData (
   IpSb     = IP4_SERVICE_FROM_IP4_CONFIG2_INSTANCE (Instance);\r
   NET_CHECK_SIGNATURE (IpSb, IP4_SERVICE_SIGNATURE);\r
 \r
-\r
   OldTpl = gBS->RaiseTPL (TPL_CALLBACK);\r
 \r
   Status = Instance->DataItem[DataType].Status;\r
   if (Status != EFI_NOT_READY) {\r
-\r
     if (Instance->DataItem[DataType].SetData == NULL) {\r
       //\r
       // This type of data is readonly.\r
       //\r
       Status = EFI_WRITE_PROTECTED;\r
     } else {\r
-\r
       Status = Instance->DataItem[DataType].SetData (Instance, DataSize, Data);\r
       if (!EFI_ERROR (Status)) {\r
         //\r
@@ -1778,10 +1769,10 @@ EfiIp4Config2GetData (
   IN VOID                       *Data   OPTIONAL\r
   )\r
 {\r
-  EFI_TPL               OldTpl;\r
-  EFI_STATUS            Status;\r
-  IP4_CONFIG2_INSTANCE  *Instance;\r
-  IP4_CONFIG2_DATA_ITEM *DataItem;\r
+  EFI_TPL                OldTpl;\r
+  EFI_STATUS             Status;\r
+  IP4_CONFIG2_INSTANCE   *Instance;\r
+  IP4_CONFIG2_DATA_ITEM  *DataItem;\r
 \r
   if ((This == NULL) || (DataSize == NULL) || ((*DataSize != 0) && (Data == NULL))) {\r
     return EFI_INVALID_PARAMETER;\r
@@ -1796,11 +1787,9 @@ EfiIp4Config2GetData (
   Instance = IP4_CONFIG2_INSTANCE_FROM_PROTOCOL (This);\r
   DataItem = &Instance->DataItem[DataType];\r
 \r
-  Status   = Instance->DataItem[DataType].Status;\r
+  Status = Instance->DataItem[DataType].Status;\r
   if (!EFI_ERROR (Status)) {\r
-\r
     if (DataItem->GetData != NULL) {\r
-\r
       Status = DataItem->GetData (Instance, DataSize, Data);\r
     } else if (*DataSize < Instance->DataItem[DataType].DataSize) {\r
       //\r
@@ -1809,7 +1798,6 @@ EfiIp4Config2GetData (
       *DataSize = Instance->DataItem[DataType].DataSize;\r
       Status    = EFI_BUFFER_TOO_SMALL;\r
     } else {\r
-\r
       *DataSize = Instance->DataItem[DataType].DataSize;\r
       CopyMem (Data, Instance->DataItem[DataType].Data.Ptr, *DataSize);\r
     }\r
@@ -1850,11 +1838,11 @@ EfiIp4Config2RegisterDataNotify (
   IN EFI_EVENT                  Event\r
   )\r
 {\r
-  EFI_TPL              OldTpl;\r
-  EFI_STATUS           Status;\r
-  IP4_CONFIG2_INSTANCE *Instance;\r
-  NET_MAP              *EventMap;\r
-  NET_MAP_ITEM         *Item;\r
+  EFI_TPL               OldTpl;\r
+  EFI_STATUS            Status;\r
+  IP4_CONFIG2_INSTANCE  *Instance;\r
+  NET_MAP               *EventMap;\r
+  NET_MAP_ITEM          *Item;\r
 \r
   if ((This == NULL) || (Event == NULL)) {\r
     return EFI_INVALID_PARAMETER;\r
@@ -1864,26 +1852,22 @@ EfiIp4Config2RegisterDataNotify (
     return EFI_UNSUPPORTED;\r
   }\r
 \r
-  OldTpl    = gBS->RaiseTPL (TPL_CALLBACK);\r
+  OldTpl = gBS->RaiseTPL (TPL_CALLBACK);\r
 \r
-  Instance  = IP4_CONFIG2_INSTANCE_FROM_PROTOCOL (This);\r
-  EventMap  = &Instance->DataItem[DataType].EventMap;\r
+  Instance = IP4_CONFIG2_INSTANCE_FROM_PROTOCOL (This);\r
+  EventMap = &Instance->DataItem[DataType].EventMap;\r
 \r
   //\r
   // Check whether this event is already registered for this DataType.\r
   //\r
   Item = NetMapFindKey (EventMap, Event);\r
   if (Item == NULL) {\r
-\r
     Status = NetMapInsertTail (EventMap, Event, NULL);\r
 \r
     if (EFI_ERROR (Status)) {\r
-\r
       Status = EFI_OUT_OF_RESOURCES;\r
     }\r
-\r
   } else {\r
-\r
     Status = EFI_ACCESS_DENIED;\r
   }\r
 \r
@@ -1915,10 +1899,10 @@ EfiIp4Config2UnregisterDataNotify (
   IN EFI_EVENT                  Event\r
   )\r
 {\r
-  EFI_TPL              OldTpl;\r
-  EFI_STATUS           Status;\r
-  IP4_CONFIG2_INSTANCE *Instance;\r
-  NET_MAP_ITEM         *Item;\r
+  EFI_TPL               OldTpl;\r
+  EFI_STATUS            Status;\r
+  IP4_CONFIG2_INSTANCE  *Instance;\r
+  NET_MAP_ITEM          *Item;\r
 \r
   if ((This == NULL) || (Event == NULL)) {\r
     return EFI_INVALID_PARAMETER;\r
@@ -1934,11 +1918,9 @@ EfiIp4Config2UnregisterDataNotify (
 \r
   Item = NetMapFindKey (&Instance->DataItem[DataType].EventMap, Event);\r
   if (Item != NULL) {\r
-\r
     NetMapRemoveItem (&Instance->DataItem[DataType].EventMap, Item, NULL);\r
     Status = EFI_SUCCESS;\r
   } else {\r
-\r
     Status = EFI_NOT_FOUND;\r
   }\r
 \r
@@ -1961,20 +1943,18 @@ Ip4Config2InitInstance (
   OUT IP4_CONFIG2_INSTANCE  *Instance\r
   )\r
 {\r
-  IP4_SERVICE           *IpSb;\r
-  IP4_CONFIG2_INSTANCE  *TmpInstance;\r
-  LIST_ENTRY            *Entry;\r
-  EFI_STATUS            Status;\r
-  UINTN                 Index;\r
-  UINT16                IfIndex;\r
-  IP4_CONFIG2_DATA_ITEM *DataItem;\r
-\r
+  IP4_SERVICE            *IpSb;\r
+  IP4_CONFIG2_INSTANCE   *TmpInstance;\r
+  LIST_ENTRY             *Entry;\r
+  EFI_STATUS             Status;\r
+  UINTN                  Index;\r
+  UINT16                 IfIndex;\r
+  IP4_CONFIG2_DATA_ITEM  *DataItem;\r
 \r
   IpSb = IP4_SERVICE_FROM_IP4_CONFIG2_INSTANCE (Instance);\r
 \r
   Instance->Signature = IP4_CONFIG2_INSTANCE_SIGNATURE;\r
 \r
-\r
   //\r
   // Determine the index of this interface.\r
   //\r
@@ -2002,7 +1982,6 @@ Ip4Config2InitInstance (
     NetMapInit (&Instance->DataItem[Index].EventMap);\r
   }\r
 \r
-\r
   //\r
   // Initialize each data type: associate storage and set data size for the\r
   // fixed size data types, hook the SetData function, set the data attribute.\r
@@ -2021,19 +2000,19 @@ Ip4Config2InitInstance (
   Instance->Policy   = Ip4Config2PolicyStatic;\r
   SET_DATA_ATTRIB (DataItem->Attribute, DATA_ATTRIB_SIZE_FIXED);\r
 \r
-  DataItem           = &Instance->DataItem[Ip4Config2DataTypeManualAddress];\r
-  DataItem->SetData  = Ip4Config2SetManualAddress;\r
-  DataItem->Status   = EFI_NOT_FOUND;\r
+  DataItem          = &Instance->DataItem[Ip4Config2DataTypeManualAddress];\r
+  DataItem->SetData = Ip4Config2SetManualAddress;\r
+  DataItem->Status  = EFI_NOT_FOUND;\r
 \r
-  DataItem           = &Instance->DataItem[Ip4Config2DataTypeGateway];\r
-  DataItem->SetData  = Ip4Config2SetGateway;\r
-  DataItem->Status   = EFI_NOT_FOUND;\r
+  DataItem          = &Instance->DataItem[Ip4Config2DataTypeGateway];\r
+  DataItem->SetData = Ip4Config2SetGateway;\r
+  DataItem->Status  = EFI_NOT_FOUND;\r
 \r
-  DataItem           = &Instance->DataItem[Ip4Config2DataTypeDnsServer];\r
-  DataItem->SetData  = Ip4Config2SetDnsServer;\r
-  DataItem->Status   = EFI_NOT_FOUND;\r
+  DataItem          = &Instance->DataItem[Ip4Config2DataTypeDnsServer];\r
+  DataItem->SetData = Ip4Config2SetDnsServer;\r
+  DataItem->Status  = EFI_NOT_FOUND;\r
 \r
-  Instance->Configured  = TRUE;\r
+  Instance->Configured = TRUE;\r
 \r
   //\r
   // Try to read the config data from NV variable.\r
@@ -2060,7 +2039,6 @@ Ip4Config2InitInstance (
   return Ip4Config2FormInit (Instance);\r
 }\r
 \r
-\r
 /**\r
   Release an IP4_CONFIG2_INSTANCE.\r
 \r
@@ -2072,19 +2050,18 @@ Ip4Config2CleanInstance (
   IN OUT IP4_CONFIG2_INSTANCE  *Instance\r
   )\r
 {\r
-  UINTN                 Index;\r
-  IP4_CONFIG2_DATA_ITEM *DataItem;\r
+  UINTN                  Index;\r
+  IP4_CONFIG2_DATA_ITEM  *DataItem;\r
 \r
   if (Instance->DeclineAddress != NULL) {\r
     FreePool (Instance->DeclineAddress);\r
   }\r
 \r
   if (!Instance->Configured) {\r
-    return ;\r
+    return;\r
   }\r
 \r
   if (Instance->Dhcp4Handle != NULL) {\r
-\r
     Ip4Config2DestroyDhcp4 (Instance);\r
   }\r
 \r
@@ -2097,13 +2074,13 @@ Ip4Config2CleanInstance (
   }\r
 \r
   for (Index = 0; Index < Ip4Config2DataTypeMaximum; Index++) {\r
-\r
     DataItem = &Instance->DataItem[Index];\r
 \r
     if (!DATA_ATTRIB_SET (DataItem->Attribute, DATA_ATTRIB_SIZE_FIXED)) {\r
       if (DataItem->Data.Ptr != NULL) {\r
         FreePool (DataItem->Data.Ptr);\r
       }\r
+\r
       DataItem->Data.Ptr = NULL;\r
       DataItem->DataSize = 0;\r
     }\r
@@ -2126,12 +2103,12 @@ Ip4Config2CleanInstance (
 VOID\r
 EFIAPI\r
 Ip4AutoReconfigCallBackDpc (\r
-  IN VOID                   *Context\r
+  IN VOID  *Context\r
   )\r
 {\r
-  IP4_SERVICE               *IpSb;\r
+  IP4_SERVICE  *IpSb;\r
 \r
-  IpSb      = (IP4_SERVICE *) Context;\r
+  IpSb = (IP4_SERVICE *)Context;\r
   NET_CHECK_SIGNATURE (IpSb, IP4_SERVICE_SIGNATURE);\r
 \r
   if (IpSb->State > IP4_SERVICE_UNSTARTED) {\r
@@ -2142,10 +2119,9 @@ Ip4AutoReconfigCallBackDpc (
 \r
   Ip4StartAutoConfig (&IpSb->Ip4Config2Instance);\r
 \r
-  return ;\r
+  return;\r
 }\r
 \r
-\r
 /**\r
   Request Ip4AutoReconfigCallBackDpc as a DPC at TPL_CALLBACK.\r
 \r
@@ -2156,8 +2132,8 @@ Ip4AutoReconfigCallBackDpc (
 VOID\r
 EFIAPI\r
 Ip4AutoReconfigCallBack (\r
-  IN EFI_EVENT              Event,\r
-  IN VOID                   *Context\r
+  IN EFI_EVENT  Event,\r
+  IN VOID       *Context\r
   )\r
 {\r
   //\r
@@ -2165,4 +2141,3 @@ Ip4AutoReconfigCallBack (
   //\r
   QueueDpc (TPL_CALLBACK, Ip4AutoReconfigCallBackDpc, Context);\r
 }\r
-\r