Date
1 - 3 of 3
Submit patch to devel@edk2.groups.io.
|
Keysound Chang
Hi,
I encountered an issue when trying to send a patch mail to devel@edk2.groups.io. The SMTP server returned 250 after I issue sendmail through git, and I also received cc. But I didn't see my post appeared on edk2-devel. I also checked this: https://github.com/tianocore/tianocore.github.io/wiki/Commit-Message-Format Looks like my format is appropriate. I append my mail here. Your advice/suggestion will be greatly appreciated. Keysound =========================================================== Subject: [PATCH] NetworkPkg/TlsAuthConfigDxe: Use HiiPopUp() instead of CreatePopUp() From: Keysound Chang <Keysound_Chang@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2635 According to EDK2 Driver Writer's Guide For UEFI 2.3.1, 4.2.18 Offer alternatives to function keys. Configuration of drivers should be accomplished via HII and via OS-present interfaces. Cc: Jiaxin Wu <jiaxin.wu@...> Cc: Siyuan Fu <siyuan.fu@...> Signed-off-by: Maciej Rabeda <maciej.rabeda@...> --- .../TlsAuthConfigDxe/TlsAuthConfigDxe.inf | 1 + .../TlsAuthConfigDxeStrings.uni | 2 ++ .../TlsAuthConfigDxe/TlsAuthConfigImpl.c | 20 +++++++++++++------ .../TlsAuthConfigDxe/TlsAuthConfigImpl.h | 1 + 4 files changed, 18 insertions(+), 6 deletions(-) diff --git a/NetworkPkg/TlsAuthConfigDxe/TlsAuthConfigDxe.inf b/NetworkPkg/TlsAuthConfigDxe/TlsAuthConfigDxe.inf index 3fc924a1d4..0ada835252 100644 --- a/NetworkPkg/TlsAuthConfigDxe/TlsAuthConfigDxe.inf +++ b/NetworkPkg/TlsAuthConfigDxe/TlsAuthConfigDxe.inf @@ -52,6 +52,7 @@ [Protocols] gEfiDevicePathProtocolGuid ## PRODUCES gEfiHiiConfigAccessProtocolGuid ## PRODUCES+ gEfiHiiPopupProtocolGuid ## CONSUMES [Guids] gTlsAuthConfigGuid ## PRODUCES ## GUIDdiff --git a/NetworkPkg/TlsAuthConfigDxe/TlsAuthConfigDxeStrings.uni b/NetworkPkg/TlsAuthConfigDxe/TlsAuthConfigDxeStrings.uni index 973b8b7716..2b4a27d24e 100644 --- a/NetworkPkg/TlsAuthConfigDxe/TlsAuthConfigDxeStrings.uni +++ b/NetworkPkg/TlsAuthConfigDxe/TlsAuthConfigDxeStrings.uni @@ -28,6 +28,8 @@ SPDX-License-Identifier: BSD-2-Clause-Patent #string STR_TLS_AUTH_CONFIG_SAVE_AND_EXIT #language en-US "Commit Changes and Exit" #string STR_TLS_AUTH_CONFIG_NO_SAVE_AND_EXIT #language en-US "Discard Changes and Exit" +#string STR_TLS_AUTH_ENROLL_CERT_FAILURE #language en-US "Enroll Cert Failure!"+ #string STR_CERT_TYPE_PCKS_GUID #language en-US "GUID for CERT" #string STR_NULL #language en-US ""diff --git a/NetworkPkg/TlsAuthConfigDxe/TlsAuthConfigImpl.c b/NetworkPkg/TlsAuthConfigDxe/TlsAuthConfigImpl.c index 2481d1098f..0ef96dfaf2 100644 --- a/NetworkPkg/TlsAuthConfigDxe/TlsAuthConfigImpl.c +++ b/NetworkPkg/TlsAuthConfigDxe/TlsAuthConfigImpl.c @@ -1383,7 +1383,6 @@ TlsAuthConfigAccessCallback ( OUT EFI_BROWSER_ACTION_REQUEST *ActionRequest ) {- EFI_INPUT_KEY Key; EFI_STATUS Status; RETURN_STATUS RStatus; TLS_AUTH_CONFIG_PRIVATE_DATA *Private;@@ -1391,6 +1390,8 @@ TlsAuthConfigAccessCallback ( TLS_AUTH_CONFIG_IFR_NVDATA *IfrNvData; UINT16 LabelId; EFI_DEVICE_PATH_PROTOCOL *File;+ EFI_HII_POPUP_PROTOCOL *HiiPopUp;+ EFI_HII_POPUP_SELECTION PopUpSelect; Status = EFI_SUCCESS; File = NULL;@@ -1402,6 +1403,11 @@ TlsAuthConfigAccessCallback ( Private = TLS_AUTH_CONFIG_PRIVATE_FROM_THIS (This); mTlsAuthPrivateData = Private;+ Status = gBS->LocateProtocol (&gEfiHiiPopupProtocolGuid, NULL, &HiiPopUp);+ if (EFI_ERROR (Status)) {+ DEBUG ((DEBUG_ERROR, "Can't find Form PopUp protocol. Exit (%r)\n", Status));+ return Status;+ } // // Retrieve uncommitted data from Browser@@ -1460,11 +1466,13 @@ TlsAuthConfigAccessCallback ( if (EFI_ERROR (Status)) { CleanFileContext (Private); - CreatePopUp (- EFI_LIGHTGRAY | EFI_BACKGROUND_BLUE,- &Key,- L"ERROR: Enroll Cert Failure!",- NULL+ HiiPopUp->CreatePopup(+ HiiPopUp,+ EfiHiiPopupStyleError,+ EfiHiiPopupTypeOk,+ Private->RegisteredHandle,+ STRING_TOKEN(STR_TLS_AUTH_ENROLL_CERT_FAILURE),+ &PopUpSelect ); } break;diff --git a/NetworkPkg/TlsAuthConfigDxe/TlsAuthConfigImpl.h b/NetworkPkg/TlsAuthConfigDxe/TlsAuthConfigImpl.h index e9af492893..68c16845b3 100644 --- a/NetworkPkg/TlsAuthConfigDxe/TlsAuthConfigImpl.h +++ b/NetworkPkg/TlsAuthConfigDxe/TlsAuthConfigImpl.h @@ -14,6 +14,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent #include <Protocol/HiiConfigAccess.h> #include <Protocol/SimpleFileSystem.h>+#include <Protocol/HiiPopup.h> // // Libraries-- 2.23.0.windows.1 |
|
|
|
Guomin Jiang
Hi keysound,
toggle quoted message
Show quoted text
I have ever encountered the issue. The reason is that you are the NuM(New user Moderated). you will need be approved by moderator and then you can see you mail normally. Please be patient for the process. Best Regards guomin -----Original Message----- |
|
|
|
Keysound Chang
Hi Guomin,
toggle quoted message
Show quoted text
Looks like it works. Thanks for your friendly reminder. Keysound -----Original Message-----
From: Jiang, Guomin <guomin.jiang@...> Sent: Thursday, April 9, 2020 3:50 PM To: discuss@edk2.groups.io; Keysound Chang <Keysound_Chang@...> Subject: RE: [edk2-discuss] Submit patch to devel@edk2.groups.io. Hi keysound, I have ever encountered the issue. The reason is that you are the NuM(New user Moderated). you will need be approved by moderator and then you can see you mail normally. Please be patient for the process. Best Regards guomin -----Original Message----- |
|
|