Ok, I will do it. -----Original Message----- From: Roy Franz [mailto:roy.fr...@linaro.org] Sent: Thursday, July 9, 2015 8:38 AM To: Tian, Feng Cc: edk2-devel@lists.sourceforge.net; ler...@redhat.com; ard.biesheu...@linaro.org; heyi....@linaro.org Subject: Re: [PATCH V4 0/5] Add TtyTerm terminal type
On Wed, Jul 8, 2015 at 5:25 PM, Tian, Feng <feng.t...@intel.com> wrote: > Roy, > > For patch 1, 2, 3, I would sign Reviewed-by: Feng Tian <feng.t...@intel.com> > > Thanks for your contribution. Please go ahead to check them in. > I'm not a committer, so I'll need someone to commit this for me. Thanks, Roy > -----Original Message----- > From: Roy Franz [mailto:roy.fr...@linaro.org] > Sent: Thursday, July 9, 2015 3:03 AM > To: edk2-devel@lists.sourceforge.net; Tian, Feng; ler...@redhat.com; > ard.biesheu...@linaro.org > Cc: heyi....@linaro.org; Roy Franz > Subject: [PATCH V4 0/5] Add TtyTerm terminal type > > This patchset adds a new terminal type "TtyTerm", that better handles the > typical Linux terminal (xterm/rxvt/gnome terminal/tmux/screen/etc.) These > often treat backspace/delete differently than the existing EDK2 terminal > types expect, particularly in emulated environments where the emulated serial > port is connected to a graphical terminal rather than a serial console. Some > terminals use a mix of vt100 and vt200 escape codes for function key > handling. Adding > vt220 function key support fixes the use of "F10" to save changes in BDS > configuration menus. > > This patchset includes changes to make the terminal type build time > configurable for the QEMU Aarch64 configuration. Once this patchset goes in > I will submit patches for the other emulated ARM platforms as appropriate. > > Note: While working on the function keys, I could not find any documentation > that matched the code/comments in TerminalConIn.c regarding VT100 function > keys. The VT100 only had 4 function keys (PF1-PF4), which are generally > mapped to F1-F4. In the code, F3/F4 don't match the VT100 documentation I > found, and I have no idea where the values for F5-F10 are from. I left the > existing > VT100 terminal type escape code handling unchanged. > > This patchset is also available as the ttyterm-v4 branch on > https://git.linaro.org/people/roy.franz/linaro-edk2.git > > Changes since v3: > * Convert !ifdef to !if for TTY_TERMINAL and INTEL_BDS defines (patch 5/5) > > Changes since v2: > * Comment typo fixes, contrib ordering fixes > * Moved static ttyTerm escape state variables to TerminalDevice structure > * Add support for changing terminal type on ARM BDS QEMU build (new patch 5/5) > * Add reviewed-by lines. (not to patch 3 that had changes) > > Changes since v1: > * Added handling of VT220 escape codes for function keys > * Fixed attribution PCD patch > * Removed '[' from patch 3 description line > > Changes from Linuxterm RFC patchset: > * Change to ttyTerm name - nothing linux specific in patchset > * remove changes to Mde module, as changes not part of UEFI specification > > Laszlo Ersek (1): > Add PCD for selecting terminal type at build time > > Roy Franz (4): > Add "TtyTerm" terminal type to TerminalDxe > Treat ASCII 0x7F as backspace for TtyTerm terminals > Accept VT220 DEL and function keys for TTY terminal type > add TTY_TERMINAL build option for ARM BDS > > ArmVirtPkg/ArmVirt.dsc.inc | 6 ++ > ArmVirtPkg/ArmVirtPkg.dec | 7 ++ > ArmVirtPkg/ArmVirtQemu.dsc | 11 +- > ArmVirtPkg/ArmVirtQemu.fdf | 4 +- > ArmVirtPkg/Library/PlatformIntelBdsLib/IntelBdsPlatform.c | 12 ++- > ArmVirtPkg/Library/PlatformIntelBdsLib/PlatformIntelBdsLib.inf | 4 + > MdeModulePkg/Include/Guid/TtyTerm.h | 25 +++++ > MdeModulePkg/MdeModulePkg.dec | 3 + > MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c | 50 > +++++++-- > MdeModulePkg/Universal/Console/TerminalDxe/Terminal.h | 5 + > MdeModulePkg/Universal/Console/TerminalDxe/TerminalConIn.c | 107 > +++++++++++++++++++- > MdeModulePkg/Universal/Console/TerminalDxe/TerminalConOut.c | 2 + > MdeModulePkg/Universal/Console/TerminalDxe/TerminalDxe.inf | 1 + > 13 files changed, 217 insertions(+), 20 deletions(-) create mode 100644 > MdeModulePkg/Include/Guid/TtyTerm.h > > -- > 2.1.4 > ------------------------------------------------------------------------------ Don't Limit Your Business. Reach for the Cloud. GigeNET's Cloud Solutions provide you with the tools and support that you need to offload your IT needs and focus on growing your business. Configured For All Businesses. Start Your Cloud Today. https://www.gigenetcloud.com/ _______________________________________________ edk2-devel mailing list edk2-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/edk2-devel
