From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 134.134.136.100, mailfrom: zhichao.gao@intel.com) Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by groups.io with SMTP; Tue, 17 Sep 2019 00:16:26 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga004.jf.intel.com ([10.7.209.38]) by orsmga105.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 17 Sep 2019 00:16:25 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,515,1559545200"; d="scan'208";a="337909591" Received: from fmsmsx103.amr.corp.intel.com ([10.18.124.201]) by orsmga004.jf.intel.com with ESMTP; 17 Sep 2019 00:16:25 -0700 Received: from fmsmsx153.amr.corp.intel.com (10.18.125.6) by FMSMSX103.amr.corp.intel.com (10.18.124.201) with Microsoft SMTP Server (TLS) id 14.3.439.0; Tue, 17 Sep 2019 00:16:25 -0700 Received: from shsmsx106.ccr.corp.intel.com (10.239.4.159) by FMSMSX153.amr.corp.intel.com (10.18.125.6) with Microsoft SMTP Server (TLS) id 14.3.439.0; Tue, 17 Sep 2019 00:16:25 -0700 Received: from shsmsx101.ccr.corp.intel.com ([169.254.1.92]) by SHSMSX106.ccr.corp.intel.com ([169.254.10.86]) with mapi id 14.03.0439.000; Tue, 17 Sep 2019 15:16:23 +0800 From: "Gao, Zhichao" To: "Gao, Liming" , Ard Biesheuvel CC: edk2-devel-groups-io , "Wang, Jian J" , "Wu, Hao A" , "Ni, Ray" , Laszlo Ersek Subject: Re: [edk2-devel] [PATCH V2 0/3] MdeModulePkg/TerminalConsole: Extend the support terminal types Thread-Topic: [edk2-devel] [PATCH V2 0/3] MdeModulePkg/TerminalConsole: Extend the support terminal types Thread-Index: AQHVbSEmbXzr2TYnEUmgGxeImGdI6Kcu5zMAgACGgqA= Date: Tue, 17 Sep 2019 07:16:23 +0000 Message-ID: <3CE959C139B4C44DBEA1810E3AA6F9000B83AEA9@SHSMSX101.ccr.corp.intel.com> References: <20190917061912.17420-1-zhichao.gao@intel.com> <4A89E2EF3DFEDB4C8BFDE51014F606A14E4FD578@SHSMSX104.ccr.corp.intel.com> In-Reply-To: <4A89E2EF3DFEDB4C8BFDE51014F606A14E4FD578@SHSMSX104.ccr.corp.intel.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Return-Path: zhichao.gao@intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Thanks Liming to clarify my confusion. I just want to know which broken pat= ch is in my patch set. > -----Original Message----- > From: Gao, Liming > Sent: Tuesday, September 17, 2019 2:48 PM > To: Ard Biesheuvel ; Gao, Zhichao > > Cc: edk2-devel-groups-io ; Wang, Jian J > ; Wu, Hao A ; Ni, Ray > ; Laszlo Ersek > Subject: RE: [edk2-devel] [PATCH V2 0/3] MdeModulePkg/TerminalConsole: > Extend the support terminal types >=20 > Ard: > Thanks for your quick report. The break is caused by the patch [edk2-d= evel] > [PATCH 2/2] MdeModulePkg/SerialDxe: Update the file Guid in SerialDxe.in= f. > This patch updates FILE_GUID only. But, its GUID value format is wrong= . I > just send the fix for it. >=20 > Thanks > Liming > >-----Original Message----- > >From: Ard Biesheuvel [mailto:ard.biesheuvel@linaro.org] > >Sent: Tuesday, September 17, 2019 2:29 PM > >To: Gao, Zhichao > >Cc: edk2-devel-groups-io ; Wang, Jian J > >; Wu, Hao A ; Ni, Ray > >; Laszlo Ersek ; Gao, Liming > > > >Subject: Re: [edk2-devel] [PATCH V2 0/3] MdeModulePkg/TerminalConsole: > >Extend the support terminal types > > > >On Tue, 17 Sep 2019 at 07:22, Gao, Zhichao wrot= e: > >> > >> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D2186 > >> > >> Putty is a very popular terminal tool in windows. So add the whole > >> support terminal keyboard type for it. The new introduced type is > >> Linux, XtermR6, > >> VT400 and SCO. And enhance the support for VT100+. > >> This patch set only add the support of function key. Refer to the lin= k: > >> https://www.ssh.com/ssh/putty/putty- > >manuals/0.68/Chapter4.html#config-funkeys > >> > >> V2: > >> Fix typo. > >> Merge the type guid defination into TtyTerm.h. > >> > > > >Hello Zhichao, > > > >Since you already pushed the broken patch, you should really be posting > >an updated patch with just the fix. > > > >In any case, this makes me wonder how you tested this code, since it > >won't even build. Could you elaborate? I usually build the patch on my local machine. I didn't catch that fault b= ecause my code base isn't up to date. And I have already ported it to a rea= l platform to verify the new introduced terminal type's function key. It wo= rks fine. Here is the common steps: 1. Apply this patch set to the edk repo 2. change the BdsBootMaintainUI lib of the platform (some platform add thi= s to its own setup driver) to support the new terminal type 3. change the PlatformBds library to make the whole terminal type legal be= cause some platform only judge the types in PcAnsi.h as legal ones 4. register F1 to F12 at the UiApp entry point (for test only) 5. build bios image and flash to the platform 6. boot to setup and change the console terminal type 7. boot the SUT(system under test) with a UART connect to the host 8. open Putty in the host and change the terminal keyboard type to match w= ith the setting in the setup 9. type function key to view if it can recognize the function correctly 10. go to 6 to verify the other terminal type Thanks, Zhichao > > > >-- > >Ard. > > > > > >> Cc: Jian J Wang > >> Cc: Hao A Wu > >> Cc: Ray Ni > >> Cc: Ard Biesheuvel > >> Cc: Laszlo Ersek > >> Cc: Liming Gao > >> Signed-of-by: Zhichao Gao > >> > >> Zhichao Gao (3): > >> MdeModulePkg: Extend the support keyboard type of Terminal console > >> MdeModulePkg/TerminalDxe: Extend the terminal console support > types > >> MdeModulePkg/BM_UI: Add the new terminal types to related menu > >> > >> MdeModulePkg/Include/Guid/TtyTerm.h | 13 + > >> .../BootMaintenanceManager.h | 12 +- > >> .../BootMaintenanceManagerStrings.uni | 10 +- > >> .../ConsoleOption.c | 35 +-- > >> .../BootMaintenanceManagerUiLib/Data.c | 16 +- > >> MdeModulePkg/MdeModulePkg.dec | 4 + > >> .../Universal/Console/TerminalDxe/Terminal.c | 17 +- > >> .../Universal/Console/TerminalDxe/Terminal.h | 37 ++- > >> .../Console/TerminalDxe/TerminalConIn.c | 281 ++++++++++++++++= -- > >> .../Console/TerminalDxe/TerminalConOut.c | 4 + > >> .../Console/TerminalDxe/TerminalDxe.inf | 6 +- > >> 11 files changed, 375 insertions(+), 60 deletions(-) > >> > >> -- > >> 2.21.0.windows.1 > >> > >> > >>=20 > >>