reg: [edk2-devel] [PATCH] Adding the MAC in the HTTP Boot Title

Posted by Sivaraman Nainar 3 years, 4 months ago

Hello all:

Here with added the patch for displaying the MAC Address along with the HTTP Boot Option Title. Please review and comment.

---
 NetworkPkg/HttpBootDxe/HttpBootConfig.c | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/NetworkPkg/HttpBootDxe/HttpBootConfig.c b/NetworkPkg/HttpBootDxe/HttpBootConfig.c
index 646c907b12..677d913a09 100644
--- a/NetworkPkg/HttpBootDxe/HttpBootConfig.c
+++ b/NetworkPkg/HttpBootDxe/HttpBootConfig.c
@@ -629,6 +629,18 @@ HttpBootConfigFormInit (
       NULL
       );

+    OldMenuString = HiiGetString (
+                          CallbackInfo->RegisteredHandle,
+                          STRING_TOKEN (STR_HTTP_BOOT_CONFIG_FORM_TITLE),
+                          NULL
+                          );
+        UnicodeSPrint (MenuString, 128, L"MAC:%s-%s", MacString,OldMenuString);
+        HiiSetString (
+          CallbackInfo->RegisteredHandle,
+          STRING_TOKEN (STR_HTTP_BOOT_CONFIG_FORM_TITLE),
+          MenuString,
+          NULL
+          );
     FreePool (MacString);
     FreePool (OldMenuString);


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#68492): https://edk2.groups.io/g/devel/message/68492
Mute This Topic: https://groups.io/mt/78823951/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: reg: [edk2-devel] [PATCH] Adding the MAC in the HTTP Boot Title

Posted by Maciej Rabeda 3 years, 4 months ago

Please provide a patch in the way described in the process.
Formal: 
https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Development-Process
Informal: 
https://github.com/tianocore/tianocore.github.io/wiki/Laszlo's-unkempt-git-guide-for-edk2-contributors-and-maintainers

Thanks,
Maciej

On 09-Dec-20 07:36, Sivaraman Nainar wrote:
>
> Hello all:
>
> Here with added the patch for displaying the MAC Address along with 
> the HTTP Boot Option Title. Please review and comment.
>
> ---
>  NetworkPkg/HttpBootDxe/HttpBootConfig.c | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
>
> diff --git a/NetworkPkg/HttpBootDxe/HttpBootConfig.c 
> b/NetworkPkg/HttpBootDxe/HttpBootConfig.c
> index 646c907b12..677d913a09 100644
> --- a/NetworkPkg/HttpBootDxe/HttpBootConfig.c
> +++ b/NetworkPkg/HttpBootDxe/HttpBootConfig.c
> @@ -629,6 +629,18 @@ HttpBootConfigFormInit (
>        NULL
>        );
>
> +    OldMenuString = HiiGetString (
> + CallbackInfo->RegisteredHandle,
> + STRING_TOKEN (STR_HTTP_BOOT_CONFIG_FORM_TITLE),
> + NULL
> +                          );
> +        UnicodeSPrint (MenuString, 128, L"MAC:%s-%s", 
> MacString,OldMenuString);
> +        HiiSetString (
> + CallbackInfo->RegisteredHandle,
> +          STRING_TOKEN (STR_HTTP_BOOT_CONFIG_FORM_TITLE),
> +          MenuString,
> +          NULL
> +          );
>      FreePool (MacString);
>      FreePool (OldMenuString);
>



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#68555): https://edk2.groups.io/g/devel/message/68555
Mute This Topic: https://groups.io/mt/78823951/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-