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.

-----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

Reply via email to