]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Universal/Network/Ip4ConfigDxe/Ip4ConfigNv.c
Fix spelling typo in the NIC_IP4_CONFIG_INFO.Permanent variable name, which was incor...
[mirror_edk2.git] / MdeModulePkg / Universal / Network / Ip4ConfigDxe / Ip4ConfigNv.c
index b7de203881f8a7136f2ee5a5cda983a619197c8c..89e1ddaf39063cb49df33f03993f80291bab6667 100644 (file)
@@ -1,8 +1,8 @@
 /** @file\r
   Helper functions for configuring or getting the parameters relating to Ip4.\r
 \r
-Copyright (c) 2009, Intel Corporation.<BR>\r
-All rights reserved. This program and the accompanying materials\r
+Copyright (c) 2009 - 2012, Intel Corporation. All rights reserved.<BR>\r
+This program and the accompanying materials\r
 are licensed and made available under the terms and conditions of the BSD License\r
 which accompanies this distribution.  The full text of the license may be found at\r
 http://opensource.org/licenses/bsd-license.php\r
@@ -13,68 +13,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 **/\r
 \r
 #include "Ip4ConfigNv.h"\r
-\r
-GLOBAL_REMOVE_IF_UNREFERENCED CONST CHAR8  Ip4HexString[] = "0123456789ABCDEFabcdef";\r
-\r
-UINTN           mNumberOfIp4Devices      = 0;\r
-\r
-IP4_FORM_CALLBACK_INFO  *mCallbackInfo = NULL;\r
-\r
-LIST_ENTRY      mIp4ConfigFormList = {\r
-  &mIp4ConfigFormList,\r
-  &mIp4ConfigFormList\r
-};\r
-\r
-HII_VENDOR_DEVICE_PATH  mIp4ConifgHiiVendorDevicePath = {\r
-  {\r
-    {\r
-      HARDWARE_DEVICE_PATH,\r
-      HW_VENDOR_DP,\r
-      {\r
-        (UINT8) (sizeof (VENDOR_DEVICE_PATH)),\r
-        (UINT8) ((sizeof (VENDOR_DEVICE_PATH)) >> 8)\r
-      }\r
-    },\r
-    //\r
-    // {6D3FD906-42B9-4220-9E63-9D8C972D58EE}\r
-    //\r
-    { 0x6d3fd906, 0x42b9, 0x4220, { 0x9e, 0x63, 0x9d, 0x8c, 0x97, 0x2d, 0x58, 0xee } }\r
-  },\r
-  {\r
-    END_DEVICE_PATH_TYPE,\r
-    END_ENTIRE_DEVICE_PATH_SUBTYPE,\r
-    {\r
-      (UINT8) (END_DEVICE_PATH_LENGTH),\r
-      (UINT8) ((END_DEVICE_PATH_LENGTH) >> 8)\r
-    }\r
-  }\r
-};\r
-\r
-/**\r
-  Convert the mac address into a hexadecimal encoded "-" seperated string.\r
-\r
-  @param[in]  Mac The mac address.\r
-  @param[in]  Len  Length in bytes of the mac address.\r
-  @param[out] Str The storage to return the mac string.\r
-\r
-**/\r
-VOID\r
-Ip4MacAddrToStr (\r
-  IN  EFI_MAC_ADDRESS  *Mac,\r
-  IN  UINT32           Len,\r
-  OUT CHAR16           *Str\r
-  )\r
-{\r
-  UINT32  Index;\r
-\r
-  for (Index = 0; Index < Len; Index++) {\r
-    Str[3 * Index]      = (CHAR16) Ip4HexString[(Mac->Addr[Index] >> 4) & 0x0F];\r
-    Str[3 * Index + 1]  = (CHAR16) Ip4HexString[Mac->Addr[Index] & 0x0F];\r
-    Str[3 * Index + 2]  = L'-';\r
-  }\r
-\r
-  Str[3 * Index - 1] = L'\0';\r
-}\r
+#include "NicIp4Variable.h"\r
 \r
 /**\r
   Calculate the prefix length of the IPv4 subnet mask.\r
@@ -95,7 +34,7 @@ GetSubnetMaskPrefixLength (
   //\r
   // The SubnetMask is in network byte order.\r
   //\r
-  ReverseMask = (SubnetMask->Addr[0] << 24) | (SubnetMask->Addr[1] << 16) | (SubnetMask->Addr[2] << 8) | (SubnetMask->Addr[3]);\r
+  ReverseMask = SwapBytes32 (*(UINT32 *)&SubnetMask[0]);\r
 \r
   //\r
   // Reverse it.\r
@@ -120,14 +59,14 @@ GetSubnetMaskPrefixLength (
   Convert the decimal dotted IPv4 address into the binary IPv4 address.\r
 \r
   @param[in]   Str             The UNICODE string.\r
-  @param[out]  Ip              The storage to return the ASCII string.\r
+  @param[out]  Ip              The storage to return the IPv4 address.\r
 \r
   @retval EFI_SUCCESS           The binary IP address is returned in Ip.\r
   @retval EFI_INVALID_PARAMETER The IP string is malformatted.\r
 **/\r
 EFI_STATUS\r
-Ip4AsciiStrToIp (\r
-  IN  CHAR            *Str,\r
+Ip4StrToIp (\r
+  IN  CHAR16            *Str,\r
   OUT EFI_IPv4_ADDRESS  *Ip\r
   )\r
 {\r
@@ -136,15 +75,15 @@ Ip4AsciiStrToIp (
 \r
   Index = 0;\r
 \r
-  while (*Str != 0) {\r
+  while (*Str != L'\0') {\r
 \r
     if (Index > 3) {\r
       return EFI_INVALID_PARAMETER;\r
     }\r
 \r
     Number = 0;\r
-    while (NET_IS_DIGIT (*Str)) {\r
-      Number = Number * 10 + (*Str - '0');\r
+    while ((*Str >= L'0') && (*Str <= L'9')) {\r
+      Number = Number * 10 + (*Str - L'0');\r
       Str++;\r
     }\r
 \r
@@ -154,7 +93,7 @@ Ip4AsciiStrToIp (
 \r
     Ip->Addr[Index] = (UINT8) Number;\r
 \r
-    if ((*Str != '\0') && (*Str != '.')) {\r
+    if ((*Str != L'\0') && (*Str != L'.')) {\r
       //\r
       // The current character should be either the NULL terminator or\r
       // the dot delimiter.\r
@@ -162,7 +101,7 @@ Ip4AsciiStrToIp (
       return EFI_INVALID_PARAMETER;\r
     }\r
 \r
-    if (*Str == '.') {\r
+    if (*Str == L'.') {\r
       //\r
       // Skip the delimiter.\r
       //\r
@@ -198,8 +137,8 @@ Ip4ConfigIpToStr (
 /**\r
   Convert the network configuration data into the IFR data.\r
 \r
-  @param[in]   ConfigFormEntry The IP4 configuration form entry.\r
-  @param[out]  IfrNvData       The IFR nv data.\r
+  @param[in]   Ip4ConfigInstance The IP4Config instance\r
+  @param[out]  IfrFormNvData     The IFR nv data.\r
 **/\r
 VOID\r
 Ip4ConfigConvertDeviceConfigDataToIfrNvData (\r
@@ -207,58 +146,135 @@ Ip4ConfigConvertDeviceConfigDataToIfrNvData (
   OUT IP4_CONFIG_IFR_NVDATA     *IfrFormNvData\r
   )\r
 {\r
-  EFI_STATUS                    Status;\r
-  NIC_IP4_CONFIG_INFO           *NicConfig;\r
-  UINTN                         ConfigLen;\r
-\r
-  IfrFormNvData->DhcpEnable = 1;\r
+  NIC_IP4_CONFIG_INFO  *NicConfig;\r
 \r
-  ConfigLen = sizeof (NIC_IP4_CONFIG_INFO) + sizeof (EFI_IP4_ROUTE_TABLE) * 2;\r
-  NicConfig = AllocateZeroPool (ConfigLen);\r
-  ASSERT (NicConfig != NULL);\r
-  Status = EfiNicIp4ConfigGetInfo (Ip4ConfigInstance, &ConfigLen, NicConfig);\r
-  if (!EFI_ERROR (Status)) {\r
+  NicConfig = EfiNicIp4ConfigGetInfo (Ip4ConfigInstance);\r
+  if (NicConfig != NULL) {\r
+    IfrFormNvData->Configure = 1;\r
+    Ip4ConfigInstance->Ip4ConfigCallbackInfo.Configured = TRUE;\r
     if (NicConfig->Source == IP4_CONFIG_SOURCE_DHCP) {\r
       IfrFormNvData->DhcpEnable = 1;\r
+      Ip4ConfigInstance->Ip4ConfigCallbackInfo.DhcpEnabled = TRUE;\r
     } else {\r
       IfrFormNvData->DhcpEnable = 0;\r
       Ip4ConfigIpToStr (&NicConfig->Ip4Info.StationAddress, IfrFormNvData->StationAddress);\r
       Ip4ConfigIpToStr (&NicConfig->Ip4Info.SubnetMask, IfrFormNvData->SubnetMask);\r
       Ip4ConfigIpToStr (&NicConfig->Ip4Info.RouteTable[1].GatewayAddress, IfrFormNvData->GatewayAddress);\r
+\r
+      Ip4ConfigInstance->Ip4ConfigCallbackInfo.DhcpEnabled = FALSE;\r
+      CopyMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo.LocalIp, &NicConfig->Ip4Info.StationAddress, sizeof (EFI_IPv4_ADDRESS));\r
+      CopyMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo.SubnetMask, &NicConfig->Ip4Info.SubnetMask, sizeof (EFI_IPv4_ADDRESS));\r
+      CopyMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo.Gateway, &NicConfig->Ip4Info.RouteTable[1].GatewayAddress, sizeof (EFI_IPv4_ADDRESS));\r
     }\r
+\r
+    FreePool (NicConfig);\r
+  } else {\r
+    IfrFormNvData->Configure = 0;\r
+    Ip4ConfigInstance->Ip4ConfigCallbackInfo.Configured = FALSE;\r
   }\r
-  FreePool (NicConfig);\r
 }\r
 \r
 /**\r
-  Get the IP4 configuration form entry by the index of the goto opcode actived.\r
+  Convert the IFR data into the network configuration data and set the IP\r
+  configure parameters for the NIC.\r
+\r
+  @param[in]       IfrFormNvData     The IFR NV data.\r
+  @param[in, out]  Ip4ConfigInstance The IP4Config instance.\r
 \r
-  @param[in]  Index The 0-based index of the goto opcode actived.\r
+  @retval EFI_SUCCESS            The configure parameter for this NIC was\r
+                                 set successfully.\r
+  @retval EFI_ALREADY_STARTED    There is a pending auto configuration.\r
+  @retval EFI_NOT_FOUND          No auto configure parameter is found.\r
 \r
-  @return The IP4 configuration form entry found.\r
 **/\r
-IP4CONFIG_FORM_ENTRY *\r
-Ip4GetConfigFormEntryByIndex (\r
-  IN UINT32 Index\r
+EFI_STATUS\r
+Ip4ConfigConvertIfrNvDataToDeviceConfigData (\r
+  IN     IP4_CONFIG_IFR_NVDATA     *IfrFormNvData,\r
+  IN OUT IP4_CONFIG_INSTANCE       *Ip4ConfigInstance\r
   )\r
 {\r
-  UINT32                  CurrentIndex;\r
-  LIST_ENTRY              *Entry;\r
-  IP4CONFIG_FORM_ENTRY    *ConfigFormEntry;\r
+  EFI_STATUS                Status;\r
+  EFI_IP_ADDRESS            HostIp;\r
+  EFI_IP_ADDRESS            SubnetMask;\r
+  EFI_IP_ADDRESS            Gateway;\r
+  EFI_INPUT_KEY             Key;\r
+  NIC_IP4_CONFIG_INFO       *NicInfo;\r
+  EFI_IP_ADDRESS            Ip;\r
 \r
-  CurrentIndex    = 0;\r
-  ConfigFormEntry = NULL;\r
+  ZeroMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo, sizeof (IP4_SETTING_INFO));\r
 \r
-  NET_LIST_FOR_EACH (Entry, &mIp4ConfigFormList) {\r
-    if (CurrentIndex == Index) {\r
-      ConfigFormEntry = NET_LIST_USER_STRUCT (Entry, IP4CONFIG_FORM_ENTRY, Link);\r
-      break;\r
+  Ip4ConfigInstance->Ip4ConfigCallbackInfo.Configured = IfrFormNvData->Configure;\r
+  Ip4ConfigInstance->Ip4ConfigCallbackInfo.DhcpEnabled = IfrFormNvData->DhcpEnable;\r
+  Ip4StrToIp (IfrFormNvData->StationAddress, &Ip4ConfigInstance->Ip4ConfigCallbackInfo.LocalIp);\r
+  Ip4StrToIp (IfrFormNvData->SubnetMask, &Ip4ConfigInstance->Ip4ConfigCallbackInfo.SubnetMask);\r
+  Ip4StrToIp (IfrFormNvData->GatewayAddress, &Ip4ConfigInstance->Ip4ConfigCallbackInfo.Gateway);\r
+\r
+  if (!Ip4ConfigInstance->Ip4ConfigCallbackInfo.Configured) {\r
+    //\r
+    // Clear the variable\r
+    //\r
+    ZeroMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo, sizeof (IP4_SETTING_INFO));\r
+\r
+    Status = EfiNicIp4ConfigSetInfo (Ip4ConfigInstance, NULL, TRUE);\r
+    if (Status == EFI_NOT_FOUND) {\r
+      return EFI_SUCCESS;\r
     }\r
 \r
-    CurrentIndex++;\r
+    return Status;\r
   }\r
 \r
-  return ConfigFormEntry;\r
+  NicInfo = AllocateZeroPool (sizeof (NIC_IP4_CONFIG_INFO) + 2 * sizeof (EFI_IP4_ROUTE_TABLE));\r
+  ASSERT (NicInfo != NULL);\r
+\r
+  NicInfo->Ip4Info.RouteTable = (EFI_IP4_ROUTE_TABLE *) (NicInfo + 1);\r
+\r
+  if (!Ip4ConfigInstance->Ip4ConfigCallbackInfo.DhcpEnabled) {\r
+    CopyMem (&HostIp.v4, &Ip4ConfigInstance->Ip4ConfigCallbackInfo.LocalIp, sizeof (HostIp.v4));\r
+    CopyMem (&SubnetMask.v4, &Ip4ConfigInstance->Ip4ConfigCallbackInfo.SubnetMask, sizeof (SubnetMask.v4));\r
+    CopyMem (&Gateway.v4, &Ip4ConfigInstance->Ip4ConfigCallbackInfo.Gateway, sizeof (Gateway.v4));\r
+\r
+    if (!NetIp4IsUnicast (NTOHL (HostIp.Addr[0]), 0)) {\r
+      CreatePopUp (EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE, &Key, L"Invalid IP address!", NULL);\r
+      return EFI_INVALID_PARAMETER;\r
+    }\r
+    if (EFI_IP4_EQUAL (&SubnetMask, &mZeroIp4Addr)) {\r
+      CreatePopUp (EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE, &Key, L"Invalid Subnet Mask!", NULL);\r
+      return EFI_INVALID_PARAMETER;\r
+    }\r
+\r
+    if ((Gateway.Addr[0] != 0)) {\r
+      if (SubnetMask.Addr[0] == 0) {\r
+        CreatePopUp (EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE, &Key, L"Gateway address is set but subnet mask is zero.", NULL);\r
+        return EFI_INVALID_PARAMETER;\r
+\r
+      } else if (!IP4_NET_EQUAL (HostIp.Addr[0], Gateway.Addr[0], SubnetMask.Addr[0])) {\r
+        CreatePopUp (EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE, &Key, L"Local IP and Gateway are not in the same subnet.", NULL);\r
+        return EFI_INVALID_PARAMETER;      }\r
+    }\r
+\r
+    NicInfo->Source = IP4_CONFIG_SOURCE_STATIC;\r
+    NicInfo->Ip4Info.RouteTableSize = 2;\r
+\r
+    CopyMem (&NicInfo->Ip4Info.StationAddress, &HostIp.v4, sizeof (EFI_IPv4_ADDRESS));\r
+    CopyMem (&NicInfo->Ip4Info.SubnetMask, &SubnetMask.v4, sizeof (EFI_IPv4_ADDRESS));\r
+\r
+    Ip.Addr[0] = HostIp.Addr[0] & SubnetMask.Addr[0];\r
+\r
+    CopyMem (&NicInfo->Ip4Info.RouteTable[0].SubnetAddress, &Ip.v4, sizeof (EFI_IPv4_ADDRESS));\r
+    CopyMem (&NicInfo->Ip4Info.RouteTable[0].SubnetMask, &SubnetMask.v4, sizeof (EFI_IPv4_ADDRESS));\r
+    CopyMem (&NicInfo->Ip4Info.RouteTable[1].GatewayAddress, &Gateway.v4, sizeof (EFI_IPv4_ADDRESS));\r
+\r
+  } else {\r
+    NicInfo->Source = IP4_CONFIG_SOURCE_DHCP;\r
+    ZeroMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo.LocalIp, sizeof (EFI_IPv4_ADDRESS));\r
+    ZeroMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo.SubnetMask, sizeof (EFI_IPv4_ADDRESS));\r
+    ZeroMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo.Gateway, sizeof (EFI_IPv4_ADDRESS));\r
+  }\r
+\r
+  NicInfo->Permanent = TRUE;\r
+  CopyMem (&NicInfo->NicAddr, &Ip4ConfigInstance->NicAddr, sizeof (NIC_ADDR));\r
+\r
+  return EfiNicIp4ConfigSetInfo (Ip4ConfigInstance, NicInfo, TRUE);\r
 }\r
 \r
 /**\r
@@ -302,12 +318,6 @@ Ip4GetConfigFormEntryByIndex (
                                   parts of the results that must be\r
                                   stored awaiting possible future\r
                                   protocols.\r
-  @retval EFI_INVALID_PARAMETER   For example, passing in a NULL\r
-                                  for the Request parameter\r
-                                  would result in this type of\r
-                                  error. In this case, the\r
-                                  Progress parameter would be\r
-                                  set to NULL.\r
   @retval EFI_NOT_FOUND           Routing data doesn't match any\r
                                   known driver. Progress set to the\r
                                   first character in the routing header.\r
@@ -333,53 +343,173 @@ Ip4DeviceExtractConfig (
   )\r
 {\r
   EFI_STATUS                       Status;\r
-  UINTN                            ConfigLen;\r
   NIC_IP4_CONFIG_INFO              *IfrDeviceNvData;\r
-  IP4_FORM_CALLBACK_INFO           *Private;\r
-  EFI_HII_CONFIG_ROUTING_PROTOCOL  *HiiConfigRouting;\r
+  NIC_IP4_CONFIG_INFO              *NicConfig;\r
   IP4_CONFIG_INSTANCE              *Ip4ConfigInstance;\r
+  IP4_CONFIG_IFR_NVDATA            *IfrFormNvData;\r
+  EFI_STRING                       ConfigRequestHdr;\r
+  EFI_STRING                       ConfigRequest;\r
+  EFI_STRING                       DeviceResult;\r
+  EFI_STRING                       FormResult;\r
+  CHAR16                           *StrPointer;\r
+  BOOLEAN                          AllocatedRequest;\r
+  UINTN                            Size;\r
+  UINTN                            BufferSize;\r
 \r
-  if (Request == NULL) {\r
+  if (Progress == NULL || Results == NULL) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
-  *Progress = Request;\r
+  *Progress     = Request;\r
+  Size          = 0;\r
+  DeviceResult  = NULL;\r
+  FormResult    = NULL;\r
+  ConfigRequest = NULL;\r
+  Status        = EFI_SUCCESS;\r
+  AllocatedRequest  = FALSE;\r
+  Ip4ConfigInstance = IP4_CONFIG_INSTANCE_FROM_CONFIG_ACCESS (This);\r
 \r
   //\r
   // Check Request data in <ConfigHdr>.\r
   //\r
-  if (!HiiIsConfigHdrMatch (Request, &gEfiNicIp4ConfigVariableGuid, EFI_NIC_IP4_CONFIG_VARIABLE)) {\r
-    return EFI_NOT_FOUND;\r
+  if ((Request == NULL) || HiiIsConfigHdrMatch (Request, &gEfiNicIp4ConfigVariableGuid, EFI_NIC_IP4_CONFIG_VARIABLE)) {\r
+    IfrDeviceNvData = AllocateZeroPool (NIC_ITEM_CONFIG_SIZE);\r
+    if (IfrDeviceNvData == NULL) {\r
+      return EFI_OUT_OF_RESOURCES;\r
+    }\r
+\r
+    NicConfig = EfiNicIp4ConfigGetInfo (Ip4ConfigInstance);\r
+    if (NicConfig == NULL) {\r
+      return EFI_NOT_FOUND;\r
+    }\r
+    CopyMem (IfrDeviceNvData, NicConfig, SIZEOF_NIC_IP4_CONFIG_INFO (NicConfig));\r
+    FreePool (NicConfig);\r
+\r
+    ConfigRequest = Request;\r
+    if ((Request == NULL) || (StrStr (Request, L"OFFSET") == NULL)) {\r
+      //\r
+      // Request has no request element, construct full request string.\r
+      // Allocate and fill a buffer large enough to hold the <ConfigHdr> template\r
+      // followed by "&OFFSET=0&WIDTH=WWWWWWWWWWWWWWWW" followed by a Null-terminator\r
+      //\r
+      ConfigRequestHdr = HiiConstructConfigHdr (&gEfiNicIp4ConfigVariableGuid, EFI_NIC_IP4_CONFIG_VARIABLE, Ip4ConfigInstance->ChildHandle);\r
+      Size = (StrLen (ConfigRequestHdr) + 32 + 1) * sizeof (CHAR16);\r
+      ConfigRequest = AllocateZeroPool (Size);\r
+      ASSERT (ConfigRequest != NULL);\r
+      AllocatedRequest = TRUE;\r
+      BufferSize = NIC_ITEM_CONFIG_SIZE;\r
+      UnicodeSPrint (ConfigRequest, Size, L"%s&OFFSET=0&WIDTH=%016LX", ConfigRequestHdr, (UINT64)BufferSize);\r
+      FreePool (ConfigRequestHdr);\r
+    }\r
+\r
+    //\r
+    // Convert buffer data to <ConfigResp> by helper function BlockToConfig()\r
+    //\r
+    Status = gHiiConfigRouting->BlockToConfig (\r
+                                  gHiiConfigRouting,\r
+                                  ConfigRequest,\r
+                                  (UINT8 *) IfrDeviceNvData,\r
+                                  NIC_ITEM_CONFIG_SIZE,\r
+                                  &DeviceResult,\r
+                                  Progress\r
+                                  );\r
+\r
+    FreePool (IfrDeviceNvData);\r
+    //\r
+    // Free the allocated config request string.\r
+    //\r
+    if (AllocatedRequest) {\r
+      FreePool (ConfigRequest);\r
+      ConfigRequest = NULL;\r
+    }\r
+\r
+    if (EFI_ERROR (Status)) {\r
+      goto Failure;\r
+    }\r
   }\r
 \r
-  Private = IP4CONFIG_FORM_CALLBACK_INFO_FROM_FORM_CALLBACK (This);\r
-  Ip4ConfigInstance = IP4_CONFIG_INSTANCE_FROM_IP4FORM_CALLBACK_INFO (Private);\r
+  if ((Request == NULL) || HiiIsConfigHdrMatch (Request, &gNicIp4ConfigNvDataGuid, EFI_NIC_IP4_CONFIG_VARIABLE)) {\r
 \r
-  IfrDeviceNvData = AllocateZeroPool (NIC_ITEM_CONFIG_SIZE);\r
-  if (IfrDeviceNvData == NULL) {\r
-    return EFI_OUT_OF_RESOURCES;\r
+    IfrFormNvData = AllocateZeroPool (sizeof (IP4_CONFIG_IFR_NVDATA));\r
+    if (IfrFormNvData == NULL) {\r
+      return EFI_OUT_OF_RESOURCES;\r
+    }\r
+\r
+    Ip4ConfigConvertDeviceConfigDataToIfrNvData (Ip4ConfigInstance, IfrFormNvData);\r
+\r
+    ConfigRequest = Request;\r
+    if ((Request == NULL) || (StrStr (Request, L"OFFSET") == NULL)) {\r
+      //\r
+      // Request has no request element, construct full request string.\r
+      // Allocate and fill a buffer large enough to hold the <ConfigHdr> template\r
+      // followed by "&OFFSET=0&WIDTH=WWWWWWWWWWWWWWWW" followed by a Null-terminator\r
+      //\r
+      ConfigRequestHdr = HiiConstructConfigHdr (&gNicIp4ConfigNvDataGuid, EFI_NIC_IP4_CONFIG_VARIABLE, Ip4ConfigInstance->ChildHandle);\r
+      Size = (StrLen (ConfigRequestHdr) + 32 + 1) * sizeof (CHAR16);\r
+      ConfigRequest = AllocateZeroPool (Size);\r
+      ASSERT (ConfigRequest != NULL);\r
+      AllocatedRequest = TRUE;\r
+      BufferSize = sizeof (IP4_CONFIG_IFR_NVDATA);\r
+      UnicodeSPrint (ConfigRequest, Size, L"%s&OFFSET=0&WIDTH=%016LX", ConfigRequestHdr, (UINT64)BufferSize);\r
+      FreePool (ConfigRequestHdr);\r
+    }\r
+\r
+    //\r
+    // Convert buffer data to <ConfigResp> by helper function BlockToConfig()\r
+    //\r
+    Status = gHiiConfigRouting->BlockToConfig (\r
+                                  gHiiConfigRouting,\r
+                                  ConfigRequest,\r
+                                  (UINT8 *) IfrFormNvData,\r
+                                  sizeof (IP4_CONFIG_IFR_NVDATA),\r
+                                  &FormResult,\r
+                                  Progress\r
+                                  );\r
+\r
+    FreePool (IfrFormNvData);\r
+    //\r
+    // Free the allocated config request string.\r
+    //\r
+    if (AllocatedRequest) {\r
+      FreePool (ConfigRequest);\r
+      ConfigRequest = NULL;\r
+    }\r
+\r
+    if (EFI_ERROR (Status)) {\r
+      goto Failure;\r
+    }\r
   }\r
 \r
-  Status = EfiNicIp4ConfigGetInfo (Ip4ConfigInstance, &ConfigLen, IfrDeviceNvData);\r
-  if (EFI_ERROR (Status)) {\r
-    FreePool (IfrDeviceNvData);\r
+  if (Request == NULL) {\r
+    Size = StrLen (DeviceResult);\r
+    Size = Size + 1;\r
+    Size = Size + StrLen (FormResult) + 1;\r
+    *Results = AllocateZeroPool (Size * sizeof (CHAR16));\r
+    ASSERT (*Results != NULL);\r
+    StrPointer  = *Results;\r
+    StrCpy (StrPointer, DeviceResult);\r
+    StrPointer  = StrPointer + StrLen (StrPointer);\r
+    *StrPointer = L'&';\r
+    StrCpy (StrPointer + 1, FormResult);\r
+    FreePool (DeviceResult);\r
+    FreePool (FormResult);\r
+  } else if (HiiIsConfigHdrMatch (Request, &gEfiNicIp4ConfigVariableGuid, EFI_NIC_IP4_CONFIG_VARIABLE)) {\r
+    *Results = DeviceResult;\r
+  } else if (HiiIsConfigHdrMatch (Request, &gNicIp4ConfigNvDataGuid, EFI_NIC_IP4_CONFIG_VARIABLE)) {\r
+    *Results = FormResult;\r
+  } else {\r
     return EFI_NOT_FOUND;\r
   }\r
 \r
+Failure:\r
   //\r
-  // Convert buffer data to <ConfigResp> by helper function BlockToConfig()\r
+  // Set Progress string to the original request string.\r
   //\r
-  HiiConfigRouting = Private->ConfigRouting;\r
-  Status = HiiConfigRouting->BlockToConfig (\r
-                               HiiConfigRouting,\r
-                               Request,\r
-                               (UINT8 *) IfrDeviceNvData,\r
-                               NIC_ITEM_CONFIG_SIZE,\r
-                               Results,\r
-                               Progress\r
-                               );\r
-\r
-  FreePool (IfrDeviceNvData);\r
+  if (Request == NULL) {\r
+    *Progress = NULL;\r
+  } else if (StrStr (Request, L"OFFSET") == NULL) {\r
+    *Progress = Request + StrLen (Request);\r
+  }\r
 \r
   return Status;\r
 }\r
@@ -428,191 +558,90 @@ Ip4DeviceRouteConfig (
   EFI_STATUS                       Status;\r
   UINTN                            BufferSize;\r
   NIC_IP4_CONFIG_INFO              *IfrDeviceNvData;\r
+  IP4_CONFIG_IFR_NVDATA            *IfrFormNvData;\r
   NIC_IP4_CONFIG_INFO              *NicInfo;\r
-  IP4_FORM_CALLBACK_INFO           *Private;\r
-  EFI_HII_CONFIG_ROUTING_PROTOCOL  *HiiConfigRouting;\r
   IP4_CONFIG_INSTANCE              *Ip4ConfigInstance;\r
   EFI_MAC_ADDRESS                  ZeroMac;\r
 \r
-  if (Configuration == NULL) {\r
+  if (Configuration == NULL || Progress == NULL) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
-  *Progress = Configuration;\r
-\r
   //\r
-  // Check Routing data in <ConfigHdr>.\r
+  // Reclaim Ip4Config variable\r
   //\r
-  if (!HiiIsConfigHdrMatch (Configuration, &gEfiNicIp4ConfigVariableGuid, EFI_NIC_IP4_CONFIG_VARIABLE)) {\r
-    return EFI_NOT_FOUND;\r
-  }\r
+  Ip4ConfigReclaimVariable ();\r
 \r
-  Private = IP4CONFIG_FORM_CALLBACK_INFO_FROM_FORM_CALLBACK (This);\r
-  Ip4ConfigInstance = IP4_CONFIG_INSTANCE_FROM_IP4FORM_CALLBACK_INFO (Private);\r
+  *Progress = Configuration;\r
 \r
-  IfrDeviceNvData = AllocateZeroPool (NIC_ITEM_CONFIG_SIZE);\r
-  if (IfrDeviceNvData == NULL) {\r
-    return EFI_OUT_OF_RESOURCES;\r
-  }\r
+  Ip4ConfigInstance = IP4_CONFIG_INSTANCE_FROM_CONFIG_ACCESS (This);\r
 \r
   //\r
-  // Convert buffer data to <ConfigResp> by helper function BlockToConfig()\r
+  // Check Routing data in <ConfigHdr>.\r
   //\r
-  HiiConfigRouting = Private->ConfigRouting;\r
-  BufferSize = NIC_ITEM_CONFIG_SIZE;\r
-  Status = HiiConfigRouting->ConfigToBlock (\r
-                               HiiConfigRouting,\r
-                               Configuration,\r
-                               (UINT8 *) IfrDeviceNvData,\r
-                               &BufferSize,\r
-                               Progress\r
-                               );\r
-  if (!EFI_ERROR (Status)) {\r
-    ZeroMem (&ZeroMac, sizeof (EFI_MAC_ADDRESS));\r
-    if (CompareMem (&IfrDeviceNvData->NicAddr.MacAddr, &ZeroMac, IfrDeviceNvData->NicAddr.Len) != 0) {\r
-      BufferSize = sizeof (NIC_IP4_CONFIG_INFO) + sizeof (EFI_IP4_ROUTE_TABLE) * IfrDeviceNvData->Ip4Info.RouteTableSize;\r
-      NicInfo = AllocateCopyPool (BufferSize, IfrDeviceNvData); \r
-      Status = EfiNicIp4ConfigSetInfo (Ip4ConfigInstance, NicInfo, TRUE);\r
-    } else {\r
-      Status = EfiNicIp4ConfigSetInfo (Ip4ConfigInstance, NULL, TRUE);\r
+  if (HiiIsConfigHdrMatch (Configuration, &gNicIp4ConfigNvDataGuid, EFI_NIC_IP4_CONFIG_VARIABLE)) {\r
+    //\r
+    // Convert buffer data to <ConfigResp> by helper function BlockToConfig()\r
+    //\r
+    IfrFormNvData = AllocateZeroPool (sizeof (IP4_CONFIG_IFR_NVDATA));\r
+    if (IfrFormNvData == NULL) {\r
+      return EFI_OUT_OF_RESOURCES;\r
     }\r
-  }\r
 \r
-  FreePool (IfrDeviceNvData);\r
-  return Status;\r
-\r
-}\r
+    BufferSize = NIC_ITEM_CONFIG_SIZE;\r
+    Status = gHiiConfigRouting->ConfigToBlock (\r
+                                  gHiiConfigRouting,\r
+                                  Configuration,\r
+                                  (UINT8 *) IfrFormNvData,\r
+                                  &BufferSize,\r
+                                  Progress\r
+                                  );\r
+    if (!EFI_ERROR (Status)) {\r
+      Status = Ip4ConfigConvertIfrNvDataToDeviceConfigData (IfrFormNvData, Ip4ConfigInstance);\r
+    }\r
 \r
-/**\r
-  This function allows the caller to request the current\r
-  configuration for one or more named elements. The resulting\r
-  string is in <ConfigAltResp> format. Any and all alternative\r
-  configuration strings shall also be appended to the end of the\r
-  current configuration string. If they are, they must appear\r
-  after the current configuration. They must contain the same\r
-  routing (GUID, NAME, PATH) as the current configuration string.\r
-  They must have an additional description indicating the type of\r
-  alternative configuration the string represents,\r
-  "ALTCFG=<StringToken>". That <StringToken> (when\r
-  converted from Hex UNICODE to binary) is a reference to a\r
-  string in the associated string pack.\r
+    FreePool (IfrFormNvData);\r
 \r
-  @param[in] This       Points to the EFI_HII_CONFIG_ACCESS_PROTOCOL.\r
-  @param[in] Request    A null-terminated Unicode string in\r
-                        <ConfigRequest> format. Note that this\r
-                        includes the routing information as well as\r
-                        the configurable name / value pairs. It is\r
-                        invalid for this string to be in\r
-                        <MultiConfigRequest> format.\r
-  @param[out] Progress  On return, points to a character in the\r
-                        Request string. Points to the string's null\r
-                        terminator if request was successful. Points\r
-                        to the most recent "&" before the first\r
-                        failing name / value pair (or the beginning\r
-                        of the string if the failure is in the first\r
-                        name / value pair) if the request was not\r
-                        successful.\r
-  @param[out] Results   A null-terminated Unicode string in\r
-                        <ConfigAltResp> format which has all values\r
-                        filled in for the names in the Request string.\r
-                        String to be allocated by the called function.\r
+  } else if (HiiIsConfigHdrMatch (Configuration, &gEfiNicIp4ConfigVariableGuid, EFI_NIC_IP4_CONFIG_VARIABLE)) {\r
 \r
-  @retval EFI_SUCCESS             The Results string is filled with the\r
-                                  values corresponding to all requested\r
-                                  names.\r
-  @retval EFI_OUT_OF_RESOURCES    Not enough memory to store the\r
-                                  parts of the results that must be\r
-                                  stored awaiting possible future\r
-                                  protocols.\r
-  @retval EFI_INVALID_PARAMETER   For example, passing in a NULL\r
-                                  for the Request parameter\r
-                                  would result in this type of\r
-                                  error. In this case, the\r
-                                  Progress parameter would be\r
-                                  set to NULL.\r
-  @retval EFI_NOT_FOUND           Routing data doesn't match any\r
-                                  known driver. Progress set to the\r
-                                  first character in the routing header.\r
-                                  Note: There is no requirement that the\r
-                                  driver validate the routing data. It\r
-                                  must skip the <ConfigHdr> in order to\r
-                                  process the names.\r
-  @retval EFI_INVALID_PARAMETER   Illegal syntax. Progress set\r
-                                  to most recent & before the\r
-                                  error or the beginning of the\r
-                                  string.\r
-  @retval EFI_INVALID_PARAMETER   Unknown name. Progress points\r
-                                  to the & before the name in\r
-                                  question.Currently not implemented.\r
-**/\r
-EFI_STATUS\r
-EFIAPI\r
-Ip4FormExtractConfig (\r
-  IN  CONST EFI_HII_CONFIG_ACCESS_PROTOCOL   *This,\r
-  IN  CONST EFI_STRING                       Request,\r
-  OUT EFI_STRING                             *Progress,\r
-  OUT EFI_STRING                             *Results\r
-  )\r
-{\r
-  if (Request == NULL || Progress == NULL || Results == NULL) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-  *Progress = Request;\r
-  return EFI_NOT_FOUND;\r
-}\r
+    IfrDeviceNvData = AllocateZeroPool (NIC_ITEM_CONFIG_SIZE);\r
+    if (IfrDeviceNvData == NULL) {\r
+      return EFI_OUT_OF_RESOURCES;\r
+    }\r
 \r
-/**\r
-  This function applies changes in a driver's configuration.\r
-  Input is a Configuration, which has the routing data for this\r
-  driver followed by name / value configuration pairs. The driver\r
-  must apply those pairs to its configurable storage. If the\r
-  driver's configuration is stored in a linear block of data\r
-  and the driver's name / value pairs are in <BlockConfig>\r
-  format, it may use the ConfigToBlock helper function (above) to\r
-  simplify the job. Currently not implemented.\r
+    BufferSize = NIC_ITEM_CONFIG_SIZE;\r
+    Status = gHiiConfigRouting->ConfigToBlock (\r
+                                  gHiiConfigRouting,\r
+                                  Configuration,\r
+                                  (UINT8 *) IfrDeviceNvData,\r
+                                  &BufferSize,\r
+                                  Progress\r
+                                  );\r
+    if (!EFI_ERROR (Status)) {\r
+      ZeroMem (&ZeroMac, sizeof (EFI_MAC_ADDRESS));\r
+      if (CompareMem (&IfrDeviceNvData->NicAddr.MacAddr, &ZeroMac, IfrDeviceNvData->NicAddr.Len) != 0) {\r
+        BufferSize = SIZEOF_NIC_IP4_CONFIG_INFO (IfrDeviceNvData);\r
+        NicInfo = AllocateCopyPool (BufferSize, IfrDeviceNvData);\r
+        if (NicInfo == NULL) {\r
+          return EFI_OUT_OF_RESOURCES;\r
+        }\r
+        Status = EfiNicIp4ConfigSetInfo (Ip4ConfigInstance, NicInfo, TRUE);\r
+        FreePool (NicInfo);\r
+      } else {\r
+        ZeroMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo, sizeof (IP4_SETTING_INFO));\r
+        Status = EfiNicIp4ConfigSetInfo (Ip4ConfigInstance, NULL, TRUE);\r
+      }\r
+    }\r
 \r
-  @param[in]  This           Points to the EFI_HII_CONFIG_ACCESS_PROTOCOL.\r
-  @param[in]  Configuration  A null-terminated Unicode string in\r
-                             <ConfigString> format.\r
-  @param[out] Progress       A pointer to a string filled in with the\r
-                             offset of the most recent '&' before the\r
-                             first failing name / value pair (or the\r
-                             beginn ing of the string if the failure\r
-                             is in the first name / value pair) or\r
-                             the terminating NULL if all was\r
-                             successful.\r
+    FreePool (IfrDeviceNvData);\r
 \r
-  @retval EFI_SUCCESS             The results have been distributed or are\r
-                                  awaiting distribution.\r
-  @retval EFI_OUT_OF_MEMORY       Not enough memory to store the\r
-                                  parts of the results that must be\r
-                                  stored awaiting possible future\r
-                                  protocols.\r
-  @retval EFI_INVALID_PARAMETERS  Passing in a NULL for the\r
-                                  Results parameter would result\r
-                                  in this type of error.\r
-  @retval EFI_NOT_FOUND           Target for the specified routing data\r
-                                  was not found.\r
-**/\r
-EFI_STATUS\r
-EFIAPI\r
-Ip4FormRouteConfig (\r
-  IN  CONST EFI_HII_CONFIG_ACCESS_PROTOCOL   *This,\r
-  IN  CONST EFI_STRING                       Configuration,\r
-  OUT EFI_STRING                             *Progress\r
-  )\r
-{\r
-  if (Configuration == NULL || Progress == NULL) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
+  } else {\r
 \r
-  *Progress = Configuration;\r
-  if (!HiiIsConfigHdrMatch (Configuration, &gEfiNicIp4ConfigVariableGuid, EFI_NIC_IP4_CONFIG_VARIABLE)) {\r
     return EFI_NOT_FOUND;\r
   }\r
 \r
-  *Progress = Configuration + StrLen (Configuration);\r
-  return EFI_SUCCESS;\r
+  return Status;\r
+\r
 }\r
 \r
 /**\r
@@ -653,163 +682,83 @@ Ip4FormCallback (
   )\r
 {\r
   IP4_CONFIG_INSTANCE       *Ip4ConfigInstance;\r
-  IP4_FORM_CALLBACK_INFO    *Private;\r
-  CHAR8                     Ip4String[IP4_STR_MAX_SIZE];\r
-  CHAR16                    PortString[128];\r
-  EFI_STRING_ID             DeviceFormTitleToken;\r
   IP4_CONFIG_IFR_NVDATA     *IfrFormNvData;\r
-  IP4CONFIG_FORM_ENTRY      *ConfigFormEntry;\r
   EFI_IP_ADDRESS            HostIp;\r
   EFI_IP_ADDRESS            SubnetMask;\r
   EFI_IP_ADDRESS            Gateway;\r
   EFI_STATUS                Status;\r
   EFI_INPUT_KEY             Key;\r
-  NIC_IP4_CONFIG_INFO       *NicInfo;\r
-  EFI_IP_ADDRESS            Ip;\r
-\r
-  ConfigFormEntry = NULL;\r
 \r
-  Private   = IP4CONFIG_FORM_CALLBACK_INFO_FROM_FORM_CALLBACK (This);\r
+  if (Action == EFI_BROWSER_ACTION_CHANGED) {\r
+    Ip4ConfigInstance = IP4_CONFIG_INSTANCE_FROM_CONFIG_ACCESS (This);\r
 \r
-  IfrFormNvData = AllocateZeroPool (sizeof (IP4_CONFIG_IFR_NVDATA));\r
-  if (IfrFormNvData == NULL) {\r
-    return EFI_OUT_OF_RESOURCES;\r
-  }\r
-\r
-  //\r
-  // Retrive uncommitted data from Browser\r
-  //\r
-  if (!HiiGetBrowserData (&gEfiNicIp4ConfigVariableGuid, EFI_NIC_IP4_CONFIG_VARIABLE, sizeof (IP4_CONFIG_IFR_NVDATA), (UINT8 *) IfrFormNvData)) {\r
-    FreePool (IfrFormNvData);\r
-    return EFI_NOT_FOUND;\r
-  }\r
-\r
-  Status = EFI_SUCCESS;\r
-\r
-  switch (QuestionId) {\r
-\r
-  case KEY_DHCP_ENABLE:\r
-    if (IfrFormNvData->DhcpEnable == 0) {\r
-      Private->Current->SessionConfigData.Enabled = FALSE;\r
-    } else {\r
-      Private->Current->SessionConfigData.Enabled = TRUE;\r
+    IfrFormNvData = AllocateZeroPool (sizeof (IP4_CONFIG_IFR_NVDATA));\r
+    if (IfrFormNvData == NULL) {\r
+      return EFI_OUT_OF_RESOURCES;\r
     }\r
 \r
-    break;\r
-\r
-  case KEY_LOCAL_IP:\r
-    UnicodeStrToAsciiStr (IfrFormNvData->StationAddress, Ip4String);\r
-    Status = Ip4AsciiStrToIp (Ip4String, &HostIp.v4);\r
-    if (EFI_ERROR (Status) || !Ip4IsUnicast (NTOHL (HostIp.Addr[0]), 0)) {\r
-      CreatePopUp (EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE, &Key, L"Invalid IP address!", NULL);\r
-      Status = EFI_INVALID_PARAMETER;\r
-    } else {\r
-      CopyMem (&Private->Current->SessionConfigData.LocalIp, &HostIp.v4, sizeof (HostIp.v4));\r
-    }\r
-\r
-    break;\r
-\r
-  case KEY_SUBNET_MASK:\r
-    UnicodeStrToAsciiStr (IfrFormNvData->SubnetMask, Ip4String);\r
-    Status = Ip4AsciiStrToIp (Ip4String, &SubnetMask.v4);\r
-    if (EFI_ERROR (Status) || ((SubnetMask.Addr[0] != 0) && (GetSubnetMaskPrefixLength (&SubnetMask.v4) == 0))) {\r
-      CreatePopUp (EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE, &Key, L"Invalid SubnetMask!", NULL);\r
-      Status = EFI_INVALID_PARAMETER;\r
-    } else {\r
-      CopyMem (&Private->Current->SessionConfigData.SubnetMask, &SubnetMask.v4, sizeof (SubnetMask.v4));\r
+    //\r
+    // Retrive uncommitted data from Browser\r
+    //\r
+    if (!HiiGetBrowserData (&gNicIp4ConfigNvDataGuid, EFI_NIC_IP4_CONFIG_VARIABLE, sizeof (IP4_CONFIG_IFR_NVDATA), (UINT8 *) IfrFormNvData)) {\r
+      FreePool (IfrFormNvData);\r
+      return EFI_NOT_FOUND;\r
     }\r
 \r
-    break;\r
+    Status = EFI_SUCCESS;\r
 \r
-  case KEY_GATE_WAY:\r
-    UnicodeStrToAsciiStr (IfrFormNvData->GatewayAddress, Ip4String);\r
-    Status = Ip4AsciiStrToIp (Ip4String, &Gateway.v4);\r
-    if (EFI_ERROR (Status) || ((Gateway.Addr[0] != 0) && !Ip4IsUnicast (NTOHL (Gateway.Addr[0]), 0))) {\r
-      CreatePopUp (EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE, &Key, L"Invalid Gateway!", NULL);\r
-      Status = EFI_INVALID_PARAMETER;\r
-    } else {\r
-      CopyMem (&Private->Current->SessionConfigData.Gateway, &Gateway.v4, sizeof (Gateway.v4));\r
-    }\r
-\r
-    break;\r
-\r
-  case KEY_SAVE_CHANGES:\r
-    Ip4ConfigInstance = Private->Current->Ip4ConfigInstance;\r
-    NicInfo = AllocateZeroPool (sizeof (NIC_IP4_CONFIG_INFO) + 2 * sizeof (EFI_IP4_ROUTE_TABLE));\r
-    NicInfo->Ip4Info.RouteTable = (EFI_IP4_ROUTE_TABLE *) (NicInfo + 1);\r
-\r
-    if (!Private->Current->SessionConfigData.Enabled) {\r
-      CopyMem (&HostIp.v4, &Private->Current->SessionConfigData.LocalIp, sizeof (HostIp.v4));\r
-      CopyMem (&SubnetMask.v4, &Private->Current->SessionConfigData.SubnetMask, sizeof (SubnetMask.v4));\r
-      CopyMem (&Gateway.v4, &Private->Current->SessionConfigData.Gateway, sizeof (Gateway.v4));\r
-\r
-      if ((Gateway.Addr[0] != 0)) {\r
-        if (SubnetMask.Addr[0] == 0) {\r
-          CreatePopUp (EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE, &Key, L"Gateway address is set but subnet mask is zero.", NULL);\r
-          Status = EFI_INVALID_PARAMETER;\r
-          break;\r
-        } else if (!IP4_NET_EQUAL (HostIp.Addr[0], Gateway.Addr[0], SubnetMask.Addr[0])) {\r
-          CreatePopUp (EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE, &Key, L"Local IP and Gateway are not in the same subnet.", NULL);\r
-          Status = EFI_INVALID_PARAMETER;\r
-          break;\r
-        }\r
+    switch (QuestionId) {\r
+    case KEY_LOCAL_IP:\r
+      Status = Ip4StrToIp (IfrFormNvData->StationAddress, &HostIp.v4);\r
+      if (EFI_ERROR (Status) || !NetIp4IsUnicast (NTOHL (HostIp.Addr[0]), 0)) {\r
+        CreatePopUp (EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE, &Key, L"Invalid IP address!", NULL);\r
+        Status = EFI_INVALID_PARAMETER;\r
+      } else {\r
+        CopyMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo.LocalIp, &HostIp.v4, sizeof (HostIp.v4));\r
       }\r
 \r
-      NicInfo->Source = IP4_CONFIG_SOURCE_STATIC;\r
-      NicInfo->Ip4Info.RouteTableSize = 2;\r
-\r
-      CopyMem (&NicInfo->Ip4Info.StationAddress, &HostIp.v4, sizeof (EFI_IPv4_ADDRESS));\r
-      CopyMem (&NicInfo->Ip4Info.SubnetMask, &SubnetMask.v4, sizeof (EFI_IPv4_ADDRESS));\r
-\r
-      Ip.Addr[0] = HostIp.Addr[0] & SubnetMask.Addr[0];\r
-\r
-      CopyMem (&NicInfo->Ip4Info.RouteTable[0].SubnetAddress, &Ip.v4, sizeof (EFI_IPv4_ADDRESS));\r
-      CopyMem (&NicInfo->Ip4Info.RouteTable[0].SubnetMask, &SubnetMask.v4, sizeof (EFI_IPv4_ADDRESS));\r
-      CopyMem (&NicInfo->Ip4Info.RouteTable[1].GatewayAddress, &Gateway.v4, sizeof (EFI_IPv4_ADDRESS));\r
-    } else {\r
-      NicInfo->Source = IP4_CONFIG_SOURCE_DHCP;\r
-    }\r
-\r
-    NicInfo->Perment = TRUE;\r
-    CopyMem (&NicInfo->NicAddr, &Ip4ConfigInstance->NicAddr, sizeof (NIC_ADDR));\r
-\r
-    EfiNicIp4ConfigSetInfo (Ip4ConfigInstance, NicInfo, TRUE);\r
+      break;\r
 \r
-    *ActionRequest = EFI_BROWSER_ACTION_REQUEST_SUBMIT;\r
-    break;\r
+    case KEY_SUBNET_MASK:\r
+      Status = Ip4StrToIp (IfrFormNvData->SubnetMask, &SubnetMask.v4);\r
+      if (EFI_ERROR (Status) || ((SubnetMask.Addr[0] != 0) && (GetSubnetMaskPrefixLength (&SubnetMask.v4) == 0))) {\r
+        CreatePopUp (EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE, &Key, L"Invalid Subnet Mask!", NULL);\r
+        Status = EFI_INVALID_PARAMETER;\r
+      } else {\r
+        CopyMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo.SubnetMask, &SubnetMask.v4, sizeof (SubnetMask.v4));\r
+      }\r
 \r
-  default:\r
-    if ((QuestionId >= KEY_DEVICE_ENTRY_BASE) && (QuestionId < (mNumberOfIp4Devices + KEY_DEVICE_ENTRY_BASE))) {\r
-      //\r
-      // In case goto the device configuration form, update the device form title.\r
-      //\r
-      ConfigFormEntry = Ip4GetConfigFormEntryByIndex ((UINT32) (QuestionId - KEY_DEVICE_ENTRY_BASE));\r
-      ASSERT (ConfigFormEntry != NULL);\r
+      break;\r
 \r
-      Ip4ConfigInstance = ConfigFormEntry->Ip4ConfigInstance;\r
+    case KEY_GATE_WAY:\r
+      Status = Ip4StrToIp (IfrFormNvData->GatewayAddress, &Gateway.v4);\r
+      if (EFI_ERROR (Status) || ((Gateway.Addr[0] != 0) && !NetIp4IsUnicast (NTOHL (Gateway.Addr[0]), 0))) {\r
+        CreatePopUp (EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE, &Key, L"Invalid Gateway!", NULL);\r
+        Status = EFI_INVALID_PARAMETER;\r
+      } else {\r
+        CopyMem (&Ip4ConfigInstance->Ip4ConfigCallbackInfo.Gateway, &Gateway.v4, sizeof (Gateway.v4));\r
+      }\r
 \r
-      UnicodeSPrint (PortString, (UINTN) 128, L"Port %s", ConfigFormEntry->MacString);\r
-      DeviceFormTitleToken = (EFI_STRING_ID) STR_IP4_DEVICE_FORM_TITLE;\r
-      HiiSetString (Private->RegisteredHandle, DeviceFormTitleToken, PortString, NULL);\r
+      break;\r
 \r
-      Ip4ConfigConvertDeviceConfigDataToIfrNvData (Ip4ConfigInstance, IfrFormNvData);\r
+    case KEY_SAVE_CHANGES:\r
+      Status = Ip4ConfigConvertIfrNvDataToDeviceConfigData (IfrFormNvData, Ip4ConfigInstance);\r
+      *ActionRequest = EFI_BROWSER_ACTION_REQUEST_SUBMIT;\r
+      break;\r
 \r
-      Private->Current = ConfigFormEntry;\r
+    default:\r
+      break;\r
     }\r
 \r
-    break;\r
-  }\r
-\r
-  if (!EFI_ERROR (Status)) {\r
+    FreePool (IfrFormNvData);\r
 \r
-    //\r
-    // Pass changed uncommitted data back to Form Browser\r
-    //\r
-    HiiSetBrowserData (&gEfiNicIp4ConfigVariableGuid, EFI_NIC_IP4_CONFIG_VARIABLE, sizeof (IP4_CONFIG_IFR_NVDATA), (UINT8 *) IfrFormNvData, NULL);\r
+    return Status;\r
   }\r
 \r
-  FreePool (IfrFormNvData);\r
-  return Status;\r
+  //\r
+  // All other action return unsupported.\r
+  //\r
+  return EFI_UNSUPPORTED;\r
 }\r
 \r
 /**\r
@@ -823,44 +772,96 @@ Ip4FormCallback (
 **/\r
 EFI_STATUS\r
 Ip4ConfigDeviceInit (\r
-    IN IP4_CONFIG_INSTANCE              *Instance\r
+  IN IP4_CONFIG_INSTANCE         *Instance\r
   )\r
 {\r
-  EFI_STATUS                  Status;\r
-  EFI_HII_DATABASE_PROTOCOL   *HiiDatabase;\r
-  IP4_FORM_CALLBACK_INFO      *CallbackInfo;\r
-\r
-  Status = gBS->LocateProtocol (&gEfiHiiDatabaseProtocolGuid, NULL, (VOID **)&HiiDatabase);\r
-  if (EFI_ERROR (Status)) {\r
-    return Status;\r
-  }\r
+  EFI_STATUS                     Status;\r
+  EFI_HII_CONFIG_ACCESS_PROTOCOL *ConfigAccess;\r
+  VENDOR_DEVICE_PATH             VendorDeviceNode;\r
+  EFI_SERVICE_BINDING_PROTOCOL   *MnpSb;\r
+  CHAR16                         *MacString;\r
+  CHAR16                         MenuString[128];\r
+  CHAR16                         PortString[128];\r
+  CHAR16                         *OldMenuString;\r
 \r
-  CallbackInfo = &Instance->Ip4FormCallbackInfo;\r
+  ConfigAccess = &Instance->HiiConfigAccessProtocol;\r
+  ConfigAccess->ExtractConfig = Ip4DeviceExtractConfig;\r
+  ConfigAccess->RouteConfig   = Ip4DeviceRouteConfig;\r
+  ConfigAccess->Callback      = Ip4FormCallback;\r
 \r
-  CallbackInfo->Signature   = IP4CONFIG_FORM_CALLBACK_INFO_SIGNATURE;\r
-  CallbackInfo->HiiDatabase = HiiDatabase;\r
+  //\r
+  // Construct device path node for EFI HII Config Access protocol,\r
+  // which consists of controller physical device path and one hardware\r
+  // vendor guid node.\r
+  //\r
+  ZeroMem (&VendorDeviceNode, sizeof (VENDOR_DEVICE_PATH));\r
+  VendorDeviceNode.Header.Type = HARDWARE_DEVICE_PATH;\r
+  VendorDeviceNode.Header.SubType = HW_VENDOR_DP;\r
 \r
-  CallbackInfo->ConfigAccess.ExtractConfig = Ip4DeviceExtractConfig;\r
-  CallbackInfo->ConfigAccess.RouteConfig = Ip4DeviceRouteConfig;\r
-  CallbackInfo->ConfigAccess.Callback = NULL;\r
+  CopyGuid (&VendorDeviceNode.Guid, &gEfiNicIp4ConfigVariableGuid);\r
 \r
-  Status = gBS->LocateProtocol (&gEfiHiiConfigRoutingProtocolGuid, NULL, (VOID **)&CallbackInfo->ConfigRouting);\r
-  if (EFI_ERROR (Status)) {\r
-    return Status;\r
-  }\r
+  SetDevicePathNodeLength (&VendorDeviceNode.Header, sizeof (VENDOR_DEVICE_PATH));\r
+  Instance->HiiVendorDevicePath = AppendDevicePathNode (\r
+                                    Instance->ParentDevicePath,\r
+                                    (EFI_DEVICE_PATH_PROTOCOL *) &VendorDeviceNode\r
+                                    );\r
 \r
-  CallbackInfo->DriverHandle = Instance->Controller;\r
+  Instance->ChildHandle = NULL;\r
   //\r
-  // Install Device Path Protocol and Config Access protocol to driver handle\r
+  // Install Device Path Protocol and Config Access protocol on new handle\r
   //\r
   Status = gBS->InstallMultipleProtocolInterfaces (\r
-                  &CallbackInfo->DriverHandle,\r
+                  &Instance->ChildHandle,\r
+                  &gEfiDevicePathProtocolGuid,\r
+                  Instance->HiiVendorDevicePath,\r
                   &gEfiHiiConfigAccessProtocolGuid,\r
-                  &CallbackInfo->ConfigAccess,\r
+                  ConfigAccess,\r
                   NULL\r
                   );\r
+  if (!EFI_ERROR (Status)) {\r
+    //\r
+    // Open the Parent Handle for the child\r
+    //\r
+    Status = gBS->OpenProtocol (\r
+                    Instance->Controller,\r
+                    &gEfiManagedNetworkServiceBindingProtocolGuid,\r
+                    (VOID **) &MnpSb,\r
+                    Instance->Image,\r
+                    Instance->ChildHandle,\r
+                    EFI_OPEN_PROTOCOL_BY_CHILD_CONTROLLER\r
+                    );\r
+  }\r
+\r
   ASSERT_EFI_ERROR (Status);\r
 \r
+  //\r
+  // Publish our HII data\r
+  //\r
+  Instance->RegisteredHandle = HiiAddPackages (\r
+                                 &gNicIp4ConfigNvDataGuid,\r
+                                 Instance->ChildHandle,\r
+                                 Ip4ConfigDxeStrings,\r
+                                 Ip4ConfigDxeBin,\r
+                                 NULL\r
+                                 );\r
+  if (Instance->RegisteredHandle == NULL) {\r
+    return EFI_OUT_OF_RESOURCES;\r
+  }\r
+\r
+  //\r
+  // Append MAC string in the menu help string and tile help string\r
+  //\r
+  Status = NetLibGetMacString (Instance->Controller, Instance->Image, &MacString);\r
+  if (!EFI_ERROR (Status)) {\r
+    OldMenuString = HiiGetString (Instance->RegisteredHandle, STRING_TOKEN (STR_IP4_CONFIG_FORM_HELP), NULL);\r
+    UnicodeSPrint (MenuString, 128, L"%s (MAC:%s)", OldMenuString, MacString);\r
+    HiiSetString (Instance->RegisteredHandle, STRING_TOKEN (STR_IP4_CONFIG_FORM_HELP), MenuString, NULL);\r
+\r
+    UnicodeSPrint (PortString, 128, L"MAC:%s", MacString);\r
+    HiiSetString (Instance->RegisteredHandle, STRING_TOKEN (STR_IP4_DEVICE_FORM_HELP), PortString, NULL);\r
+    FreePool (MacString);\r
+  }\r
+\r
   return Status;\r
 }\r
 \r
@@ -877,292 +878,32 @@ Ip4ConfigDeviceUnload (
   IN IP4_CONFIG_INSTANCE              *Instance\r
   )\r
 {\r
-  IP4_FORM_CALLBACK_INFO      *CallbackInfo;\r
-\r
-  CallbackInfo = &Instance->Ip4FormCallbackInfo;\r
-\r
-  Ip4ConfigUpdateForm (Instance, FALSE);\r
-\r
   //\r
-  // Uninstall EFI_HII_CONFIG_ACCESS_PROTOCOL\r
+  // Remove HII package list\r
   //\r
-  gBS->UninstallMultipleProtocolInterfaces (\r
-         CallbackInfo->DriverHandle,\r
-         &gEfiHiiConfigAccessProtocolGuid,\r
-         &CallbackInfo->ConfigAccess,\r
-         NULL\r
-         );\r
-\r
-  return EFI_SUCCESS;\r
-}\r
-\r
-/**\r
-  Unload the network configuration form, this includes: delete all the network\r
-  device configuration entries, uninstall the form callback protocol and\r
-  free the resources used.\r
-\r
-  @retval EFI_SUCCESS             The network configuration form is unloaded.\r
-**/\r
-EFI_STATUS\r
-Ip4ConfigFormUnload (\r
-  VOID\r
-  )\r
-{\r
-  IP4CONFIG_FORM_ENTRY     *ConfigFormEntry;\r
-\r
-  while (!IsListEmpty (&mIp4ConfigFormList)) {\r
-    //\r
-    // Uninstall the device forms as the network driver instance may fail to\r
-    // control the controller but still install the device configuration form.\r
-    // In such case, upon driver unloading, the driver instance's driverbinding.\r
-    // stop () won't be called, so we have to take this chance here to uninstall\r
-    // the device form.\r
-    //\r
-    ConfigFormEntry = NET_LIST_USER_STRUCT (mIp4ConfigFormList.ForwardLink, IP4CONFIG_FORM_ENTRY, Link);\r
-    Ip4ConfigUpdateForm (ConfigFormEntry->Ip4ConfigInstance, FALSE);\r
-  }\r
+  HiiRemovePackages (Instance->RegisteredHandle);\r
 \r
   //\r
-  // Remove HII package list\r
+  // Close the child handle\r
   //\r
-  mCallbackInfo->HiiDatabase->RemovePackageList (\r
-                                mCallbackInfo->HiiDatabase,\r
-                                mCallbackInfo->RegisteredHandle\r
-                                );\r
+  gBS->CloseProtocol (\r
+         Instance->Controller,\r
+         &gEfiManagedNetworkServiceBindingProtocolGuid,\r
+         Instance->Image,\r
+         Instance->ChildHandle\r
+         );\r
 \r
   //\r
   // Uninstall EFI_HII_CONFIG_ACCESS_PROTOCOL\r
   //\r
   gBS->UninstallMultipleProtocolInterfaces (\r
-         mCallbackInfo->DriverHandle,\r
+         Instance->ChildHandle,\r
          &gEfiDevicePathProtocolGuid,\r
-         &mIp4ConifgHiiVendorDevicePath,\r
+         Instance->HiiVendorDevicePath,\r
          &gEfiHiiConfigAccessProtocolGuid,\r
-         &mCallbackInfo->ConfigAccess,\r
+         &Instance->HiiConfigAccessProtocol,\r
          NULL\r
          );\r
 \r
-  FreePool (mCallbackInfo);\r
-\r
-  return EFI_SUCCESS;\r
-}\r
-\r
-/**\r
-  Updates the network configuration form to add/delete an entry for the network\r
-  device specified by the Instance.\r
-\r
-  @param[in]  Instance            The IP4 Config instance.\r
-  @param[in]  AddForm             Whether to add or delete a form entry.\r
-\r
-  @retval EFI_SUCCESS             The network configuration form is updated.\r
-  @retval EFI_OUT_OF_RESOURCES    Failed to allocate memory.\r
-  @retval Others                  Other errors as indicated.\r
-**/\r
-EFI_STATUS\r
-Ip4ConfigUpdateForm (\r
-  IN IP4_CONFIG_INSTANCE              *Instance,\r
-  IN BOOLEAN                          AddForm\r
-  )\r
-{\r
-  LIST_ENTRY                  *Entry;\r
-  IP4CONFIG_FORM_ENTRY        *ConfigFormEntry;\r
-  BOOLEAN                     EntryExisted;\r
-  EFI_STATUS                  Status;\r
-  EFI_SIMPLE_NETWORK_PROTOCOL *Snp;\r
-  CHAR16                      PortString[128];\r
-  UINT16                      FormIndex;\r
-  VOID                        *StartOpCodeHandle;\r
-  VOID                        *EndOpCodeHandle;\r
-  EFI_IFR_GUID_LABEL          *StartLabel;\r
-  EFI_IFR_GUID_LABEL          *EndLabel;\r
-\r
-  ConfigFormEntry = NULL;\r
-  EntryExisted    = FALSE;\r
-\r
-  NET_LIST_FOR_EACH (Entry, &mIp4ConfigFormList) {\r
-    ConfigFormEntry = NET_LIST_USER_STRUCT (Entry, IP4CONFIG_FORM_ENTRY, Link);\r
-\r
-    if (ConfigFormEntry->Controller == Instance->Controller) {\r
-      EntryExisted = TRUE;\r
-      break;\r
-    }\r
-  }\r
-\r
-  if (AddForm) {\r
-    if (EntryExisted) {\r
-      return EFI_SUCCESS;\r
-    } else {\r
-      //\r
-      // Add a new form.\r
-      //\r
-      ConfigFormEntry = (IP4CONFIG_FORM_ENTRY *) AllocateZeroPool (sizeof (IP4CONFIG_FORM_ENTRY));\r
-      if (ConfigFormEntry == NULL) {\r
-        return EFI_OUT_OF_RESOURCES;\r
-      }\r
-\r
-      ConfigFormEntry->Ip4ConfigInstance = Instance;\r
-      InitializeListHead (&ConfigFormEntry->Link);\r
-      ConfigFormEntry->Controller = Instance->Controller;\r
-\r
-      //\r
-      // Get the simple network protocol and convert the MAC address into\r
-      // the formatted string.\r
-      //\r
-      Status = gBS->HandleProtocol (\r
-                      Instance->Controller,\r
-                      &gEfiSimpleNetworkProtocolGuid,\r
-                      (VOID **)&Snp\r
-                      );\r
-      ASSERT (Status == EFI_SUCCESS);\r
-\r
-      Ip4MacAddrToStr (&Snp->Mode->PermanentAddress, Snp->Mode->HwAddressSize, ConfigFormEntry->MacString);\r
-\r
-      //\r
-      // Compose the Port string and create a new EFI_STRING_ID.\r
-      //\r
-      UnicodeSPrint (PortString, 128, L"%s %s", Instance->NicName, ConfigFormEntry->MacString);\r
-      ConfigFormEntry->PortTitleToken = HiiSetString (mCallbackInfo->RegisteredHandle, 0, PortString, NULL);\r
-\r
-      //\r
-      // Compose the help string of this port and create a new EFI_STRING_ID.\r
-      //\r
-      UnicodeSPrint (PortString, 128, L"Set the network parameters on eth%d %s", 0, ConfigFormEntry->MacString);\r
-      ConfigFormEntry->PortTitleHelpToken = HiiSetString (mCallbackInfo->RegisteredHandle, 0, PortString, NULL);\r
-\r
-      InsertTailList (&mIp4ConfigFormList, &ConfigFormEntry->Link);\r
-      mNumberOfIp4Devices++;\r
-    }\r
-  } else {\r
-    ASSERT (EntryExisted);\r
-\r
-    mNumberOfIp4Devices--;\r
-    RemoveEntryList (&ConfigFormEntry->Link);\r
-    gBS->FreePool (ConfigFormEntry);\r
-  }\r
-\r
-  //\r
-  // Init OpCode Handle\r
-  //\r
-  StartOpCodeHandle = HiiAllocateOpCodeHandle ();\r
-  ASSERT (StartOpCodeHandle != NULL);\r
-\r
-  EndOpCodeHandle = HiiAllocateOpCodeHandle ();\r
-  ASSERT (EndOpCodeHandle != NULL);\r
-\r
-  //\r
-  // Create Hii Extend Label OpCode as the start opcode\r
-  //\r
-  StartLabel = (EFI_IFR_GUID_LABEL *) HiiCreateGuidOpCode (StartOpCodeHandle, &gEfiIfrTianoGuid, NULL, sizeof (EFI_IFR_GUID_LABEL));\r
-  StartLabel->ExtendOpCode = EFI_IFR_EXTEND_OP_LABEL;\r
-  StartLabel->Number       = DEVICE_ENTRY_LABEL;\r
-\r
-  //\r
-  // Create Hii Extend Label OpCode as the end opcode\r
-  //\r
-  EndLabel = (EFI_IFR_GUID_LABEL *) HiiCreateGuidOpCode (EndOpCodeHandle, &gEfiIfrTianoGuid, NULL, sizeof (EFI_IFR_GUID_LABEL));\r
-  EndLabel->ExtendOpCode = EFI_IFR_EXTEND_OP_LABEL;\r
-  EndLabel->Number       = LABEL_END;\r
-\r
-  FormIndex = 0;\r
-\r
-  FormIndex = 0;\r
-  NET_LIST_FOR_EACH (Entry, &mIp4ConfigFormList) {\r
-    ConfigFormEntry = NET_LIST_USER_STRUCT (Entry, IP4CONFIG_FORM_ENTRY, Link);\r
-\r
-    HiiCreateGotoOpCode (\r
-      StartOpCodeHandle,                            // Container for dynamic created opcodes\r
-      FORMID_DEVICE_FORM,                           // Target Form ID\r
-      ConfigFormEntry->PortTitleToken,              // Prompt text\r
-      ConfigFormEntry->PortTitleHelpToken,          // Help text\r
-      EFI_IFR_FLAG_CALLBACK,                        // Question flag\r
-      (UINT16)(KEY_DEVICE_ENTRY_BASE + FormIndex)   // Question ID\r
-      );\r
-\r
-    FormIndex++;\r
-  }\r
-\r
-  HiiUpdateForm (\r
-    mCallbackInfo->RegisteredHandle,\r
-    &gEfiNicIp4ConfigVariableGuid,\r
-    FORMID_MAIN_FORM,\r
-    StartOpCodeHandle, // Label DEVICE_ENTRY_LABEL\r
-    EndOpCodeHandle    // LABEL_END\r
-    );\r
-\r
-  HiiFreeOpCodeHandle (StartOpCodeHandle);\r
-  HiiFreeOpCodeHandle (EndOpCodeHandle);\r
-\r
   return EFI_SUCCESS;\r
 }\r
-\r
-/**\r
-  Initialize the network configuration form, this includes: delete all the network\r
-  device configuration entries, install the form callback protocol and\r
-  allocate the resources used.\r
-\r
-  @retval EFI_SUCCESS             The network configuration form is unloaded.\r
-  @retval EFI_OUT_OF_RESOURCES    Failed to allocate memory.\r
-**/\r
-EFI_STATUS\r
-Ip4ConfigFormInit (\r
-    VOID\r
-  )\r
-{\r
-  EFI_STATUS                  Status;\r
-  EFI_HII_DATABASE_PROTOCOL   *HiiDatabase;\r
-  IP4_FORM_CALLBACK_INFO      *CallbackInfo;\r
-\r
-  Status = gBS->LocateProtocol (&gEfiHiiDatabaseProtocolGuid, NULL, (VOID **)&HiiDatabase);\r
-  if (EFI_ERROR (Status)) {\r
-    return Status;\r
-  }\r
-\r
-  CallbackInfo = (IP4_FORM_CALLBACK_INFO *) AllocateZeroPool (sizeof (IP4_FORM_CALLBACK_INFO));\r
-  if (CallbackInfo == NULL) {\r
-    return EFI_OUT_OF_RESOURCES;\r
-  }\r
-\r
-  CallbackInfo->Signature   = IP4CONFIG_FORM_CALLBACK_INFO_SIGNATURE;\r
-  CallbackInfo->HiiDatabase = HiiDatabase;\r
-\r
-  CallbackInfo->ConfigAccess.ExtractConfig = Ip4FormExtractConfig;\r
-  CallbackInfo->ConfigAccess.RouteConfig = Ip4FormRouteConfig;\r
-  CallbackInfo->ConfigAccess.Callback = Ip4FormCallback;\r
-\r
-  Status = gBS->LocateProtocol (&gEfiHiiConfigRoutingProtocolGuid, NULL, (VOID **)&CallbackInfo->ConfigRouting);\r
-  if (EFI_ERROR (Status)) {\r
-    return Status;\r
-  }\r
-\r
-  CallbackInfo->DriverHandle = NULL;\r
-  //\r
-  // Install Device Path Protocol and Config Access protocol to driver handle\r
-  //\r
-  Status = gBS->InstallMultipleProtocolInterfaces (\r
-                  &CallbackInfo->DriverHandle,\r
-                  &gEfiDevicePathProtocolGuid,\r
-                  &mIp4ConifgHiiVendorDevicePath,\r
-                  &gEfiHiiConfigAccessProtocolGuid,\r
-                  &CallbackInfo->ConfigAccess,\r
-                  NULL\r
-                  );\r
-  ASSERT_EFI_ERROR (Status);\r
-\r
-  //\r
-  // Publish our HII data\r
-  //\r
-  CallbackInfo->RegisteredHandle = HiiAddPackages (\r
-                                     &gEfiNicIp4ConfigVariableGuid,\r
-                                     CallbackInfo->DriverHandle,\r
-                                     Ip4ConfigDxeStrings,\r
-                                     Ip4ConfigDxeBin,\r
-                                     NULL\r
-                                     );\r
-  if (CallbackInfo->RegisteredHandle == NULL) {\r
-    return EFI_OUT_OF_RESOURCES;\r
-  }\r
-\r
-  mCallbackInfo = CallbackInfo;\r
-\r
-  return Status;\r
-}\r